Skip to content

ci: github actions validate workflow - #3381

Merged
thaJeztah merged 6 commits into
docker:masterfrom
crazy-max:gha-validate
Feb 25, 2022
Merged

ci: github actions validate workflow#3381
thaJeztah merged 6 commits into
docker:masterfrom
crazy-max:gha-validate

Conversation

@crazy-max

Copy link
Copy Markdown
Member

follow-up #3365

Switch to GitHub Actions for validate target:

  • authors
  • vendor
  • yamldocs
  • manpages

Bake targets have also been added for each of them so we only need to set targets to the current matrix in the validate GHA workflow.

CircleCI is now removed. In a follow-up we could fix:

  • # - windows-latest # FIXME: some tests are failing on the Windows runner, as well as on Appveyor since June 24, 2018: https://ci.appveyor.com/project/docker/cli/history
  • # - 20.10-dind # FIXME: Fails on 20.10
    - stable-dind # TODO: Use 20.10-dind, stable-dind is deprecated

Open questions:

  • Switch to cli-docs-tool for yaml docs generation but we can pick that commit in a follow-up if you prefer. For man pages generation with cli-docs-tool, needs to work on Man support cli-docs-tool#14 first.
  • Some Dockerfiles have been added but we might be able to merge them to the Dockerfile.dev one.
  • de8d569 was added to have an unique place to set the GO_VERSION but this would enforce buildx bake usage.

@crazy-max

Copy link
Copy Markdown
MemberAuthor

image

This check fails because CircleCI config has been removed.

image

When this PR is merged we should remove the CircleCI integration in this repository.

@crazy-max
crazy-max marked this pull request as ready for review December 9, 2021 10:15
@crazy-max
crazy-maxforce-pushed the gha-validate branch 2 times, most recently from 0097b27 to 910979dCompareJanuary 13, 2022 14:49
Comment threadvendor.mod Outdated
@crazy-max
crazy-maxforce-pushed the gha-validate branch 3 times, most recently from e6134bf to 5127f8fCompareJanuary 14, 2022 08:06
@crazy-max
crazy-maxforce-pushed the gha-validate branch 7 times, most recently from f10b743 to 5b1ae5dCompareJanuary 18, 2022 16:18
Comment threaddockerfiles/Dockerfile.docs Outdated
@crazy-max

Copy link
Copy Markdown
MemberAuthor

PTAL @thaJeztah@silvin-lubecki

@crazy-max
crazy-maxforce-pushed the gha-validate branch 2 times, most recently from d757a2c to 554489bCompareJanuary 28, 2022 17:53
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
switch to cli-docs-tool and validate yamldocs
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

circle-ci failure is expected (as it's removed in this PR); we could've keep it in this one, and remove after, but probably not worth the hassle

@thaJeztahthaJeztah added area/testing kind/refactor PR's that refactor, or clean-up code status/2-code-review labels Feb 25, 2022
@thaJeztahthaJeztah added this to the 21.xx milestone Feb 25, 2022

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

okay... here we go!

@thaJeztah
thaJeztah merged commit 6401786 into docker:masterFeb 25, 2022
@crazy-max
crazy-max deleted the gha-validate branch February 25, 2022 18:46
@crazy-maxcrazy-max mentioned this pull request Feb 25, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/testingkind/refactorPR's that refactor, or clean-up codestatus/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@crazy-max@thaJeztah@silvin-lubecki