Skip to content

Backport of docs: Remove YAML service registration examples into release/1.16.x#18906

Merged
hc-github-team-consul-core merged 1 commit into
release/1.16.xfrom
backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole
Sep 20, 2023
Merged

Backport of docs: Remove YAML service registration examples into release/1.16.x#18906
hc-github-team-consul-core merged 1 commit into
release/1.16.xfrom
backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole

Conversation

@hc-github-team-consul-core
Copy link
Copy Markdown
Collaborator

Backport

This PR is auto-generated from #18877 to be assessed for backporting due to the inclusion of the label backport/1.16.

The below text is copied from the body of the original PR.


Description

Remove YAML service registration examples and replace them with JSON. This is because YAML is not a supported configuration format for the Consul's agent configuration, nor is it supported by the HTTP API.

This commit replaces the YAML examples with JSON and adds additional JSON examples where they were missing.

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

Overview of commits

@hc-github-team-consul-core hc-github-team-consul-core requested a review from a team as a code owner September 19, 2023 18:41
@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch from c38b9bf to 2bd90bd Compare September 19, 2023 18:41
@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch from 0af2dcf to cfb031f Compare September 19, 2023 18:41
@github-actions github-actions Bot added the type/docs Documentation needs to be created/updated/clarified label Sep 19, 2023
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto approved Consul Bot automated PR

@vercel vercel Bot temporarily deployed to Preview – consul September 19, 2023 18:52 Inactive
@blake blake force-pushed the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch from cfb031f to fe7cc84 Compare September 19, 2023 19:22
@blake blake force-pushed the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch from fe7cc84 to 25266dc Compare September 20, 2023 18:31
@blake blake added the pr/no-changelog PR does not need a corresponding .changelog entry label Sep 20, 2023
@blake blake force-pushed the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch from 25266dc to 3a4616a Compare September 20, 2023 23:04
@hc-github-team-consul-core hc-github-team-consul-core merged commit 750317c into release/1.16.x Sep 20, 2023
@hc-github-team-consul-core hc-github-team-consul-core deleted the backport/docs/remove-k8s-yaml-service-registration/nominally-assuring-mole branch September 20, 2023 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-changelog PR does not need a corresponding .changelog entry type/docs Documentation needs to be created/updated/clarified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants