Skip to content

Conversation

@wangzihao3
Copy link

@wangzihao3 wangzihao3 commented Dec 18, 2019

  • Add docker.arm64 in Makefile, use make docker.arm64 to build images
    for arm.
  • Add opensds-all-arm-dev.yaml to test opensds with noauth

Signed-off-by: wangzihao wangzihao18@huawei.com

What this PR does / why we need it:
This PR add the support for building opensds on ARM64. There are some ARM servers like huawei cloud is used to storage. Opensds may need to run on ARM.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
fixes #1130
Special notes for your reviewer:

Release note:

NONE 
- Add docker.arm64 in Makefile, use make docker.arm64 to build images for arm. - Add opensds-all-arm-dev.yaml to test opensds with noauth Signed-off-by: wangzihao <wangzihao18@huawei.com>
@codecov
Copy link

codecov bot commented Dec 18, 2019

Codecov Report

Merging #1133 into development will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@ ## development #1133 +/- ## ============================================ Coverage 34.88% 34.88% ============================================ Files 97 97 Lines 17583 17583 ============================================ Hits 6133 6133 Misses 10580 10580 Partials 870 870
type: Directory
---
##################################################################################################
# Authchecker service
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove unused code here

---
##################################################################################################
# Dashboard service
##################################################################################################
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove unused code

Makefile Outdated
cp $(BUILD_DIR)/bin/osdsdock ./cmd/osdsdock
cp $(BUILD_DIR)/bin/osdslet ./cmd/osdslet
cp $(BUILD_DIR)/bin/osdsapiserver ./cmd/osdsapiserver
docker build cmd/osdsdock -t opensdsio/opensds-dock-arm64:ci
Copy link
Collaborator

Choose a reason for hiding this comment

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

what does ci mean?

Copy link
Author

Choose a reason for hiding this comment

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

Hi @wisererik . I used ci tag to test on my machine ,and sorry for that . I have update it to latest. Please take a look, thanks!

golang.org/x/crypto v0.0.0-20190510104115-cbcb75029529
golang.org/x/time v0.0.0-20191024005414-555d28b269f0 // indirect
google.golang.org/appengine v1.4.0 // indirect
google.golang.org/grpc v1.24.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

please don't go.mod if there is no change of go code.

Copy link
Author

Choose a reason for hiding this comment

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

have restore it

- remove unused code - change the tag of arm images - restore go.mod Signed-off-by: wangzihao <wangzihao18@huawei.com>
@wangzihao3 wangzihao3 requested a review from wisererik December 19, 2019 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants