8000 Refactor WLS on AKS to cover two domain home source type scenarios. by edburns · Pull Request #2215 · oracle/weblogic-kubernetes-operator · GitHub
[go: up one dir, main page]
Skip to content

Refactor WLS on AKS to cover two domain home source type scenarios. #2215

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Refactor WLS on AKS to cover two domain home source type scenarios. #2215

wants to merge 1 commit into from

Conversation

edburns
Copy link
Contributor
@edburns edburns commented Feb 22, 2021

  • Model in Image

  • Domain on PV

This refactoring uses hugo shortcodes, like so:

{{< readfile file="/samples/simple/azure-kubernetes-service/includes/prerequisites-01.txt" >}}

This shortcode is defined in docs-source/layouts/shortcodes/readfile.html, which is simply:

{{ $toRead := .Get "file" }}
{{ readFile $toRead | markdownify }}

By using hugo shortcodes, we can keep the text DRY, but maintain readability.

Before this change, the WLS on AKS arrangement looks like:

3. Samples

   Simple samples

      Credentials

      ...

      Azure Kubernetes Service

      Tanzu Kubernetes Service

After this change, the WLS on AKS arrangement looks like:

3. Samples

   Simple samples

      Credentials

      ...

      Azure Kubernetes Service

        Domain home on a PV

        Model in image

        Troubleshooting

      Tanzu Kubernetes Service

Naturally, the steps in the two domain home source types have significant overlap. Hugho shortcode readfile allows us to account for that overlap while maintaining DRY.

Running WLS on AKS via "Model in image"

On branch wls-on-aks-model-in-image
modified: docs-source/content/samples/simple/azure-kubernetes-service/_index.md

  • Use "Domain home source type" instead of "Domain home source scenario" for consistency with other parts of the documentation.

  • Update into to account for other values of Domain home source type.

  • Use "Domain in PV" instead of "Domain on a PV" for consistency with other parts of the docs.

modified: docs-source/content/userguide/managing-domains/_index.md

  • Add "Azure Container Registry".

modified: docs-source/content/userguide/managing-domains/domain-resource.md

  • Use the term domainUID consistently.

modified: docs-source/content/userguide/managing-domains/prepare.md

  • Briefly define the term domainUID in place.

modified: docs-source/content/userguide/overview/k8s-setup.md

modified: docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

  • Add automation step for domain-on-pv.

  • Validate service account

  • Complete PENDING activities.

  • Additional TOC entry

Remove steps to build image.

Use readfile with shortcode to allow content reuse

On branch wls-on-aks-model-in-image Completed refactoring to verify domain-on-pv is correct. Next step is to revisit model-in-image.

  • Moved info block "The following sections of the" to the top, to get it out of the include.

  • Section Prerequisites.

    • Add dependency on Docker for Desktop.

  • Section Oracle Container Registry.

    • Simplified steps.

    • 12.2.1.4 pull.

  • Moved section Clone WebLogic Server Kubernetes Operator repository lower, to get it out of the include.

  • Added correctness check at end of section on setting up AKS. Starts with "After your Kubernetes cluster is up and running".

  • See above about info block.

  • Section Generate configuration files has been renamed Create PV and PVC The table in this section has been greatly simplified.

  • Things start to diverge greatly when the upstream says "Now let’s
    ask the operator to create a WebLogic Server domain within the AKS
    cluster." and the fork says "Follow Domain home on a PV - Use the
    script to create a domain to create the WebLogic domain home within
    the AKS cluster."

    We get back on track with the text "You may observe error-related
    output during the creation of the domain."

  • In the fork, in the section that starts with the text "You must create
    LoadBalancer services" the YAML snippets for the admin and managed
    servers are adjacent, and the apply commands are in the same execute block.

  • Troubleshooting is a separate section.

modified: docs-source/content/samples/simple/azure-kubernetes-service/_index.md

  • Remove TOC entries to non-existent content.

  • Capitalize D in domain when appropriate.

  • Make it clear the two sub-sections are independent.

deleted: docs-source/content/samples/simple/azure-kubernetes-service/create-aks-cluster.md

  • Content absorbed into files included with readFile shortcode.

modified: docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

  • See above.

new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources.md
modified: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body.md
modified: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md

  • Extractions.

modified: docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

  • Start first validation work.

On branch wls-on-aks-model-in-image Complete refactoring.
modified: docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

  • Use 3.1.1 instead of 3.0.3.

  • Move prerequisites into include.

new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites.md

  • Move prerequisites into include.

modified: docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

modified: docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

  • Add JDK to prereqs.

  • Add "--admin-enabled".

  • Fix typos in table.

On branch wls-on-aks-model-in-image
modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/mii-initial.yaml
modified: kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/JRF/mii-initial-d1-JRF-v1.yaml
modified: kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/WLS/mii-initial-d1-WLS-v1.yaml

  • Correct admin/managed.

On branch wls-on-aks-model-in-image
modified: docs-source 10000 /content/samples/simple/azure-kubernetes-service/domain-on-pv.md

  • Move notice to correct place.

  • Make headings be same as in live docs.

new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.md
new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.md

  • Break up files to separate parts.

deleted: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body.md
modified: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md

  • Re-add rows removed from table.

modified: docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

  • Use two parts.

  • Remove unnecessary heading.

    modified: docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md
    new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.md
    new file: docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-02.md
    deleted: docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources.md
    modified: docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md
    modified: docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/azure-file-pv-template.yaml
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/azure-file-pvc-template.yaml
    renamed: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-aks-cluster-inputs.yaml -> kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks-inputs.yaml
    renamed: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-aks-cluster.sh -> kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks.sh
    deleted: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/admin-lb.yaml
    deleted: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/cluster-lb.yaml
    deleted: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/create-domain-inputs.yaml
    new file: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/loadbalancer-template.yaml
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/admin-lb.yaml
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/cluster-lb.yaml
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/helm-sa-cluster-admin-role.yaml
    modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/mii-initial.yaml

Fix the menu issue.
The live page will show included files in the menu.
Change the included files as txt can solve this problem.

Changes to be committed:
modified: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.txt
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-02.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-02.txt
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.txt
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.txt
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.txt
renamed: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites-01.txt
new file: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites-02.txt
modified: ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

  • Validate model-in-image

On branch wls-on-aks-model-in-image
modified: docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

  • Added note to skip to sample deployment.

  • Use 1. for numbered lists.

modified: kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks-inputs.yaml

  • Update weblogic version number in image to 12.2.1.4.

Add troubeshooting for WebLogic Kubernetes Operator ErrImagePull

Troubleshooting for VM size error

Update troubleshooting.md

Update domain-on-pv.md

@edburns
Copy link
Contributor Author
edburns commented Feb 22, 2021

The previous big WLS on AKS PR #1873, had the following individuals as reviewers. Perhaps the set of reviewers for this PR could be a subset of those individuals?

@tbarnes-us
Copy link

Just some general comments:
1 - The readFile short code is cool. +1
2 - Rosemary is not listed as a reviewer. She should probably be asked to make an edit pass on the new changes once review is complete.
3 - Is it practical for the include files to have a '.md' suffix instead of '.txt' suffix? Since they don't end in '.md', they will be easily missed by doc source searches that simply rely on the commonly expected extension.

@edburns
Copy link
Contributor Author
edburns commented Feb 22, 2021

@tbarnes-us Thanks for the prompt response. I am unable to set reviewers. This is why I have suggested a list of reviewers at this comment. Can you please choose the right reviewers based on my comment and your knowledge of who is appropriate?

3 - Is it practical for the include files to have a '.md' suffix instead of '.txt' suffix? Since they don't end in '.md', they will be easily missed by doc source searches that simply rely on the commonly expected extension.

We tried including the .md but we observed it disrupted the flow of the TOC. We observed changing it to .txt avoided this disruption.

rosemarymarano
rosemarymarano reviewed Feb 23, 2021
View reviewed changes
Copy link
Member
@rosemarymarano rosemarymarano left a comment

Choose a reason for hiding this comment

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

@edburns , I was able to get through only your first two docs, Azure Kubernetes Service and Domain home on a PV. I will come back to it later today or tomorrow and finish up the others.

