Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 25, 2022

With stores locally available we don't need to fetch the latest state
from kube-apiserver. Changed all calls to use the stores instead.

Signed-off-by: André Martins andre@cilium.io

@aanm aanm added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.12 labels Aug 25, 2022
@aanm
Copy link
Member Author

aanm commented Aug 25, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from 442c3ff to 38c7024 Compare September 5, 2022 21:13
@aanm
Copy link
Member Author

aanm commented Sep 5, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from 38c7024 to 21a7744 Compare September 8, 2022 18:56
@aanm
Copy link
Member Author

aanm commented Sep 8, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from 21a7744 to 5e1398d Compare September 8, 2022 22:14
@aanm
Copy link
Member Author

aanm commented Sep 8, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from 5e1398d to 7993240 Compare September 9, 2022 12:21
@aanm
Copy link
Member Author

aanm commented Sep 9, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from 7993240 to b320af8 Compare September 9, 2022 12:27
@aanm
Copy link
Member Author

aanm commented Sep 9, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch from b320af8 to 2625eb2 Compare September 9, 2022 15:34
@aanm
Copy link
Member Author

aanm commented Sep 9, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch 2 times, most recently from aafb4a0 to 1be8a19 Compare September 9, 2022 17:55
@aanm
Copy link
Member Author

aanm commented Sep 9, 2022

/test

@aanm aanm closed this Sep 15, 2022
@aanm aanm reopened this Sep 15, 2022
@aanm aanm force-pushed the pr/optimize-api-calls branch from 083eb0a to 3b82af2 Compare September 15, 2022 23:11
@aanm
Copy link
Member Author

aanm commented Sep 15, 2022

/test

@aanm aanm force-pushed the pr/optimize-api-calls branch 2 times, most recently from 297c8b2 to ef20933 Compare September 20, 2022 11:45
@aanm
Copy link
Member Author

aanm commented Sep 20, 2022

/test

@aanm
Copy link
Member Author

aanm commented Sep 20, 2022

/test-1.25-net-next

@aanm aanm marked this pull request as ready for review September 22, 2022 09:27
@aanm aanm requested review from a team as code owners September 22, 2022 09:27
@aanm aanm requested a review from a team as a code owner September 22, 2022 09:27
With stores locally available we don't need to fetch the latest state
from kube-apiserver. Changed all calls to use the stores instead.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/optimize-api-calls branch from ef20933 to 5025cb0 Compare October 12, 2022 07:45
@aanm
Copy link
Member Author

aanm commented Oct 12, 2022

The CI was green and the only change was a code comment. Merging

@aanm aanm added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 12, 2022
@qmonnet qmonnet merged commit 69e4c69 into cilium:master Oct 12, 2022
@qmonnet qmonnet mentioned this pull request Oct 19, 2022
6 tasks
@joestringer joestringer added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Nov 18, 2022
@aanm aanm deleted the pr/optimize-api-calls branch November 21, 2022 20:25
@christarazi christarazi added the area/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. label Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants