Repository navigation
update domain from hybrid-cloud-patterns to validatedpatterns.io #625
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -41,7 +41,7 @@ In this blog, we’re going to use OpenShift Gitops to deploy the Argo Rollouts | |
|
|
||
| ## Preparation | ||
|
|
||
| Let’s start by deploying the argo-rollouts pattern from the [argo-rollouts](https://github.com/hybrid-cloud-patterns/argo-rollouts). For this demo, I have deployed a 3-Node compact cluster using `m5.2xlarge` machine types in AWS. This demo will only use rollouts to deploy onto a single cluster. | ||
| Let’s start by deploying the argo-rollouts pattern from the [argo-rollouts](https://github.com/validatedpatterns/argo-rollouts). For this demo, I have deployed a 3-Node compact cluster using `m5.2xlarge` machine types in AWS. This demo will only use rollouts to deploy onto a single cluster. | ||
|
|
||
| ```sh | ||
|
|
||
|
|
@@ -65,15 +65,15 @@ argo-rollouts-7d9dd-master-2 Running m5.2xlarge us-east-2 us-east-2c 1 | |
|
|
||
| If you've never deployed OpenShift before, you could try [ROSA](https://cloud.redhat.com/learn/getting-started-red-hat-openshift-service-aws-rosa/deploy-rosa-cluster) the pay-as-you-go OpenShift managed service. | ||
|
|
||
| Next, you'll need to create a fork of the [argo-rollouts](https://github.com/hybrid-cloud-patterns/argo-rollouts/) repo. | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As above - needs to stay the same. |
||
| Next, you'll need to create a fork of the [argo-rollouts](https://github.com/validatedpatterns/argo-rollouts/) repo. | ||
| Go there in a browser, make sure you’re logged in to GitHub, click the “Fork” button, and confirm the destination by clicking the big green "Create fork" button. | ||
|
|
||
| Next, [install the Validated Patterns operator](https://validatedpatterns.io/infrastructure/using-validated-pattern-operator/) from Operator Hub. | ||
|
|
||
| And finally, click through to the installed operator, and select the `Create | ||
| instance` button and fill out the Create a Pattern form. Most of the defaults | ||
| are fine, but make sure you update the GitSpec URL to point to your fork of | ||
| `argo-rollouts`, rather than `https://github.com/hybrid-cloud-patterns/argo-rollouts`. | ||
| `argo-rollouts`, rather than `https://github.com/validatedpatterns/argo-rollouts`. | ||
|
|
||
| To see what’s going on, click on “Installed Operators” and then change the Project to “All Projects”. After a bit, you will see the following operators installed: | ||
| Advanced Cluster Manager for Kubernetes | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -29,7 +29,7 @@ You will need to login into GitHub and be able to fork two repositories. | |
| * validatedpatterns/multicluster-devsecops | ||
|
|
||
| <!--There is no such repo as chat-client in either orgs --> | ||
| * hybrid-cloud-patterns/chat-client | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This doesn't exist in hybrid-cloud-patterns, validatedpatterns, or validatedpatterns-sandbox as far as I can tell. |
||
| * validatedpatterns/chat-client | ||
|
|
||
| ## Pipeline Demos | ||
|
|
||
|
|
||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm confused about what's not correct? I have rhvp.cluster_utils PR that needs to be merged which will resolve ( at least I think it will) what you're seeing
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ah got you. then let's polish up that cluster_utils PR and then we merge this. thanks!
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like the other PR was closed and not merged, but it also looks as though both paths are checked based on current readme (https://github.com/validatedpatterns/rhvp.cluster_utils/blob/main/roles/vault_utils/README.md) and push_secrets.yml config: https://github.com/validatedpatterns/rhvp.cluster_utils/blob/5ef364d10c55112f784651c34101158a186e0f29/roles/vault_utils/tasks/push_secrets.yaml#L67 so... the purpose of the other PR was addressed, but this line change is still incorrect, I think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't change, since it's a point-in-time blog post AND since the updated URL doesn't exist. The URL in the existing post does still exist as an archive: https://github.com/hybrid-cloud-patterns/argo-rollouts