Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Condense test details into one config file #102

Merged
merged 1 commit into from
Mar 18, 2021
Merged

Condense test details into one config file #102

merged 1 commit into from
Mar 18, 2021

Conversation

M-Davies
Copy link
Contributor

@M-Davies M-Davies commented Mar 17, 2021 •
edited
Loading

  • Now easier for third parties to avoid release cycle test disabling and specify their own default test lists

Closes: #93
Signed-off-by: Morgan Davies morgandavies2020@gmail.com

@M-Davies M-Davies added this to the March 2021 milestone Mar 17, 2021
@M-Davies M-Davies self-assigned this Mar 17, 2021
@github-actions
Copy link

Thank you for creating a pull request!
In order to run the pipeline tests I require an admin to post the following comment: run tests

@M-Davies
Copy link
Contributor Author

Test / Groovy is expected to fail (See prTester/ documentation for why)

@M-Davies
Copy link
Contributor Author

run tests

M-Davies added a commit to M-Davies/openjdk-build that referenced this pull request Mar 17, 2021
@M-Davies
Update doc to account for ci changes
e662589 
* Twin PR -> adoptium/ci-jenkins-pipelines#102

Signed-off-by: Morgan Davies <morgandavies2020@gmail.com>
@M-Davies M-Davies mentioned this pull request Mar 17, 2021
Merged
@adoptopenjdk-github-bot
Copy link
Contributor

🟠 PR TESTER RESULT 🟠

❎ Some pipelines failed or the job was aborted! ❎
See the pipeline-build-check below for more information...

@M-Davies M-Davies force-pushed the test_list_config branch 2 times, most recently from 4531617 to b62f4a7 Compare March 18, 2021 13:05
@M-Davies
Copy link
Contributor Author

run tests

@M-Davies M-Davies changed the title Condense test disabling into one config file Condense test details into one config file Mar 18, 2021
@adoptopenjdk-github-bot
Copy link
Contributor

🟠 PR TESTER RESULT 🟠

❎ Some pipelines failed or the job was aborted! ❎
See the pipeline-build-check below for more information...

@M-Davies
Condense test details into one config file
8a9288b 
* Now easier for third parties to avoid release cycle test disabling and specify their own default test lists

Signed-off-by: Morgan Davies <morgandavies2020@gmail.com>
@M-Davies
Copy link
Contributor Author

run tests

@M-Davies M-Davies marked this pull request as ready for review March 18, 2021 13:17
@adoptopenjdk-github-bot
Copy link
Contributor

🟠 PR TESTER RESULT 🟠

❎ Some pipelines failed or the job was aborted! ❎
See the pipeline-build-check below for more information...

@M-Davies
Copy link
Contributor Author

Tester failures unrelated

andrew-m-leonard
andrew-m-leonard approved these changes Mar 18, 2021
View reviewed changes
Copy link
Contributor

@andrew-m-leonard andrew-m-leonard left a comment

Choose a reason for hiding this comment

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

change lgtm

karianna
karianna approved these changes Mar 18, 2021
View reviewed changes
Copy link
Contributor

@karianna karianna left a comment

Choose a reason for hiding this comment

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

LGTM

@karianna karianna merged commit a035e1a into adoptium:master Mar 18, 2021
@M-Davies
Copy link
Contributor Author

Running https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/1443/console to test

@M-Davies M-Davies mentioned this pull request Mar 19, 2021
Merged
karianna pushed a commit to adoptium/temurin-build that referenced this pull request Mar 23, 2021
@M-Davies
Update doc to account for ci changes (#2534)
d4391ff 
* Twin PR -> adoptium/ci-jenkins-pipelines#102

Signed-off-by: Morgan Davies <morgandavies2020@gmail.com>
@sxa sxa mentioned this pull request Apr 19, 2021
Merged
AdamBrousseau added a commit to AdamBrousseau/ci-jenkins-pipelines that referenced this pull request Oct 14, 2022
@AdamBrousseau
Merge pull request adoptium#102 from AdamBrousseau/arty_build_name
6fe6280 
Prepend buildInfo with sys-rt and printout buildInfo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karianna karianna karianna approved these changes

@andrew-m-leonard andrew-m-leonard andrew-m-leonard approved these changes

Assignees

@M-Davies M-Davies

Labels
documentation enhancement jenkins-pipeline testing
Projects
None yet
Milestone
March 2021
Development

Successfully merging this pull request may close these issues.

Propose moving Test lists and Enable Test bool to a config file
4 participants
@M-Davies @adoptopenjdk-github-bot @karianna @andrew-m-leonard