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

network: Add nmstate examples #531

Closed
wants to merge 1 commit into from

Conversation

qinqon
Copy link

@qinqon qinqon commented Apr 18, 2023

Now that nmstate is part of fcos the network configuration examples should be extended with it.

Depends on:

Open as implementation of:

@qinqon
Copy link
Author

qinqon commented Apr 18, 2023

/cc @bgilbert

Now that nmstate is part of fcos the network configuration examples
should be extended with it.

Signed-off-by: Enrique Llorente <ellorent@redhat.com>
@qinqon qinqon force-pushed the add-nmstate-network-configuration branch from 6f0583e to bfab777 Compare April 18, 2023 13:51
@dustymabe
Copy link
Member

Thanks for opening this. I want to discuss the implications of this amongst the team in our meeting tomorrow.

- path: /etc/nmstate/ens2.yml
mode: 0600
contents:
inline: |

Choose a reason for hiding this comment

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

I think the original example are expecting a static hostname. So we can state here in nmstate like:

hostname:
  config: myhost

@dustymabe
Copy link
Member

Thanks for opening this. I want to discuss the implications of this amongst the team in our meeting tomorrow.

Finally got around to discussing this. Results over in coreos/fedora-coreos-tracker#1175 (comment). I'll look at this and #536 tomorrow.

@qinqon
Copy link
Author

qinqon commented May 4, 2023

I will close this since @cathay4t is taking over the task, thanks @dustymabe

@qinqon qinqon closed this May 4, 2023
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 this pull request may close these issues.

3 participants