Skip to content

BUILD-12290 Move Windows GitHub-hosted jobs to WarpBuild - #5966

Merged
SamirM-BE merged 1 commit into
masterfrom
feat/smarini/BUILD-12290-windows-warp
Aug 20, 2026
Merged

BUILD-12290 Move Windows GitHub-hosted jobs to WarpBuild#5966
SamirM-BE merged 1 commit into
masterfrom
feat/smarini/BUILD-12290-windows-warp

Conversation

@SamirM-BE

@SamirM-BE SamirM-BE commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move the Windows CI jobs to WarpBuild:
    • qa-os-win and the only-sonarqube-project ruling profile use warp-custom-windows-2022-m.
    • The full without-sonarqube-project ruling profile uses warp-custom-windows-2022-l, which provides the capacity required by that analysis.
  • Preserve the existing ruling profiles, Maven goals, SonarQube runtime, and expected ruling dumps.
  • Use Windows-safe workflow settings: LF checkout, explicit Java 21 for ruling, cleared inherited JVM options, an 8 GiB Maven heap for the Windows unit-test job, and serial nested Maven execution to avoid local-repository write races.

Behavior and trade-offs

  • The full Windows ruling profile remains functionally equivalent, but uses the large runner and runs its nested Maven tests without method-level parallelism. This can increase capacity usage and elapsed time.
  • On a Windows ruling failure, the raw actual-results artifact is still uploaded. The optional HTML diff report is skipped because its generator adds a separate npm dependency step to the failure path.
  • No source analysis scope, quality profiles, expected ruling results, or release behavior is changed.

Validation

  • Windows unit tests pass on warp-custom-windows-2022-m.
  • Windows only-sonarqube-project ruling passes on warp-custom-windows-2022-m.
  • Windows without-sonarqube-project ruling passes on warp-custom-windows-2022-l.
  • Linux ruling, plugin QA, sanity, Test Analyze, custom-rules/license, SCA, quality analysis, and required security checks pass.

The full ruling profile needs the large runner: the medium runner made 3 GiB available to the scanner, while the large runner completed with the shared expected ruling results.

Parent: https://sonarsource.atlassian.net/browse/BUILD-12260
Sub-task: https://sonarsource.atlassian.net/browse/BUILD-12290

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

BUILD-12290

@SamirM-BE
SamirM-BE force-pushed the feat/smarini/BUILD-12290-windows-warp branch from 57d07b0 to f868f3b Compare August 19, 2026 14:17
@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Ruling Diff Summary

Detected changes in 4 rule files: 0 issues removed, 36 issues added.

S9355 (java) on eclipse-jetty - 0 issues removed, 16 issues added - new ruling file

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 56)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 75)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 105)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 1115)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 1994)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 2002)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 596)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 609)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 626)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ScopedHandler.java (line 204)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ScopedHandler.java)

Added jetty-util/src/main/java/org/eclipse/jetty/util/IteratingCallback.java (line 420)

(source file not found at this revision: jetty-util/src/main/java/org/eclipse/jetty/util/IteratingCallback.java)

Added jetty-util/src/main/java/org/eclipse/jetty/util/ssl/X509.java (line 41)

(source file not found at this revision: jetty-util/src/main/java/org/eclipse/jetty/util/ssl/X509.java)

Added jetty-util/src/main/java/org/eclipse/jetty/util/ssl/X509.java (line 46)

(source file not found at this revision: jetty-util/src/main/java/org/eclipse/jetty/util/ssl/X509.java)
S9355 (java) on eclipse-jetty-similar-to-main - 0 issues removed, 13 issues added - new ruling file

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 56)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 75)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java (line 105)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 1115)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 1994)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/Request.java (line 2002)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/Request.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 596)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 609)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java (line 626)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java)

Added jetty-server/src/main/java/org/eclipse/jetty/server/handler/ScopedHandler.java (line 204)

(source file not found at this revision: jetty-server/src/main/java/org/eclipse/jetty/server/handler/ScopedHandler.java)
S9355 (java) on guava - 0 issues removed, 5 issues added - new ruling file

Added src/com/google/common/collect/ImmutableEnumSet.java (line 46)

        41 |       default:
        42 |         return new ImmutableEnumSet(set);
        43 |     }
        44 |   }
        45 | 
