-
Notifications
You must be signed in to change notification settings - Fork 336
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: trigger ApisixRoute event when service is created #1152
Conversation
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #1152 +/- ##
==========================================
- Coverage 30.45% 29.99% -0.46%
==========================================
Files 81 81
Lines 9839 9976 +137
==========================================
- Hits 2996 2992 -4
- Misses 6517 6660 +143
+ Partials 326 324 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. |
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
pkg/ingress/apisix_route.go
Outdated
// svc meta key -> ApisixRoute name -> empty | ||
// map[string]map[string]struct{} | ||
svcLock sync.RWMutex | ||
svcMap map[string]map[string]struct{} //*sync.Map |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can just use sync.Map ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to use more specific types instead of the untyped sync.Map.
test/e2e/suite-ingress/suite-ingress-resource/resourcepushing.go
Outdated
Show resolved
Hide resolved
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
test/e2e/suite-ingress/suite-ingress-resource/resourcepushing.go
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
let's move forward.
Type of change:
Fixes #926