Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare for v1.15 development cycle #26516

Merged
merged 1 commit into from Jul 7, 2023
Merged

Prepare for v1.15 development cycle #26516

merged 1 commit into from Jul 7, 2023

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Jun 27, 2023

Depends on: #26237 (which introduces per-branch comment-trigger workflows, so this PR doesn't need them)

This should be merged once we prepare the v1.14 branch.

Differences from the previous dev cycle preparation PR:

  • Don't include stable branch CI workflows in the main branch.
  • Rely on renovate over dependabot on main branch now.
  • Use the v1.15-dev version during the dev cycle.

@joestringer joestringer requested review from a team as code owners June 27, 2023 22:13
@joestringer joestringer requested a review from squeed June 27, 2023 22:13
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 27, 2023
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Jun 27, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 27, 2023
@nbusseneau nbusseneau self-requested a review June 28, 2023 15:23
@joestringer joestringer mentioned this pull request Jun 28, 2023
51 tasks
Copy link
Contributor

@squeed squeed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed for my files.

Copy link
Member

@bimmlerd bimmlerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

contributing changes LGTM

Copy link
Member

@jschwinger233 jschwinger233 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Datapath looks good, thanks!

Copy link
Member

@nbusseneau nbusseneau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed with Joe offline:

@@ -19,6 +19,6 @@ jobs:
- uses: docker://quay.io/cilium/scruffy:v0.0.3@sha256:ca997451b739cbf03c204cb2523a671c31c61edc606aa5d20dc3560bc7f25bc7
with:
entrypoint: scruffy
args: --git-repository=./ --stable-branches=origin/main,origin/v1.9,origin/v1.10,origin/v1.11,origin/v1.12,origin/v1.13
args: --git-repository=./ --stable-branches=origin/main,origin/v1.9,origin/v1.10,origin/v1.11,origin/v1.12,origin/v1.13,origin/v1.14
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Nit] Should we remove v1.9 and v1.10 from here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah we probably should. I have another PR in the works to drop all 1.11 or earlier references, which should wait until the release. I can add this change into that one.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -14,26 +14,31 @@ on:
- Cilium L4LB XDP (ci-l4lb-1.11)
- Cilium L4LB XDP (ci-l4lb-1.12)
- Cilium L4LB XDP (ci-l4lb-1.13)
- Cilium L4LB XDP (ci-l4lb-1.14)
Copy link
Member

@nbusseneau nbusseneau Jul 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Useless nit] The ci-xxx-1.xx workflows do not exist anymore, now it's only the regular non -1.xx versions for all branches. Not that it matters considering Loki is disabled anyway...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was debating if we should just be deleting the loki workflow instead.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@michi-covalent michi-covalent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😇

.github/renovate.json5 Show resolved Hide resolved
Signed-off-by: Joe Stringer <joe@cilium.io>
@aanm aanm merged commit 2414373 into main Jul 7, 2023
55 checks passed
@aanm aanm deleted the pr/joe/v1.15-prep branch July 7, 2023 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet