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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Crn attribute #5107

Merged
merged 19 commits into from Feb 29, 2024
Merged

Add Crn attribute #5107

merged 19 commits into from Feb 29, 2024

Conversation

Diptipowervs
Copy link
Contributor

Community Note

  • Please vote on this pull request by adding a 馃憤 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccXXX'

...

@ismirlia ismirlia added the service/Power Systems Issues related to Power Systems label Feb 1, 2024
@ismirlia
Copy link
Collaborator

ismirlia commented Feb 7, 2024

Sorry I accidentally pushed to this branch thinking I was committing to a different repo. This should be fixed now.

@Diptipowervs
Copy link
Contributor Author

Test Results:--- PASS: TestAccIBMPIWorkspaceBasic (144.96s)
PASS

Copy link
Collaborator

@michaelkad michaelkad left a comment

Choose a reason for hiding this comment

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

Documentation needs update.

Diptipowervs and others added 3 commits February 21, 2024 12:37
Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>
Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>
Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>
@hkantare hkantare merged commit cf99ea3 into IBM-Cloud:master Feb 29, 2024
1 check passed
ismirlia pushed a commit to powervs-ibm/terraform-provider-ibm that referenced this pull request Mar 4, 2024
* Add return crn attribute

* Update doc

* Update website/docs/d/pi_workspace.html.markdown

Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>

* Update resource documents

* Add attribute to resource doc

* workspace attr

* Add Attr_CreationDate

* Resolve conflicts

* Add Attr_WorkspaceDetails in resource schema

* Update ibm/service/power/resource_ibm_pi_workspace_test.go

Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>

* Update ibm/service/power/resource_ibm_pi_workspace_test.go

Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>

* Update website/docs/r/pi_workspace.html.markdown

Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>

---------

Co-authored-by: michaelkad <45772690+michaelkad@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service/Power Systems Issues related to Power Systems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants