Project

General

Profile

Actions

action #124688

open

coordination #96596: [qe-core][CI] CI/CD and Coding style improvements

[qe-core] Reduce execution of compile tests

Added by szarate about 1 year ago. Updated 11 months ago.

Status:
Blocked
Priority:
Normal
Assignee:
Category:
Refactor/Code Improvements
Target version:
Start date:
2023-02-16
Due date:
% Done:

0%

Estimated time:
Difficulty:
Sprint:
QE-Core: April Sprint 23 (Apr 05 - May 03)

Description

In our CI, we have a job that calls the compile target which takes around 9 minutes

export PERL5LIB=../..:os-autoinst:lib:tests/installation:tests/x11:tests/qa_automation:tests/virt_autotest:tests/cpu_bugs:tests/sles4sap/saptune:$PERL5LIB: ; ( git ls-files "*.pm" || find . -name \*.pm|grep -v /os-autoinst/ ) | parallel perl -c 2>&1 | grep -v " OK$" && exit 2; true

And this is looking for all of the files per every PR; if we manage to find a way how to find the files that changed between a PR and what is on master, we can shave a lot of seconds there, and enable it fully after things are merged, as we don't really care about the time there


Files

Screenshot from 2023-03-06 16-04-04.png (92 KB) Screenshot from 2023-03-06 16-04-04.png required workflow rfan1, 2023-03-06 08:10
Actions #1

Updated by rfan1 about 1 year ago

  • Status changed from New to Workable
  • Assignee set to rfan1
Actions #2

Updated by rfan1 about 1 year ago

Seems we have the code there already:

.PHONY: test-compile
test-compile: check-links
        export PERL5LIB=${PERL5LIB_}:$(shell ./tools/wheel --verify) ; ( git ls-files "*.pm" || find . -name \*.pm|grep -v /os-autoinst/ ) | parallel perl -c 2>&1 | grep -v " OK$$" && exit 2; true

.PHONY: test-compile-changed
test-compile-changed: os-autoinst/
        export PERL5LIB=${PERL5LIB_}:$(shell ./tools/wheel --verify) ; for f in `git diff --name-only | grep '.pm'` ; do perl -c $$f 2>&1 | grep -v " OK$$" && exit 2; done ; true

Actions #3

Updated by rfan1 about 1 year ago

  • Status changed from Workable to In Progress
Actions #6

Updated by rfan1 about 1 year ago

Please refer to the attached file, that compile tests on pull request phase is required one so that I can not skip it.

https://docs.github.com/en/actions/using-workflows/required-workflows

We may need some github owners to change the default workflow.

Actions #7

Updated by szarate about 1 year ago

  • Sprint changed from QE-Core: February Sprint (Feb 08 - Mar 08) to QE-Core: March Sprint (Mar 08 - Apr 05)
Actions #8

Updated by szarate about 1 year ago

  • Sprint changed from QE-Core: March Sprint (Mar 08 - Apr 05) to QE-Core: April Sprint 23 (Apr 05 - May 03)
Actions #9

Updated by rfan1 11 months ago

  • Status changed from In Progress to Blocked
Actions

Also available in: Atom PDF