Skip to content
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

github action: add ARM64 test #12928

Merged
merged 1 commit into from
May 10, 2021
Merged

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented May 8, 2021

Tested in my personal fork:

Screen Shot 2021-05-08 at 1 05 42 AM

Screen Shot 2021-05-08 at 12 47 43 AM

Screen Shot 2021-05-08 at 12 48 07 AM

ref. #12856 and etcd-io/website#273

@codecov-commenter
Copy link

codecov-commenter commented May 8, 2021

Codecov Report

Merging #12928 (76c4889) into master (aeb9b5f) will decrease coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #12928      +/-   ##
==========================================
- Coverage   73.28%   73.03%   -0.25%     
==========================================
  Files         430      430              
  Lines       34182    34182              
==========================================
- Hits        25051    24966      -85     
- Misses       7202     7279      +77     
- Partials     1929     1937       +8     
Flag Coverage Δ
all 73.03% <ø> (-0.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
server/etcdserver/api/rafthttp/peer.go 79.35% <0.00%> (-16.13%) ⬇️
server/etcdserver/api/rafthttp/peer_status.go 89.28% <0.00%> (-10.72%) ⬇️
server/etcdserver/api/v3rpc/lease.go 74.64% <0.00%> (-5.64%) ⬇️
client/v3/experimental/recipes/key.go 72.13% <0.00%> (-3.28%) ⬇️
pkg/traceutil/trace.go 94.66% <0.00%> (-2.67%) ⬇️
server/proxy/grpcproxy/watch.go 95.09% <0.00%> (-2.46%) ⬇️
server/etcdserver/api/rafthttp/msgappv2_codec.go 69.72% <0.00%> (-1.84%) ⬇️
server/etcdserver/api/v2http/client.go 89.01% <0.00%> (-1.35%) ⬇️
server/mvcc/kvstore_txn.go 72.72% <0.00%> (-1.22%) ⬇️
client/v3/watch.go 91.05% <0.00%> (-1.12%) ⬇️
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aeb9b5f...76c4889. Read the comment docs.


### Step 2. Install github action on the host

SSH into the instance, and install the github action self-hosted runner (see [install scripts](https://github.com/etcd-io/etcd/settings/actions/runners/new?arch=arm64&os=linux)).
Copy link
Contributor

Choose a reason for hiding this comment

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

I cannot open the link (404). Please double check / document the permissions needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just gave you the admin access. Can you check again?

Copy link
Contributor

Choose a reason for hiding this comment

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

Still does not work. I cannot open even: https://github.com/etcd-io/etcd/settings

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ptabor Just added you to cncf-admins. Now you should have access to the settings.

And run:

```
# run this as systemd service
Copy link
Contributor

Choose a reason for hiding this comment

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

could you please provide an examplar systemd *.conf file ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Leave it as TODO. We will follow up with the example config file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually, no need to add systemd service file. Github action has built-in script to do that:

sudo ./svc.sh install
sudo ./svc.sh start
sudo ./svc.sh status

Updated the README.

@gyuho
Copy link
Contributor Author

gyuho commented May 8, 2021

/cc @mkumatag @spzala

@gyuho gyuho force-pushed the github-worker-arm64 branch 2 times, most recently from ac5dbd5 to 76c4889 Compare May 8, 2021 20:46
@gyuho gyuho added the WIP label May 8, 2021
@gyuho gyuho force-pushed the github-worker-arm64 branch 3 times, most recently from 5ba2919 to acf0f8c Compare May 8, 2021 23:29
@gyuho gyuho removed the WIP label May 8, 2021
@@ -15,7 +15,7 @@ mod="$1"
ver="$2"

function maybe_update_module {
log_callout "Processing ${module}"
log_callout "Processing ${mod}"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixing the shellcheck error. It was failing due to unassigned var.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was piggy-backing on the variable set here:

local module=${1:-"."}

with the intent of printing module that we are currently processing.

Lets remove the line at all.

README for self-hosted-linux-arm64-graviton2-tests.yml

Signed-off-by: Gyuho Lee <leegyuho@amazon.com>
@gyuho gyuho merged commit a8f38eb into etcd-io:master May 10, 2021
@gyuho gyuho deleted the github-worker-arm64 branch May 10, 2021 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants