Skip to content

Conversation

glibsm
Copy link
Member

@glibsm glibsm commented Aug 11, 2025

No description provided.

@glibsm glibsm requested a review from a team as a code owner August 11, 2025 17:10
@glibsm glibsm requested review from glrf and removed request for a team August 11, 2025 17:10
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label PR is blocked until the release note is set label Aug 11, 2025
@glibsm glibsm added the release-note/misc This PR makes changes that have no direct user impact. label Aug 11, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label PR is blocked until the release note is set label Aug 11, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 12, 2025
@@ -72,7 +72,7 @@ this reason, only the latest Hubble CLI version is maintained.

| Version | Release Date | Maintained | Supported Cilium Version | Artifacts |
|------------------------------------------------------|----------------------|------------|--------------------------|-------------------------------------------------------------------------|
| [v1.17](https://github.com/cilium/hubble/tree/main) | 2025-06-23 (v1.17.5) | Yes | Cilium 1.17 and older | [GitHub Release](https://github.com/cilium/hubble/releases/tag/v1.17.5) |
| [v1.18](https://github.com/cilium/hubble/tree/main) | 2025-08-11 (v1.18.0) | Yes | Cilium 1.17 and older | [GitHub Release](https://github.com/cilium/hubble/releases/tag/v1.18.0) |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| [v1.18](https://github.com/cilium/hubble/tree/main) | 2025-08-11 (v1.18.0) | Yes | Cilium 1.17 and older | [GitHub Release](https://github.com/cilium/hubble/releases/tag/v1.18.0) |
| [v1.18](https://github.com/cilium/hubble/tree/main) | 2025-08-11 (v1.18.0) | Yes | Cilium 1.18 and older | [GitHub Release](https://github.com/cilium/hubble/releases/tag/v1.18.0) |

Copy link
Member Author

Choose a reason for hiding this comment

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

We support more than just 1.18 cilium, so this confuses me. I thought that -1 version would make more sense here no?

Copy link
Member

Choose a reason for hiding this comment

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

Mmh, I read it as "Cilium v1.18 is unsupported". Why is "Supported Cilium version - Cilium 1.18 and older" confusing?

@maintainer-s-little-helper maintainer-s-little-helper bot removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

3 participants