rosemarymarano
rosemarymarano reviewed Feb 23, 2021
View reviewed changes
Copy link
Member
@rosemarymarano rosemarymarano left a comment

Choose a reason for hiding this comment

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

Fewer comments this time. Overall, very well written.

@galiacheng @edburns
Refactor WLS on AKS to cover two domain home source type scenarios.
9eda626 
* Model in Image

* Domain on PV

This refactoring uses hugo shortcodes, like so:

```
{{< readfile file="/samples/simple/azure-kubernetes-service/includes/prerequisites-01.txt" >}}
```

This shortcode is defined in `docs-source/layouts/shortcodes/readfile.html`, which is simply:

```
{{ $toRead := .Get "file" }}
{{ readFile $toRead | markdownify }}
```

By using hugo shortcodes, we can keep the text DRY, but maintain readability.

Before this change, the WLS on AKS arrangement looks like:

```
3. Samples

   Simple samples

      Credentials

      ...

      Azure Kubernetes Service

      Tanzu Kubernetes Service
```

After this change, the WLS on AKS arrangement looks like:

```
3. Samples

   Simple samples

      Credentials

      ...

      Azure Kubernetes Service

        Domain home on a PV

        Model in image

        Troubleshooting

      Tanzu Kubernetes Service
```

Naturally, the steps in the two domain home source types have significant overlap. Hugho shortcode readfile allows us to account for that overlap while maintaining DRY.

Running WLS on AKS via "Model in image"

On branch wls-on-aks-model-in-image
modified:   docs-source/content/samples/simple/azure-kubernetes-service/_index.md

- Use "Domain home source type" instead of "Domain home source scenario" for consistency with other parts of the documentation.

- Update into to account for other values of Domain home source type.

- Use "Domain in PV" instead of "Domain on a PV" for consistency with other parts of the docs.

modified:   docs-source/content/userguide/managing-domains/_index.md

- Add "Azure Container Registry".

modified:   docs-source/content/userguide/managing-domains/domain-resource.md

- Use the term `domainUID` consistently.

modified:   docs-source/content/userguide/managing-domains/prepare.md

- Briefly define the term `domainUID` in place.

modified:   docs-source/content/userguide/overview/k8s-setup.md

modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

- Add automation step for domain-on-pv.

- Validate service account

- Complete PENDING activities.

- Additional TOC entry

Remove steps to build image.

Use readfile with shortcode to allow content reuse

On branch wls-on-aks-model-in-image Completed refactoring to verify domain-on-pv is correct. Next step is to revisit model-in-image.

* Moved info block "The following sections of the" to the top, to get it out of the include.

* Section *Prerequisites*.

   * Add dependency on Docker for Desktop.

* Section *Oracle Container Registry*.

   * Simplified steps.

   * 12.2.1.4 pull.

* Moved section *Clone WebLogic Server Kubernetes Operator repository* lower, to get it out of the include.

* Added correctness check at end of section on setting up AKS.  Starts with "After your Kubernetes cluster is up and running".

* See above about info block.

* Section *Generate configuration files* has been renamed *Create PV and PVC*  The table in this section has been greatly simplified.

* Things start to diverge greatly when the upstream says "Now let’s
  ask the operator to create a WebLogic Server domain within the AKS
  cluster." and the fork says "Follow Domain home on a PV - Use the
  script to create a domain to create the WebLogic domain home within
  the AKS cluster."

  We get back on track with the text "You may observe error-related
  output during the creation of the domain."

* In the fork, in the section that starts with the text "You must create
  LoadBalancer services" the YAML snippets for the admin and managed
  servers are adjacent, and the apply commands are in the same execute block.

* Troubleshooting is a separate section.

modified:   docs-source/content/samples/simple/azure-kubernetes-service/_index.md

- Remove TOC entries to non-existent content.

- Capitalize D in domain when appropriate.

- Make it clear the two sub-sections are independent.

deleted:    docs-source/content/samples/simple/azure-kubernetes-service/create-aks-cluster.md

- Content absorbed into files included with `readFile` shortcode.

modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

- See above.

new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md

- Extractions.

modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

- Start first validation work.

On branch wls-on-aks-model-in-image Complete refactoring.
modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

- Use 3.1.1 instead of 3.0.3.

