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

Fix workflow job template webhook credential bug #12325

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

Tompage1994
Copy link
Contributor

@Tompage1994 Tompage1994 commented Jun 6, 2022

Signed-off-by: tompage1994@hotmail.co.uk tpage@redhat.com

SUMMARY

related #12324
Fixes issue in awx.awx.workflow_job_template where webhook_credential option was not working because the api /api/v2/webhook_credential does not exist.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • Collection
AWX VERSION
21.0.0

@john-westcott-iv john-westcott-iv added component:awx_collection issues related to the collection for controlling AWX and removed component:collection labels Jun 8, 2022
@john-westcott-iv
Copy link
Member

@Tompage1994 Thanks for your contribution. We are going to confirm we have tests which will pickup on this bug to make sure we don't end up with a regression in the future. Once those test are defined we will get this merged.

@john-westcott-iv
Copy link
Member

Validated with a test on #12468

@jay-steurer jay-steurer self-assigned this Jul 7, 2022
@CFSNM
Copy link
Contributor

CFSNM commented Jul 8, 2022

@Tompage1994 hey, could you rebase your branch using ansible/devel one? From QE we are having some issues due to the misalignment between your branch and devel. Thanks!

Signed-off-by: tompage1994@hotmail.co.uk <tpage@redhat.com>
@Tompage1994
Copy link
Contributor Author

@Tompage1994 hey, could you rebase your branch using ansible/devel one? From QE we are having some issues due to the misalignment between your branch and devel. Thanks!

Done

@jay-steurer jay-steurer merged commit 008a4b4 into ansible:devel Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:awx_collection issues related to the collection for controlling AWX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants