Skip to content

feat: load balanced web service observability#3451

Merged
mergify[bot] merged 15 commits intoaws:mainlinefrom
dannyrandall:feat/lb-ws-observability
Apr 18, 2022
Merged

feat: load balanced web service observability#3451
mergify[bot] merged 15 commits intoaws:mainlinefrom
dannyrandall:feat/lb-ws-observability

Conversation

@dannyrandall
Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.

@dannyrandall dannyrandall requested a review from a team as a code owner April 11, 2022 21:20
@dannyrandall dannyrandall removed the request for review from a team April 11, 2022 21:20
@dannyrandall dannyrandall requested a review from Lou1415926 April 11, 2022 21:20
Copy link
Copy Markdown
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

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

Looks great, thanks Danny!

Comment thread internal/pkg/template/templates/workloads/partials/cf/taskrole.yml Outdated
Comment thread internal/pkg/template/templates/workloads/partials/cf/taskrole.yml Outdated
@efekarakus efekarakus added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 11, 2022
efekarakus
efekarakus previously approved these changes Apr 11, 2022
Copy link
Copy Markdown
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢

Just added the label to validate with @Lou1415926 if that image is consistent with app runner, after that feel free to remove it

@dannyrandall dannyrandall marked this pull request as draft April 11, 2022 23:12
@huanjani huanjani removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 12, 2022
@dannyrandall dannyrandall marked this pull request as ready for review April 13, 2022 21:55
@dannyrandall dannyrandall requested a review from efekarakus April 13, 2022 21:55
@dannyrandall dannyrandall dismissed efekarakus’s stale review April 13, 2022 21:56

changed PR substantially after initial approval

Comment thread internal/pkg/template/templates/workloads/partials/cf/sidecars.yml
Comment thread internal/pkg/template/templates/workloads/partials/cf/taskrole.yml
Copy link
Copy Markdown
Contributor

@iamhopaul123 iamhopaul123 left a comment

Choose a reason for hiding this comment

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

Maybe after this we could add it to svc show like what WX does for rdws 🤔

@mergify mergify Bot merged commit a2449f5 into aws:mainline Apr 18, 2022
@dannyrandall dannyrandall deleted the feat/lb-ws-observability branch April 18, 2022 17:54
mergify Bot pushed a commit that referenced this pull request May 5, 2022
Docs follow up from: #3481, #3451

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants