-
Notifications
You must be signed in to change notification settings - Fork 24
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
config: restrict deployments to amd64 nodes #328
config: restrict deployments to amd64 nodes #328
Conversation
2978d9e
to
8b8e13c
Compare
🎉 All dependencies have been resolved ! |
8b8e13c
to
586918b
Compare
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.
It might be nice to clarify that your patch is an adaptation of the original contribution. Something like adding a 'Original-Version-By:' footer?
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.
@phlogistonjohn wrote.
Not sure it is required because the original contributor @tiferrei of #326 is still t author of this PR's commit which also carries their signoff line |
Until samba-operator supports other architectures (namely, ARM64), restrict its deployment to AMD64 nodes (e.g., in cases of non x86_64 or mixed ARCH clusters) via standard K8s node-selector mechanism [1]. [1] https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/ Signed-off-by: Tiago Ferreira <me@tiferrei.com> Co-authored-by: Shachar Sharon <ssharon@redhat.com>
586918b
to
6134073
Compare
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.
good enough for me.
/retest centos-ci/sink-clustered/mini-k8s-latest |
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.
Thanks for adjusting the commit message, @synarete ! LGTM now!
Until samba-operator supports other architectures (namely, ARM64), restrict its deployment to AMD64 nodes (e.g., in cases of non x86_64 or mixed ARCH clusters) via standard K8s node-selector mechanism [1].
[1] https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/
depends on #329