Skip to content

Simplify multitenancy guide #2173

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
renejeglinsky merged 3 commits into from
Oct 27, 2025
Merged

Simplify multitenancy guide #2173

renejeglinsky merged 3 commits into from
Oct 27, 2025

Conversation

@swaldmann
Copy link
Contributor

@swaldmann swaldmann commented Oct 24, 2025 •
edited
Loading

  • with-mtx profile config is added on cds add multitenancy by default.
  • Removed oauth-configuration snippet: was hardcoded to a specific region (bad practice!) and we add a more generic one in cds add xsuaa/ias.
  • cds add kyma is a convenient shortcut for cds add helm,containerize.
  • Removed the "Behind the Scenes" section as it's overly hard-wired to XSUAA + Service Manager. It wasn't adding a lot of value imo.

@swaldmann swaldmann changed the title Simplify multitenancy configuration Simplify multitenancy guide Oct 24, 2025
@swaldmann swaldmann requested a review from ecklie October 24, 2025 13:04
@swaldmann swaldmann marked this pull request as ready for review October 24, 2025 13:04
@renejeglinsky
Copy link
Contributor

renejeglinsky commented Oct 27, 2025

I tested what kind of oauth config is added with cds add xsuaa/ias. For me there doesn't seem to be added anything. So how would that work then?

@swaldmann
Copy link
Contributor Author

swaldmann commented Oct 27, 2025

I tested what kind of oauth config is added with cds add xsuaa/ias. For me there doesn't seem to be added anything. So how would that work then?

It's (by design) not added until you add the approuter with cds add approuter, as the generic variant is based on the approtuer-exposed URL. approuter is also automatically added with cds add portal, which is used in this guide.

@renejeglinsky renejeglinsky merged commit 965492f into main Oct 27, 2025
7 checks passed
@renejeglinsky renejeglinsky deleted the simpler-mt branch October 27, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@renejeglinsky renejeglinsky renejeglinsky approved these changes

@ecklie ecklie ecklie approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@swaldmann @renejeglinsky @ecklie