Skip to content

Commit 7e76ada

Browse files
committed
Do not expose debug section anymore
This patch updates the existing API and removes the debug section that was supposed to run the Pod in sleep mode for troubleshooting purposes. This is still possible by using the regular k8s approach, and it's described by [1]. [1] openstack-k8s-operators/dev-docs#90 Signed-off-by: Francesco Pantano <fpantano@redhat.com>
1 parent 053fb28 commit 7e76ada

13 files changed

Lines changed: 27 additions & 394 deletions

api/bases/glance.openstack.org_glanceapis.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,6 @@ spec:
6060
databaseUser:
6161
default: glance
6262
type: string
63-
debug:
64-
properties:
65-
service:
66-
default: false
67-
type: boolean
68-
type: object
6963
extraMounts:
7064
items:
7165
properties:

api/bases/glance.openstack.org_glances.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,6 @@ spec:
5454
cronJob:
5555
default: false
5656
type: boolean
57-
dbSync:
58-
default: false
59-
type: boolean
6057
type: object
6158
extraMounts:
6259
items:
@@ -825,12 +822,6 @@ spec:
825822
items:
826823
type: string
827824
type: array
828-
debug:
829-
properties:
830-
service:
831-
default: false
832-
type: boolean
833-
type: object
834825
networkAttachments:
835826
items:
836827
type: string

api/v1beta1/common_types.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,6 @@ type GlanceAPITemplate struct {
5959
// NodeSelector to target subset of worker nodes running this service
6060
NodeSelector map[string]string `json:"nodeSelector,omitempty"`
6161

62-
// +kubebuilder:validation:Optional
63-
// Debug - enable debug for different deploy stages. If an init container is used, it runs and the
64-
// actual action pod gets started with sleep infinity
65-
Debug GlanceAPIDebug `json:"debug,omitempty"`
66-
6762
// +kubebuilder:validation:Optional
6863
// Pvc - Storage claim for file-backed Glance
6964
Pvc string `json:"pvc,omitempty"`

api/v1beta1/glance_types.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -141,10 +141,6 @@ type PasswordSelector struct {
141141

142142
// GlanceDebug defines the observed state of GlanceAPIDebug
143143
type GlanceDebug struct {
144-
// +kubebuilder:validation:Optional
145-
// +kubebuilder:default=false
146-
// DBSync enable debug
147-
DBSync bool `json:"dbSync"`
148144
// +kubebuilder:validation:Optional
149145
// +kubebuilder:default=false
150146
// CronJob enable debug

api/v1beta1/glanceapi_types.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -83,14 +83,6 @@ type GlanceAPISpec struct {
8383
ImageCacheSize string `json:"imageCacheSize"`
8484
}
8585

86-
// GlanceAPIDebug defines the observed state of GlanceAPIDebug
87-
type GlanceAPIDebug struct {
88-
// +kubebuilder:validation:Optional
89-
// +kubebuilder:default=false
90-
// Service enable debug
91-
Service bool `json:"service"`
92-
}
93-
9486
// GlanceAPIStatus defines the observed state of GlanceAPI
9587
type GlanceAPIStatus struct {
9688
// ReadyCount of glance API instances

api/v1beta1/zz_generated.deepcopy.go

Lines changed: 0 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/bases/glance.openstack.org_glanceapis.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,6 @@ spec:
6060
databaseUser:
6161
default: glance
6262
type: string
63-
debug:
64-
properties:
65-
service:
66-
default: false
67-
type: boolean
68-
type: object
6963
extraMounts:
7064
items:
7165
properties:

config/crd/bases/glance.openstack.org_glances.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,6 @@ spec:
5454
cronJob:
5555
default: false
5656
type: boolean
57-
dbSync:
58-
default: false
59-
type: boolean
6057
type: object
6158
extraMounts:
6259
items:
@@ -825,12 +822,6 @@ spec:
825822
items:
826823
type: string
827824
type: array
828-
debug:
829-
properties:
830-
service:
831-
default: false
832-
type: boolean
833-
type: object
834825
networkAttachments:
835826
items:
836827
type: string

0 commit comments

Comments
 (0)