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

images: Bump Hubble CLI to v0.9.0 #18077

Merged

Conversation

gandro
Copy link
Member

@gandro gandro commented Dec 1, 2021

This bumps the Hubble CLI to the recently released version 0.9.0. Hubble
CLI v0.9.0 has been released to include the Hubble protobuf API changes
present in Cilium v1.11-rc3 and thus is intended to be bundled with the
final Cilium v1.11 release.

Due to the cyclic dependency between Hubble CLI and Cilium, this PR can
only be done after the feature freeze and thus needs to be backported to v1.11.

This bumps the Hubble CLI to the recently released version 0.9.0. Hubble
CLI v0.9.0 has been released to include the Hubble protobuf API changes
present in Cilium v1.11-rc3 and thus is intended to be bundled with the
final Cilium v1.11 release.

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
@gandro gandro added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. sig/hubble Impacts hubble server or relay needs-backport/1.11 labels Dec 1, 2021
@gandro gandro requested review from a team as code owners December 1, 2021 09:59
@gandro gandro requested review from glibsm and qmonnet December 1, 2021 09:59
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Dec 1, 2021
@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/test

Job 'Cilium-PR-K8s-1.23-kernel-4.9' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDemosTest Tests Star Wars Demo

Failure Output

FAIL: failed to ensure kubectl version: failed to run kubectl version

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.23-kernel-4.9 so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.22-kernel-4.19' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks service across nodes Tests NodePort BPF Tests with secondary NodePort device

Failure Output

FAIL: Request from k8s1 to service http://[fd04::11]:31135 failed

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.22-kernel-4.19 so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.16-net-next' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig MonitorAggregation Checks that monitor aggregation restricts notifications

Failure Output

FAIL: failed to ensure kubectl version: failed to download kubectl

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.16-net-next so I can create a new GitHub issue to track it.

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

Test triage (will edit this post as more tests finish)

Test [no-policies]:
  ❌ no-policies/pod-to-pod/curl-1: cilium-test/client2-5998d566b4-88f7s (10.12.0.111) -> cilium-test/echo-other-node-f4d46f75b-2bzqj (10.20.1.83:8080)
  ❌ no-policies/pod-to-pod/curl-3: cilium-test/client-6488dcf5d4-ngwcz (10.12.0.40) -> cilium-test/echo-other-node-f4d46f75b-2bzqj (10.20.1.83:8080)
Test [allow-all]:
  ❌ allow-all/pod-to-service/curl-0: cilium-test/client-6488dcf5d4-ngwcz (10.12.0.40) -> cilium-test/echo-other-node (echo-other-node:8080)
  ❌ allow-all/pod-to-service/curl-2: cilium-test/client2-5998d566b4-88f7s (10.12.0.111) -> cilium-test/echo-other-node (echo-other-node:8080)
Connectivity test failed: 2 tests failed
  ❌ allow-all/pod-to-pod/curl-1: cilium-test/client-6488dcf5d4-ngwcz (10.12.0.40) -> cilium-test/echo-other-node-f4d46f75b-2bzqj (10.20.1.83:8080)
  ❌ allow-all/pod-to-pod/curl-3: cilium-test/client2-5998d566b4-88f7s (10.12.0.111) -> cilium-test/echo-other-node-f4d46f75b-2bzqj (10.20.1.83:8080)

@gandro gandro removed the request for review from glibsm December 1, 2021 12:19
@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/ci-aks

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/ci-multicluster

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/test-1.22-4.19

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/test-1.23-4.9

Job 'Cilium-PR-K8s-1.23-kernel-4.9' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sCustomCalls Basic test with byte-counter

Failure Output

FAIL: failed to ensure kubectl version: failed to download kubectl

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.23-kernel-4.9 so I can create a new GitHub issue to track it.

Edit: Another kubectl flake. https://jenkins.cilium.io/job/Cilium-PR-K8s-1.23-kernel-4.9/35/

@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/test-1.23-4.9

@gandro gandro added the release-priority/best-effort The project for target version is not a hard requirement. label Dec 1, 2021
@gandro
Copy link
Member Author

gandro commented Dec 1, 2021

/test-1.16-netnext

Job 'Cilium-PR-K8s-1.16-net-next' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig MonitorAggregation Checks that monitor aggregation restricts notifications

Failure Output

FAIL: failed to ensure kubectl version: failed to download kubectl

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.16-net-next so I can create a new GitHub issue to track it.

https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-net-next/2104/

@gandro
Copy link
Member Author

gandro commented Dec 2, 2021

/test-1.16-netnext

Job 'Cilium-PR-K8s-1.16-net-next' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sKafkaPolicyTest Kafka Policy Tests KafkaPolicies

Failure Output

FAIL: failed to ensure kubectl version: failed to download kubectl

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.16-net-next so I can create a new GitHub issue to track it.

@gandro
Copy link
Member Author

gandro commented Dec 2, 2021

Previous netnext flakes all are #18060 (which has been fixed on master just now)

Therefore marking this ready to merge so there still is a chance this makes it into v1.11.0.

@gandro gandro added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 2, 2021
@nathanjsweet nathanjsweet merged commit 6bd3833 into cilium:master Dec 3, 2021
@joestringer joestringer added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Dec 5, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.11 in 1.11.0 Dec 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. release-priority/best-effort The project for target version is not a hard requirement. sig/hubble Impacts hubble server or relay
Projects
No open projects
1.11.0
Backport done to v1.11
Development

Successfully merging this pull request may close these issues.

None yet

6 participants