Nick Carboni

Results 58 comments of Nick Carboni

I'm going to close this and rebase onto https://github.com/openshift/assisted-service/pull/6283 since they conflict so heavily.

>as I have discovered that the infrastructure operator will not function correctly unless we handle this issue Can you elaborate a bit more? Ideally we'd have the failing behavior explained...

While I think we should consider this, I'm still split. All customers would install the operator through MCE and thus would definitely have ManagedCluster, but we also publish our operator...

Closing this as the issue was fixed in the most recent 4.16 payloads

So this looks okay, but is it really what you want to do? As far as I can tell this just adds another way to provide the same token, so...

I think it would be fine to add the authz stuff in this same PR. It gives context that makes these changes worthwhile. Also I wouldn't want to have to...