Skip to content
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.

Sign up for GitHub

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

Jump to bottom

added reference to helm chart usage on core,horizon install #216

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

added reference to helm chart usage on core,horizon install #216

wants to merge 9 commits into from

Conversation

sreuland
Copy link
Contributor

@sreuland sreuland commented Aug 24, 2023 •
edited
Loading

Suggest the new helm chart option available for installing horizon and core.

Once published, the quickstart Readme can refer to these docs for production installation recommendations per stellar/quickstart#478

@sreuland sreuland mentioned this pull request Aug 24, 2023
Closed
@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@leighmcculloch leighmcculloch requested review from a team and MonsieurNicolas and removed request for leighmcculloch August 24, 2023 19:54
@sreuland sreuland requested a review from jacekn August 24, 2023 20:10
@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@sreuland sreuland marked this pull request as draft August 29, 2023 17:32
@sreuland
Copy link
Contributor Author

sreuland commented Aug 29, 2023 •
edited
Loading

keeping this in draft until helm chart issues are resolved:

@sreuland sreuland mentioned this pull request Aug 29, 2023
Open
@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@sreuland sreuland requested a review from jacekn August 31, 2023 15:57
@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

jacekn
jacekn reviewed Sep 5, 2023
View reviewed changes
docs/run-api-server/installing.mdx Outdated Show resolved Hide resolved
docs/run-api-server/running.mdx Outdated Show resolved Hide resolved
@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@stellar-jenkins
Copy link

Preview is available here:
http://developers-pr216.previews.kube001.services.stellar-ops.com

@sreuland
Copy link
Contributor Author

sreuland commented Sep 5, 2023

@jacekn , I have addressed feedback on helm usages, at this point the remaining aspect is the core helm docs have a dependency on values.yaml's for testnet/pubnet that haven't been created yet:

https://github.com/stellar/stellar-docs/pull/216/files#diff-e0a15f4578c42a6d9909a0ed0007c7ced8c399449f8acc8d87d9a6d10dcbd5a2R40-R42

Once stellar/helm-charts#65 is done, I think this PR will be ready to merge.

@sreuland sreuland mentioned this pull request Sep 12, 2023
Closed
@sreuland sreuland mentioned this pull request May 3, 2024
Merged
@briwylde08
Copy link
Contributor

Hi @sreuland! Just checking on this one again. Is this still a WIP?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MonsieurNicolas MonsieurNicolas Awaiting requested review from MonsieurNicolas

@jacekn jacekn Awaiting requested review from jacekn

@tsachiherman tsachiherman Awaiting requested review from tsachiherman

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
DO_NOT_MERGE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sreuland @stellar-jenkins @briwylde08 @jacekn @tsachiherman