Skip to content

Modified DomainInImage Upgrade usecase #2534

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
Sep 15, 2021
Merged

Modified DomainInImage Upgrade usecase #2534

merged 6 commits into from
Sep 15, 2021

Conversation

anpanigr
Copy link
Member

@anpanigr anpanigr commented Sep 13, 2021

Modified the DomainInImage usecases to
a) use k8s API to create domain resource, instead of hard-coded domain.yaml file
b) removed the commented Operator Release upgrade such as 2.5.0/2.6.0

Jenkin Upgrade Test

https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/6423/ ( Operator WLS upgrade)
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/6424 ( All upgrade )

ANTARYAMI.PANIGRAHI@ORACLE.COM added 2 commits September 10, 2021 19:55
Copy link
Member

@vanajamukkara vanajamukkara left a comment

Choose a reason for hiding this comment

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

LG

Copy link
Member

@sankarpn sankarpn left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@maggiehe00 maggiehe00 left a comment

Choose a reason for hiding this comment

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

LGTM

@rjeberhard rjeberhard merged commit e74938f into main Sep 15, 2021
@tbarnes-us
Copy link

Just verifying: Note that this change did not remove the domain.yaml file.

@anpanigr
Copy link
Member Author

Just verifying: Note that this change did not remove the domain.yaml file.

The file has been removed an earlier #2526
Due to this reason the DomainInImage upgrade tests are failing, since it was using the hard-coded domain.yaml file

@rjeberhard rjeberhard deleted the upgrade-fix branch January 31, 2022 14:15
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.

6 participants