Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Implements using ~/.config/tanzu/tkg/local as the config dir for local clusters #2305

Merged

Conversation

jpmcb
Copy link
Contributor

@jpmcb jpmcb commented Oct 21, 2021

What this PR does / why we need it

All work related to standalone-cluster overhaul and re-implementation

  • Adds getLocalBomPath() function
  • Renames bundle to bomImage in getTkrBom function (which makes more sense since that image isn't a bundle)

Details for the Release Notes (PLEASE PROVIDE)

N/a

Which issue(s) this PR fixes

Related to proposal #2266

Describe testing done for PR

  • rm -rf ~/.config/tanzu directory
  • Build binary and run create cluster
  • Bom now located at ~/.config/tanzu/tkg/local/bom and cluster deploys successfully

Special notes for your reviewer

Open to any and all suggestions regarding variable naming etc!

- Adds `getLocalBomPath()` function
- Renames `bundle` to `bomImage` in `getTkrBom` function

Signed-off-by: John McBride <[email protected]>
@jpmcb jpmcb requested a review from a team as a code owner October 21, 2021 14:53
Copy link
Contributor

@stmcginnis stmcginnis left a comment

Choose a reason for hiding this comment

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

Looks good!

@stmcginnis stmcginnis merged commit 7114a6c into vmware-tanzu:standalone-overhaul Oct 21, 2021
joshrosso pushed a commit that referenced this pull request Jan 18, 2022
- Adds `getLocalBomPath()` function
- Renames `bundle` to `bomImage` in `getTkrBom` function

Signed-off-by: John McBride <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants