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

oss compoments for multi-vault namespaces #8453

Merged
merged 1 commit into from
Jul 27, 2020
Merged

Conversation

drewbailey
Copy link
Contributor

Adds oss components to support enterprise multi-vault namespace feature

@@ -101,6 +101,11 @@ Run Options:
the job file. This overrides the token found in $VAULT_TOKEN environment
variable and that found in the job.

-vault-namespace
If set, the passed Vault namespace is stored in the job before sending to the
Nomad servers. This overrides the namespace found in $VAULT_NAMESPACE environment
Copy link
Member

Choose a reason for hiding this comment

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

I just realized this is a bit of a backwards incompatibility:

Before Nomad ignored this environment variable and would have used the agent-configured Vault namespace.

Now the operator's environment will override the agent-configuration.

I could see this being an issue for CI/CD environments which may use a distinct Vault namespace from the app being deployed.

However, VAULT_TOKEN is already prior art for nomad run's environment to be app specific.

To be safe, let's call out this change in the Version Specific Upgrade docs.

helper/funcs.go Outdated Show resolved Hide resolved
helper/funcs_test.go Outdated Show resolved Hide resolved
nomad/job_endpoint_oss.go Outdated Show resolved Hide resolved
@drewbailey drewbailey force-pushed the oss-multi-vault-ns branch 2 times, most recently from 3d287a7 to 9e9f7c2 Compare July 21, 2020 16:46
adds in oss components to support enterprise multi-vault namespace feature

upgrade specific doc on vault multi-namespaces

vault docs

update test to reflect new error
@drewbailey drewbailey merged commit 940817a into master Jul 27, 2020
@drewbailey drewbailey deleted the oss-multi-vault-ns branch July 27, 2020 12:45
greut added a commit to greut/nomad that referenced this pull request Sep 5, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
greut added a commit to greut/nomad that referenced this pull request Sep 5, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
greut added a commit to greut/nomad that referenced this pull request Sep 6, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
greut added a commit to greut/nomad that referenced this pull request Sep 8, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
tgross pushed a commit that referenced this pull request Sep 9, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
fredrikhgrelland pushed a commit to fredrikhgrelland/nomad that referenced this pull request Sep 28, 2020
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants