Skip to content

add GitHub action to trigger Docker Desktop e2e tests with Compose edge version#10625

Merged
glours merged 1 commit intodocker:v2from
glours:compose-edge-desktop-integration
Jun 9, 2023
Merged

add GitHub action to trigger Docker Desktop e2e tests with Compose edge version#10625
glours merged 1 commit intodocker:v2from
glours:compose-edge-desktop-integration

Conversation

@glours
Copy link
Copy Markdown
Contributor

@glours glours commented May 26, 2023

What I did
Add a GitHub action job to trigger Docker Desktop e2e tests with the latest edge version of Compose to early detect integration issue.

Related issue
https://docker.atlassian.net/browse/ENV-120

(not mandatory) A picture of a cute animal, if possible in relation to what you did
image

@glours glours self-assigned this May 26, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented May 26, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.86 🎉

Comparison is base (599723f) 58.91% compared to head (4f97c84) 59.78%.

❗ Current head 4f97c84 differs from pull request most recent head ec31d3c. Consider uploading reports for the commit ec31d3c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##               v2   #10625      +/-   ##
==========================================
+ Coverage   58.91%   59.78%   +0.86%     
==========================================
  Files         111      107       -4     
  Lines        9709     9481     -228     
==========================================
- Hits         5720     5668      -52     
+ Misses       3401     3232     -169     
+ Partials      588      581       -7     

see 15 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@glours glours force-pushed the compose-edge-desktop-integration branch 2 times, most recently from ed2cb57 to ef8d233 Compare June 2, 2023 07:47
Comment thread .github/workflows/merge.yml Outdated
name: Trigger Docker Desktop e2e with edge version
uses: actions/github-script@v6
with:
github-token: ${{ secrets.DOCKERPUBLICBOT_WRITE_PAT }}
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.

That’s a token for Docker Hub, isn’t it? I’m not sure it’s going to help there.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, that's why this PR is still a draft, I also need to update the way I'll get the image tag to pass to the workflow

@glours glours force-pushed the compose-edge-desktop-integration branch 2 times, most recently from 2a84a3b to 0155002 Compare June 2, 2023 13:38
@glours glours marked this pull request as ready for review June 2, 2023 13:41
@glours glours requested review from crazy-max, mat007, milas and ndeloof June 2, 2023 13:42
Copy link
Copy Markdown
Member

@mat007 mat007 left a comment

Choose a reason for hiding this comment

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

Makes sense!

@glours glours force-pushed the compose-edge-desktop-integration branch from 0155002 to 4f97c84 Compare June 2, 2023 14:20
Copy link
Copy Markdown
Contributor

@milas milas left a comment

Choose a reason for hiding this comment

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

🎉

@glours glours force-pushed the compose-edge-desktop-integration branch from 4f97c84 to ec31d3c Compare June 9, 2023 19:07
@glours glours enabled auto-merge June 9, 2023 19:15
@glours glours merged commit 58c5ea8 into docker:v2 Jun 9, 2023
@glours glours deleted the compose-edge-desktop-integration branch June 30, 2025 21:18
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.

4 participants