Skip to content

docs: clean up quickstart#6051

Merged
arkodg merged 1 commit intoenvoyproxy:mainfrom
arkodg:cleanup-quickstart
May 14, 2025
Merged

docs: clean up quickstart#6051
arkodg merged 1 commit intoenvoyproxy:mainfrom
arkodg:cleanup-quickstart

Conversation

@arkodg
Copy link
Contributor

@arkodg arkodg commented May 14, 2025

No description provided.

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from a team as a code owner May 14, 2025 00:34
@arkodg arkodg requested a review from melsal13 May 14, 2025 00:35
__Note:__ In case your Kubernetes cluster does not have a LoadBalancer implementation, we recommend installing one
so the `Gateway` resource has an Address associated with it. We recommend using [MetalLB](https://metallb.universe.tf/installation/).

__Note:__ For Mac user, you need install and run [Docker Mac Net Connect](https://github.com/chipmk/docker-mac-net-connect) to make the Docker network work.
Copy link
Member

Choose a reason for hiding this comment

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

why remove this?

Copy link
Contributor Author

@arkodg arkodg May 14, 2025

Choose a reason for hiding this comment

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

its distracting for an end user, its tied to mac and docker and kind, for e.g. on mac, i use docker desktop and this isnt needed, this note is needed in developer docs, when running e2e on kind on mac

@codecov
Copy link

codecov bot commented May 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.36%. Comparing base (943c961) to head (990ac08).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6051      +/-   ##
==========================================
- Coverage   70.40%   70.36%   -0.04%     
==========================================
  Files         217      217              
  Lines       36171    36171              
==========================================
- Hits        25465    25453      -12     
- Misses       9185     9193       +8     
- Partials     1521     1525       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arkodg arkodg requested a review from zirain May 14, 2025 00:44
@arkodg arkodg requested review from a team May 14, 2025 00:51
@arkodg arkodg merged commit 40a5147 into envoyproxy:main May 14, 2025
28 of 29 checks passed
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.

3 participants