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

Rename to exported services #902

Merged
merged 10 commits into from
Dec 8, 2021
Merged

Conversation

thisisnotashwin
Copy link
Contributor

@thisisnotashwin thisisnotashwin commented Dec 6, 2021

Changes proposed in this PR:

  • Update the partition-exports config entry to exported-services.
  • Delete the "autogenerated" config created by operator SDK that does not get regenerated. Most of the configs were stale and this will reduce the chaos.
  • Update the API module to the latest and update AdminPartitions to Partitions where necessary.
  • Create anonymous policy only in the default partition.

How I've tested this PR:

  • Acceptance tests.

How I expect reviewers to test this PR:

  • 👀

Checklist:

  • Tests added
  • CHANGELOG entry added

    HashiCorp engineers only, community PRs should not add a changelog entry.
    Entries should use present tense (e.g. Add support for...)

@thisisnotashwin thisisnotashwin force-pushed the rename-to-exported-services branch 6 times, most recently from 3f7c5eb to c90e830 Compare December 7, 2021 15:11
@@ -70,7 +70,7 @@ commands:
type: string
consul-k8s-image:
type: string
default: "docker.mirror.hashicorp.services/hashicorpdev/consul-k8s-control-plane:latest"
default: "ashwinvenkatesh/consul-k8s@sha256:ca2bde89a28563d40100039c217df6f750cc8b3bd72b31036bb1aaaff74f23fc"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove this before merging to master.

@thisisnotashwin thisisnotashwin requested review from a team, ndhanushkodi and t-eckert and removed request for a team December 7, 2021 20:41
@thisisnotashwin thisisnotashwin marked this pull request as ready for review December 7, 2021 20:41
@thisisnotashwin thisisnotashwin requested review from lkysow and removed request for ndhanushkodi December 7, 2021 20:42
CHANGELOG.md Outdated Show resolved Hide resolved
@thisisnotashwin thisisnotashwin force-pushed the rename-to-exported-services branch from 1c0de55 to c7782a8 Compare December 7, 2021 20:56
@thisisnotashwin thisisnotashwin force-pushed the rename-to-exported-services branch from c7782a8 to b45f341 Compare December 7, 2021 21:52
@thisisnotashwin thisisnotashwin force-pushed the rename-to-exported-services branch from b45f341 to ed81558 Compare December 8, 2021 15:58
Ashwin Venkatesh and others added 2 commits December 8, 2021 12:28
Co-authored-by: Luke Kysow <1034429+lkysow@users.noreply.github.com>
@thisisnotashwin thisisnotashwin merged commit 6408674 into main Dec 8, 2021
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.

3 participants