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

mgr/cephadm: allow setting insecure_skip_verify for alertmanager #45860

Merged
merged 1 commit into from Apr 18, 2022

Conversation

adk3798
Copy link
Contributor

@adk3798 adk3798 commented Apr 11, 2022

Fixes: https://tracker.ceph.com/issues/55272

Signed-off-by: Adam King adking@redhat.com

Changelog: Changed it from always applying this setting to only applying it if an "insecure" property is set to true in the spec

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@adk3798
Copy link
Contributor Author

adk3798 commented Apr 13, 2022

http://pulpito.ceph.com/adking-2022-04-13_05:16:48-orch:cephadm-wip-adk-testing-2022-04-12-2231-distro-basic-smithi/

2 Failures

To summarize, run was passing. Will start tracking this type of lvm batch failure to see if it happens on the same hosts (which would imply a possible hardware issue) in https://tracker.ceph.com/issues/55319

@mgfritch mgfritch requested review from s0nea and p-se April 13, 2022 15:52
@adk3798 adk3798 changed the title mgr/cephadm: set insecure_skip_verify to true for alertmanager mgr/cephadm: allow setting insecure_skip_verify to true for alertmanager Apr 13, 2022
@adk3798 adk3798 requested a review from a team as a code owner April 13, 2022 18:17
Add a "secure" parameter to alertmanager spec that will cause it
to deploy alertmanagers with insecure_skip_verify as true or false
depending on the value given for "secure".

NOTE: alertmanager must still be reconfigured after applying a yaml
with this option changed.

Fixes: https://tracker.ceph.com/issues/55272
Fixes: https://tracker.ceph.com/issues/55333

Signed-off-by: Adam King <adking@redhat.com>
@adk3798 adk3798 changed the title mgr/cephadm: allow setting insecure_skip_verify to true for alertmanager mgr/cephadm: allow setting insecure_skip_verify for alertmanager Apr 14, 2022
@adk3798
Copy link
Contributor Author

adk3798 commented Apr 14, 2022

jenkins test make check

@adk3798
Copy link
Contributor Author

adk3798 commented Apr 15, 2022

@adk3798 adk3798 mentioned this pull request Apr 17, 2022
14 tasks
Copy link
Contributor

@p-se p-se left a comment

Choose a reason for hiding this comment

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

lgtm

@adk3798 adk3798 merged commit b6f42e6 into ceph:master Apr 18, 2022
15 checks passed
@epuertat
Copy link
Member

hi @adk3798: is this going to be backported to quincy-pacific? (as cephadm doesn't follow the regular Ceph backport process it's not clear to me)

@adk3798
Copy link
Contributor Author

adk3798 commented Apr 19, 2022

hi @adk3798: is this going to be backported to quincy-pacific? (as cephadm doesn't follow the regular Ceph backport process it's not clear to me)

already backported this to pacific. Quincy backport will happen soon as part of a large batch of cephadm things to backport there once the initial release completes.

@adk3798 adk3798 mentioned this pull request Apr 27, 2022
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants