Skip to content

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Jun 9, 2023

[ upstream commit d90803c ]
[ upstream commit 2c367de ]
[ Backporter's notes:

  • Removed github workflow. Workflow operates from main branch.
  • Applied /usr/bin/env hunk to the script (was treewide). ]

Related: #25205

$ for pr in 25205; do contrib/backporting/set-labels.py $pr done v1.13; done

[ upstream commit d90803c ]
[ upstream commit 2c367de ]
[ Backporter's notes:
  - Removed github workflow. Workflow operates from main branch.
  - Applied /usr/bin/env hunk to the script (was treewide). ]

Signed-off-by: Chance Zibolski <chance.zibolski@gmail.com>
Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested a review from a team as a code owner June 9, 2023 17:21
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Jun 9, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.13 kind/backports This PR provides functionality previously merged into master. labels Jun 9, 2023
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.

let's get #26086 merged first. otherwise the push chart workflow will start failing on v1.13 branch.

edit: oh it's the other way around. this one needs to go first so that the script exists 👀

@michi-covalent
Copy link
Contributor

✅ confirmed the chart version here: https://github.com/cilium/cilium/actions/runs/5224577019 (1.13.3-dev.35-pr-joe-version-script-v1-13-e76d38d1d6)

https://quay.io/repository/cilium-charts-dev/cilium?tab=tags

no need for the full CI run. let's just wait for the basic CI to finish before merging.

@michi-covalent michi-covalent merged commit 843403d into v1.13 Jun 9, 2023
@michi-covalent michi-covalent deleted the pr/joe/version-script-v1.13 branch June 9, 2023 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
Status: Released
Development

Successfully merging this pull request may close these issues.

3 participants