-
Notifications
You must be signed in to change notification settings - Fork 131
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
Add new common version file for projects that have release streams ou… #1150
Conversation
c518da6
to
5c679c8
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.
Looking good, a few things.
A change is needed in the main.go
in the root of the repo around Line 109 in the printImages code, the CommonComponents needs to be added like the Calico and Enterprise Components loops.
0ab7c3a
to
b7661c0
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.
Looks good, one final thing can you add to the RELEASING.md doc about updating calico/common_versions.yaml
where we talk about the calico and enterprise ones.
b7661c0
to
944a37a
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.
LGTM
0c7ceb1
to
8278a06
Compare
…tside of OS and EE
8278a06
to
427d788
Compare
…tside of OS and EE
Description
For PR author
make gen-files
make gen-versions
For PR reviewers
A note for code reviewers - all pull requests must have the following:
kind/bug
if this is a bugfix.kind/enhancement
if this is a a new feature.enterprise
if this PR applies to Calico Enterprise only.