Skip to content

Use Candlepin project official image#104

Closed
ehelms wants to merge 1 commit intotheforeman:masterfrom
ehelms:candlepin-container
Closed

Use Candlepin project official image#104
ehelms wants to merge 1 commit intotheforeman:masterfrom
ehelms:candlepin-container

Conversation

@ehelms
Copy link
Copy Markdown
Member

@ehelms ehelms commented Mar 7, 2025

Attempting to use the Candlepin project's official container, but running into errors I do not understand:

2025-03-07 02:22:21,154 [thread=Thread-0 (ActiveMQ-server-ActiveMQServerImpl::name=localhost)] [=, org=, csid=] WARN  org.apache.activemq.artemis.core.server - AMQ222216: Security problem while authenticating: AMQ229031: Unable to validate user from invm:0. Username: null; SSL certificate subject DN: unavailable
2025-03-07 02:22:21,156 [thread=main] [=, org=, csid=] WARN  org.candlepin.audit.ArtemisMessageSource - Unable to reconnect message listeners. Messages will not be received: event.default
org.apache.activemq.artemis.api.core.ActiveMQSecurityException: AMQ229031: Unable to validate user from invm:0. Username: null; SSL certificate subject DN: unavailable

@ekohl ekohl added the candlepin label Mar 7, 2025
candlepin_db_ssl_ca: None
candlepin_db_ssl_verify: true

candlepin_tomcat_home: /opt/tomcat/conf
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

what was the reason for this change? Once I undid this change and let the config files be under /etc/tomcat then it worked

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think I ran into errors originally with this path. And then I assumed since tomcat is in /opt/tomcat and there is a conf directory in there that it should use that. Since it wouldn't want to clash with system tomcat from package.

What is the expectation for where configuration files should live with this image?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I tested this locally and got a green result. Pushed update to this PR and we'll see what automation says.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I see in the candlepin image we build we use /opt/tomcat/conf, so I have no idea why that doesn't work. I'll try to tinker a bit more on monday to see why

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@ehelms so it looks like the reason it worked previously by using /etc/tomcat/conf was because the wrong container was still being used (the one with the rpms). I had to do a few changes to get it working. Should I push a commit to this PR?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You can find the diff here: nikosmoum@94d127d

@ehelms ehelms force-pushed the candlepin-container branch from 6409bed to 981bbae Compare March 13, 2025 18:18
@ehelms
Copy link
Copy Markdown
Member Author

ehelms commented Mar 13, 2025

@ekohl How do you see your config in containers work (#102) squaring with this kind of change?

@ekohl
Copy link
Copy Markdown
Member

ekohl commented Mar 14, 2025

@ekohl How do you see your config in containers work (#102) squaring with this kind of change?

We can still create a container that is FROM quay.io/candlepin/candlepin:4.4.20-1 and adds the config files & expose env vars. Maintenance wise we'll have to see what's best. I think quay can rebuild containers automatically if the base layer changes so that's something to investigate IMHO.

@ehelms
Copy link
Copy Markdown
Member Author

ehelms commented Mar 28, 2025 via email

Signed-off-by: Eric D. Helms <ericdhelms@gmail.com>
@ehelms ehelms force-pushed the candlepin-container branch from 981bbae to abefc94 Compare April 1, 2025 17:45
@ehelms
Copy link
Copy Markdown
Member Author

ehelms commented Apr 1, 2025

@nikosmoum Updated based on your diff

@ehelms
Copy link
Copy Markdown
Member Author

ehelms commented May 8, 2025

Moved: theforeman/candlepin-oci-images#2

@ehelms ehelms closed this May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants