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.

Sign up for GitHub

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

Jump to bottom

Warn if value is overriden both in secrets and the environment #28

Closed
frlan opened this issue Feb 17, 2020 · 0 comments · Fixed by #190
Closed

Warn if value is overriden both in secrets and the environment #28

frlan opened this issue Feb 17, 2020 · 0 comments · Fixed by #190
Assignees
@icemac
Labels
enhancement New feature or request gocept
Milestone
batou 2.3

Comments

@frlan
Copy link
Member

frlan commented Feb 17, 2020

Scenario:

A overwrite is done inside environments/myenv.cfg:

[component:example]
myvar = True

as well as in secrets secrets/myenv.cfg:


[component:example]
myvar = False

Obviously the overwrites do conflict.

Expectation

Batou should discover this scenario and warn the user
@ctheune ctheune changed the title Warn at multiple overwrites Warn if value is overriden both in secrets and the environment Apr 16, 2020
@ctheune ctheune added the enhancement New feature or request label Apr 16, 2020
@ctheune ctheune modified the milestones: batou 2, batou 2.1 May 7, 2020
@ctheune ctheune self-assigned this Nov 3, 2020
@ctheune ctheune modified the milestones: batou 2.1, batou 2.3 Dec 7, 2020
@ctheune ctheune added the gocept label May 26, 2021
This was referenced Jun 18, 2021
Merged
Merged
@icemac icemac assigned sallner and icemac and unassigned ctheune and sallner Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@icemac icemac

Labels
enhancement New feature or request gocept
Projects
None yet
Milestone
batou 2.3
Development

Successfully merging a pull request may close this issue.

Fail if an attribute is set both in environment and via secrets.
4 participants
@icemac @frlan @ctheune @sallner