Conversation
|
This looks related to #141 but is not an aspect I have incorporated. I was wanting to get in a version of remote database testing and then harden it with more of the options. |
|
It is, @Gauravtalreja1 ran into this when testing ext db stuff |
evgeni
force-pushed
the
ssldb
branch
5 times, most recently
from
f8a1e26 to
1ba7910
Compare
evgeni
force-pushed
the
ssldb
branch
4 times, most recently
from
921e621 to
7355577
Compare
| containers.podman.podman_secret: | ||
| state: present | ||
| name: candlepin-db-ca | ||
| data: "{{ lookup('ansible.builtin.file', candlepin_database_ssl_ca) if candlepin_database_ssl_ca else 'empty' }}" |
There was a problem hiding this comment.
This is creating an empty secret if there is no database SSL cert? Why not use a when conditional on the sercret?
There was a problem hiding this comment.
Because then I need to also conditionally mount it, and that's painful ;)
There was a problem hiding this comment.
I get that, I worry about this being a red herring while debugging.
There was a problem hiding this comment.
what kind of red herring? the file being present?
There was a problem hiding this comment.
Correct. The file / secret being present but empty raises the "should it be empty? or is it accidentally empty?"
There was a problem hiding this comment.
That's why the string is "empty", or should I do "this secret was intentionally left blank"?
|
What else do you think is needed to take it out of draft? |
|
I wanted to write up some tests to validate it. |
evgeni
force-pushed
the
ssldb
branch
2 times, most recently
from
46aeafa to
3a61d77
Compare
evgeni
force-pushed
the
ssldb
branch
7 times, most recently
from
481ca09 to
d157f1a
Compare
evgeni
force-pushed
the
ssldb
branch
3 times, most recently
from
a0572e5 to
9adce9d
Compare
|
@ehelms look, no draft! |
Gauravtalreja1
left a comment
Gauravtalreja1
left a comment
There was a problem hiding this comment.
ACK 🍏
Tested this with foremanctl-1.2.0-1.20260303084744080355.pr345.20.g01afd69.el9.noarch
It works as expected to deploy Foreman with external DB for both SSL and non-SSL
Thank you @evgeni 🚀
3 participants
No description provided.