kcp icon indicating copy to clipboard operation
kcp copied to clipboard

Generation number doesn't seem correctly set on cached builtin resources.

Open davidfestal opened this issue 1 year ago • 7 comments

When completing the replication_test.go e2e cache test with replication scenarios of rbac resources (CR / CRB objects), the generation is not equal when comparing original and cached objects.

I added a TODO in the completed test: https://github.com/kcp-dev/kcp/blob/cf8a625b0266f761e70e0eb5dcde0817b6ac89e8/test/e2e/reconciler/cache/replication_test.go#L706-L716

Maybe the work done for CRD-backed resources in this PR should also be done for built-in resources that support replication ?

davidfestal avatar Mar 30 '23 12:03 davidfestal

@p0lyn0mial do you happen to remember if this issue is still up to date and/or valid?

embik avatar Aug 10 '23 15:08 embik

@p0lyn0mial do you happen to remember if this issue is still up to date and/or valid?

I think so, we wanted to fix it with https://github.com/kcp-dev/kubernetes/pull/132

p0lyn0mial avatar Aug 11 '23 10:08 p0lyn0mial

and the test still (https://github.com/kcp-dev/kcp/blob/main/test/e2e/reconciler/cache/replication_test.go#L683) seems to be removing the "generation" field before comparison.

p0lyn0mial avatar Aug 11 '23 11:08 p0lyn0mial

Issues go stale after 90d of inactivity. After a furter 30 days, they will turn rotten. Mark the issue as fresh with /remove-lifecycle stale.

If this issue is safe to close now please do so with /close.

/lifecycle stale

kcp-ci-bot avatar Apr 16 '24 20:04 kcp-ci-bot

/remove-lifecycle stale

embik avatar Apr 17 '24 06:04 embik

Issues go stale after 90d of inactivity. After a furter 30 days, they will turn rotten. Mark the issue as fresh with /remove-lifecycle stale.

If this issue is safe to close now please do so with /close.

/lifecycle stale

kcp-ci-bot avatar Jul 16 '24 08:07 kcp-ci-bot

/remove-lifecycle stale

embik avatar Jul 16 '24 11:07 embik