Skip to content

Fix APT repository instructions to work#26

Merged
phlax merged 2 commits intoenvoyproxy:mainfrom
slovdahl:patch-1
Nov 9, 2024
Merged

Fix APT repository instructions to work#26
phlax merged 2 commits intoenvoyproxy:mainfrom
slovdahl:patch-1

Conversation

@slovdahl
Copy link
Copy Markdown
Contributor

@slovdahl slovdahl commented Nov 8, 2024

The instructions on https://www.envoyproxy.io/docs/envoy/v1.32.1/start/install#install-envoy-on-debian-based-linux are correct, but the instructions here did not work out of the box.

The instructions on https://www.envoyproxy.io/docs/envoy/v1.32.1/start/install#install-envoy-on-debian-based-linux are correct, but the instructions here did not work out of the box.

Signed-off-by: Sebastian Lövdahl <slovdahl@hibox.fi>
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 8, 2024

Deploy Preview for envoy-apt failed.

Name Link
🔨 Latest commit de104c4
🔍 Latest deploy log https://app.netlify.com/projects/envoy-apt/deploys/672e7473f22791000866bcbe

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for this @slovdahl

i forgot to update these instructions when i fixed the other

Signed-off-by: Sebastian Lövdahl <slovdahl@hibox.fi>
@slovdahl
Copy link
Copy Markdown
Contributor Author

slovdahl commented Nov 9, 2024

Removed a comma that shouldn't have been there.

@phlax phlax merged commit 91585df into envoyproxy:main Nov 9, 2024
@slovdahl slovdahl deleted the patch-1 branch November 10, 2024 06:17
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