Skip to content

Fixes #31119 - run katello-certs-check when installing with custom certs#579

Merged
wbclark merged 1 commit intotheforeman:developfrom
wbclark:katello-certs-check
Oct 23, 2020
Merged

Fixes #31119 - run katello-certs-check when installing with custom certs#579
wbclark merged 1 commit intotheforeman:developfrom
wbclark:katello-certs-check

Conversation

@wbclark
Copy link
Copy Markdown
Contributor

@wbclark wbclark commented Sep 15, 2020

Based on #514

@ehelms
Copy link
Copy Markdown
Member

ehelms commented Sep 24, 2020

I think you can rebase this now @wbclark

@wbclark wbclark force-pushed the katello-certs-check branch from a28af7a to 7cdcb70 Compare October 17, 2020 17:29
@wbclark wbclark marked this pull request as ready for review October 17, 2020 17:29
@wbclark
Copy link
Copy Markdown
Contributor Author

wbclark commented Oct 17, 2020

Rebased and ready for review

@ekohl
Copy link
Copy Markdown
Member

ekohl commented Oct 18, 2020

Looks like the tests don't like the expired certificate, other than that I think this makes sense.

Edit: perhaps this also deserves a Redmine issue for the changelog?

Copy link
Copy Markdown
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Please open a Redmine issue for this and use a more descriptive commit message, other than that 👍

@wbclark wbclark force-pushed the katello-certs-check branch from 7cdcb70 to aa3a6c3 Compare October 21, 2020 14:15
@wbclark wbclark changed the title Katello certs check Fixes #31119 - run katello-certs-check when installing with custom certs Oct 21, 2020
@wbclark wbclark force-pushed the katello-certs-check branch from aa3a6c3 to aaee541 Compare October 21, 2020 15:02
@wbclark wbclark force-pushed the katello-certs-check branch from aaee541 to bed69f0 Compare October 21, 2020 19:43
@wbclark wbclark merged commit 608b5f9 into theforeman:develop Oct 23, 2020
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.

4 participants