This repository was archived by the owner on Jul 21, 2021. It is now read-only.
Integration Test Pilot #205
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com> Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Co-authored-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com> Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
| sh 'cmake --build builddir-integration --target flash' | ||
|
|
||
| sh 'rm -rf testing/integration/test-coordinator/kiso-testing' | ||
| sh 'git clone --depth 1 --branch prototype/integrate-kiso-demo https://github.com/ChiefGokhlayehBosch/kiso-testing.git testing/integration/test-coordinator/kiso-testing' |
Contributor
Author
There was a problem hiding this comment.
This needs to be changed as soon as the fixes I made to kiso-testing are merged back into eclipse/kiso-testing
- Merge
kiso-testingfixes back to mainline.
Signed-off-by: ChiefGokhlayehBosch <fixed-term.andreas.baulig@bosch.com>
ChiefGokhlayehBosch
force-pushed
the
feature/#17-add-integration-test-framework
branch
from
dcd7790 to
29c78ac
Compare
Contributor
Author
|
After discussion with @sebastianpfischer, it makes more sense to remove the test-entry lookup from the embedded side and revert these changes on kiso-testing side. Each test-app only ever contains a single test-entry. Assigning a ID to those might be neat for debugging purposes, but it doesn't make sense to have a lookup-mechanism for it. The relevant bits in the CCMsg structure shall be marked reserved.
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR combines #159 and #105, as well as expanding on the two, by adding the necessary CI/CD configuration to run integration tests on decentralized Jenkins agents.
A guide on how to set up new agents has been added to the hugo doc. There you'll also find an overview on how it's intended to work.
Docker Hub images have been uploaded. I made them multi-arch images, meaning a Raspberry-Pi 3 (ARM 32-bit) can pull them straight from the hub. The agent image is not published to Docker Hub, as it almost always requires JLink to be built-in. Instead it can be built locally. I added a disclaimer to the agent's
Dockerfile, printing the terms of use and asking the user to accept (or decline) the agreement via build-args during image creation.To report build status back to GitHub, agents require an access token with
repo:status-flag set. Updating the commit status, can be done without reverse proxy, meaning an agent may reside behind a NAT or corporate firewall, with one-way access to the internet.Code-wise
core/testingand the simple integration test incore/essentialsremain basically untouched from what was there in #159 and #105 (some superficial touch-ups soclang-tidyis happy and bugfixes in the UART integration test).Closes #159 #105