Skip to content

Conversation

mum4k
Copy link
Collaborator

@mum4k mum4k commented Jan 25, 2021

Changing github repository paths from oschaaf to envoyproxy.

Signed-off-by: Jakub Sobon mumak@google.com

Changing github repository paths from oschaaf to envoyproxy.

Signed-off-by: Jakub Sobon <mumak@google.com>
@mum4k mum4k requested a review from oschaaf January 25, 2021 05:14
@mum4k mum4k added the waiting-for-review A PR waiting for a review. label Jan 25, 2021
@mum4k
Copy link
Collaborator Author

mum4k commented Jan 25, 2021

@oschaaf there is also a reference to oschaaf/benchmark-dev:latest in the same readme. Is that the correct path for us to use?

Copy link
Member

@oschaaf oschaaf left a comment

Choose a reason for hiding this comment

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

Ah, yeah, good idea.

@oschaaf oschaaf removed the waiting-for-review A PR waiting for a review. label Jan 25, 2021
@mum4k
Copy link
Collaborator Author

mum4k commented Jan 26, 2021

@oschaaf please see my question about the reference to oschaaf/benchmark-dev:latest in the same file.

@oschaaf
Copy link
Member

oschaaf commented Jan 26, 2021

@oschaaf there is also a reference to oschaaf/benchmark-dev:latest in the same readme. Is that the correct path for us to use?

Oh, sorry I missed that. I think that for the time being it's the best option, because we don't push the image in CI to a proper location. There's https://hub.docker.com/r/envoyproxy/nighthawk-dev, maybe we should try to also obtain https://hub.docker.com/r/envoyproxy/nighthawk-benchmark-dev (or some such) as a location to push to.

@oschaaf
Copy link
Member

oschaaf commented Jan 26, 2021

(Filed #611 to track)

@mum4k
Copy link
Collaborator Author

mum4k commented Jan 26, 2021

Thank you @oschaaf!

@mum4k mum4k merged commit 9ade1a5 into envoyproxy:main Jan 26, 2021
@mum4k mum4k deleted the benchmark-rename branch January 26, 2021 19:05
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.

2 participants