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

feat(clouds): add initial implementation for clouds client #593

Merged

Conversation

anvial
Copy link
Member

@anvial anvial commented Sep 25, 2024

Description

This PR:

  • Introduced cloudsClient struct with methods for creating, reading, updating, and destroying clouds.
  • Added input and output structs for CreateCloud, ReadCloud, UpdateCloud, and DestroyCloud operations.
  • Integrated cloudsClient with SharedClient for shared functionality.

This change sets up the basic structure for managing clouds within the Juju client.

Type of change

  • Add new resource

QA steps

# just build terraform plugin

@anvial anvial requested a review from hmlanigan September 25, 2024 09:55
@anvial anvial force-pushed the JUJU-6547-prepare-schema-form-cloud-resource branch from 249f5a7 to b92dcc9 Compare September 25, 2024 09:55
- Introduced `cloudsClient` struct with methods for creating, reading, updating, and destroying
  clouds.
- Added input and output structs for `CreateCloud`, `ReadCloud`, `UpdateCloud`, and `DestroyCloud`
  operations.
- Integrated `cloudsClient` with `SharedClient` for shared functionality.

This change sets up the basic structure for managing clouds within the Juju client.
@anvial anvial force-pushed the JUJU-6547-prepare-schema-form-cloud-resource branch from b92dcc9 to e8ac402 Compare September 25, 2024 09:58
internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
internal/provider/resource_kubernetes_cloud_test.go Outdated Show resolved Hide resolved
internal/juju/clouds_test.go Outdated Show resolved Hide resolved
internal/juju/clouds.go Outdated Show resolved Hide resolved
Update internal/provider/resource_kubernetes_cloud.go

chore: extend cloud name description

Co-authored-by: hmlanigan <[email protected]>

Update internal/provider/resource_kubernetes_cloud.go

chore: extend parent cloud name description

Co-authored-by: hmlanigan <[email protected]>

Update internal/provider/resource_kubernetes_cloud.go

chore: extend parent cloud region description

Co-authored-by: hmlanigan <[email protected]>

chore: added method comment headers

chore: added "id" to the schema to handle imports

chore: delete test file (until it's used)

chore: delete clouds_test unit it's used

chore: remane cloudsClient --> kubernetesCloudClient
@anvial anvial force-pushed the JUJU-6547-prepare-schema-form-cloud-resource branch from a61410c to c7d2617 Compare September 27, 2024 08:15
@anvial anvial requested a review from hmlanigan September 27, 2024 08:16
Copy link
Member

@hmlanigan hmlanigan left a comment

Choose a reason for hiding this comment

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

LGTM - a couple of small comments and linter issues to fix.

internal/provider/resource_kubernetes_cloud.go Outdated Show resolved Hide resolved
@Aflynn50 Aflynn50 self-requested a review October 1, 2024 11:43
@anvial anvial merged commit 1fcd5cb into juju:main Oct 2, 2024
24 of 30 checks passed
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