Skip to content

Conversation

aditighag
Copy link
Member

@aditighag aditighag commented Feb 25, 2021

Commit 02320e added support for using backporting scripts with Cilium forks. Document the
additional steps that are required with this new workflow.

Additionally, workflow scope might be needed with the GitHub token in
some cases that update the corresponding yamls -

(refusing to allow a Personal Access Token to create or update workflow `.github/workflows/go-check.yaml` without `workflow` scope

Commit 02320e added support for working with Cilium forks. Document the
additional steps that are required with this new workflow.

Additionally, workflow scope might be needed with the GitHub token in
some cases that update the corresponding yamls -

``(refusing to allow a Personal Access Token to create or update workflow `.github/workflows/go-check.yaml` without `workflow` scope``

Signed-off-by: Aditi Ghag <aditi@cilium.io>
@aditighag aditighag requested review from a team as code owners February 25, 2021 21:53
@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 Feb 25, 2021
@aditighag aditighag requested review from nebril and qmonnet February 25, 2021 21:53
@aditighag aditighag added the release-note/misc This PR makes changes that have no direct user impact. label Feb 25, 2021
@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 Feb 25, 2021
@aditighag aditighag requested a review from a team February 25, 2021 21:53
Copy link
Member

@christarazi christarazi left a comment

Choose a reason for hiding this comment

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

🚀

@aditighag aditighag added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 25, 2021
@qmonnet
Copy link
Member

qmonnet commented Apr 14, 2021

Due to a mistake when updating the command for changing labels in a backport PR, this PR has NOT been backported to v1.7. Marking for backports again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. 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.

9 participants