Skip to content

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"

@cla-bot
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
Member

@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
Member

@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.

@cla-bot
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
Member

@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
Member

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

@bobapple
Copy link

@cla-bot check

@cla-bot
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

@cla-bot check

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

Thanks a lot for your contribution @fl0wx!

@mocdaniel mocdaniel merged commit 12c4ac1 into NETWAYS:main Dec 31, 2023
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