>>>     46 |   /*
        47 |    * Notes on EnumSet and <E extends Enum<E>>:
        48 |    *
        49 |    * This class isn't an arbitrary ForwardingImmutableSet because we need to
        50 |    * know that calling {@code clone()} during deserialization will return an
        51 |    * object that no one else has a reference to, allowing us to guarantee

Added src/com/google/common/collect/LinkedListMultimap.java (line 105)

       100 |  * @since 2.0
       101 |  */
       102 | @GwtCompatible(serializable = true, emulated = true)
       103 | public class LinkedListMultimap<K, V> extends AbstractMultimap<K, V>
       104 |     implements ListMultimap<K, V>, Serializable {
>>>    105 |   /*
       106 |    * Order is maintained using a linked list containing all key-value pairs. In
       107 |    * addition, a series of disjoint linked lists of "siblings", each containing
       108 |    * the values for a specific key, is used to implement {@link
       109 |    * ValueForKeyIterator} in constant time.
       110 |    */

Added src/com/google/common/net/PercentEscaper.java (line 136)

       131 |       octets[c] = true;
       132 |     }
       133 |     return octets;
       134 |   }
       135 | 
>>>    136 |   /*
       137 |    * Overridden for performance. For unescaped strings this improved the
       138 |    * performance of the uri escaper from ~760ns to ~400ns as measured by
       139 |    * {@link CharEscapersBenchmark}.
       140 |    */
       141 |   @Override

Added src/com/google/common/net/PercentEscaper.java (line 153)

       148 |       }
       149 |     }
       150 |     return index;
       151 |   }
       152 | 
>>>    153 |   /*
       154 |    * Overridden for performance. For unescaped strings this improved the
       155 |    * performance of the uri escaper from ~400ns to ~170ns as measured by
       156 |    * {@link CharEscapersBenchmark}.
       157 |    */
       158 |   @Override

