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