Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

manage_ctr_mgr.sh assumes amd64 #76

Closed
jodh-intel opened this issue Feb 8, 2018 · 1 comment
Closed

manage_ctr_mgr.sh assumes amd64 #76

jodh-intel opened this issue Feb 8, 2018 · 1 comment
Assignees
@chavafg

Comments

@jodh-intel
Copy link
Contributor

Once #71 lands, we need to update https://github.com/kata-containers/tests/blob/master/cmd/container-manager/manage_ctr_mgr.sh so that it doesn't assume amd64 when setting up the Ubuntu Docker repo.

I think we could just add in some logic that does:

goarch=$(go env GOARCH)

... and make use of that as it's the correct value (whereas arch(1) isn't).
@jodh-intel jodh-intel mentioned this issue Feb 8, 2018
Closed
@jodh-intel
Copy link
Contributor Author

Any progress on this one?

chavafg added a commit to chavafg/tests-1 that referenced this issue Mar 20, 2018
@chavafg
manage_ctr_mgr: use system arch for installing repos
7d6f013 
get system architecture and use it for adding the docker
repositories for Ubuntu.

Fixes: kata-containers#76.

Signed-off-by: Salvador Fuentes <[email protected]>
@chavafg chavafg mentioned this issue Mar 20, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@chavafg chavafg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chavafg @jodh-intel