Skip to content

Update k8s user guide to use deployments - #474

Merged
andygrove merged 2 commits into
apache:masterfrom
edrevo:k8s-deployments
Jun 2, 2021
Merged

Update k8s user guide to use deployments#474
andygrove merged 2 commits into
apache:masterfrom
edrevo:k8s-deployments

Conversation

@edrevo

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes#473.

What changes are included in this PR?

  • Rename port to bind-port since the configure_me create was trying to parse an env var that was being autopopulated by Kubernetes
  • Change the user guide for k8s

@edrevoedrevo changed the title K8s deploymentsUpdate k8s user guide to use deploymentsJun 1, 2021
@edrevo

Copy link
Copy Markdown
ContributorAuthor

cc @andygrove

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #474 (56aeaea) into master (1601112) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@ Coverage Diff @@## master #474 +/- ##
=======================================
Coverage 75.84% 75.84% =======================================
Files 153 153 Lines 25872 25872 =======================================
Hits 19622 19622 Misses 6250 6250 
Impacted FilesCoverage Δ
ballista/rust/executor/src/main.rs0.00% <0.00%> (ø)
ballista/rust/scheduler/src/main.rs0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1601112...56aeaea. Read the comment docs.

@andygroveandygrove left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks @edrevo

@andygrove
andygrove merged commit 01b57f7 into apache:masterJun 2, 2021
@edrevo
edrevo deleted the k8s-deployments branch June 11, 2021 09:35
@houqphouqp added ballista documentation Improvements or additions to documentation labels Jul 30, 2021
HairstonE pushed a commit to HairstonE/datafusion that referenced this pull request Oct 7, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Ballista] Use deployments in k8s user guide

4 participants

@edrevo@codecov-commenter@andygrove@houqp