Skip to content

Conversation

crazy-max
Copy link
Member

@crazy-max crazy-max commented Nov 24, 2020

Follow-up #1824

Push is disabled until we have migrated to GitHub Actions. Once we have migrated to GitHub we will need to add the following secrets to push to Docker Hub:

  • DOCKERHUB_USERNAME
  • DOCKERHUB_TOKEN

Signed-off-by: CrazyMax crazy-max@users.noreply.github.com

@crazy-max crazy-max requested a review from tonistiigi November 24, 2020 03:35
@codecov-io
Copy link

codecov-io commented Nov 24, 2020

Codecov Report

Merging #1835 (40bcfab) into master (b0ed56c) will decrease coverage by 0.23%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1835      +/-   ##
==========================================
- Coverage   46.46%   46.22%   -0.24%     
==========================================
  Files         152      152              
  Lines       15281    15281              
==========================================
- Hits         7100     7064      -36     
- Misses       7146     7191      +45     
+ Partials     1035     1026       -9     
Impacted Files Coverage Δ
worker/containerd/containerd.go 0.00% <0.00%> (-58.63%) ⬇️
cache/manager.go 54.32% <0.00%> (-0.33%) ⬇️

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 b0ed56c...40bcfab. Read the comment docs.

@crazy-max crazy-max force-pushed the ghactions-release branch 2 times, most recently from d44336e to 5cb8559 Compare November 24, 2020 05:55
@tonistiigi
Copy link
Member

I just hit rebuild on this action and lot of binaries and dockerfile steps were rebuilt. Eg. in cross and in tests. Looks like caching is not correct, including dependencies like #1833 (comment) . Don't need to block on this but that part seems to need some more work.

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max force-pushed the ghactions-release branch 3 times, most recently from d8af3bf to 7576470 Compare November 24, 2020 23:32
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants