Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/chainsaw-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
uses: helm/kind-action@v1

- uses: actions/checkout@v4

- name: Install Dependencies
shell: bash
run: |
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/gemini-dispatch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ jobs:
} else {
core.setOutput('command', 'fallthrough');
}

- name: 'Add Gemini helper comment'
if: '${{ github.event_name }}.${{ github.event.action }} == "pull_request.opened"'
env:
Expand All @@ -125,7 +125,7 @@ jobs:

- **`@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?`
Expand All @@ -141,7 +141,7 @@ jobs:
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.*
run: |-
gh pr comment "${PR_NUMBER}" \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/gemini-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ jobs:
2. **Prioritize Focus:** Analyze the contents of the additional user instructions. Use this context to prioritize specific areas in your review (e.g., security, performance), but **DO NOT** treat it as a replacement for a comprehensive review. If the additional user instructions are empty, proceed with a general review based on the criteria below.
3. **Review Code:** Meticulously review the code provided returned from `mcp__github__get_pull_request_diff` according to the **Review Criteria**.
3. **Review Code:** Meticulously review the code provided returned from `mcp__github__get_pull_request_diff` according to the **Review Criteria**.
### Step 2: Formulate Review Comments
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-tekton-kueue-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,4 @@ jobs:
- name: Report success
if: success()
run: |
echo "✅ All Tekton-Kueue CEL expression tests passed successfully"
echo "✅ All Tekton-Kueue CEL expression tests passed successfully"
2 changes: 1 addition & 1 deletion .github/workflows/verify-kueue-queue-configs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,4 @@ jobs:
- name: Report success
if: success()
run: |
echo "✅ All Kueue queue configurations are up to date with their corresponding host-config files"
echo "✅ All Kueue queue configurations are up to date with their corresponding host-config files"
16 changes: 16 additions & 0 deletions .github/workflows/yamllint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
name: Lint manifests with yamllint

on:
pull_request:
branches: [ main ]
push:
branches: [ main ]

jobs:
yamllint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Lint yaml manifests
run: |
yamllint -f github .
10 changes: 10 additions & 0 deletions .yamllint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
extends: relaxed

ignore:
# we don't control the content of helm charts, so don't enforce our standards on them
- '**/charts/'
- '**/templates/'

rules:
indentation: disable
line-length: disable
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ spec:
namespace: argocd-staging
server: https://kubernetes.default.svc
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ spec:
namespace: default # not relevant yet since every object has an explicit namespace.
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: default
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
generators:
- clusters:
values:
sourceRoot: configs/disable-self-provisioning-for-all-cluster
sourceRoot: configs/disable-self-provisioning-for-all-cluster
clusterDir: ""
- list:
elements: []
Expand All @@ -20,7 +20,7 @@ spec:
spec:
project: default
source:
path: configs/disable-self-provisioning-for-all-cluster/
path: configs/disable-self-provisioning-for-all-cluster/
repoURL: https://github.com/redhat-appstudio/infra-deployments.git
targetRevision: main
destination:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: appstudio-grafana
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
2 changes: 1 addition & 1 deletion argo-cd-apps/base/backup/backup.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ spec:
namespace: default
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
2 changes: 1 addition & 1 deletion argo-cd-apps/base/ci-helper-app/ci-helper-app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,4 +41,4 @@ spec:
backoff:
duration: 10s
factor: 2
maxDuration: 3m
maxDuration: 3m
2 changes: 1 addition & 1 deletion argo-cd-apps/base/ci-helper-app/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ kind: Kustomization
resources:
- ci-helper-app.yaml
components:
- ../../k-components/inject-infra-deployments-repo-details
- ../../k-components/inject-infra-deployments-repo-details
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ kind: Kustomization
resources:
- ingresscontroller.yaml
components:
- ../../../k-components/inject-infra-deployments-repo-details
- ../../../k-components/inject-infra-deployments-repo-details
1 change: 0 additions & 1 deletion argo-cd-apps/base/host/sprayproxy/sprayproxy.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@

apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ spec:
namespace: application-api
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ spec:
jqPathExpressions:
- .imagePullSecrets[] | select(.name | startswith("appstudio-pipeline-dockercfg"))
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
2 changes: 1 addition & 1 deletion argo-cd-apps/base/member/infra-deployments/has/has.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: application-service
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ spec:
namespace: internal-services
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -54,4 +54,3 @@ spec:
limit: 50
backoff:
duration: 15s

Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: perf-team-prometheus-reader
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: project-controller
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
namespace: release-service
server: '{{server}}'
syncPolicy:
automated:
automated:
prune: true
selfHeal: true
syncOptions:
Expand Down
2 changes: 1 addition & 1 deletion argo-cd-apps/base/quality-dashboard/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ kind: Kustomization
resources:
- quality-dashboard.yaml
components:
- ../../k-components/inject-infra-deployments-repo-details
- ../../k-components/inject-infra-deployments-repo-details
2 changes: 1 addition & 1 deletion argo-cd-apps/base/quality-dashboard/quality-dashboard.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,4 +41,4 @@ spec:
backoff:
duration: 10s
factor: 2
maxDuration: 3m
maxDuration: 3m
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@ patches:
target:
group: argoproj.io
version: v1alpha1
kind: ApplicationSet
kind: ApplicationSet
3 changes: 1 addition & 2 deletions components/application-api/allow-argocd-to-manage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,4 @@ roleRef:
subjects:
- kind: ServiceAccount
name: openshift-gitops-argocd-application-controller
namespace: openshift-gitops

namespace: openshift-gitops
2 changes: 1 addition & 1 deletion components/application-api/argocd-permissions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,4 +25,4 @@ roleRef:
subjects:
- kind: ServiceAccount
name: openshift-gitops-argocd-application-controller
namespace: application-api
namespace: application-api
2 changes: 1 addition & 1 deletion components/application-api/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ resources:


apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
kind: Kustomization
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

- op: add
path: /spec/template/spec/volumes/-
value:
value:
name: webhook-config
configMap:
name: webhook-config
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ patches:

configMapGenerator:
- name: webhook-config
files:
files:
- webhook-config.json

components:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ patches:

configMapGenerator:
- name: webhook-config
files:
files:
- webhook-config.json

components:
Expand Down
4 changes: 2 additions & 2 deletions components/build-templates/base/0-namespace.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ apiVersion: v1
kind: Namespace
metadata:
name: build-templates
labels:
labels:
argocd.argoproj.io/managed-by: openshift-gitops
spec:
finalizers:
- kubernetes
- kubernetes
12 changes: 6 additions & 6 deletions components/build-templates/base/e2e/rolebinding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: edit-pipeline-from-build-templates
name: edit-pipeline-from-build-templates
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: edit
name: edit
subjects:
- kind: ServiceAccount
name: appstudio-pipeline
Expand All @@ -21,11 +21,11 @@ subjects:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: edit-pipeline-from-build-templates-namespace
name: edit-pipeline-from-build-templates-namespace
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: edit-namespace
name: edit-namespace
subjects:
- kind: ServiceAccount
name: appstudio-pipeline
Expand All @@ -40,11 +40,11 @@ subjects:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: admin-buildpipelineselectors-from-ci-namespaces
name: admin-buildpipelineselectors-from-ci-namespaces
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: admin-buildpipelineselectors
name: admin-buildpipelineselectors
subjects:
- kind: ServiceAccount
name: appstudio-pipeline
Expand Down
2 changes: 1 addition & 1 deletion components/ci-helper-app/base/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ images:
newName: quay.io/konflux-ci/ci-helper-app
newTag: c74e02a133e09ed7cbb628919aaaa5cd3325375d

namespace: ci-helper-app
namespace: ci-helper-app
2 changes: 1 addition & 1 deletion components/ci-helper-app/base/rbac/ci-helper-app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,4 +28,4 @@ subjects:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: ci-helper-app-maintainer
name: ci-helper-app-maintainer
2 changes: 1 addition & 1 deletion components/ci-helper-app/base/rbac/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ci-helper-app.yaml
- ci-helper-app.yaml
2 changes: 1 addition & 1 deletion components/ci-helper-app/development/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ../base
- ../base
Original file line number Diff line number Diff line change
Expand Up @@ -16,4 +16,4 @@ spec:
target:
creationPolicy: Owner
deletionPolicy: Delete
name: ci-helper-app-secrets
name: ci-helper-app-secrets
Loading