Skip to content

[WIP] Blog post on read-cache-after-write consistency#3194

Draft
csviri wants to merge 32 commits intonextfrom
blog-read-after-write
Draft

[WIP] Blog post on read-cache-after-write consistency#3194
csviri wants to merge 32 commits intonextfrom
blog-read-after-write

Conversation

@csviri
Copy link
Collaborator

@csviri csviri commented Mar 1, 2026

Signed-off-by: Attila Mészáros a_meszaros@apple.com

csviri and others added 30 commits February 28, 2026 10:54
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
simplified the javadoc

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <metacosm@gmail.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
- Adds utility that provides methods to update resources using comparable resource versions
- Integrates this utility to the core of the framework (thus simplifying `ReconciliationDispatcher`)
  -  note that this introduces a change in the behavior for the `UpdateControl.patchStatus` (and others), since it won't trigger the reconiliation for the event in that update.
-  Renames former `ReconcilerUtils` to `ReconcilerUtilsInternal`, this is breaking but that utils was never advertised for non-internal usage 
- Includes also fixes for ControllerEventSource event filtering
- Improves TemporaryResourceCache event filtering algorithm

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Co-authored-by: Steve Hawkins <shawkins@redhat.com>
)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
This should not be user facing. At least not in any obvious scenerio.

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
…om Context (#3142)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <metacosm@gmail.com>
Co-authored-by: Chris Laprun <metacosm@gmail.com>
#3146)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
)

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Co-authored-by: Attila Mészáros <a_meszaros@apple.com>
* refactor: avoid creating intermediate collections when unneeded

Also use constant filters and collectors

Signed-off-by: Chris Laprun <metacosm@gmail.com>

* fix: incorrect test

Signed-off-by: Chris Laprun <metacosm@gmail.com>

---------

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
)

* improve: prepare for removal of exitOnStopLeading from public API

Signed-off-by: Chris Laprun <metacosm@gmail.com>

* fix: missed deprecated annotation

[skip ci]

Signed-off-by: Chris Laprun <metacosm@gmail.com>

---------

Signed-off-by: Chris Laprun <metacosm@gmail.com>
[skip ci]

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Fixes #3175

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Co-authored-by: Chris Laprun <metacosm@gmail.com>
* fix: unify how resource information is added, prevent NPEs

Fixes #3183

Some (all?) MDC implementations prevent adding null values so default
should be provided or the key omitted.

Signed-off-by: Chris Laprun <metacosm@gmail.com>

* fix: prefix name

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: cosmetic

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* refactor: remove unused methods

Signed-off-by: Chris Laprun <metacosm@gmail.com>

---------

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* feat: emit MDCUtils.NO_NAMESPACE value when namespace is null

Fixes #3184

Signed-off-by: Chris Laprun <metacosm@gmail.com>

* fix: improve wording

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: Chris Laprun <metacosm@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 1, 2026
@csviri csviri changed the title [WIP] BLog post on read-cache-after-write consistency [WIP] Blog post on read-cache-after-write consistency Mar 1, 2026
csviri added 2 commits March 5, 2026 22:10
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 6, 2026
@openshift-merge-robot
Copy link

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update docs and add blogpost for comparable resource version related changes

4 participants