-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
prepare for v1.25 as istio source is already branched #16261
Conversation
Signed-off-by: Faseela K <faseela.k@est.tech>
Signed-off-by: Faseela K <faseela.k@est.tech>
Signed-off-by: Faseela K <faseela.k@est.tech>
/retest |
@@ -77,7 +77,7 @@ baseurl := "$(URL)" | |||
endif | |||
|
|||
# Which branch of the Istio source code do we fetch stuff from | |||
export SOURCE_BRANCH_NAME ?= master | |||
export SOURCE_BRANCH_NAME ?= release-1.25 |
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.
Shouldn't we leave master
refs as-is? Typically we do, right?
Or is this a temporary chore we need to do, and then afterwards flip it back from release-1.25
to master
?
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.
Yes, I think we move this back later
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.
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.
Moves back after the branch cut of istio.io happens (day .0 is released)
@dhawton when should this PR be merged? |
Reviewers