Added src/com/google/common/util/concurrent/AbstractIdleService.java (line 39)

        34 |  * @since 1.0
        35 |  */
        36 | @Beta
        37 | public abstract class AbstractIdleService implements Service {
        38 | 
>>>     39 |   /* Thread names will look like {@code "MyService STARTING"}. */
        40 |   private final Supplier<String> threadNameSupplier = new Supplier<String>() {
        41 |     @Override public String get() {
        42 |       return serviceName() + " " + state();
        43 |     }
        44 |   };
S9353 (java) on regex-examples - 0 issues removed, 2 issues added - new ruling file

Added src/main/java/org/regex/examples/RegexDatabase2.java (line 457)

       452 |     // https://github.com/googleapis/google-api-java-client-services/blob/81e2a6698f5f960ae493c0acdefe830514531bf5/clients/google-api-services-accessapproval/v1/1.31.0/com/google/api/services/accessapproval/v1/AccessApproval.java#L2442
       453 |     Pattern.compile("^projects/[^/]+/accessApprovalSettings$"),
       454 |     // https://github.com/googleapis/google-api-java-client-services/blob/81e2a6698f5f960ae493c0acdefe830514531bf5/clients/google-api-services-accessapproval/v1/1.31.0/com/google/api/services/accessapproval/v1/AccessApproval.java#L2920
       455 |     Pattern.compile("^projects/[^/]+/approvalRequests/[^/]+$"),
       456 |     // https://github.com/atp-mipt/jsyntrax/blob/4ca79decf441cba9ab67928935c57efdd33a348c/jsyntrax/src/main/java/org/atpfivt/jsyntrax/styles/NodeTokenStyle.java#L11
>>>    457 |     Pattern.compile("."),
       458 |     // https://github.com/atp-mipt/jsyntrax/blob/4ca79decf441cba9ab67928935c57efdd33a348c/jsyntrax/src/main/java/org/atpfivt/jsyntrax/styles/NodeHexStyle.java#L11
       459 |     Pattern.compile("^\\w"),
       460 |     // https://github.com/atp-mipt/jsyntrax/blob/4ca79decf441cba9ab67928935c57efdd33a348c/jsyntrax/src/main/java/org/atpfivt/jsyntrax/styles/NodeBoxStyle.java#L10
       461 |     Pattern.compile("^/"),
       462 |     // https://github.com/saiprakash774/Userregistration/blob/33ca8b01235c6da18ebe0380b950118bd1789eea/UserRegistration.java#L15

Added src/main/java/org/regex/examples/RegexDatabase8.java (line 1129)

      1124 |     // https://github.com/LmaaMiracle/StudentAssistantBot/blob/ad0fb69c2e5f3bec786dbf949879448b21c95be9/documentation/4_Software_Construction/4.2_Construction_of_Program%20Modules/4.2.3%20%D0%9E%D1%81%D0%BE%D0%B1%D0%BB%D0%B8%D0%B2%D0%BE%D1%81%D1%82%D1%96%20%D1%81%D1%82%D0%B2%D0%BE%D1%80%D0%B5%D0%BD%D0%BD%D1%8F%20%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BD%D0%B8%D1%85%20%D0%BA%D0%BB%D0%B0%D1%81%D1%96%D0%B2/BotState.java#L374
      1125 |     Pattern.compile("([01][0-9]|2[0-3]):[0-5][0-9]"),
      1126 |     // https://github.com/dearcode2018/file/blob/857c774411c3689f2613d2c8d945b8b78e3214d2/poi/src/main/java/com/hua/util/POIUtil.java#L512
      1127 |     Pattern.compile("\\(.+\\)"),
      1128 |     // https://github.com/raise-isayan/YaguraExtender/blob/04d52bd802c91ef4ffb957898badb03248fa0724/src/main/java/extend/util/external/TransUtil.java#L166
>>>   1129 |     Pattern.compile(".", Pattern.DOTALL),
      1130 |     // https://github.com/raise-isayan/YaguraExtender/blob/04d52bd802c91ef4ffb957898badb03248fa0724/src/main/java/extend/util/external/TransUtil.java#L168
      1131 |     Pattern.compile("[^A-Za-z0-9!\"$'()*,/:<>@\\[\\\\\\]^`{|}~]"),
      1132 |     // https://github.com/raise-isayan/YaguraExtender/blob/04d52bd802c91ef4ffb957898badb03248fa0724/src/main/java/extend/util/external/TransUtil.java#L169
      1133 |     Pattern.compile("[^A-Za-z0-9\"<>\\[\\\\\\]^`{|}]"),
      1134 |     // https://github.com/raise-isayan/YaguraExtender/blob/04d52bd802c91ef4ffb957898badb03248fa0724/src/main/java/extend/util/external/TransUtil.java#L170

@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

5 similar comments
@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@SamirM-BE
SamirM-BE marked this pull request as ready for review August 20, 2026 08:06
@SamirM-BE
SamirM-BE force-pushed the feat/smarini/BUILD-12290-windows-warp branch 3 times, most recently from 1166677 to 7f8f67b Compare August 20, 2026 09:19
@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5971

Please review and merge it into your branch.

@SamirM-BE
SamirM-BE force-pushed the feat/smarini/BUILD-12290-windows-warp branch from 7f8f67b to f15ad9a Compare August 20, 2026 09:22
@github-actions

Copy link
Copy Markdown
Contributor

Ruling needs updating. A fix PR has been created: #5971

Please review and merge it into your branch.

Run Windows CI workloads on WarpBuild and give the full ruling analysis the required runner capacity.
@SamirM-BE
SamirM-BE force-pushed the feat/smarini/BUILD-12290-windows-warp branch from f15ad9a to 011ab2c Compare August 20, 2026 09:41
@sonarqube-next

Copy link
Copy Markdown
Contributor

@SamirM-BE
SamirM-BE merged commit 4b2830d into master Aug 20, 2026
17 checks passed
@SamirM-BE
SamirM-BE deleted the feat/smarini/BUILD-12290-windows-warp branch August 20, 2026 11:50
@gitar-bot

gitar-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Migrates Windows GitHub-hosted CI jobs to WarpBuild's custom runner with updated Maven options and disabled parallel methods for ruling tests. No issues found.

Implementation Status 🟡 0 / 1 issues implemented
BUILD-12290 — 0 / 1 objectives

The PR does not modify any workflow files to change the Windows GitHub-hosted runners to WarpBuild.

  • ⬜ Replace Windows GitHub-hosted runners in sonar-java Build with WarpBuild warp-custom-windows-2022-m
Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

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.

2 participants