Skip to content

fix: pass in optional arguments to sqs#421

Merged
mkanoor merged 3 commits into
ansible:mainfrom
mkanoor:amazon_sqs_changes
Apr 16, 2025
Merged

fix: pass in optional arguments to sqs#421
mkanoor merged 3 commits into
ansible:mainfrom
mkanoor:amazon_sqs_changes

Conversation

@mkanoor
Copy link
Copy Markdown
Contributor

@mkanoor mkanoor commented Apr 11, 2025

Based on feedback we received we are missing some key parameters from sqs queue processing. Adding additional optional parameters

  • queue_owner_aws_account_id
  • max_number_of_messages

https://issues.redhat.com/browse/AAP-42623

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/sqs/client/get_queue_url.html

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/sqs/client/receive_message.html

hsong-rh
hsong-rh previously approved these changes Apr 14, 2025
Comment thread extensions/eda/plugins/event_source/aws_sqs_queue.py Outdated
bzwei
bzwei previously approved these changes Apr 14, 2025
Comment thread extensions/eda/plugins/event_source/aws_sqs_queue.py Outdated
Comment thread extensions/eda/plugins/event_source/aws_sqs_queue.py Outdated
Comment thread extensions/eda/plugins/event_source/aws_sqs_queue.py Outdated
Comment thread extensions/eda/plugins/event_source/aws_sqs_queue.py Outdated
Based on feedback we received we are missing some key parameters
from sqs queue processing. Adding additional optional parameters

* queue_owner_aws_account_id
* max_number_of_messages
@github-project-automation github-project-automation Bot moved this from In Progress to Review in 🧰 devtools project board Apr 16, 2025
@mkanoor mkanoor merged commit 711e0b5 into ansible:main Apr 16, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants