Skip to content

Conversation

rosemarymarano
Copy link
Contributor

Add x-ref from the Quick Start guide to the create-domain script details as per suggestion from @tbarnes-us.

@tbarnes-us tbarnes-us requested a review from mriccell May 14, 2021 15:55
Copy link
Member

@ddsharpe ddsharpe left a comment

Choose a reason for hiding this comment

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

This is a simple change. If we need a bigger change, file a Jira. This change accomplished what it set out to do. LGTM

1. Create a new image with a domain home, plus create a domain resource that the operator will use to deploy the image, by running the [create-domain](http://github.com/oracle/weblogic-kubernetes-operator/blob/main/kubernetes/samples/scripts/create-weblogic-domain/domain-home-in-image/create-domain.sh) script.

The script's behavior is controlled by an inputs file plus command-line options. The script downloads the [WebLogic Image Tool](https://oracle.github.io/weblogic-image-tool/) and [WebLogic Deploy Tool](https://oracle.github.io/weblogic-deploy-tooling/), and uses these tools to create a new image with a domain home, a domain resource YAML file that references the image, and, if the `-e` option is specified, deploy the domain resource to Kubernetes. For a detailed understanding of the steps that the `create-domain.sh` script performs for you, see the bulleted items under [Use the script to create a domain]({{< relref "/samples/simple/domains/domain-home-in-image/#use-the-script-to-create-a-domain" >}})

This comment was marked as resolved.

@rosemarymarano
Copy link
Contributor Author

Yep, good idea.

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.

Looks good! Thanks!

@rjeberhard rjeberhard merged commit e9accfc into main May 18, 2021
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