Skip to content

Add k8s contiv/vpp Helm support for arm64 platform#1002

Merged
rastislavs merged 3 commits intocontiv:masterfrom
TrevorTaoARM:arm64
Aug 13, 2018
Merged

Add k8s contiv/vpp Helm support for arm64 platform#1002
rastislavs merged 3 commits intocontiv:masterfrom
TrevorTaoARM:arm64

Conversation

@TrevorTaoARM
Copy link
Contributor

Add k8s Helm template and makefile support for the generation of the deployment yaml file
of arm64 platform.

Give only different parts in values-arm64.yaml for arm64 from values.yaml.

A sample file named contiv-vpp-arm64.yaml is also given.

Add a typical contiv-vpp-arm64.yaml file for deployment on arm64 platform.
The deployment way for that on arm64 is the same as the contiv-vpp.yaml
for x86_64 platform for Kubernetes, which is like:
kubectl apply -f ./contiv-vpp-arm64.yaml

Signed-off-by: trevortao <trevor.tao@arm.com>
Add k8s Helm template and makefile support for
the generation of the deployment yaml file
of arm64 platform.

Give only different parts in values-arm64.yaml for arm64
from values.yaml.
Update the template/vpp.yaml accordingly.

A sample file named contiv-vpp-arm64.yaml is also
given.

Signed-off-by: trevortao <trevor.tao@arm.com>
@coveralls
Copy link

coveralls commented Aug 8, 2018

Coverage Status

Coverage remained the same at 76.477% when pulling fa2d8f6 on TrevorTaoARM:arm64 into d0eaa53 on contiv:master.

To install without tiller for arm64, you can generate the manifest from this chart and install via kubectl:

```console
helm template --name my-release ../contiv-vpp -f ./values-arm64.yaml > manifest-arm64.yaml
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't this contain both: -f ./values-arm64.yaml,./values.yaml ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it should be added.
Updated.

Signed-off-by: trevortao <trevor.tao@arm.com>
@rastislavs rastislavs merged commit 876daa5 into contiv:master Aug 13, 2018
@TrevorTaoARM TrevorTaoARM deleted the arm64 branch August 13, 2018 08:51
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.

3 participants