-
Notifications
You must be signed in to change notification settings - Fork 6.4k
Closed
Labels
bugSomething isn't workingSomething isn't workingmodel-behaviorIssues related to behaviors exhibited by the modelIssues related to behaviors exhibited by the model
Description
Uploaded thread: 019a3723-0153-7cf1-8d2a-6cff3813e735
I used /review and noticed the reasoning outputs had repeated words. Some examples here:
• Reviewinging potential potential config config map map issues issues****
This investigation investigation focuses focuses on on subtle subtle bugs bugs
around around config config map map reconciliation reconciliation,, especially
especially handling handling race race conditions conditions when when creating
creating config config maps maps that that may may already already exist exist due
due to to eventual eventual consistency consistency in in caching caching.. I'm I'm
exploring exploring how how stale stale caches caches or or owner owner references
references might might prevent prevent proper proper updates updates or or event
event triggering triggering,, and and suspect suspect recon reconcilcilinging
logic logic should should re re-fetch-fetch or or re requeuequeue on on Already
AlreadyExistsExists rather rather than than silently silently ignoring ignoring..
Also Also considering considering metric metric registration registration conflicts
conflicts and and concurrency concurrency risks risks related related to to config
config map map management management in in the the new new controller controller
package package.. Running Running tests tests next next will will help help confirm
confirm these these susp suspicionsicions..
Let me know if I can provide any other information.
Shakabrahh, nachoalonso, dpolivaev and matsfinsas
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingmodel-behaviorIssues related to behaviors exhibited by the modelIssues related to behaviors exhibited by the model