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

Add workflows for stable branches #16944

Merged
merged 2 commits into from Jul 21, 2021
Merged

Add workflows for stable branches #16944

merged 2 commits into from Jul 21, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented Jul 20, 2021

Since issue_comment is only triggered in GH workflows of the default
branch, we need to add a copy of the conformance GH workflows in the
default branch as well. This will allow the conformance tests to be
tested against stable branches, which right now only 1.10 has support
for this.

aanm added 2 commits July 20, 2021 19:13
This helps developers to know which trigger they can use to retest the
l4lb GH workflow.

Signed-off-by: André Martins <andre@cilium.io>
Since `issue_comment` is only triggered in GH workflows of the default
branch, we need to add a copy of the conformance GH workflows in the
default branch as well. This will allow the conformance tests to be
tested against stable branches, which right now only 1.10 has support
for this.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added the release-note/ci This PR makes changes to the CI. label Jul 20, 2021
@aanm aanm requested review from a team as code owners July 20, 2021 17:23
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Issue template makes sense. LMK if you want me to take a closer look at any workflows.

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.

I'm very reluctant to go forward with having to maintain multiple versions of the same workflow but I don't have a better solution at the moment.

rm eksctl_$(uname -s)_amd64.tar.gz

- name: Set up AWS CLI credentials
uses: aws-actions/configure-aws-credentials@13d241b293754004c80624b5567555c4a39ffbe3
Copy link
Member

Choose a reason for hiding this comment

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

Nit: #16942

Suggested change
uses: aws-actions/configure-aws-credentials@13d241b293754004c80624b5567555c4a39ffbe3
uses: aws-actions/configure-aws-credentials@0d9a5be0dceea74e09396820e1e522ba4a110d2f

Copy link
Member Author

@aanm aanm Jul 21, 2021

Choose a reason for hiding this comment

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

Let's wait for the dependabot to do this automatically

rm eksctl_$(uname -s)_amd64.tar.gz

- name: Set up AWS CLI credentials
uses: aws-actions/configure-aws-credentials@13d241b293754004c80624b5567555c4a39ffbe3
Copy link
Member

Choose a reason for hiding this comment

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

Nit: #16942

Suggested change
uses: aws-actions/configure-aws-credentials@13d241b293754004c80624b5567555c4a39ffbe3
uses: aws-actions/configure-aws-credentials@0d9a5be0dceea74e09396820e1e522ba4a110d2f

@aanm aanm merged commit 54afa8c into master Jul 21, 2021
@aanm aanm deleted the pr/add-stable-workflows branch July 21, 2021 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants