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

ResourceController service update #196

Merged
merged 5 commits into from
Jul 27, 2022
Merged

ResourceController service update #196

merged 5 commits into from
Jul 27, 2022

Conversation

apoorva9s14
Copy link
Contributor

Signed-off-by: apoorva9s14 [email protected]

PR summary

For the resource controller service update as per this PR - https://github.ibm.com/cloud-api-docs/resource-controller/pull/244, resource-controller-v2.yaml is the updated yaml, I have regenerated the SDK files with openapi-sdkgen.
We are looking to address:

Looping - https://github.ibm.com/cloud-api-docs/resource-controller/issues/226
Credentials redaction

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • [ Yes] The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • [ Yes] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

The examples and the SDK code snippet in the apidocs would be updated for credentials redaction

Does this PR introduce a breaking change?

  • Yes
  • [. ] No

Other information

Signed-off-by: apoorva9s14 <[email protected]>
Signed-off-by: apoorva9s14 <[email protected]>
Signed-off-by: apoorva9s14 <[email protected]>
Signed-off-by: apoorva9s14 <[email protected]>
Copy link
Member

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

I think the changes look good but I left a question about the credentials redaction piece to review before approving

// string value identifies the level of access that's required to view the credential. For additional information, see
// [viewing a
// credential](https://cloud.ibm.com/docs/account?topic=account-service_credentials&interface=ui#viewing-credentials-ui).
Redacted *string `json:"REDACTED,omitempty"`
Copy link
Member

Choose a reason for hiding this comment

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

So where is this coming from? Was it added by hand? I don't see it in the API definition

Copy link
Contributor Author

@apoorva9s14 apoorva9s14 Jul 27, 2022

Choose a reason for hiding this comment

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

We have updated the api definition, the PR is merged in test, It is also merged in production - https://github.ibm.com/cloud-api-docs/resource-controller/pull/248

Signed-off-by: apoorva9s14 <[email protected]>
Copy link
Member

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@dpopp07 dpopp07 merged commit 5272cdc into main Jul 27, 2022
@dpopp07 dpopp07 deleted the rc_fix branch July 27, 2022 20:42
ibm-devx-sdk pushed a commit that referenced this pull request Jul 27, 2022
# [0.27.0](v0.26.1...v0.27.0) (2022-07-27)

### Features

* **Resource Controller:** regen to add new method ([#196](#196)) ([5272cdc](5272cdc))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 0.27.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants