Skip to content

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

Merged
merged 5 commits into from
Feb 17, 2023
Merged

Prepare for WKO 3.4.6 #3993

merged 5 commits into from
Feb 17, 2023

Conversation

rjeberhard
Copy link
Member

No description provided.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 16, 2023
Copy link
Member

@rosemarymarano rosemarymarano left a 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
Copy link
Member

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`.
Copy link
Member

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`.
Copy link
Member

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`.
Copy link
Member

Choose a reason for hiding this comment

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

-> Provided...

Copy link
Member

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.
Copy link
Member

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`.
Copy link
Member

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.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@rosemarymarano rosemarymarano left a 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`.
Copy link
Member

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. |
Copy link
Member

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`.
Copy link
Member

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".

Copy link
Member

@ankedia ankedia left a 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.

@rjeberhard rjeberhard merged commit df769f5 into release/3.4 Feb 17, 2023
@rjeberhard rjeberhard deleted the prepare-346 branch February 17, 2023 17:49
robertpatrick pushed a commit that referenced this pull request Apr 12, 2023
* Prepare for WKO 3.4.6
rjeberhard added a commit to rjeberhard/weblogic-kubernetes-operator that referenced this pull request Apr 14, 2023
* Prepare for WKO 3.4.6
rjeberhard added a commit to rjeberhard/weblogic-kubernetes-operator that referenced this pull request Apr 14, 2023
* Prepare for WKO 3.4.6
rjeberhard added a commit that referenced this pull request Apr 19, 2023
* Prepare for WKO 3.4.6
rjeberhard added a commit that referenced this pull request Apr 19, 2023
* Prepare for WKO 3.4.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants