Skip to content

Conversation

@imprateeksh
Copy link
Member

@imprateeksh imprateeksh commented Dec 17, 2025

Description

PR to include pulumi examples which will be referred as part of Pulumi tutorial in cloud docs.

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

@imprateeksh imprateeksh changed the title chore: pulumi examplesPulumi egs chore: add pulumi examples Dec 17, 2025
@imprateeksh imprateeksh self-assigned this Dec 17, 2025
Copy link
Member

@maheshwarishikha maheshwarishikha left a comment

Choose a reason for hiding this comment

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

As per Vincent's comment here, I think we should not show resources creation directly using ibmcloud-provider:
image

Copy link
Member

@maheshwarishikha maheshwarishikha left a comment

Choose a reason for hiding this comment

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

Is pulumi/main.py same as pulumi/terraform-ibm-modules/watson_discovery/main.py?

Code structure is not quiet clear. Can we add a directory named something like additional-examples if user is interested to check. But one should be in main path - that will be referred in the tutorial.

@imprateeksh
Copy link
Member Author

imprateeksh commented Dec 17, 2025

As per Vincent's comment here, I think we should not show resources creation directly using ibmcloud-provider: image

Can we include this at last under reference links? Else will remove it completely.

@maheshwarishikha
Copy link
Member

As per Vincent's comment here, I think we should not show resources creation directly using ibmcloud-provider: image

Can we include this at last under reference links? Else will remove it completely.

You can add if you want. But in repository, also please move to different directory.

@imprateeksh
Copy link
Member Author

imprateeksh commented Dec 18, 2025

Is pulumi/main.py same as pulumi/terraform-ibm-modules/watson_discovery/main.py?

Code structure is not quiet clear. Can we add a directory named something like additional-examples if user is interested to check. But one should be in main path - that will be referred in the tutorial.

The initial code structure was given only for understanding but in pulumi it expects one file named __main__.py at root level.
So I thought of providing two examples, instead it would be good if I use one __main__.py file having both watson discovery and COS modules together so that anyone can run it directly.

I have kept the code simple for Terraform Modules only.

@imprateeksh
Copy link
Member Author

Followed the instructions provided in Readme.md in a separate directory to verify the steps. Here are the results:

When using an existing Resource Group -
image

When using a new Resource Group -
image

Saw issues with KMS
image

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

imprateeksh commented Dec 19, 2025

Successful provisioning:
image

Validation

Watson Discovery:

image

Cos bucket instance:

image

Cos bucket with sample file uploaded:

image

Destroy
image
image

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

/run pipeline

@maheshwarishikha maheshwarishikha merged commit 0c7fd7a into main Dec 22, 2025
2 checks passed
@maheshwarishikha maheshwarishikha deleted the pulumi-egs branch December 22, 2025 12:09
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.

3 participants