- Move prerequisites into include.

new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites.md

- Move prerequisites into include.

modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

- Move prerequisites into include.

- Remove OCR. It is in a file that is included.

- Move clone steps up.

- Apply lessons from Jianguo from Azure/AKS#1517 (comment)

modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

- Add JDK to prereqs.

- Add "--admin-enabled".

- Fix typos in table.

On branch wls-on-aks-model-in-image
modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/mii-initial.yaml
modified:   kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/JRF/mii-initial-d1-JRF-v1.yaml
modified:   kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/WLS/mii-initial-d1-WLS-v1.yaml

- Correct admin/managed.

On branch wls-on-aks-model-in-image
modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

- Move notice to correct place.

- Make headings be same as in live docs.

new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.md
new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.md

- Break up files to separate parts.

deleted:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md

- Re-add rows removed from table.

modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

- Use two parts.

- Remove unnecessary heading.

	modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md
	new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.md
	new file:   docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-02.md
	deleted:    docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources.md
	modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md
	modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/azure-file-pv-template.yaml
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/azure-file-pvc-template.yaml
	renamed:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-aks-cluster-inputs.yaml -> kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks-inputs.yaml
	renamed:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-aks-cluster.sh -> kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks.sh
	deleted:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/admin-lb.yaml
	deleted:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/cluster-lb.yaml
	deleted:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/domain-on-pv/create-domain-inputs.yaml
	new file:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/loadbalancer-template.yaml
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/admin-lb.yaml
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/cluster-lb.yaml
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/helm-sa-cluster-admin-role.yaml
	modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/mii-initial.yaml

Fix the menu issue.
The live page will show included files in the menu.
Change the included files as txt can solve this problem.

 Changes to be committed:
	modified:   ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-01.txt
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service
584E
/includes/clean-up-resources-body-02.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/clean-up-resources-body-02.txt
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-01.txt
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-body-02.txt
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.txt
	renamed:    ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites.md -> ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites-01.txt
	new file:   ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites-02.txt
	modified:   ../../../../../docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md

- Validate model-in-image

On branch wls-on-aks-model-in-image
modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md

- Added note to skip to sample deployment.

- Use 1. for numbered lists.

modified:   kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/create-domain-on-aks-inputs.yaml

- Update weblogic version number in image to 12.2.1.4.

Add troubeshooting for WebLogic Kubernetes Operator ErrImagePull

Troubleshooting for VM size error

Update troubleshooting.md

Update domain-on-pv.md

Remove diff markers

Thanks @tbarnes-us.

On branch wls-on-aks-model-in-image Apply suggestions from @rosemarymarano and @tbarnes-us.
modified:   docs-source/content/samples/simple/azure-kubernetes-service/_index.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/domain-on-pv.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/create-aks-cluster-storage.txt
modified:   docs-source/content/samples/simple/azure-kubernetes-service/includes/prerequisites-02.txt
modified:   docs-source/content/samples/simple/azure-kubernetes-service/model-in-image.md
modified:   docs-source/content/samples/simple/azure-kubernetes-service/troubleshooting.md
modified:   docs-source/content/userguide/managing-domains/prepare.md
modified:   kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/JRF/mii-initial-d1-JRF-v1.yaml
modified:   kubernetes/samples/scripts/create-weblogic-domain/model-in-image/domain-resources/WLS/mii-initial-d1-WLS-v1.yaml
modified:   operator/integration-tests/model-in-image/mii-sample-wrapper/mii-domain.yaml.template-JRF
modified:   operator/integration-tests/model-in-image/mii-sample-wrapper/mii-domain.yaml.template-WLS
deleted:    kubernetes/samples/scripts/create-weblogic-domain-on-azure-kubernetes-service/model-in-image/mii-initial.yaml
@edburns edburns mentioned this pull request Feb 23, 2021
Merged
@edburns
Copy link
Contributor Author
edburns commented Feb 23, 2021

Obsoleted by #2219 .

@edburns edburns closed this Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tbarnes-us tbarnes-us tbarnes-us left review comments

@rosemarymarano rosemarymarano rosemarymarano left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@edburns @tbarnes-us @rosemarymarano @galiacheng
0