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

cmd/observe: move flows specific code to flows*.go files #578

Merged
merged 1 commit into from
Jun 25, 2021

Conversation

tklauser
Copy link
Member

Second round of refactoring for #543.

Follow-up for commit 700d6bc ("cmd/observe: rename flow-related
functions and types"), separated to ease review.

This separates the generic observe functionality from the functionality
specific to flows. This follows the file naming scheme used for agent
and debug events.

No functional changes. This is preparatory refactoring for the
introduction of the hubble observe flows subcommand.

Follow-up for commit 700d6bc ("cmd/observe: rename flow-related
functions and types"), separated to ease review.

This separates the generic observe functionality from the functionality
specific to flows. This follows the file naming scheme used for agent
and debug events.

No functional changes. This is preparatory refactoring for the
introduction of the `hubble observe flows` subcommand.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser added the area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. label Jun 25, 2021
@tklauser tklauser requested a review from a team June 25, 2021 10:21
@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 Jun 25, 2021
@tklauser tklauser requested review from glibsm and removed request for a team June 25, 2021 10:21
@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Jun 25, 2021
@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 Jun 25, 2021
@tklauser tklauser merged commit d8f172d into master Jun 25, 2021
@tklauser tklauser deleted the pr/tklauser/cmd-observe-flows-file-rename branch June 25, 2021 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. 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