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

added missing quotes to ticket delegation #250

Merged
merged 2 commits into from
Dec 31, 2023
Merged

Conversation

fl0wx
Copy link
Contributor

@fl0wx fl0wx commented Dec 29, 2023

Added missing quotes to ticket delegation. Currently, the ticket creation fails with the following error message:

TASK [icinga.icinga.icinga2 : delegate ticket request to master] ***********************************************************************************************************************************************************************************************************************************************************
changed: [host1.local -> host0.local] => {"changed": true, "cmd": "icinga2 pki ticket --cn \"host1.local --salt XXXXXXXX\"", "delta": "0:00:00.062976", "end": "2023-12-29 11:33:07.761888", "msg": "", "rc": 0, "start": "2023-12-29 11:33:07.698912", "stderr": "", "stderr_lines": [], "stdout": "fc043b935aacd292b742b0aaecd63e9055c16c95", "stdout_lines": ["fc043b935aacd292b742b0aaecd63e9055c16c95"]}

TASK [icinga.icinga.icinga2 : get certificate] *****************************************************************************************************************************************************************************************************************************************************************************
fatal: [host1.local]: FAILED! => {"changed": true, "cmd": "icinga2 pki  request --ticket \"fc043b935aacd292b742b0aaecd63e9055c16c95\" --host \"host0.local\" --port \"5665\" --ca \"/var/lib/icinga2/certs/ca.crt\" --key \"/var/lib/icinga2/certs/host1.local.key\" --trustedcert \"/var/lib/icinga2/certs/trusted-master.crt\"  --cert \"/var/lib/icinga2/certs/host1.local.crt\"", "delta": "0:00:00.130217", "end": "2023-12-29 11:33:09.363278", "msg": "non-zero return code", "rc": 1, "start": "2023-12-29 11:33:09.233061", "stderr": "", "stderr_lines": [], "stdout": "information/cli: Writing CA certificate to file '/var/lib/icinga2/certs/ca.crt'.\ncritical/cli: !!! Invalid ticket for CN 'host1.local'.", "stdout_lines": ["information/cli: Writing CA certificate to file '/var/lib/icinga2/certs/ca.crt'.", "critical/cli: !!! Invalid ticket for CN 'host1.local'."]}

the addition of the new quotes changes the command to an actual working command:

OLD: icinga2 pki ticket --cn "host1.local --salt XXXXXXXX"
NEW: icinga2 pki ticket --cn "host1.local" --salt "XXXXXXXX"

Copy link

cla-bot bot commented Dec 29, 2023

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @fl0wx

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

  • If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

Copy link
Contributor

@mocdaniel mocdaniel left a comment

Choose a reason for hiding this comment

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

Thanks a lot, this slipped our attention.
LGTM! Edit: Please see comment below.

@mocdaniel mocdaniel self-requested a review December 29, 2023 15:08
Copy link
Contributor

@mocdaniel mocdaniel left a comment

Choose a reason for hiding this comment

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

Almost forgot: Could you please add a changelog fragment? This makes it easier for us to track changes, fixes, etc. and attribute them to contributors.

Information on how to do it can be found here, with some recent examples here.

Copy link

cla-bot bot commented Dec 29, 2023

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @fl0wx

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

  • If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

@fl0wx
Copy link
Contributor Author

fl0wx commented Dec 29, 2023

done :)

@mocdaniel
Copy link
Contributor

@bobapple can you check whether the CLA has been signed successfully?

@fl0wx
Copy link
Contributor Author

fl0wx commented Dec 29, 2023

@mocdaniel i signed it just now. can you rerun the check?

@mocdaniel mocdaniel linked an issue Dec 29, 2023 that may be closed by this pull request
@mocdaniel mocdaniel added this to the v0.4.0 milestone Dec 29, 2023
@mocdaniel mocdaniel added the bug Something isn't working label Dec 29, 2023
@fl0wx fl0wx requested a review from mocdaniel December 29, 2023 15:30
@mocdaniel
Copy link
Contributor

Unfortunately, I can't, hence the ping above :) This will get merged after the weekend, probably.

@bobapple
Copy link
Member

@cla-bot check

Copy link

cla-bot bot commented Dec 31, 2023

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @fl0wx

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

  • If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

@bobapple
Copy link
Member

@cla-bot check

@cla-bot cla-bot bot added the cla/signed label Dec 31, 2023
@mocdaniel
Copy link
Contributor

Thanks a lot for your contribution @fl0wx!

@mocdaniel mocdaniel merged commit 12c4ac1 into Icinga:main Dec 31, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla/signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow passing Ticketsalts for ticket generation
3 participants