-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
build, ci: extend API checks to include Hubble API #14091
Conversation
ca6ed53
to
230db2f
Compare
Similar to the existing infrastructure to detect OpenAPI based changes, detect whether any of the Hubble API file need to be regenerated. This change also includes regeneration of `api/v1/flow/flow.pb.go` which was previously missed and the GitHub action would otherwise fail. Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
230db2f
to
52904a6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks!
The newly added target is used by the "Go-related checks" GitHub action which passed. Full CI run isn't necessary, marking as ready to merge. Also marked for backport to 1.8 and 1.9 as it might help to catch missing API re-generation in backport PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Similar to the existing infrastructure to detect OpenAPI based changes,
detect whether any of the Hubble API file need to be regenerated.
This change also includes regeneration of
api/v1/flow/flow.pb.go
whichwas previously missed and the GitHub action would otherwise fail.