Skip to content

Catalog onboarding pipeline configuration #69

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 40 commits into from
Nov 27, 2023
Merged

Catalog onboarding pipeline configuration #69

merged 40 commits into from
Nov 27, 2023

Conversation

@digvijay-ukirde
Copy link
Contributor

Description

Addition of catalog config file to enable catalog pipeline

Release required?

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

Feature: Enable catalog pipeline

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.

digvijay-ukirde and others added 30 commits October 19, 2023 13:39
@digvijay-ukirde
Initial commit
c360220
@digvijay-ukirde
Update unused variables
c4f444e
@digvijay-ukirde
Update PR test
6b045cf
@digvijay-ukirde
Rebase to main
0a6b943
@digvijay-ukirde
Temp fix to run the piepline
9f4121b
@digvijay-ukirde
Minor fix (semantics bug)
93bbe3c
@digvijay-ukirde
Update PR test with env variable
1f1a6c0
@digvijay-ukirde
Minor fix (Default value updates)
8d12f19
@digvijay-ukirde
Minor fix (skip_iam_authorization_policy set to false)
f78d050
@digvijay-ukirde
Conditional skip_iam_authorization_policy setting
c419936
@digvijay-ukirde
Merge branch 'main' into develop
b9a8db4
@digvijay-ukirde
go minor updates
86f3a3a
@digvijay-ukirde
go minor updates
a3ef635
@danielbu-ibm
run go mod tidy
c35d620
@digvijay-ukirde
CRA scan updates
a2e2db1
@digvijay-ukirde
CRA scan update ignore rules
becbc6d
@digvijay-ukirde
Ansible path updating
e6618ba
@digvijay-ukirde
Revert "Ansible path updating"
d022091 
This reverts commit e6618ba.
@digvijay-ukirde
Code optimization
bff8f9d
@digvijay-ukirde
Merge branch 'main' into develop
7393350
@digvijay-ukirde
Merge branch 'main' into develop
a226f7d
@digvijay-ukirde
Update IAM account settings
98de8c2
@digvijay-ukirde
Setting replayable to false for verification
e49d8ac
@digvijay-ukirde
Merge branch 'main' into develop
5b9e508
@digvijay-ukirde
Minor changes
dfaffc5
@digvijay-ukirde
Minor fix (skip_iam_authorization_policy set to false)
702fc3e
@digvijay-ukirde
Conditional skip_iam_authorization_policy setting
ab718f2
@digvijay-ukirde
Code optimization
d85912a
@digvijay-ukirde
Minor changes
553605f
@digvijay-ukirde
Merge remote-tracking branch 'origin/develop' into develop
93d3684 
# Conflicts:
#	modules/landing_zone/main.tf
@digvijay-ukirde
Copy link
Contributor Author

/run pipeline

@digvijay-ukirde
Copy link
Contributor Author

/run pipeline

ocofaigh
ocofaigh requested changes Nov 20, 2023
View reviewed changes
Copy link
Contributor

@ocofaigh ocofaigh left a comment

Choose a reason for hiding this comment

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

left some comments

ocofaigh
ocofaigh reviewed Nov 20, 2023
View reviewed changes
@digvijay-ukirde
Copy link
Contributor Author

/run pipeline

ocofaigh
ocofaigh previously requested changes Nov 24, 2023
View reviewed changes
Copy link
Contributor

@ocofaigh ocofaigh left a comment

Choose a reason for hiding this comment

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

see latest comments

ocofaigh
ocofaigh reviewed Nov 24, 2023
View reviewed changes
ibm_catalog.json Outdated
"label": "Advanced",
"name": "advanced",
"install_type": "fullstack",
"working_directory": "solution/hpc",
Copy link
Contributor

Choose a reason for hiding this comment

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

solutions/hpc

@digvijay-ukirde
Copy link
Contributor Author

/run pipeline

@digvijay-ukirde digvijay-ukirde self-assigned this Nov 27, 2023
@digvijay-ukirde digvijay-ukirde merged commit ede19f2 into main Nov 27, 2023
@digvijay-ukirde digvijay-ukirde deleted the develop branch November 27, 2023 19:01
@digvijay-ukirde digvijay-ukirde restored the develop branch November 27, 2023 19:01
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 1.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@bksagar bksagar deleted the develop branch January 8, 2025 07:40
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ocofaigh ocofaigh ocofaigh approved these changes

@go-davidng go-davidng Awaiting requested review from go-davidng

Assignees

@digvijay-ukirde digvijay-ukirde

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@digvijay-ukirde @terraform-ibm-modules-ops @ocofaigh @danielbu-ibm