-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Simplify gce inventory plugin injector #3754
Simplify gce inventory plugin injector #3754
Conversation
Build failed.
|
@@ -18,8 +18,6 @@ def gce(cred, env, private_data_dir): | |||
project = cred.get_input('project', default='') | |||
username = cred.get_input('username', default='') | |||
|
|||
env['GCE_EMAIL'] = username | |||
env['GCE_PROJECT'] = project |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See https://docs.ansible.com/ansible/latest/modules/gce_module.html
I can find no evidence that these are employed by the gce module anywhere, and if not, they shouldn't be here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
b) GCE_PROJECT, GCE_CREDENTIAL_FILE_PATH, GCE_EMAIL (legacy, not recommended; req'd if using p12 key)
Maybe I'm wrong about this and GCE_EMAIL is still needed...
This consumes the change made in Ansible core ansible/ansible#54407 which is in Ansible 2.8, allowing the plugin injection logic to share the script logic and to be simplified
Build succeeded.
|
Build succeeded.
|
Build succeeded (gate pipeline).
|
Upload directly 353
SUMMARY
This consumes the change made in Ansible core:
ansible/ansible#54407
Which is in Ansible 2.8
This allows the plugin injection logic to share the script logic.
ISSUE TYPE
COMPONENT NAME
AWX VERSION
ADDITIONAL INFORMATION
This entry becomes congruent with the script environment now, see:
awx/awx/main/tests/data/inventory/scripts/gce/env.json
Line 4 in 864fef4
This is a very subtle change which will keep open many important possibilities if #3749 also makes it in.
build_env
methods access to the credential file paths