Skip to content

fix: swallow rejected execution after shutdown#6063

Merged
manusa merged 1 commit intofabric8io:mainfrom
shawkins:iss6059
Jun 25, 2024
Merged

fix: swallow rejected execution after shutdown#6063
manusa merged 1 commit intofabric8io:mainfrom
shawkins:iss6059

Conversation

@shawkins
Copy link
Copy Markdown
Contributor

Description

The handling to make the informer behavior deterministic allows for the cache call to happen after shutdown. The simplest thing to do is to just swallow the rejected execution exception.

closes: #6059

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change
  • Chore (non-breaking change which doesn't affect codebase;
    test, version modification, documentation, etc.)

Checklist

  • Code contributed by me aligns with current project license: Apache 2.0
  • I Added CHANGELOG entry regarding this change
  • I have implemented unit tests to cover my changes
  • I have added/updated the javadocs and other documentation accordingly
  • No new bugs, code smells, etc. in SonarCloud report
  • I tested my code in Kubernetes
  • I tested my code in OpenShift

@sonarqubecloud
Copy link
Copy Markdown

@manusa manusa added this to the 7.0.0 milestone Jun 25, 2024 — with automated-tasks
Copy link
Copy Markdown
Member

@manusa manusa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thx!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

io.fabric8.kubernetes.client.informers.impl.cache.Reflector - listSyncAndWatch failed

5 participants