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

Namespace Specific Resources Should Not Require Namespaces in Their Names #142

Open
devinlyons opened this issue Jun 14, 2023 · 0 comments · May be fixed by #143
Open

Namespace Specific Resources Should Not Require Namespaces in Their Names #142

devinlyons opened this issue Jun 14, 2023 · 0 comments · May be fixed by #143

Comments

@devinlyons
Copy link

devinlyons commented Jun 14, 2023

Chef Version

$ chef-client --version
Chef: 17.10.3

Input

sensu_secret 'prod_foo' do
    id '/secrets/prod/foo'
    namespace 'prod'
    secrets_provider 'vault'
end

sensu_secret 'test_foo' do
    id '/secrets/test/foo'
    namespace 'test'
    secrets_provider 'vault'
end

Output

This creates a secret called prod_foo in the namespace prod and test_foo in the namespace test.

Impact

This creates a more complicated environment where every asset in Sensu has to be aware of its own namespace.

Expected Behavior

It would be better if they could both be called foo. However, that is impossible because every resource in Chef must have a unique name and that name is used as the name of the resource in Sensu.

sensu_secret 'prod_foo' do
    id '/secrets/prod/foo'
    namespace 'prod'
    secrets_provider 'vault'
    secret_name 'foo'
end

sensu_secret 'test_foo' do
    id '/secrets/prod/foo'
    namespace 'prod'
    secrets_provider 'vault'
    secret_name 'foo'
end

Actual Behavior

As described above.

Steps to Reproduce your problem

As described above.

@devinlyons devinlyons changed the title Assets With the Same Name in Different Namespaces Do Not Work Namespace Specific Resources Should Not Require Namespaces in Their Names Jun 15, 2023
@devinlyons devinlyons linked a pull request Jun 15, 2023 that will close this issue
7 tasks
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 a pull request may close this issue.

1 participant