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

sysdump: remove support for standalone hubble #514

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bmcustodio
Copy link
Contributor

Standalone Hubble is pre-1.8, and hence not supported anymore. This commit removes support for dumping the daemonset + logs + etc..

@bmcustodio bmcustodio requested a review from a team as a code owner September 2, 2021 16:12
@bmcustodio bmcustodio temporarily deployed to ci September 2, 2021 16:12 Inactive
@bmcustodio bmcustodio force-pushed the pr/bruno/sysdump-no-standalone-hubble branch from 949cdb0 to 0fff256 Compare September 2, 2021 16:26
@bmcustodio bmcustodio temporarily deployed to ci September 2, 2021 16:26 Inactive
@bmcustodio bmcustodio force-pushed the pr/bruno/sysdump-no-standalone-hubble branch from 0fff256 to 9875337 Compare September 2, 2021 16:46
@bmcustodio bmcustodio temporarily deployed to ci September 2, 2021 16:46 Inactive
@tklauser tklauser added the needs-rebase This PR needs to be rebased because it has merge conflicts. label Sep 6, 2021
@tklauser
Copy link
Member

tklauser commented Sep 6, 2021

@bmcustodio looks like this PR needs a rebase after #513 was merged.

Standalone Hubble is pre-1.8, and hence not supported anymore. This
commit removes support for dumping the daemonset + logs + etc..

Signed-off-by: Bruno Miguel Custódio <brunomcustodio@gmail.com>
@tklauser tklauser force-pushed the pr/bruno/sysdump-no-standalone-hubble branch from 9875337 to bdba5f8 Compare September 9, 2021 13:26
@tklauser tklauser removed the needs-rebase This PR needs to be rebased because it has merge conflicts. label Sep 9, 2021
@tklauser tklauser temporarily deployed to ci September 9, 2021 13:26 Inactive
@tklauser
Copy link
Member

tklauser commented Sep 9, 2021

Rebased and force pushed because I want to merge this before opening another PR for sysdump, i.e. to avoid merge conflicts on the new PR as well.

@tklauser
Copy link
Member

tklauser commented Sep 9, 2021

After quickly syncing with @michi-covalent out-of-band, it seems we still have some 1.8 users of standalone Hubble. For them it would be useful to still be able to collect standalone hubble info in their sysdump, so l'll postpone merging this PR for a bit to give them a chance to migrate. Thus moving this PR to draft for now.

@tklauser tklauser marked this pull request as draft September 9, 2021 14:58
@tklauser tklauser added the dont-merge/blocked Another PR must be merged before this one. label Sep 9, 2021
@tklauser tklauser added the needs-rebase This PR needs to be rebased because it has merge conflicts. label Jul 6, 2022
@michi-covalent michi-covalent removed their request for review April 8, 2023 02:42
@michi-covalent michi-covalent removed their assignment Apr 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge/blocked Another PR must be merged before this one. needs-rebase This PR needs to be rebased because it has merge conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants