Skip to content

Recreate OWLS-86115 work #2344

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 6 commits into from
Apr 29, 2021
Merged

Recreate OWLS-86115 work #2344

merged 6 commits into from
Apr 29, 2021

Conversation

rjeberhard
Copy link
Member

@rjeberhard rjeberhard commented Apr 27, 2021

I'm not sure what happened with GitHub, but when I tried to change the base for @lennyphan's work GitHub created a PR with hundreds of changes. This PR is recreated with just his deltas, but targeted to the main branch. (Original pull: #2285).

In this PR:

  • For Domain home source type MII, configuration overrides are now generated using WDT model filter. The model filter implementation follows the same logic for generating situational configuration in introspectDomain.py's SitConfigGenerator. Domain home source types Domain-on-PV and Domain-in-Image will continue to use situational configuration overrides.

  • Added ability to run python based unit tests for model filter implementation.

Copy link

@tbarnes-us tbarnes-us left a comment

Choose a reason for hiding this comment

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

Way cool!

@tbarnes-us
Copy link

General comment: I think this change will likely lift a restriction or two that we document in the user guide 'model updates' chapter. We (@jshum2479 @lennyphan and I) should meet to discuss updating the chapter, and whether we should test first.

@lennyphan
Copy link
Member

Successful run of all integration tests on this branch: https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/4895/

@rjeberhard rjeberhard merged commit 88ee5ce into main Apr 29, 2021
@rjeberhard rjeberhard deleted the OWLS-86115-recreated branch January 31, 2022 14:22
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.

3 participants