Skip to content

Feature/arm experimental#527

Merged
Maleware merged 29 commits intomainfrom
feature/arm-experimental
Feb 27, 2024
Merged

Feature/arm experimental#527
Maleware merged 29 commits intomainfrom
feature/arm-experimental

Conversation

@Maleware
Copy link
Copy Markdown
Member

First draft of an small article about early access to our arm64 images

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 16, 2024

Deploy Preview for stackable-docs ready!

Name Link
🔨 Latest commit 5f4069c
🔍 Latest deploy log https://app.netlify.com/sites/stackable-docs/deploys/65dda405b0b68600085e5859
😎 Deploy Preview https://deploy-preview-527--stackable-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Copy link
Copy Markdown
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

Sorry for all the comments/suggestions!

Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Accepting comments of Andrew, thank you!

Co-authored-by: Andrew Kenworthy <andrew.kenworthy@stackable.de>
@Maleware Maleware requested a review from adwk67 February 27, 2024 07:29
adwk67
adwk67 previously approved these changes Feb 27, 2024
Copy link
Copy Markdown
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Copy Markdown
Contributor

@fhennig fhennig left a comment

Choose a reason for hiding this comment

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

I'd suggest to rename the file to "arm64-support.adoc" (so no experimental) but add the experimental to the title (i.e. "Multi-architecture for SDP (experimental").

Otherwise if we stabilize this feature, the link will change, and that will be annoying.

Comment thread modules/concepts/pages/experimental-arm64-support.adoc Outdated
Copy link
Copy Markdown
Contributor

@fhennig fhennig left a comment

Choose a reason for hiding this comment

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

perfect, thanks!

@Maleware Maleware added this pull request to the merge queue Feb 27, 2024
Merged via the queue into main with commit 20cb785 Feb 27, 2024
@Maleware Maleware deleted the feature/arm-experimental branch February 27, 2024 09:07

By declaring `--set image.repository=docker.stackable.tech/stackable-experimental/commons-operator` you will overwrite the image selected by default with one located in the organization `stackable-experimental`. The other operators can be installed in the same way.

WARNING: You have to install every operator you need for a given demo individually with helm using `--skip-release` with stackablectl. Otherwise, it will try to install x86 operators and pull ARM64 product images afterwards.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure what this sentence means? The docs so far only mention helm and helm is used to install the operators? This is confusing.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, we have to install operators using helm and using stackablectl --skip-release to install a demo. This is because stackablectl operator install kafka=23.11.0 would point to docker.stackable.tech/stackable/kafka-operator:23.11.0 .

I see that this is bad timing to hint it and I overlooked helm *and* using. Would this make it more clear?

I want to inform about --skip-release as it would only be mentioned in the command on the bottom.


SDP release 23.11.0 has been ported to the ARM64 architecture. These images are stored within the `stackable-experimental` organization in our https://repo.stackable.tech/#browse/browse:docker:v2%2Fstackable-experimental[Nexus].

As they are not available by default so you need to first switch to the `stackable-experimental` organization before pulling them for use with the data platform.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not available to whom? By what? How do I switch?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Through our default access ways. Default refers to docker.stackable.tech/stackable/....

I understand you need to somehow know what I mean. Is this more clear? :

switch from docker.stackable.tech/stackable to docker.stackable.tech/stackable-experimental before pulling images from the repository.

Comment thread modules/concepts/pages/arm64-support.adoc
@lfrancke
Copy link
Copy Markdown
Member

lfrancke commented Mar 1, 2024

Can you please add a link to the generated documentation?

@Maleware
Copy link
Copy Markdown
Member Author

Maleware commented Mar 4, 2024

https://deploy-preview-558--stackable-docs.netlify.app/home/nightly/concepts/arm64-support current WIP

Currently deployed:
https://docs.stackable.tech/home/nightly/concepts/arm64-support

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants