Skip to content
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

Owls 87209 - Domain lifecycle scripts fix in cases where domain/cluster specs are overridden at the individual managed server level #2222

Merged
merged 4 commits into from
Feb 24, 2021

Conversation

ankedia
Copy link
Member

@ankedia ankedia commented Feb 24, 2021

Owls 87209 - Domain lifecycle scripts fix in cases where domain/cluster specs are overridden at the individual managed server level. The script doesn't work correctly when .spec.managedServers section has pre-existing details for a particular managed server. For e.g Consider the below stanza where the serverStartState attribute is overridden for soa_server1.
spec:
managedServers:

  • serverName: soa_server1
    serverStartState: RUNNING

This change adds the server start policy element for the managed server if the policy doesn't exist or replaces the policy if the policy already exists. Similarly, it unsets the policy and/or deletes the managed server element if managed server spec only has serverStartPolicy under it.

@tbarnes-us
Copy link

General comment: IIRC, there are integration tests for these scripts. If so, please file a JIRA to enhance them to cover the new use case...

Copy link
Member

@anpanigr anpanigr left a comment

Choose a reason for hiding this comment

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

Can you enhance integration test (ItServerStartPolicy.java) to add this usecase.

@ankedia
Copy link
Member Author

ankedia commented Feb 24, 2021

General comment: IIRC, there are integration tests for these scripts. If so, please file a JIRA to enhance them to cover the new use case...

Thanks. I have made the suggested changes and also created JIRA OWLS-87819 to track the enhancement of the integration tests to cover the new use case.

@ankedia
Copy link
Member Author

ankedia commented Feb 24, 2021

Can you enhance integration test (ItServerStartPolicy.java) to add this usecase.
I have created JIRA OWLS-87819 to track the integration test enhancement and it can be done in a separate PR. Do we have any other integration tests where domain/cluster specs are overridden at the managed server level?

@tbarnes-us
Copy link

tbarnes-us commented Feb 24, 2021

.. Can you enhance integration test (ItServerStartPolicy.java) to add this usecase.
. I have created JIRA OWLS-87819 to track the integration test enhancement and it can be done in a separate PR.
Thanks.
. Do we have any other integration tests where domain/cluster specs are overridden at the managed server level?
I don't know, Pani might.

@rjeberhard rjeberhard merged commit 30a51ce into develop Feb 24, 2021
@ankedia ankedia deleted the owls_87209_fix branch March 3, 2021 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants