Skip to content
This repository has been archived by the owner on Dec 23, 2024. It is now read-only.

feat: API Sync by GitHub Action (2024-03-01) #87

Merged
merged 4 commits into from
Mar 6, 2024
Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Mar 1, 2024

This API Sync PR was automated through GitHub Actions workflow_displatch
on 2024-03-01.

  • latest Equinix Metal API spec have been fetched
  • patches have been applied
  • generated client has been updated

Closes #86

@displague
Copy link
Member

displague commented Mar 4, 2024

https://github.com/equinix-labs/metal-python/actions/runs/8112298204/job/22173240639

TASK [metal_connection : create connection for test] ***************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was:   str type expected (type=type_error.str)
fatal: [testhost]: FAILED! => {"changed": false, "msg": "Error in metal_connection: 1 validation error for DedicatedPortCreateInput\nspeed\n  str type expected (type=type_error.str)"}

(Same error on the last sync attempt: #86 (comment))

@ctreatma
Copy link
Contributor

ctreatma commented Mar 5, 2024

I think this is a case where the Ansible tests are correct and so is the SDK update. We should move forward with this sync / release, and address the type mismatch in Ansible as part of pulling in the updated SDK.

@t0mk t0mk merged commit d3077de into main Mar 6, 2024
2 of 3 checks passed
@t0mk t0mk deleted the sync/gh-1709298300 branch March 6, 2024 15:40
Copy link
Contributor Author

github-actions bot commented Mar 6, 2024

This PR is included in version 0.8.0 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants