Skip to content

Commit 05b8bf6

Browse files
authored
change event recorder cluster namespace to kube-system (#871)
because publishing to "default" is a fallback thing. fixes #870 Signed-off-by: clux <[email protected]>
1 parent 7b2d145 commit 05b8bf6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

kube-runtime/src/events.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ impl Recorder {
176176
/// Cluster scoped objects will publish events in the "default" namespace.
177177
#[must_use]
178178
pub fn new(client: Client, reporter: Reporter, reference: ObjectReference) -> Self {
179-
let default_namespace = "default".to_owned();
179+
let default_namespace = "kube-system".to_owned(); // default does not work on k8s < 1.22
180180
let events = Api::namespaced(client, reference.namespace.as_ref().unwrap_or(&default_namespace));
181181
Self {
182182
events,
@@ -294,7 +294,7 @@ mod test {
294294
secondary: None,
295295
})
296296
.await?;
297-
let events: Api<CoreEvent> = Api::namespaced(client, "default");
297+
let events: Api<CoreEvent> = Api::namespaced(client, "kube-system");
298298

299299
let event_list = events.list(&Default::default()).await?;
300300
let found_event = event_list

0 commit comments

Comments
 (0)