You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "adoroszlai (via GitHub)" <gi...@apache.org> on 2025/01/28 08:34:25 UTC

[PR] HDDS-12143. Generate list of splits for integration check dynamically [ozone]

adoroszlai opened a new pull request, #7766:
URL: https://github.com/apache/ozone/pull/7766

   ## What changes were proposed in this pull request?
   
   Integration check runs subsets of tests concurrently in splits.  The list of splits is currently defined by both root POM (as profiles) and in the workflow definition (as matrix variable).  The goal of this change is to remove the hard-coded list from the workflow definition, and generate it dynamically as part of the `build-info` check (like acceptance check suites).
   
   Add `test-` prefix to be able to identify test suite profiles.  Add `-tests` suffix for other type of test profiles (`stable`/`native`).
   
   https://issues.apache.org/jira/browse/HDDS-12143
   
   ## How was this patch tested?
   
   _integration_ check has the same splits as on `master`:
   https://github.com/adoroszlai/ozone/actions/runs/13005461028
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


Re: [PR] HDDS-12143. Generate list of splits for integration check dynamically [ozone]

Posted by "adoroszlai (via GitHub)" <gi...@apache.org>.
adoroszlai commented on PR #7766:
URL: https://github.com/apache/ozone/pull/7766#issuecomment-2623966183

   Thanks @devmadhuu for the review.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


Re: [PR] HDDS-12143. Generate list of splits for integration check dynamically [ozone]

Posted by "adoroszlai (via GitHub)" <gi...@apache.org>.
adoroszlai merged PR #7766:
URL: https://github.com/apache/ozone/pull/7766


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


Re: [PR] HDDS-12143. Generate list of splits for integration check dynamically [ozone]

Posted by "devmadhuu (via GitHub)" <gi...@apache.org>.
devmadhuu commented on code in PR #7766:
URL: https://github.com/apache/ozone/pull/7766#discussion_r1935183144


##########
pom.xml:
##########
@@ -2337,7 +2337,7 @@
       </build>
     </profile>
     <profile>
-      <id>stable</id>
+      <id>stable-tests</id>

Review Comment:
   Is this profile being used as of today ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


Re: [PR] HDDS-12143. Generate list of splits for integration check dynamically [ozone]

Posted by "adoroszlai (via GitHub)" <gi...@apache.org>.
adoroszlai commented on code in PR #7766:
URL: https://github.com/apache/ozone/pull/7766#discussion_r1935210235


##########
pom.xml:
##########
@@ -2337,7 +2337,7 @@
       </build>
     </profile>
     <profile>
-      <id>stable</id>
+      <id>stable-tests</id>

Review Comment:
   I don't think so.  Probably can be removed in some cleanup task.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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