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] build and push multi-arch Docker images #518

Merged
merged 2 commits into from
Jul 16, 2020

Conversation

jwhui
Copy link
Member

@jwhui jwhui commented Jul 9, 2020

Resolves #501
Resolves #452
Resolves #289
Resolves #274

@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #518 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
+ Coverage   76.78%   76.81%   +0.02%     
==========================================
  Files          57       57              
  Lines        3851     3851              
==========================================
+ Hits         2957     2958       +1     
+ Misses        894      893       -1     
Impacted Files Coverage Δ
src/dbus/server/dbus_agent.cpp 85.07% <0.00%> (+1.49%) ⬆️

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 a9659ef...ac760f5. Read the comment docs.

@@ -82,7 +82,9 @@ dns64_install()

service_name="$(_detect_service_name)"

[ -f /.dockerenv ] || sudo sh -c "echo \"nameserver $DNS64_NAMESERVER_ADDR\" >> $RESOLV_CONF_HEAD"
if without DOCKER; then
Copy link
Member Author

Choose a reason for hiding this comment

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

I tried all of the approaches below and they did not work in this context for some reason:

  • -f /.dockerenv
  • grep docker /proc/1/cgroup -qa
  • grep 'docker\|lxc' /proc/1/cgroup -qa

@jwhui jwhui requested review from gjc13 and bukepo July 13, 2020 15:38
Copy link
Member

@bukepo bukepo left a comment

Choose a reason for hiding this comment

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

LGTM:+1:

@jwhui jwhui merged commit 250c6d6 into openthread:master Jul 16, 2020
jwhui added a commit that referenced this pull request Jul 16, 2020
@jwhui jwhui deleted the docker branch July 16, 2020 18:55
simonlingoogle pushed a commit to simonlingoogle/ot-br-posix that referenced this pull request Nov 11, 2020
Issue(s): b/162572906

* origin/github/master:
  [third_party] update OpenThread (openthread#533)
  [mdns] fix crash upon avahi access error (openthread#526)
  [doc] add GitHub Action badges for Build and Docker (openthread#525)
  [ncp] fix legacy prefix handling (openthread#524)
  [github-actions] fix repo name in docker workflow (openthread#523)
  [github-action] add building multi-arch Docker images (openthread#518)
  [scripts] update to support docker buildx (openthread#518)
  [build] fix build with BUILD_SHARED_LIBS=ON (openthread#522)
Change-Id: I98b2b4bbbd556484eb53d09121f33b5d1a201cd2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants