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

[on-prem] Node group names don't support dashes (-) #350

Open
ralgozino opened this issue Feb 3, 2025 · 0 comments
Open

[on-prem] Node group names don't support dashes (-) #350

ralgozino opened this issue Feb 3, 2025 · 0 comments
Assignees
Labels
size/S ~1d | Epic:~10d

Comments

@ralgozino
Copy link
Member

ralgozino commented Feb 3, 2025

Describe the bug

Using a dash (hyphen) - the name of a node group, for example:

spec:
  kubernetes:
    nodes:
      - name: ingress-external
        hosts:
          ...

results in the following Ansible error:

[WARNING]: Invalid characters were found in group names but not replaced, use -vvvv to see details

To Reproduce

Steps to reproduce the behavior:

  1. Create a furyctl.yaml
  2. Add a node group with the name containing a dash -.
  3. Apply the configuration
  4. See error

Expected behavior

A think using - should be supported, it is a very common use case. Maybe we should replace the dashes for underscores in the hosts.yaml template file.

If we can't support them, we should update the schema to fail at validation-time instead of at apply-time.

Software versions

Please complete the following information:

  • Kubernetes Fury Distribution version: 1.31.0
  • furyctl version: 0.31.0

Additional context

This seems to be a recurrent error for Ansible users too.

Note

The node group name that you set in the furyctl.yaml file is also used to determine the nodes role.

@ralgozino ralgozino changed the title [on-prem] Node group names don't support - [on-prem] Node group names don't support dashes (-) Feb 4, 2025
@nutellinoit nutellinoit added the size/S ~1d | Epic:~10d label Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S ~1d | Epic:~10d
Projects
None yet
Development

No branches or pull requests

3 participants