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

simplify monitor address setting #7479

Merged
merged 2 commits into from
Mar 8, 2024
Merged

simplify monitor address setting #7479

merged 2 commits into from
Mar 8, 2024

Conversation

guits
Copy link
Collaborator

@guits guits commented Feb 16, 2024

this drops the following parameters:

  • monitor_address_block
  • monitor_interface
  • monitor_address

The monitor address will be automatically set from public_network parameter.

@guits guits force-pushed the public_network branch 2 times, most recently from 68f8d0c to 9247b1c Compare March 8, 2024 07:07
@guits guits changed the title simplify monitor address setting [skip ci] simplify monitor address setting Mar 8, 2024
@guits guits changed the title [skip ci] simplify monitor address setting simplify monitor address setting Mar 8, 2024
@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-non_container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-non_container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-non_container-lvm_batch

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-container-lvm_batch

@guits
Copy link
Collaborator Author

guits commented Mar 8, 2024

jenkins test centos-non_container-switch_to_containers

this drops the following parameters:

- monitor_address_block
- monitor_interface
- monitor_address

The monitor address will be automatically set from `public_network` parameter.

Signed-off-by: Guillaume Abrioux <[email protected]>
2b72ea9 and
c8eeae2 have missed the resync.

This addresses that.

Signed-off-by: Guillaume Abrioux <[email protected]>
@guits guits merged commit aeba991 into main Mar 8, 2024
27 checks passed
@guits guits deleted the public_network branch March 8, 2024 12:02
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.

2 participants