Skip to content

Commit ded4989

Browse files
authored
Bump operator to v1.0.212 (#3197)
## Description This also adds an operator `OWNERS` file and will request review the usernames in that file if the operator manifests or rbac check fail ## How Has This Been Tested? <!-- Describe the tests you ran and how you verified your changes. --> - [ ] Added Unit Tests - [ ] Updated e2e Tests - [ ] Manual Testing - [ ] Manual Load Test ## Kubernetes Checklist <!-- If this PR affects how Odigos interacts with Kubernetes, check the relevant boxes below and provide more details --> - [ ] Changes how Odigos interacts with Kubernetes - [ ] Introduces additional calls to the API Server (potential performance impact) - [ ] New Query/feature supported in all the k8s versions supported by Odigos - [ ] Modifies Odigos manifests (addressed in both CLI and Helm) - [ ] Changes RBAC permissions ## User Facing Changes <!-- Any changes that users will notice or need to be aware of --> - [ ] Users need to take action before upgrading - [ ] Automatic migration will modify existing objects (backward compatible) - [ ] Changes UI, CLI, or K8s Manifests aspects in a way that users need to be aware of - [ ] Documentation updated accordingly
1 parent 4baae1c commit ded4989

10 files changed

+465
-444
lines changed

.github/workflows/check-operator-manifests.yml

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ jobs:
1717
go-version: '1.24'
1818

1919
- name: Check operator manifests are up to date
20+
id: check-manifests
2021
run: |
2122
cd operator
2223
USE_IMAGE_DIGESTS=true make generate manifests
@@ -26,4 +27,20 @@ jobs:
2627
echo "::error::Please run cd operator/ && USE_IMAGE_DIGESTS=true make generate manifests"
2728
git diff
2829
exit 1
29-
fi
30+
fi
31+
32+
- name: Add operator owners as reviewers if check-operator-manifests failed
33+
if: failure() && steps.check-manifests.outcome == 'failure'
34+
uses: actions/github-script@v7
35+
with:
36+
script: |
37+
const fs = require('fs');
38+
const ownersText = fs.readFileSync('operator/OWNERS', 'utf8');
39+
const usernames = ownersText.trim().split('\n').filter(line => line.trim() !== '');
40+
41+
github.rest.pulls.requestReviewers({
42+
owner: context.repo.owner,
43+
repo: context.repo.repo,
44+
pull_number: context.issue.number,
45+
reviewers: usernames
46+
});

.github/workflows/check-operator-rbac.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,3 +48,19 @@ jobs:
4848
echo "::error::Component RBAC updated without corresponding permissions changed in Operator, please see https://github.com/odigos-io/odigos/tree/main/operator#permissions or apply the 'operator-rbac-approved' label to bypass this check."
4949
exit 1
5050
fi
51+
52+
- name: Add operator owners as reviewers if check-operator-rbac failed
53+
if: failure()
54+
uses: actions/github-script@v7
55+
with:
56+
script: |
57+
const fs = require('fs');
58+
const ownersText = fs.readFileSync('operator/OWNERS', 'utf8');
59+
const usernames = ownersText.trim().split('\n').filter(line => line.trim() !== '');
60+
61+
github.rest.pulls.requestReviewers({
62+
owner: context.repo.owner,
63+
repo: context.repo.repo,
64+
pull_number: context.issue.number,
65+
reviewers: usernames
66+
});

operator/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
# To re-generate a bundle for another specific version without changing the standard setup, you can:
44
# - use the VERSION as arg of the bundle target (e.g make bundle VERSION=0.0.2)
55
# - use environment variables to overwrite this value (e.g export VERSION=0.0.2)
6-
VERSION ?= 1.0.206
6+
VERSION ?= 1.0.212
77

88
# CHANNELS define the bundle channels used in the bundle.
99
# Add a new line here if you would like to change its default config. (E.g CHANNELS = "candidate,fast,stable")

operator/OWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
damemi
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
apiVersion: v1
2+
data:
3+
ODIGOS_VERSION: 1.0.212
4+
kind: ConfigMap
5+
metadata:
6+
name: odigos-operator-odigos-version-5472996t4d

operator/bundle/manifests/odigos-operator-odigos-version-f982d26dtt_v1_configmap.yaml

Lines changed: 0 additions & 6 deletions
This file was deleted.

0 commit comments

Comments
 (0)