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

Makefile: Reduce the amount of vendoring calls #2670

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

lambdanis
Copy link
Contributor

When generating Go code for proto API or k8s CRDs, we don't need to run vendoring for all modules before and after. It should be enough if we run vendoring only for the relevant module (api or pkg/k8s) before, and only for the main module after.

@lambdanis lambdanis added the release-note/misc This PR makes changes that have no direct user impact. label Jul 12, 2024
@lambdanis lambdanis force-pushed the pr/lambdanis/less-vendor branch from c4f32bd to 5633f7e Compare July 12, 2024 19:01
When generating Go code for proto API or k8s CRDs, we don't need to run
vendoring for all modules before and after. It should be enough if we run
vendoring only for the relevant module (api or pkg/k8s) before, and only for
the main module after.

Signed-off-by: Anna Kapuscinska <anna@isovalent.com>
@lambdanis lambdanis force-pushed the pr/lambdanis/less-vendor branch from 5633f7e to ccfd9ea Compare July 12, 2024 19:04
@lambdanis lambdanis marked this pull request as ready for review July 12, 2024 22:18
@lambdanis lambdanis requested a review from a team as a code owner July 12, 2024 22:18
@lambdanis lambdanis requested a review from olsajiri July 12, 2024 22:18
Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

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

Nice, if this is enough, it will make the generation steps more lightweight.

@lambdanis lambdanis merged commit 5482c2d into cilium:main Jul 15, 2024
39 checks passed
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.

2 participants