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

feat(neighbor): support announce all processes #1180

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

ahmet2mir
Copy link
Contributor

Hi,

As discused on slack, currently there is no way to tell neighbours to announce all defined processes.

Adding "all" inside api.processes allow announcing all of them.

@ahmet2mir ahmet2mir force-pushed the feat/processall branch 3 times, most recently from 43e16fc to 4e54fd4 Compare September 27, 2023 19:00
@thomas-mangin
Copy link
Member

Good idea and the only issue with with patch would be the missing second run file. That said, I would rather have a new keyword “processes-matching” taking a regex expression which would be used to decide which process names to include and add a alias for processes called processes-named perhaps too

@thomas-mangin thomas-mangin merged commit d7e6fef into Exa-Networks:main Sep 29, 2023
5 checks passed
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