Skip to content

hello-world and wordpress examples#1

Open
PatrickBladeeman wants to merge 4 commits into
cloud-ark:mainfrom
PatrickBladeeman:main
Open

hello-world and wordpress examples#1
PatrickBladeeman wants to merge 4 commits into
cloud-ark:mainfrom
PatrickBladeeman:main

Conversation

@PatrickBladeeman
Copy link
Copy Markdown

Added hello-world and wordpress provider/tenant gitops examples with ArgoCD

name: kubeplus-wordpress-tenants
namespace: argocd
spec:
project: kubeplus-tenants
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the project name to "wordpress-tenants"

apiVersion: argoproj.io/v1alpha1
kind: AppProject
metadata:
name: kubeplus-tenants
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the name to "wordpress-tenants"

apiVersion: argoproj.io/v1alpha1
kind: AppProject
metadata:
name: kubeplus-tenants
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the name to "helloworld-tenants"

name: kubeplus-hello-world-consumer
namespace: argocd
spec:
project: kubeplus-tenants
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

helloworld-tenants

name: kubeplus-hello-world-provider
namespace: argocd
spec:
project: kubeplus-provider
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change this to helloworld-provider

apiVersion: argoproj.io/v1alpha1
kind: AppProject
metadata:
name: kubeplus-provider
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change this to helloworld-provider

git add multitenancy/application-hosting/hello-world-argocd/provider/hello-world-service-composition.yaml
git commit -m "Test hello world provider Argo CD sync"
git push
kubectl annotate application kubeplus-hello-world-provider -n argocd argocd.argoproj.io/refresh=hard --overwrite
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we have to annotate argocd application?
Add an explanation.

name: kubeplus-wordpress-provider
namespace: argocd
spec:
project: kubeplus-provider
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the project name to wordpress-provider

apiVersion: argoproj.io/v1alpha1
kind: AppProject
metadata:
name: kubeplus-provider
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wordpress-provider

git add multitenancy/application-hosting/wordpress-argocd/provider/wordpress-service-composition.yaml
git commit -m "Test wordpress provider Argo CD sync"
git push
kubectl annotate application kubeplus-wordpress-provider -n argocd argocd.argoproj.io/refresh=hard --overwrite
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add a comment as to why this step is added.

- Tenant apps can sync as `kubeplus-saas-consumer`, which can create `GitopsWPService` instances but cannot create `ResourceComposition`.
- Both ArgoCD applications point at `cloud-ark/kubeplus-examples`, but
use different subpaths under `multitenancy/application-hosting/wordpress-argocd/`.
- The shorter custom kind `GitopsWPService` avoids KubePlus's Helm release-name length limit and allows tenant names like `wp-tenant1`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good point.

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.

2 participants