Skip to content

Conversation

@marinakog
Copy link
Contributor

Create a file on the source image that cannot be accessed by the oracle user, check error in introspector
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/5711/

@marinakog marinakog requested a review from anpanigr July 19, 2021 23:28
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.

LGTM

@rjeberhard rjeberhard merged commit a732a21 into main Jul 20, 2021
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.

few comments

* Negative Test to create domain with file , created by user tester with permission read only
* and not accessible by oracle user in auxiliary image
* via provided Dockerfile.
* Check the error msg is in introspector pod log, domain events and operator pod log.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Check the error msg is in introspector pod log, domain events and operator pod log.
* Check the error message is in introspector pod log, domain events and operator pod log.


// create stage dir for auxiliary image
Path errorpathAIPath1 = Paths.get(RESULTS_ROOT, "errorpathauxiimage4");
assertDoesNotThrow(() -> FileUtils.deleteDirectory(errorpathAIPath1.toFile()));
Copy link
Member

Choose a reason for hiding this comment

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

Please assert failure messages in all asserts below.

// push image to repo for multi node cluster
if (!DOMAIN_IMAGES_REPO.isEmpty()) {
logger.info("docker push image {0} to registry {1}", errorPathAuxiliaryImage1, DOMAIN_IMAGES_REPO);
assertTrue(dockerPush(errorPathAuxiliaryImage1),
Copy link
Member

Choose a reason for hiding this comment

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

I guess you need to use dockerLoginAndPushImageToRegistry() method

@rjeberhard rjeberhard deleted the cmerror branch January 31, 2022 14:19
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