Skip to content

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

Draft
SamirM-BE wants to merge 8 commits into
masterfrom
feat/smarini/BUILD-12290-windows-warp
Draft

BUILD-12290 Move Windows GitHub-hosted jobs to WarpBuild#5966
SamirM-BE wants to merge 8 commits 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 Windows GitHub-hosted Build jobs to WarpBuild warp-custom-windows-2022-m.
  • Covers ruling-qa Windows matrix entries and qa-os-win.

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

Test plan

  • Confirm Windows ruling QA and qa-os-win schedule on warp-custom-windows-2022-m
  • Confirm those jobs succeed
  • Watch runtime, queue time, and memory on the medium Windows jobs

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Aug 19, 2026 •
edited by atlassian Bot
Loading

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

github-actions Bot commented Aug 19, 2026

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 mentioned this pull request Aug 19, 2026
Closed
@github-actions

github-actions Bot commented Aug 19, 2026 •
edited
Loading

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

github-actions Bot commented Aug 19, 2026

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 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 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 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 needs updating. A fix PR has been created: #5968

Please review and merge it into your branch.

@SamirM-BE
BUILD-12290 Check out ruling sources with LF on Windows Warp
1bda849 
Match qa-os-win git line endings and skip npx/npmjs when generating ruling diffs on Windows.
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

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

Please review and merge it into your branch.

@gitar-bot

gitar-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Migrates Windows GitHub-hosted build jobs to WarpBuild runners and aligns JVM encoding, line endings, and resource settings for the ruling and QA workflows. No issues found.

Implementation Status ✅ 1 / 1 issues implemented
✅ BUILD-12290 — 1 / 1 objectives

The PR successfully replaces Windows GitHub-hosted runners with WarpBuild warp-custom-windows-2022-m in the build workflow.

✅ 1 complete
  • ✅ 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

@sonarqube-next

sonarqube-next Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@SamirM-BE