-
Notifications
You must be signed in to change notification settings - Fork 218
Prepare for WKO 3.4.6 #3993
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 WKO 3.4.6 #3993
Conversation
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.
Minor edits
@@ -57,6 +58,15 @@ draft: false | |||
|
|||
### Change log | |||
|
|||
#### Operator 3.4.6 | |||
|
|||
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image |
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.
Period at the end of the sentence.
#### Operator 3.4.6 | ||
|
||
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image | ||
* Support or provide clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. |
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.
We generally use past tense in these statements, so:
- Added support or provided...
|
||
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image | ||
* Support or provide clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. | ||
* Improve support for dynamic clusters that do not specify `ServerTemplate` or `DynamicClusterSize`. |
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.
-> Improved support...
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image | ||
* Support or provide clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. | ||
* Improve support for dynamic clusters that do not specify `ServerTemplate` or `DynamicClusterSize`. | ||
* Provide clearer validation errors for configured clusters that have no member servers defined under `Server`. |
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.
-> Provided...
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.
Minor: The validation error pertains to the dynamic cluster that have no ServerTemplate defined under DynamicServers
.
* Support or provide clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. | ||
* Improve support for dynamic clusters that do not specify `ServerTemplate` or `DynamicClusterSize`. | ||
* Provide clearer validation errors for configured clusters that have no member servers defined under `Server`. | ||
* Resolved an issue related to an NPE generated while ensuring container port names are not longer than the Kubernetes enforced maximum of 15 characters. |
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.
Minor: The issue with container port names was causing StringIndexOutOfBoundsException instead of NPE.
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image | ||
* Support or provide clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. | ||
* Improve support for dynamic clusters that do not specify `ServerTemplate` or `DynamicClusterSize`. | ||
* Provide clearer validation errors for configured clusters that have no member servers defined under `Server`. |
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.
Minor: The validation error pertains to the dynamic cluster that have no ServerTemplate defined under DynamicServers
.
Kudos, SonarCloud Quality Gate passed!
|
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.
Just some wording suggestions
#### Operator 3.4.6 | ||
|
||
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image. | ||
* Added Support or provided clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. |
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.
-> Added support
@@ -8,6 +8,7 @@ draft: false | |||
|
|||
| Date | Version | Introduces backward incompatibilities? | Change - See also, [Change log](#change-log). | | |||
|--------------------|----------|---------------------------------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | |||
| February 17, 2023 | v3.4.6 | no | Resolved several issues related to WDT models for Model in Image, resolved an issue related to an exception while correcting container port names, and updates including the WebLogic Monitoring Exporter. | |
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.
-> updates including the WebLogic Monitoring Exporter
Every time I read this, I have a hard time discerning what you mean, until I read the Change Log. What I would suggest is:
-> updates including the WebLogic Monitoring Exporter version. OR
-> updates including the version of WebLogic Monitoring Exporter.
* Resolved several issues related to WebLogic Deploy Tooling (WDT) models for Model in Image. | ||
* Added Support or provided clearer validation errors for cases where `AdminServerName` is set or missing and where there is or is not a matching entry under `Server`. | ||
* Improved support for dynamic clusters that do not specify `ServerTemplate` or `DynamicClusterSize`. | ||
* Provided clearer validation errors for configured clusters that have no member servers defined under `DynamicServers`. |
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.
Minor - The clear error message is for dynamic cluster i.e. "Provided clearer validation errors for dynamic clusters".
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.
I just have a minor comment, otherwise LGTM.
* Prepare for WKO 3.4.6
* Prepare for WKO 3.4.6
* Prepare for WKO 3.4.6
No description provided.