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.

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

Update weekly test list #2203

Merged
merged 6 commits into from
Nov 5, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 12 additions & 11 deletions pipelines/build/common/build_base_file.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -56,28 +56,29 @@ class Builder implements Serializable {
def context
def currentBuild

/*
Test targets triggered in 'nightly' build pipelines running 6 days per week
nightly + weekly to be run during a 'release' pipeline
*/
final List<String> nightly = [
'sanity.openjdk',
'sanity.system',
'extended.system',
'sanity.perf',
'sanity.external'
'sanity.functional',
'extended.functional'
]

// Temporarily remove weekly tests due to lack of machine capacity during release
final List<String> weekly = []
/*
/*
Test targets triggered in 'weekly' build pipelines running once per week
nightly + weekly to be run during a 'release' pipeline
*/
final List<String> weekly = [
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we get a short explanation or link about the test selection? Will daily and weekly be combined during release? Will both run on a weekend?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've pushed some comments - and will update the "Graduated Testing wiki, once we actually schedule a 1x/week weekly build pipeline

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks!

'extended.openjdk',
'extended.perf',
'extended.external',
'special.openjdk',
'special.functional',
'special.system',
'special.perf'
'sanity.external'
]
*/


/*
Returns an IndividualBuildConfig that is passed down to the downstream job.
It uses several helper functions to pull in and parse the build configuration for the job.
Expand Down
10 changes: 2 additions & 8 deletions pipelines/jobs/configurations/jdk11u_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -112,21 +112,15 @@ class Config11 {
arm32Linux : [
os : 'linux',
arch : 'arm',
test : [
nightly: ['sanity.openjdk'],
weekly : []
],
test : 'default',
configureArgs : '--enable-dtrace=auto'
],

aarch64Linux : [
os : 'linux',
arch : 'aarch64',
dockerImage : 'adoptopenjdk/centos7_build_image',
test : [
nightly: ['sanity.functional', 'extended.functional', 'sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.external'],
weekly : ['extended.openjdk', 'extended.perf', 'extended.external', 'special.openjdk','special.functional', 'special.system', 'special.perf']
],
test : 'default',
configureArgs : '--enable-dtrace=auto'
],

Expand Down
5 changes: 1 addition & 4 deletions pipelines/jobs/configurations/jdk15u_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -131,10 +131,7 @@ class Config15 {
arm32Linux : [
os : 'linux',
arch : 'arm',
test : [
nightly: ['sanity.openjdk'],
weekly : []
],
test : 'default',
configureArgs : '--enable-dtrace'
],

Expand Down
5 changes: 1 addition & 4 deletions pipelines/jobs/configurations/jdk8u_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -111,10 +111,7 @@ class Config8 {
arm32Linux : [
os : 'linux',
arch: 'arm',
test: [
nightly: ['sanity.openjdk'],
weekly : []
]
test: 'default'
],

aarch64Linux : [
Expand Down