Skip to content

ci: publish image for testservice [WPB-21987]#3758

Closed
markbrockhoff wants to merge 1 commit into
wireapp:developfrom
markbrockhoff:ci/publish-docker-image-of-testservice
Closed

ci: publish image for testservice [WPB-21987]#3758
markbrockhoff wants to merge 1 commit into
wireapp:developfrom
markbrockhoff:ci/publish-docker-image-of-testservice

Conversation

@markbrockhoff

@markbrockhoff markbrockhoff commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

The e2e tests of the wire webapp use the testservice in some test cases. Currently it manually builds it for each test execution.

Solutions

To save on CI time and resources a docker image with the prebuilt testservice should be published for others to just consume.

Testing

How to Test

You can build and run the new docker image on your system using the following commands:

docker build -t testservice -f ./tools/testservice/Dockerfile .
docker run -p 8080:8080 -p 8081:8081 testservice

Notes (Optional)

The PR is currently set up to execute the new Workflow for the PR. This is to test it before merging. Once it ran once successfully I'll remove the pull_request trigger.


PR Post Submission Checklist for internal contributors (Optional)

  • Wire's Github Workflow has automatically linked the PR to a JIRA issue

PR Post Merge Checklist for internal contributors

  • If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References
  1. https://sparkbox.com/foundry/semantic_commit_messages
  2. https://github.com/wireapp/.github#usage
  3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@markbrockhoff markbrockhoff changed the title ci: publish image for testservice ci(WPB-21987): publish image for testservice Dec 5, 2025
@MohamadJaara MohamadJaara changed the title ci(WPB-21987): publish image for testservice ci: publish image for testservice [WPB-21987] Dec 5, 2025
@markbrockhoff markbrockhoff force-pushed the ci/publish-docker-image-of-testservice branch from f865d1b to e89957d Compare December 8, 2025 11:00
@markbrockhoff markbrockhoff force-pushed the ci/publish-docker-image-of-testservice branch from e89957d to 6376647 Compare December 8, 2025 12:04
@sonarqubecloud

sonarqubecloud Bot commented Dec 8, 2025

Copy link
Copy Markdown

@markbrockhoff

Copy link
Copy Markdown
Contributor Author

PR was closed in favor of #3759 since I'm not able to run CI for a PR from a fork

@markbrockhoff markbrockhoff deleted the ci/publish-docker-image-of-testservice branch December 8, 2025 12:38
@markbrockhoff markbrockhoff restored the ci/publish-docker-image-of-testservice branch December 8, 2025 12:38
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