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

[manual backport stable-5] lambda_info: remove unnecessary snake_casing of env_vars when querying lambda config (#1457) #1479

Conversation

alinabuzachis
Copy link
Collaborator

lambda_info: remove unnecessary snake_casing of env_vars when querying lambda config

SUMMARY

This PR modifies code to avoid converting environment variables to snake_case when querying config. Fixes #1412

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

lambda_info
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

…g lambda config (ansible-collections#1457)

lambda_info: remove unnecessary snake_casing of env_vars when querying lambda config

SUMMARY

This PR modifies code to avoid converting environment variables to snake_case when querying config.
Fixes ansible-collections#1412

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

lambda_info
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/27a71b5331544bc1a2fa133bc491bd66

✔️ ansible-galaxy-importer SUCCESS in 3m 38s
✔️ build-ansible-collection SUCCESS in 12m 44s
✔️ ansible-test-splitter SUCCESS in 5m 11s
✔️ integration-amazon.aws-1 SUCCESS in 9m 37s
Skipped 43 jobs

@alinabuzachis alinabuzachis changed the title [manual backport] lambda_info: remove unnecessary snake_casing of env_vars when querying lambda config (#1457) [manual backport stable-5] lambda_info: remove unnecessary snake_casing of env_vars when querying lambda config (#1457) Apr 26, 2023
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 26, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/1bbed625efab474bb68d94aeb96238d3

✔️ ansible-galaxy-importer SUCCESS in 4m 44s
✔️ build-ansible-collection SUCCESS in 13m 51s
✔️ ansible-test-splitter SUCCESS in 5m 03s
✔️ integration-amazon.aws-1 SUCCESS in 11m 18s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 9f8f2ee into ansible-collections:stable-5 Apr 26, 2023
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Migrate rds_cluster* modules and tests

Depends-On: ansible-collections#1027
Depends-On: ansible-collections#1478
Remove rds_cluster* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Mike Graves <mgraves@redhat.com>
Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Migrate rds_cluster* modules and tests

Depends-On: ansible-collections#1027
Depends-On: ansible-collections#1478
Remove rds_cluster* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Mike Graves <mgraves@redhat.com>
Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Migrate rds_cluster* modules and tests

Depends-On: ansible-collections#1027
Depends-On: ansible-collections#1478
Remove rds_cluster* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Mike Graves <mgraves@redhat.com>
Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants