Skip to content
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.

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

Updating postConfigure() #236

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Updating postConfigure() #236

wants to merge 1 commit into from

Conversation

atef23
Copy link

@atef23 atef23 commented Apr 22, 2021

Updating postConfigure to check if delayedpostconfigure.sh is present and execute it if present. Otherwise check if postconfigure.sh is present and execute it.

Hi,

The launch scripts for container images call postConfigure. Currently, if there is no delayedpostconfigure.sh present under $JBOSS_HOME/extensions then postconfigure.sh does not get invoked even it is there. I observed this behavior in the kieserver-rhel8 image:

https://catalog.redhat.com/software/containers/rhpam-7/rhpam-kieserver-rhel8/5db03b6cbed8bd164af2a202?container-tabs=dockerfile

The SSO image script does not have a delayed post configure but does directly call postconfigure. A colleague of mine was able to invoke postconfigure.sh by placing it under /opt/eap/extensions on a container running the SSO image. When I attempted to deploy the kieserver-rhel8 image, I needed to add an empty placeholder delayedpostconfigure.sh to get postconfigure.sh to execute. I'm not sure what the intent is, but please let me know if I can help with any modifications.

Thanks.

Updating postConfigure to check if delayedpostconfigure.sh is present and execute it if present. Otherwise check if postconfigure.sh is present and execute it.
@atef23
Copy link
Author

atef23 commented Apr 22, 2021

I'm not sure if the intent is to check for delayedpostconfigure.sh and execute that instead of postconfigure.sh or to execute both if they are present. I don't see the delayedpostconfigure.sh function called:

https://github.com/jboss-container-images/jboss-kie-modules/blob/283e0d3f077d9aa3968f36a55762e6a0102715ab/jboss-kie-smartrouter/added/launch/configure.sh

https://github.com/jboss-container-images/jboss-kie-modules/blob/283e0d3f077d9aa3968f36a55762e6a0102715ab/os-eap-legacy/os-eap-launch/added/launch/configure.sh

https://github.com/jboss-container-images/jboss-kie-modules/blob/283e0d3f077d9aa3968f36a55762e6a0102715ab/jboss-kie-process-migration/added/launch/configure.sh

So I assumed delayedpostconfigure.sh should be executed through a call to the postConfigure function

@jfdenise
Copy link
Contributor

@atef23 in EAP and WildFly images we are calling delayedPostConfigure that calls postconfigure.sh script if delayedpostconfigure.sh is not present. That is done there: https://github.com/wildfly/wildfly-cekit-modules/blob/master/jboss/container/wildfly/launch-config/config/added/launch/launch.sh#L87
I suspect your images are not using the same entry point.
So we can't integrate your PR in the master branch, it would break WildFly images. We need to understand your setup. Are you using a branch of wildfly-cekit-modules?

@jfdenise jfdenise added the hold label Apr 26, 2021
@atef23
Copy link
Author

atef23 commented Apr 27, 2021

Ah I see. Let me try to find the upstream source for the kieserver-rhel8 image:

https://catalog.redhat.com/software/containers/rhpam-7/rhpam-kieserver-rhel8/5db03b6cbed8bd164af2a202?container-tabs=dockerfile

That is the one that is calling a configure.sh which I suspect is not calling delayedPostConfigure like one of those scripts I liked above. The workaround I'm using right now is to place an empty delayedPostConfigure.sh under /opt/eap/extensions along with a functional postConfigure.sh which causes the postConfigure.sh to get executed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants