-
Notifications
You must be signed in to change notification settings - Fork 341
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
aws_rds inventory plugin does not return any results due to regression in 478022695b333043857a6929b350a2a3c07ae567 #1304
Labels
bug
This issue/PR relates to a bug
easyfix
Good for new comers and easy to start with contribution
python3
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
Comments
Files identified in the description:
If these files are inaccurate, please update the |
HI @levinse Thanks for taking the time to report this issue. As you found the fix, could you please take some time to submit a PR? Thanks |
abikouo
added
easyfix
Good for new comers and easy to start with contribution
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
and removed
needs_triage
labels
Dec 21, 2022
softwarefactory-project-zuul bot
pushed a commit
that referenced
this issue
Jan 10, 2023
[stable-5] aws_rds - RDS inventory plugin didn't read config SUMMARY fixes #1304 Partial reversion of #958 (ish) _read_config_data() has side effects, namely that it sets all of the options data, as well as returning data. ISSUE TYPE Bugfix Pull Request COMPONENT NAME aws_rds ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None>
@tremble Can we close this? |
fixed by #1332 |
abikouo
pushed a commit
to abikouo/amazon.aws
that referenced
this issue
Sep 18, 2023
Rename cloudwatch metrics_alarm SUMMARY In line with the new naming guidelines rename ec2_metric_alarm to cloudwatch_metric_alarm ISSUE TYPE Feature Pull Request COMPONENT NAME plugins/modules/cloudwatch_metric_alarm.py plugins/modules/ec2_metric_alarm.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None>
abikouo
pushed a commit
to abikouo/amazon.aws
that referenced
this issue
Sep 18, 2023
Rename cloudwatch metrics_alarm SUMMARY In line with the new naming guidelines rename ec2_metric_alarm to cloudwatch_metric_alarm ISSUE TYPE Feature Pull Request COMPONENT NAME plugins/modules/cloudwatch_metric_alarm.py plugins/modules/ec2_metric_alarm.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None>
abikouo
pushed a commit
to abikouo/amazon.aws
that referenced
this issue
Oct 24, 2023
Rename cloudwatch metrics_alarm SUMMARY In line with the new naming guidelines rename ec2_metric_alarm to cloudwatch_metric_alarm ISSUE TYPE Feature Pull Request COMPONENT NAME plugins/modules/cloudwatch_metric_alarm.py plugins/modules/ec2_metric_alarm.py ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis <None>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
This issue/PR relates to a bug
easyfix
Good for new comers and easy to start with contribution
python3
waiting_on_contributor
Needs help. Feel free to engage to get things unblocked
Summary
aws_rds inventory plugin does not return any results due to regression in 4780226.
This commit, ostensibly named "linting", has actually removed the following line in plugins/inventory/aws_rds.py:
This causes the inventory plugin to return no data. Restoring this line fixes the expected output.
Issue Type
Bug Report
Component Name
aws_rds inventory plugin
Ansible Version
Collection Versions
AWS SDK versions
Configuration
OS / Environment
No response
Steps to Reproduce
Expected Results
I expected some inventory results to appear
Actual Results
Code of Conduct
The text was updated successfully, but these errors were encountered: