Skip to content

Conversation

@JoaoPedroPP
Copy link
Contributor

@JoaoPedroPP JoaoPedroPP commented Dec 11, 2025

Deploy kite to stone-stage-p01 cluster (Private konflux instance)

KFLUXUI-943

@openshift-ci openshift-ci bot requested review from Katka92 and testcara December 11, 2025 13:27
@github-actions
Copy link
Contributor

🤖 Gemini AI Assistant Available

Hi @JoaoPedroPP! I'm here to help with your pull request. You can interact with me using the following commands:

Available Commands

  • @gemini-cli /review - Request a comprehensive code review

    • Example: @gemini-cli /review Please focus on security and performance
  • @gemini-cli <your question> - Ask me anything about the codebase

    • Example: @gemini-cli How can I improve this function?
    • Example: @gemini-cli What are the best practices for error handling here?

How to Use

  1. Simply type one of the commands above in a comment on this PR
  2. I'll analyze your code and provide detailed feedback
  3. You can track my progress in the workflow logs

Permissions

Only OWNER, MEMBER, or COLLABORATOR users can trigger my responses. This ensures secure and appropriate usage.


This message was automatically added to help you get started with the Gemini AI assistant. Feel free to delete this comment if you don't need assistance.

@github-actions
Copy link
Contributor

🤖 Hi @JoaoPedroPP, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@JoaoPedroPP
Copy link
Contributor Author

Pinging @gbenhaim @hugares since I am not sure if there is any process to deploy it to a new cluster and I don't want to blow everything up :)

@@ -0,0 +1,33 @@
apiVersion: kustomize.config.k8s.io/v1beta1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kite application is not deployed in staging-downtream overlay, you need to remove the patch deleting the application but also change the applicatset to generate application with right path. Here is the diff of what is missing:

git diff
diff --git a/argo-cd-apps/base/member/infra-deployments/konflux-kite/konflux-kite.yaml b/argo-cd-apps/base/member/infra-deployments/konflux-kite/konflux-kite.yaml
index d53caf967..63a1e29f0 100644
--- a/argo-cd-apps/base/member/infra-deployments/konflux-kite/konflux-kite.yaml
+++ b/argo-cd-apps/base/member/infra-deployments/konflux-kite/konflux-kite.yaml
@@ -19,6 +19,8 @@ spec:
                 # Staging clusters
                 - nameNormalized: stone-stg-rh01
                   values.clusterDir: stone-stg-rh01
+                - nameNormalized: stone-stage-p01
+                  values.clusterDir: stone-stage-p01
   template:
     metadata:
       name: konflux-kite-{{nameNormalized}}
diff --git a/argo-cd-apps/overlays/staging-downstream/delete-applications.yaml b/argo-cd-apps/overlays/staging-downstream/delete-applications.yaml
index 95c9b33ab..44583d45e 100644
--- a/argo-cd-apps/overlays/staging-downstream/delete-applications.yaml
+++ b/argo-cd-apps/overlays/staging-downstream/delete-applications.yaml
@@ -10,10 +10,3 @@ kind: ApplicationSet
 metadata:
   name: quality-dashboard
 $patch: delete
----
-# At this time we're starting with external staging
-apiVersion: argoproj.io/v1alpha1
-kind: ApplicationSet
-metadata:
-  name: konflux-kite
-$patch: delete

Copy link
Contributor Author

@JoaoPedroPP JoaoPedroPP Dec 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated in new version.

Out of curiosity, why did we need to delete konflux-kite ApplicationSet if we only had deploy set?

Deploy kite to stone-stage-p01 cluster (Private konflux instance)

KFLUXUI-943
@openshift-ci
Copy link

openshift-ci bot commented Dec 12, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: JoaoPedroPP
Once this PR has been reviewed and has the lgtm label, please assign hugares for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot removed the approved label Dec 12, 2025
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