-
Notifications
You must be signed in to change notification settings - Fork 346
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
Use VMware Harbor registry for flow-aggregator Docker image #1650
Use VMware Harbor registry for flow-aggregator Docker image #1650
Conversation
0cab885
to
a3f0042
Compare
85d6737
to
65893e8
Compare
Codecov Report
@@ Coverage Diff @@
## feature/flow-aggregator #1650 +/- ##
==========================================================
Coverage ? 62.00%
==========================================================
Files ? 181
Lines ? 15411
Branches ? 0
==========================================================
Hits ? 9556
Misses ? 4836
Partials ? 1019
Flags with carried forward coverage won't be shown. Click here to find out more. |
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, thanks!
Changes are the same as in antrea-io#1617 but for the antrea/flow-aggregator Docker image. There was also a typo in the flow-aggregator YAML: the image key didn't match the one used in generate-manifest-flow-aggregator.sh when invoking kustomize.
65893e8
to
3fc6580
Compare
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.
Thanks for the PR. LGTM
…o#1650) Changes are the same as in antrea-io#1617 but for the antrea/flow-aggregator Docker image. There was also a typo in the flow-aggregator YAML: the image key didn't match the one used in generate-manifest-flow-aggregator.sh when invoking kustomize.
…o#1650) Changes are the same as in antrea-io#1617 but for the antrea/flow-aggregator Docker image. There was also a typo in the flow-aggregator YAML: the image key didn't match the one used in generate-manifest-flow-aggregator.sh when invoking kustomize.
…o#1650) Changes are the same as in antrea-io#1617 but for the antrea/flow-aggregator Docker image. There was also a typo in the flow-aggregator YAML: the image key didn't match the one used in generate-manifest-flow-aggregator.sh when invoking kustomize.
…o#1650) Changes are the same as in antrea-io#1617 but for the antrea/flow-aggregator Docker image. There was also a typo in the flow-aggregator YAML: the image key didn't match the one used in generate-manifest-flow-aggregator.sh when invoking kustomize.
Changes are the same as in
#1617 but for the
antrea/flow-aggregator Docker image.
There was also a typo in the flow-aggregator YAML: the image key didn't
match the one used in generate-manifest-flow-aggregator.sh when invoking
kustomize.