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

Install CentOS7's OpenSSL 1.0.2 package on CentOS6 to resolve certificate issues #2344

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

sxa
Copy link
Member

@sxa sxa commented Oct 8, 2021

Fixes #2342

Checklist
  • commit message has one of the standard prefixes
  • faq.md updated if appropriate
  • other documentation is changed or added (if applicable)
  • playbook changes run through VPC or QPC (if you have access)
  • for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

@sxa sxa force-pushed the cent6_ssl102 branch 2 times, most recently from be06f0e to 9a7762c Compare October 11, 2021 15:40
@sxa sxa self-assigned this Oct 11, 2021
@sxa sxa added this to the October 2021 milestone Oct 11, 2021
@sxa
Copy link
Member Author

sxa commented Oct 11, 2021

CentOS6 playbook check has passed :-)

@sxa
Copy link
Member Author

sxa commented Oct 11, 2021

I'd really prefer not to do this, but since CentOS6 is out of support and it seems that it is not possible for this to work without OpenSSL 1.0.2, pulling the packages from CentOS7 seems a safe option and appears to work.

NOTE: Which this works for creating the docker build image, it does NOT pass in VagrantPlaybookCheck, possibly due to the presence of some 32-bit openssl packages. That should NOT block the merging of this (since things are even more broken just now) but should be addressed subsequently...

@sxa sxa marked this pull request as ready for review October 11, 2021 16:41
@jerboaa
Copy link

jerboaa commented Oct 12, 2021

This seems a very ugly hack. But I guess that's better than a broken build. An update to CentOS 7 for the docker build containers needs to be considered.

@sxa
Copy link
Member Author

sxa commented Oct 12, 2021

This seems a very ugly hack. But I guess that's better than a broken build.

My thoughts exactly :-) But at least it's a documented, repeatable and visible hack that should not have any effect on the contents of the Temurin builds.

An update to CentOS 7 for the docker build containers needs to be considered.

The issue with that is, of course, that without putting in the older libraries somewhere you're going to end up on a dependency on later versions of glibc etc. and given how close we are to the next release I really don't want to be even considering that right now.

@jerboaa
Copy link

jerboaa commented Oct 12, 2021

An update to CentOS 7 for the docker build containers needs to be considered.

The issue with that is, of course, that without putting in the older libraries somewhere you're going to end up on a dependency on later versions of glibc etc. and given how close we are to the next release I really don't want to be even considering that right now.

Yes understood. It should be discussed on the PMC for the January releases, though.

@sxa
Copy link
Member Author

sxa commented Oct 12, 2021

Solaris failure is being addressed elsewhere. Merging in the absence of any major objections (This was also circulated to #infrastructure last night) and I've raised #2351 to get VPC fixed.

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.

Ansible request for workaround to cacerts in CentOS6 playbooks
2 participants