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

Adopt custom_injectors change from awx-plugins #15638

Merged

Conversation

chrismeyersfsu
Copy link
Member

SUMMARY

Related PR ansible/awx-plugins#57

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API
AWX VERSION
devel
ADDITIONAL INFORMATION

@github-actions github-actions bot added component:api dependencies Pull requests that update a dependency file labels Nov 15, 2024
@chrismeyersfsu chrismeyersfsu force-pushed the AAP-25749-move-inject-credential branch 2 times, most recently from 387a115 to 813348b Compare November 18, 2024 20:34
@chrismeyersfsu chrismeyersfsu changed the title Adopt post_injectors change from awx-plugins Adopt custom_injectors change from awx-plugins Nov 18, 2024
@chrismeyersfsu chrismeyersfsu force-pushed the AAP-25749-move-inject-credential branch 2 times, most recently from 9d8e59e to 34229f2 Compare November 19, 2024 18:56
@AlanCoding
Copy link
Member

AlanCoding commented Nov 19, 2024

I assume checks are failing due to the dependency.

Otherwise looks good.

@chrismeyersfsu chrismeyersfsu force-pushed the AAP-25749-move-inject-credential branch 2 times, most recently from ad1fcbc to 8d57cd3 Compare December 10, 2024 16:07
@github-actions github-actions bot removed the dependencies Pull requests that update a dependency file label Dec 10, 2024
* Unit tests do not create CredentialType records for Credential
  plugins. Instead, they explicitly instantiate CredentialType(s) for
  Credential plugins. They rely on CredentialType.defaults[key] to do
  so. This change makes sure custom_injectors get bolted onto the
  created CredentialType.
@chrismeyersfsu chrismeyersfsu force-pushed the AAP-25749-move-inject-credential branch from f857235 to 8ccb32b Compare December 10, 2024 17:32
Copy link

sonarcloud bot commented Dec 10, 2024

@chrismeyersfsu chrismeyersfsu merged commit a129bc8 into ansible:devel Dec 10, 2024
26 checks passed
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.

2 participants