Skip to content

Conversation

nebril
Copy link
Member

@nebril nebril commented Sep 24, 2021

Let's merge this after cli 0.9.1 is released

@nebril nebril requested review from a team as code owners September 24, 2021 12:23
@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 Sep 24, 2021
@nebril nebril added the release-note/ci This PR makes changes to the CI. label Sep 24, 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 Sep 24, 2021
@tklauser
Copy link
Member

FYI, now that the release is actually there I've re-triggered the kind workflow which was previously failing due to the release not existing yet.

@pchaigno
Copy link
Member

We need a test commit to run the workflows with this version, right?

@nbusseneau
Copy link
Member

nbusseneau commented Sep 27, 2021

In the test PR we did get a DNS failure still, in https://github.com/cilium/cilium/actions/runs/1277574393 during the second connectivity test (with encryption enabled) :/

@pchaigno
Copy link
Member

DNS is always the first thing to fail when there are connectivity issues, so that's not really a distinguishing feature ;-)

I don't see any to-proxy traces in tests without proxy redirects, so I don't think those failures are the same issue.

Note the to-proxy failure may still happen in connectivity checks after IPsec (though it doesn't appear to be the case here) because, even though we reordered tests in the checks, the first few tests after enabling IPsec may still conflict with the last few tests before enabling IPsec. A fix for that could be to flush the conntrack map in between the two connectivity check runs in our workflows. Nevertheless, I'd rather wait to see how prevalent that issue is before sending such a fix.

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.

:shipit:

@nbusseneau nbusseneau added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 27, 2021
This change updates cilium-cli to 0.9.1 in github action workflows
files.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril force-pushed the pr/nebril/update-cli-9.1 branch from 5b1dd77 to 66732e1 Compare September 27, 2021 10:26
@nebril
Copy link
Member Author

nebril commented Sep 27, 2021

repushed a commit with proper formatting, bpf job was complaining about lack of commit description

@kaworu
Copy link
Member

kaworu commented Sep 27, 2021

Should this PR be marked with needs-backport/v1.10? See this PR that is going to be backported (backport PR).

@nbusseneau
Copy link
Member

Should this PR be marked with needs-backport/v1.10? See this PR that is going to be backported (backport PR).

Yes, it should be backported so that the Kind workflow uses the right CLI version.

@jibi jibi merged commit acf3431 into cilium:master Sep 27, 2021
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/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants