feat: module metal_plan_info #146
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
had a problem deploying
to
external
GitHub Actions
Failure
ctreatma
added a commit
to equinix-labs/metal-python
that referenced
this pull request
Dec 21, 2023
This PR updated package version to v0.5.0. We do a release after this PR and later we update metal-python version in Ansible collection. We need this in for equinix/ansible-collection-equinix#146 to work.
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
t0mk
suggested changes
Dec 22, 2023
|
the issue with Plans listing was fixed in metal-python and vesion of metal-python in the collection was updated. Please rebase this PR on current main. |
|
This is blocked by equinix-labs/metal-python#70 We will need to get that PR in and do a release befre we merge this. |
antoninrykalsky
force-pushed
the
feat/136-metal-plans
branch
from
979f764 to
2423b58
Compare
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
added 10 commits
February 19, 2024 15:10
…t mutually exclusive
antoninrykalsky
force-pushed
the
feat/136-metal-plans
branch
from
1078f9a to
f67f803
Compare
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
t0mk
suggested changes
Feb 22, 2024
Co-authored-by: Tomáš Karásek <[email protected]>
antoninrykalsky
had a problem deploying
to
external
GitHub Actions
Failure
feat: add metal_plans_info.py Co-authored-by: Tomáš Karásek <[email protected]>
antoninrykalsky
temporarily deployed
to
external
GitHub Actions
Inactive
t0mk
approved these changes
Feb 22, 2024
|
This PR is included in version 0.4.0 🎉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ansible wrapper of
https://github.com/equinix-labs/metal-python/blob/main/equinix_metal/equinix_metal/api/plans_api.py#L50
which is a wrapper of:
https://deploy.equinix.com/developers/api/metal/#tag/Plans/operation/findPlans
Temporary as draft, until i find a way to make category, type, include and exclude filters running