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

Update Travis automation to use current "metal-cli", avoid deprecated "facility" API #680

Open
vielmetti opened this issue Mar 28, 2024 · 1 comment

Comments

@vielmetti
Copy link

Description

(Describe the feature, bug, question, proposal that you are requesting)

On the Equinix Metal (formerly Packet) platform, it was observed that automation that
relies on our deprecated "facilities" feature was failing, resulting in Travis nodes not
being torn down at end of use.

Importance

(Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have))

Undeleted nodes are accumulating credits, which are part of a finite CNCF credit allocation.
In addition at some extreme if machines are being turned on without ever being turned off and
sit idle for a long time it is not a sustainable situation.

Location

(Where is the piece of code, package, or document affected by this issue?)

Code identified by the search for "packet-cli" command

https://github.com/search?q=repo%3Apravega%2Fpravega-operator%20packet-cli&type=code

Suggestions for an improvement

(How do you suggest to fix or proceed with this issue?)

  1. Replace old "packet-cli" code with current supported "metal-cli" https://github.com/equinix/metal-cli
  2. Replace "facility" API with "metro" API https://deploy.equinix.com/developers/docs/metal/locations/facilities/
  3. Explore using a different machine type / location that might have similar performance + lower cost and impact

Thanks @RaulGracia for the assistance.

@vielmetti
Copy link
Author

The "facility" API has been turned off, and so the old automation based on the "packet-cli" code will no longer work; you should get a 422 error.

Let me know if you want to schedule a time to review; I am available.

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

No branches or pull requests

1 participant