Add requestProcessor parameter

Author: rnc

java-components/cli/src/main/java/com/redhat/hacbs/cli/driver/Pipeline.java

@@ -21,14 +21,17 @@ public class Pipeline extends Base implements Runnable {
     @Inject
     Driver driver;
 
-    @CommandLine.Option(names = "-q", description = "Quay repo", defaultValue = "quay.io/redhat-user-workloads-stage/pnc-devel-tenant/pnc")
+    @CommandLine.Option(names = "--quay", description = "Quay repo", defaultValue = "quay.io/redhat-user-workloads-stage/pnc-devel-tenant/pnc")
     String quayRepo;
 
+    @CommandLine.Option(names = "--processor", description = "Request Process Image", defaultValue = "quay.io/redhat-user-workloads/konflux-jbs-pnc-tenant/jvm-build-service/build-request-processor:latest")
+    String processor;
+
     @ActivateRequestContext // https://github.com/quarkusio/quarkus/issues/8758
     @Override
     public void run() {
         logger.info("### in here with driver {}", driver);
-        driver.addValues(accessToken.orElse("NO_TOKEN"), quayRepo);
+        driver.addValues(accessToken.orElse(""), quayRepo, processor);
 
         BuildRequest request = BuildRequest.builder()
                 .namespace(namespace)

java-components/driver/src/main/java/com/redhat/hacbs/driver/Driver.java

@@ -47,10 +47,12 @@ public class Driver {
 
     private String accessToken;
     private String quayRepo;
+    private String processor;
 
-    public void addValues(String accessToken, String quayRepo) {
+    public void addValues(String accessToken, String quayRepo, String processor) {
         this.accessToken = accessToken;
         this.quayRepo = quayRepo;
+        this.processor = processor;
     }
 
     public void create(BuildRequest buildRequest) throws IOException {
@@ -63,6 +65,7 @@ public void create(BuildRequest buildRequest) throws IOException {
         }
 
         Map<String, String> templateProperties = new HashMap<>();
+        templateProperties.put("REQUEST_PROCESSOR", processor);
         templateProperties.put("QUAY_REPO", quayRepo);
         templateProperties.put("URL", buildRequest.getScmUrl());
         templateProperties.put("REVISION", buildRequest.getScmRevision());
@@ -93,7 +96,9 @@ public void create(BuildRequest buildRequest) throws IOException {
      * @return fresh access token
      */
     public String getFreshAccessToken() {
-        return oidcClient.getTokens().await().indefinitely().getAccessToken();
+        var result = oidcClient.getTokens().await().indefinitely().getAccessToken();
+        System.err.println("### result " + result);
+        return result;
     }
 
     private <T> T createModelNode(String resourceDefinition, Map<String, String> properties, Class<T> clazz) {

java-components/driver/src/main/resources/pipeline.yaml

@@ -70,8 +70,7 @@ spec:
     - name: ENABLE_INDY_PROXY
       value: "false"
     - name: JVM_BUILD_SERVICE_REQPROCESSOR_IMAGE
-      value: "quay.io/ncross/hacbs-jvm-build-request-processor:latest"
-      # "quay.io/redhat-user-workloads/konflux-jbs-pnc-tenant/jvm-build-service/build-request-processor:latest"
+      value: "%{REQUEST_PROCESSOR}"
   # TODO: Should PNC set both limits and requests? See
   #   https://home.robusta.dev/blog/kubernetes-memory-limit
   #   https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/