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

config entry: hardcode proxy-defaults name as global #14833

Merged
merged 2 commits into from
Oct 25, 2022

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Oct 2, 2022

proxy-defaults can only have the name global. Because of this, we support not even setting the name in the config file:

kind = "proxy-defaults"

Previously, writing this would result in the output:

Config entry written: proxy-defaults/

Now it will output:

Config entry written: proxy-defaults/global

This change follows what was done for the new Mesh config entry.

Fixes #10809

@github-actions github-actions bot added theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface labels Oct 2, 2022
lkysow added 2 commits October 3, 2022 15:23
proxy-defaults can only have the name global. Because of this,
we support not even setting the name in the config file:

```
kind = "proxy-defaults"
```

Previously, writing this would result in the output:

```
Config entry written: proxy-defaults/
```

Now it will output:

```
Config entry written: proxy-defaults/global
```

This change follows what was done for the new Mesh config entry.
@lkysow lkysow force-pushed the lkysow/proxy-defaults-cli branch from 5d90421 to 52bad1c Compare October 3, 2022 22:24
@lkysow lkysow marked this pull request as ready for review October 3, 2022 22:24
@lkysow lkysow requested a review from a team October 3, 2022 23:58
Copy link
Contributor

@ishustava ishustava left a comment

Choose a reason for hiding this comment

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

❤️

@lkysow lkysow merged commit fbd47e1 into main Oct 25, 2022
@lkysow lkysow deleted the lkysow/proxy-defaults-cli branch October 25, 2022 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

If you write a proxy-defaults without name set the success message is confusing
2 participants