Skip to content

feat: AWS Elemental MediaConnect L2 design#885

Merged
alvazjor merged 6 commits intoaws:mainfrom
jamiepmullan:feat/mediaconnect-l2-design
Apr 24, 2026
Merged

feat: AWS Elemental MediaConnect L2 design#885
alvazjor merged 6 commits intoaws:mainfrom
jamiepmullan:feat/mediaconnect-l2-design

Conversation

@jamiepmullan
Copy link
Copy Markdown
Contributor

@jamiepmullan jamiepmullan commented Mar 19, 2026

This is a request for comments about AWS Elemental MediaConnect L2 design. See #884 for
additional details.

APIs are signed off by @alvazjor


By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache-2.0 license

@alvazjor alvazjor self-assigned this Apr 9, 2026
Copy link
Copy Markdown
Contributor

@alvazjor alvazjor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two things that were flagged:

  1. There are several actions avaiable for media connect Flow resource: https://docs.aws.amazon.com/service-authorization/latest/reference/list_awselementalmediaconnect.html, yet there is no grants facade defined. Are they not required? what is the justification to not have it?
  2. Bridge and Router are missing fromXXXAttributes or fromXXXArn import methods. Is this on purpose?

Comment thread text/0884-aws-elemental-mediaconnect-l2.md
Comment thread text/0884-aws-elemental-mediaconnect-l2.md Outdated
Comment thread text/0884-aws-elemental-mediaconnect-l2.md Outdated
Comment thread text/0884-aws-elemental-mediaconnect-l2.md Outdated
Comment thread text/0884-aws-elemental-mediaconnect-l2.md
Comment thread text/0884-aws-elemental-mediaconnect-l2.md
Comment thread text/0884-aws-elemental-mediaconnect-l2.md
Comment thread text/0884-aws-elemental-mediaconnect-l2.md
@jamiepmullan jamiepmullan requested a review from alvazjor April 16, 2026 13:22
@alvazjor alvazjor added status/api-approved API Bar Raiser signed-off the API of this RFC pr/do-not-merge Let mergify know not to auto merge labels Apr 17, 2026
@alvazjor
Copy link
Copy Markdown
Contributor

Signing off the RFC. Will keep it open for the final comments period. If no additional comments are added, this can be merged on April 24th

@alvazjor
Copy link
Copy Markdown
Contributor

Final comments period is over. Closing this one

@alvazjor alvazjor removed the pr/do-not-merge Let mergify know not to auto merge label Apr 24, 2026
@alvazjor
Copy link
Copy Markdown
Contributor

@Mergifyio refresh

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 24, 2026

refresh

✅ Pull request refreshed

@alvazjor
Copy link
Copy Markdown
Contributor

@Mergifyio queue

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 24, 2026

Merge Queue Status

This pull request spent 20 seconds in the queue, including 3 seconds running CI.

Waiting for:
  • #commented-reviews-by=0
  • any of:
    • check-neutral = markdownlint
    • check-skipped = markdownlint
    • check-success = markdownlint
All conditions

Reason

Pull request #885 has been dequeued

merge conditions no longer match:

  • #commented-reviews-by=0
  • any of [🛡 GitHub branch protection]:
    • check-neutral = markdownlint
    • check-skipped = markdownlint
    • check-success = markdownlint

Hint

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

@mergify mergify Bot added the queued label Apr 24, 2026
mergify Bot added a commit that referenced this pull request Apr 24, 2026
@mergify mergify Bot added dequeued and removed queued labels Apr 24, 2026
@alvazjor alvazjor merged commit 04f22f8 into aws:main Apr 24, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dequeued status/api-approved API Bar Raiser signed-off the API of this RFC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants