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 Workflows implementation #948

Merged
merged 5 commits into from
Nov 19, 2020

Conversation

VonOx
Copy link
Contributor

@VonOx VonOx commented Nov 12, 2020

This PR add 2 Github workflows/actions

docker-pr-build.yml

This workflow is triggered on no draft PR.
Here we build amd64 docker image without docker login and without pushing to DockerHub

docker-dev-build.yml

This one is a manual action
It build and push multi arch image to gladys:dev tag based on master branch.

Currently there is a bug in node alpine ( i386 is unavailable for building ) #947

Tested successfully in repo => https://github.com/VonOx/GLADYS_CI


@Pierre-Gilles like circle ci you need to set repo secrets:

DOCKERHUB_PASSWORD
DOCKERHUB_USER

@VonOx VonOx marked this pull request as draft November 12, 2020 13:02
@codecov
Copy link

codecov bot commented Nov 12, 2020

Codecov Report

Merging #948 (e4b6cb8) into master (765461e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #948   +/-   ##
=======================================
  Coverage   94.22%   94.22%           
=======================================
  Files         462      462           
  Lines        6133     6133           
=======================================
  Hits         5779     5779           
  Misses        354      354           

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 765461e...e4b6cb8. Read the comment docs.

@Pierre-Gilles
Copy link
Contributor

Nice!

And how does it work for the dev build? With a manual trigger?

@VonOx
Copy link
Contributor Author

VonOx commented Nov 16, 2020

Nice!

And how does it work for the dev build? With a manual trigger?

Yep , Actions Tab and run job

@Pierre-Gilles
Copy link
Contributor

Amazing! Let me know when this is ready for review :)

@VonOx
Copy link
Contributor Author

VonOx commented Nov 16, 2020

It's ready but if you merge we cant build i386 platform

@Pierre-Gilles
Copy link
Contributor

Ah yes alright. What's the next step for that then? is i386 used right now ? (I don't remember for which platform we build it?)

@VonOx
Copy link
Contributor Author

VonOx commented Nov 16, 2020

Yes we build ( with circleci ) x86 images , I have no idea how many pulls but maybe for dev images it is acceptable to not have i386 images

@VonOx VonOx marked this pull request as ready for review November 16, 2020 13:13
@Pierre-Gilles
Copy link
Contributor

Yes we build ( with circleci ) x86 images

Which machines exactly are using those images ?

It's maybe ok to drop support for this honestly, even more for dev images :)

@VonOx
Copy link
Contributor Author

VonOx commented Nov 16, 2020

Some NAS I suppose and old 32bits processors

Majority of running instance use arm I think ( we build arm v6/v7 and arm64 ). We maybe cover about 95% of the scope

I removed the draft flag, you can see workflow running
https://github.com/GladysAssistant/Gladys/actions/runs/366056548

@VonOx VonOx changed the title [WIP] Github Workflows implementation Github Workflows implementation Nov 16, 2020
@VonOx VonOx requested a review from Pierre-Gilles November 19, 2020 12:42
@Pierre-Gilles Pierre-Gilles merged commit f26772c into GladysAssistant:master Nov 19, 2020
R6n0 pushed a commit to R6n0/Gladys that referenced this pull request Dec 2, 2020
…adysAssistant#948)

* Github Workflow for dev build

* Workflow for building amd64 images on PR

* Rename job

* Revert to old node lts (12.x)

* Update docker-dev-build.yml
@VonOx VonOx deleted the GithubActions branch April 20, 2021 08:45
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.

2 participants