Skip to content
Merged
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
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -392,8 +392,8 @@ endif
bundle: manifests kustomize operator-sdk ## Generate bundle manifests and metadata, then validate generated files.
$(OPERATOR_SDK) generate kustomize manifests -q
cd config/operator/deployment/ && $(KUSTOMIZE) edit set image controller=$(IMG) && \
$(KUSTOMIZE) edit add patch --kind Deployment --name openstack-operator-controller-operator --namespace system --patch "[{\"op\": \"replace\", \"path\": \"/spec/template/spec/containers/0/env/0\", \"value\": {\"name\": \"OPENSTACK_RELEASE_VERSION\", \"value\": \"$(OPENSTACK_RELEASE_VERSION)\"}}]" && \
$(KUSTOMIZE) edit add patch --kind Deployment --name openstack-operator-controller-operator --namespace system --patch "[{\"op\": \"replace\", \"path\": \"/spec/template/spec/containers/0/env/1\", \"value\": {\"name\": \"OPERATOR_IMAGE_URL\", \"value\": \"$(IMG)\"}}]"
$(KUSTOMIZE) edit add patch --kind Deployment --name openstack-operator-controller-init --namespace system --patch "[{\"op\": \"replace\", \"path\": \"/spec/template/spec/containers/0/env/0\", \"value\": {\"name\": \"OPENSTACK_RELEASE_VERSION\", \"value\": \"$(OPENSTACK_RELEASE_VERSION)\"}}]" && \
$(KUSTOMIZE) edit add patch --kind Deployment --name openstack-operator-controller-init --namespace system --patch "[{\"op\": \"replace\", \"path\": \"/spec/template/spec/containers/0/env/1\", \"value\": {\"name\": \"OPERATOR_IMAGE_URL\", \"value\": \"$(IMG)\"}}]"
$(KUSTOMIZE) build config/operator --load-restrictor='LoadRestrictionsNone' | $(OPERATOR_SDK) generate bundle $(BUNDLE_GEN_FLAGS)
$(OPERATOR_SDK) bundle validate ./bundle

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -436,6 +436,13 @@ spec:
- description: TLS - overrides tls parameters for public endpoint
displayName: TLS
path: telemetry.aodhApiOverride.tls
- description: CloudKittyAPIOverride, provides the ability to override the generated
manifest of several child resources.
displayName: Cloud Kitty APIOverride
path: telemetry.cloudKittyApiOverride
- description: TLS - overrides tls parameters for public endpoint
displayName: TLS
path: telemetry.cloudKittyApiOverride.tls
- description: Enabled - Whether OpenStack Telemetry services should be deployed
and managed
displayName: Enabled
Expand Down
2 changes: 1 addition & 1 deletion config/operator/default_images.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
spec:
template:
Expand Down
12 changes: 6 additions & 6 deletions config/operator/deployment/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,23 +2,23 @@ apiVersion: v1
kind: Namespace
metadata:
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
control-plane: controller-manager
name: system
---
apiVersion: apps/v1
kind: Deployment
metadata:
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
control-plane: controller-manager
openstack.org/operator-name: openstack-init
spec:
selector:
matchLabels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
control-plane: controller-manager
openstack.org/operator-name: openstack-init
replicas: 1
Expand All @@ -27,7 +27,7 @@ spec:
annotations:
kubectl.kubernetes.io/default-container: operator
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
control-plane: controller-manager
openstack.org/operator-name: openstack-init
spec:
Expand Down Expand Up @@ -89,7 +89,7 @@ spec:
requests:
cpu: 10m
memory: 128Mi
serviceAccountName: openstack-operator-controller-operator
serviceAccountName: openstack-operator-controller-init
terminationGracePeriodSeconds: 10
tolerations:
- key: "node.kubernetes.io/not-ready"
Expand Down
13 changes: 13 additions & 0 deletions config/operator/deployment/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,3 +14,16 @@ images:
- name: controller
newName: quay.io/openstack-k8s-operators/openstack-operator
newTag: latest
patches:
- patch: '[{"op": "replace", "path": "/spec/template/spec/containers/0/env/0", "value":
{"name": "OPENSTACK_RELEASE_VERSION", "value": "0.5.0-1767875480"}}]'
target:
kind: Deployment
name: openstack-operator-controller-init
namespace: system
- patch: '[{"op": "replace", "path": "/spec/template/spec/containers/0/env/1", "value":
{"name": "OPERATOR_IMAGE_URL", "value": "quay.io/openstack-k8s-operators/openstack-operator:latest"}}]'
target:
kind: Deployment
name: openstack-operator-controller-init
namespace: system
2 changes: 1 addition & 1 deletion config/operator/manager_operator_images.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
spec:
template:
Expand Down
2 changes: 1 addition & 1 deletion config/operator/rbac/leader_election_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
app.kubernetes.io/managed-by: kustomize
name: leader-election-role-operator
rules:
Expand Down
2 changes: 1 addition & 1 deletion config/operator/rbac/leader_election_role_binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,5 @@ roleRef:
name: leader-election-role-operator
subjects:
- kind: ServiceAccount
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
2 changes: 1 addition & 1 deletion config/operator/rbac/metrics_auth_role_binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ roleRef:
name: metrics-auth-role-operator
subjects:
- kind: ServiceAccount
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
4 changes: 2 additions & 2 deletions config/operator/rbac/role_binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
app.kubernetes.io/managed-by: kustomize
name: operator-rolebinding
roleRef:
Expand All @@ -11,5 +11,5 @@ roleRef:
name: operator-role
subjects:
- kind: ServiceAccount
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
4 changes: 2 additions & 2 deletions config/operator/rbac/service_account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
labels:
app.kubernetes.io/name: openstack-operator-controller-operator
app.kubernetes.io/name: openstack-operator-controller-init
app.kubernetes.io/managed-by: kustomize
name: controller-operator
name: controller-init
namespace: system
2 changes: 1 addition & 1 deletion hack/run_with_local_webhook.sh
Original file line number Diff line number Diff line change
Expand Up @@ -400,7 +400,7 @@ if [ -n "${CSV_NAME}" ]; then

oc patch "${CSV_NAME}" -n openstack-operators --type=json -p="[{'op': 'replace', 'path': '/spec/install/spec/deployments/0/spec/replicas', 'value': 0}]"
oc patch "${CSV_NAME}" -n openstack-operators --type=json -p="[{'op': 'replace', 'path': '/spec/webhookdefinitions', 'value': []}]"
oc wait -n openstack-operators --for=jsonpath='{.spec.replicas}'=0 deploy/openstack-operator-controller-operator
oc wait -n openstack-operators --for=jsonpath='{.spec.replicas}'=0 deploy/openstack-operator-controller-init
oc scale --replicas=0 -n openstack-operators deploy/openstack-operator-controller-manager
oc delete --ignore-not-found=true validatingwebhookconfiguration openstack-operator-validating-webhook-configuration
oc delete --ignore-not-found=true mutatingwebhookconfiguration openstack-operator-mutating-webhook-configuration
Expand Down
2 changes: 1 addition & 1 deletion hack/sync-bindata.sh
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,7 @@ cat > ../config/operator/manager_operator_images.yaml <<EOF_CAT
apiVersion: apps/v1
kind: Deployment
metadata:
name: openstack-operator-controller-operator
name: openstack-operator-controller-init
namespace: system
spec:
template:
Expand Down
2 changes: 1 addition & 1 deletion internal/dataplane/util/image_registry_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

package util
package util //nolint:revive // util is an acceptable package name in this context

import (
"context"
Expand Down