You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/02/09 06:59:00 UTC

[jira] [Updated] (HDDS-6186) Selective checks: skip integration check for unit test changes

     [ https://issues.apache.org/jira/browse/HDDS-6186?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

ASF GitHub Bot updated HDDS-6186:
---------------------------------
    Labels: pull-request-available  (was: )

> Selective checks: skip integration check for unit test changes
> --------------------------------------------------------------
>
>                 Key: HDDS-6186
>                 URL: https://issues.apache.org/jira/browse/HDDS-6186
>             Project: Apache Ozone
>          Issue Type: Improvement
>          Components: build, test
>            Reporter: Attila Doroszlai
>            Assignee: Attila Doroszlai
>            Priority: Major
>              Labels: pull-request-available
>
> _unit_ and _integration_ checks run tests in mutually exclusive submodules. We may skip _integration_ check if only unit *tests* are changed.  However, we need to run _integration_ if any helper is changed, since we don't know if it's used by integration tests or not.
> We can distinguish tests and test utilities by:
> # file name pattern ({{Test\*.java}} are tests, others are utilities) OR
> # extracting unit tests into separate submodule (similar to {{ratis-test}} in Ratis)
> The first approach is quick and dirty.
> The second one needs a bit more work, but is clean and has the additional benefit that all tests can be run despite any failures in dependencies.  (Currently a test failure in e.g. {{hdds-common}} causes build/test in {{hdds-server-framework}} and other modules that depend on it.)  The downside is that it may cause many merge conflicts for everyone (feature branches, other in-progress work, open PRs).
> I propose to do it now the first way, then do it properly after current feature branches are merged.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org