Skip to content

Conversation

@swaldmann
Copy link
Contributor

@swaldmann swaldmann commented Mar 27, 2025

See internal for preview

@swaldmann swaldmann requested a review from chgeo March 27, 2025 18:10
@swaldmann swaldmann marked this pull request as ready for review March 27, 2025 18:10
Copy link
Contributor

@chgeo chgeo left a comment

Choose a reason for hiding this comment

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

Really nice!

Wish I would have the time to manually go through all of that. Would it make sense that you lead us through this in the next team sync?

+ Use a Kyma-enabled [Trial Account](https://account.hanatrial.ondemand.com/) or purchase a Kyma cluster from SAP
+ You need a [Container Image Registry](#get-access-to-a-container-registry)
+ Get the required SAP BTP service entitlements
+ Install [Docker Desktop or Docker for Linux](https://docs.docker.com/get-docker/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we really need a local docker daemon or just the docker CLI? The setup would be much faster then.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Possibly, haven't tried w/o Docker Desktop. I pulled the link out of the CLI section as we currently recommend the whole thing.

Copy link
Contributor

Choose a reason for hiding this comment

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

You can test it by shutting down your local docker daemon and see if the whole flow still works.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There seems no official way to install the Docker CLI w/o Docker Desktop:
https://stackoverflow.com/questions/38675925/is-it-possible-to-install-only-the-docker-cli-and-not-the-daemon

Specify the repository where you want to push the images:

[CAP's Helm chart](#cds-add-helm) can be configured by the settings as explained below. Mandatory settings are marked with <X/>.
```yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

This is the containerize.yaml, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

Question about this section: Is it needed after all? That's all part of the initial deploy flow, or?

Copy link
Contributor Author

@swaldmann swaldmann Mar 28, 2025

Choose a reason for hiding this comment

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

I kept it for completeness' sake, so we have at least one place where the ctz tool is documented if some setup differs from cds up and they build it from scratch.

@swaldmann swaldmann enabled auto-merge March 31, 2025 08:58
@swaldmann swaldmann disabled auto-merge March 31, 2025 08:59
@swaldmann swaldmann merged commit 6e32b1e into main Mar 31, 2025
4 checks passed
@swaldmann swaldmann deleted the cds-up branch March 31, 2025 08:59
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.

4 participants