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

Allow selecting the service's ClusterIP #375

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

remram44
Copy link
Contributor

This is mostly useful to be able to set it to "None", creating a headless service that can be scraped using DNS discovery (dns_sd_configs).

No change in default behavior.

@remram44
Copy link
Contributor Author

Bump

@glowkey
Copy link
Collaborator

glowkey commented Oct 23, 2024

The commit needs to be signed (git commit -S) before it can be merged.

@remram44
Copy link
Contributor Author

The commit is signed-off (git commit -s) as requested by your contributing guide.

@glowkey
Copy link
Collaborator

glowkey commented Oct 30, 2024

Sorry for the confusion. The contribution guide needs to be updated as the repository rules prevent us from merging commits that have not been signed with "git commit -S".
rules

This is mostly useful to be able to set it to "None", creating a
headless service that can be scraped using DNS discovery
(dns_sd_configs).

Signed-off-by: Remi Rampin <[email protected]>
@remram44
Copy link
Contributor Author

Please fix your contributing guide ASAP, a guide with the wrong instructions is worse than no guide at all.

I signed the commit so you can be sure the owner of this GitHub account is the owner of this GitHub account... really confused as to what this does for security, but here you go.

@glowkey glowkey merged commit 0a543e1 into NVIDIA:main Oct 30, 2024
1 check passed
@glowkey
Copy link
Collaborator

glowkey commented Oct 30, 2024

We've asked the admins to change this rule to make future contributions easier. Thanks for your help!

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