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

Changes to adopt Consul Dataplane architecture for Consul ECS #199

Merged
merged 49 commits into from
Sep 6, 2023

Conversation

Ganeshrockz
Copy link
Contributor

Changes proposed in this PR:

How I've tested this PR:

CI, Manual deployment

How I expect reviewers to test this PR:

Most of these files were already reviewed as part of previous PRs. I would like some 👀 on the CHANGELOG because that is the only major addition to the feature branch after all the other changes.

Checklist:

  • Tests added
  • CHANGELOG entry added

@Ganeshrockz Ganeshrockz requested review from a team, sarahethompson, dlaguerta, kisunji and cthain and removed request for a team August 24, 2023 06:53
Copy link
Contributor

@cthain cthain left a comment

Choose a reason for hiding this comment

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

Just a few minor comments on the CHANGELOG and README.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Ganeshrockz Ganeshrockz requested a review from cthain August 30, 2023 17:25
Copy link
Contributor

@cthain cthain left a comment

Choose a reason for hiding this comment

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

Reviewed the CHANGELOG and README 👍

All other files were reviewed in the PRs mentioned in the changelog.

* Service sameness example config

* Added Changelog

* Export mesh gateways

* Address comments
…corp/terraform-aws-consul-ecs into ganeshrockz/dataplane-arch-changes
@Ganeshrockz Ganeshrockz merged commit e9ff3b8 into main Sep 6, 2023
@Ganeshrockz Ganeshrockz deleted the ganeshrockz/dataplane-arch-changes branch September 6, 2023 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants