-
Notifications
You must be signed in to change notification settings - Fork 3k
Separate cloud example configurations to another file #13771
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
Conversation
I don't follow, what limitation is this fixing by splitting the file?
You can still parse json and run 2 examples at the same time or not? I dont see how splitting the json works differently than as it is. |
greentea triggers test of all the examples in one shot. It doesn't separate trigger of each examples. When baudrates are different in example-applications, greentea cannot handle this and gets executed with a single baud rate.
All the examples in examples.json files are built in CI sequentially as defined in this config file. If we need to separate examples run, without touching examples.jon, we need to create another config in CI and fetch the group as needed. We could also think about moving these files to mbed-os-ci repo but I don't know how widely this is used. Also, in CI, the way these examples are built and tested are differently to other applications. There is need to insert some certificate and connect to cloud providers. |
Jenkins CI Test : ❌ FAILEDBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
@saheerb Let us know once this is ready for CI (removing do not merge label) |
CI restarted |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
This PR separate cloud examples to a separate configuration file from example configuration. This is done for the following reasons:
Impact of changes
None.
Migration actions required
Documentation
None
Pull request type
Test results
None required as changes affect CI pipeline and CI scripts are adapted for this.
Reviewers
@evedon