-
Notifications
You must be signed in to change notification settings - Fork 217
Operator version 3.0.2 #1912
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
Operator version 3.0.2 #1912
Conversation
…ator Helm chart and related doc change
…tor .out, and document log rotation for WLS .out and .log.
…Exceeded (which happens when intro job times out due to DeadlineExceeded).
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.
From the doc perspective, LGTM.
What is the process of adding bug fixes? It doesn't look as though you are cherry-picking commits merged into develop, as you've been squashing the PRs to single commits, and here you seem to be using the original commits on the branch. How will this affect merging future work from develop into master? |
I cherry-picked the commits from the original PR's. |
After we merge to master, I'll merge master back into develop to make sure that there are no conflicts (there shouldn't be). |
What is the advantage of doing that vs. cherry-picking the squashed commits? |
| --- | --- | --- | --- | | ||
| September 15, 2020 | v3.0.2 | no | This release contains several fixes, including improvements to log rotation and a fix that avoids unnecessarily updating the domain status. | |
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.
Can we list the fixes that went into 3.0.2 for user reference?
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.
One edit request/comment.
@@ -4,7 +4,7 @@ date: 2019-03-15T11:25:28-04:00 | |||
draft: false | |||
--- | |||
|
|||
### Recent changes | |||
### Releases | |||
|
|||
| Date | Version | Introduces backward incompatibilities? | Changes | |
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.
Changes -> Change (I let this go before but it was a mistake. It should be singular, Change, just like all the other headings are singular, Date, Version, Issue, and Description. The question mark after Introduces backward incompatibilities? is ok.) I looked at the Change log in the Hugo UI and it all looks fine, definitely good to go.
We are planning to release another set of bug fixes.
The following PR's were selected based on customer impact and severity:
Additionally, the following PR's were required because one or more of the PR's in the above list depended on these changes:
Based on a review request, additionally added