WASdev / azure.liberty.aks

Apache License 2.0
2 stars 8 forks source link

Support WebSphere Liberty Operator #69

Closed majguo closed 1 year ago

majguo commented 2 years ago

Description

The PR is to support WebSphere Liberty Operator by resolving the following issues:

Change summary

Testing

Verified the following 8 test cases with integration-test pipeline (Note: For all cases enabled with WLO, Edition is IBM WebSphere Application Server and Product entitlement source is Standalone): AGIC WLO OLO Sample App Pipeline run
X integration-test-65
X X integration-test-66
X X integration-test-67
X X X integration-test-68
X integration-test-69
X X integration-test-70
X X integration-test-71
X X X integration-test-72
Verified the following 5 test cases using preview link: AGIC WLO OLO Sample App Edition Product entitlement source
X X X IBM WebSphere Application Server Standalone
X X X IBM WebSphere Application Server Standalone
X X IBM WebSphere Application Server Liberty Core IBM WebSphere Hybrid Edition
X X IBM WebSphere Application Server Network Deployment IBM Cloud Pak for Applications
X X IBM WebSphere Application Server Network Deployment IBM WebSphere Application Server Family Edition

Feel free to do a live testing with preview link.

UI changes review

Besides visiting preview link for reviewing the renamed blade Operator and application (which comes from Configure application), reviewer can preview the UI changes using the Create UI Definition Sandbox:

Also attached the screenshots for quick overview:

Signed-off-by: Jianguo Ma jiangma@microsoft.com

git4rk commented 2 years ago

@majguo I have a couple of questions:

  1. Are we using OLM to install the WLO?
  2. If customer choses to use IBM Supported, will we still provide sample application CR yaml for WebSphereLibertyApplication?
majguo commented 2 years ago

@git4rk

  1. We're using kustomize to install the WLO (as well as the OLO).
  2. Yes. See websphere-liberty-application.yaml.template and websphere-liberty-application-agic.yaml.template.
git4rk commented 2 years ago

@edburns Can you please review the recent changes. Thanks.