Skip to content

Script for running test exclusion parser in sandbox#6993

Open
adamfarley wants to merge 6 commits intoadoptium:masterfrom
adamfarley:create_shell_script_for_exclude_file_parser
Open

Script for running test exclusion parser in sandbox#6993
adamfarley wants to merge 6 commits intoadoptium:masterfrom
adamfarley:create_shell_script_for_exclude_file_parser

Conversation

@adamfarley
Copy link
Copy Markdown
Contributor

A quickstart script that prepares a sandbox and runs the three core python scripts used for identifying disabled tests and determining whether their associated issue is closed.

Related to Step 4 in #1853

assisted-by: bob

assisted-by: bob

Signed-off-by: Adam Farley <adfarley@ibm.com>
Signed-off-by: Adam Farley <adfarley@ibm.com>
Signed-off-by: Adam Farley <adfarley@ibm.com>
Signed-off-by: Adam Farley <adfarley@ibm.com>
else
log_info "One or more Python requirements not found"
log_info "Installing Python requirements..."
pip3 install -q -r scripts/disabled_tests/requirements.txt
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not sure we should just go and install the required python modules, especially if they are coming from a variable? could be malware?

Signed-off-by: Adam Farley <adfarley@ibm.com>
Signed-off-by: Adam Farley <adfarley@ibm.com>
@smlambert
Copy link
Copy Markdown
Contributor

thanks @adamfarley - Remind me from our previous discussion, how this script is different than the workflow we already have https://github.com/adoptium/aqa-tests/blob/master/.github/workflows/parse-issues.yml and why it is needed or preferred.

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.

4 participants