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

docs: [ovh sd] Added missing label for OVH dedicated server in SD #13970

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jiekun
Copy link

@jiekun jiekun commented Apr 23, 2024

__meta_ovhcloud_dedicated_server_no_intervention label is available for OVH Cloud dedicated server. But it's missing in doc.

See:

dedicatedServerLabelPrefix + "no_intervention": model.LabelValue(strconv.FormatBool(server.NoIntervention)),

And previous test case:

"__meta_ovhcloud_dedicated_server_no_intervention": "false",

@jiekun jiekun force-pushed the doc/ovh-dedicated-server-label branch from da1b7a1 to 4988ac3 Compare April 23, 2024 02:07
@jiekun
Copy link
Author

jiekun commented Apr 26, 2024

@machine424 @roidelapluie Hi May I have your review of this PR? Thanks!

@@ -1467,6 +1467,7 @@ For OVHcloud's [public cloud instances](https://www.ovhcloud.com/en/public-cloud
* `__meta_ovhcloud_dedicated_server_ipv6`: the IPv6 of the server
* `__meta_ovhcloud_dedicated_server_link_speed`: the link speed of the server
* `__meta_ovhcloud_dedicated_server_name`: the name of the server
* `__meta_ovhcloud_dedicated_server_no_intervention`: the [intervention](https://support.us.ovhcloud.com/hc/en-us/articles/27991435200147-FAQ-Interventions-and-Hardware-Replacement) of the server.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we should precise whether no_intervention means:

  • interventions are disabled on the server.
    OR
  • there is no interventions currently on the server.

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for the review and sorry for my late reply, I was out of the office yesterday.

According to OVH's document, noIntervation refer to:

  • Prevent datacenter intervention.

So in this case, it means "interventions are disabled on the server". And I'd like to suggest using the following description:

  • is this server immune to datacenter intervention.

WDYT @machine424

Copy link
Collaborator

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

thanks.

@machine424
Copy link
Collaborator

For the failing DCO job:

Thanks for your PR. For legal reasons, we require that all commits are signed with a DCO before we can merge them. See this blog post for considerations around this.

This means that the last line of your commit message should read like:

Signed-Off-By: Your Name <your@email.address>

If you are using GitHub through the web interface, it's quickest to close this PR and open a new one with the appropriate line.

If you are using Git on the command line, it is probably quickest to amend and force push. You can do that with

git commit --amend --signoff
git push -f $remote $remote_branch_for_pr

As always, be careful when force-pushing.

Originally posted by @roidelapluie

@jiekun jiekun force-pushed the doc/ovh-dedicated-server-label branch from 0ed116c to 450661d Compare April 30, 2024 09:32
…e discovery doc

Signed-off-by: Jiekun <zhujiekun@52tt.com>
@jiekun jiekun force-pushed the doc/ovh-dedicated-server-label branch from 450661d to 0cd3a22 Compare April 30, 2024 09:35
@jiekun
Copy link
Author

jiekun commented Apr 30, 2024

@machine424 Thanks again. I just fixed it. And sorry for the trouble.

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.

None yet

2 participants