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

Bug: Claudie doesn't allow nodes from two different AWS provider instances in the same region #1312

Closed
bernardhalas opened this issue Apr 2, 2024 · 1 comment · Fixed by #1348
Labels
bug Something isn't working groomed Task that everybody agrees to pass the gatekeeper

Comments

@bernardhalas
Copy link
Member

bernardhalas commented Apr 2, 2024

Current Behaviour

Currently, Claudie doesn't allow the following configuration:

spec:
  providers:
    - name: aws-team1
      providerType: aws
...
    - name: aws-team2
      providerType: aws
...
  nodePools:
    dynamic:
      - name: control-aws
        providerSpec:
          name: aws-team1
          region: eu-west-1
      - name: compute-aws
        providerSpec:
          name: aws-team2
          region: eu-west-1

Expected Behaviour

Sometimes, the users might want to deploy nodepools in the same region, by two different providers (e.g. for billing, reporting or security issues).

Anything else to note

Currently, I see the options to set provider.alias to e.g.:

  • nodepool_{{ $provider.name }}_{{ $provider.region }}
  • nodepool_{{ substr(sha256($provider.name + $provider.region), 0, 4) }}
  • nodepool_{{ $random }}

References

@bernardhalas bernardhalas added the bug Something isn't working label Apr 2, 2024
@JKBGIT1 JKBGIT1 added the good first issue Good for newcomers label Apr 5, 2024
@JKBGIT1
Copy link
Contributor

JKBGIT1 commented Apr 5, 2024

Depends on #1296

@JKBGIT1 JKBGIT1 added the groomed Task that everybody agrees to pass the gatekeeper label Apr 5, 2024
@Despire Despire removed the good first issue Good for newcomers label Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working groomed Task that everybody agrees to pass the gatekeeper
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants