WIP: Support for Openshift env with non-root(oracle) user #1630
Conversation
|
Is this still a WIP or is it ready for review? |
Its still in review as the changes impact the samples used in Operator repo: https://github.com/oracle/weblogic-kubernetes-operator/tree/master/kubernetes/samples/charts/apache-webtier. |
|
|
||
| #Expose ports | ||
| EXPOSE 80 4433 | ||
| EXPOSE 8080 4433 |
There was a problem hiding this comment.
Will this cause problems for the existing customers when they upgrade to the new version?
There was a problem hiding this comment.
When customers upgrade to next version, for eg., say 12.2.1.4, then we plan to provide support for both 80 and 8080.
There was a problem hiding this comment.
I would like to make this a generic image for 12.2.1.3, 12.2.14, 14.1.1.0 can we support both 80 and 8080 now?
| # Licensed under the Universal Permissive License v 1.0 as shown at http://oss.oracle.com/licenses/upl. | ||
| LoadModule weblogic_module /root/lib/${MOD_WLS_PLUGIN} | ||
| # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl. | ||
| LoadModule weblogic_module /u01/oracle/lib/mod_wl_24.so |
There was a problem hiding this comment.
The old version uses MOD_WLS_PLUGIN env var. Is there a reason to change it to the hard-coded value?
| touch /etc/sudoers && \ | ||
| sed -i /etc/sudoers -re 's/^%sudo.*/%sudo ALL=(ALL:ALL) NOPASSWD: ALL/g' && \ | ||
| sed -i /etc/sudoers -re 's/^root.*/root ALL=(ALL:ALL) NOPASSWD: ALL/g' && \ | ||
| sed -i /etc/sudoers -re 's/^#includedir.*/## **Removed the include directive** ##"/g' && \ |
There was a problem hiding this comment.
The include directive is commented out. Do we really need to remove it?
ashageetha
changed the title
There was a problem hiding this comment.
@ashageetha Could you reply to the two outstanding comments from Dongbo?
|
|
||
| #Expose ports | ||
| EXPOSE 80 4433 | ||
| EXPOSE 8080 4433 |
There was a problem hiding this comment.
I would like to make this a generic image for 12.2.1.3, 12.2.14, 14.1.1.0 can we support both 80 and 8080 now?
Sure. We are working on to support both 80 and 8080 this week. I will update you once its ready. |
Djelibeybi
changed the title
|
I have added WIP to the title of the PR so it doesn't get merged too early. Once you are happy with the code, please remove that and the merge will be allowed. |
Hi,
This PR has fix for Apache Webtier to run in Openshift env with non-root user and with port 8080.
This sample is being used as part of Apache Webtier Setup in Openshift env as per WebLogic Kubernetes Operator Samples: "https://github.com/oracle/weblogic-kubernetes-operator/tree/master/kubernetes/samples/charts/apache-webtier".
Currently tested in below scenario: