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

InitRegion is long, lots of side effects #4265

Open
jayunit100 opened this issue Jan 10, 2023 · 0 comments
Open

InitRegion is long, lots of side effects #4265

jayunit100 opened this issue Jan 10, 2023 · 0 comments
Assignees
Labels
area/cli kind/bug PR/Issue related to a bug

Comments

@jayunit100
Copy link
Contributor

Bug description

The InitRegion function has alot of side effects and is like 500 lines long.
lets split the code up and try to get rid of global func variables so its easy to debug/add logs for stages and so on, as its one of the most common things that breaks in TKG installation.

Affected product area (please put an X in all that apply)

  • ( ) APIs
  • ( ) Addons
  • ( ) CLI
  • ( ) Docs
  • ( ) IAM
  • ( ) Installation
  • ( ) Plugin
  • ( ) Security
  • ( ) Test and Release
  • ( ) User Experience
  • ( ) Developer Experience

Expected behavior

Steps to reproduce the bug

Version (include the SHA if the version is not obvious)

Environment where the bug was observed (cloud, OS, etc)

Relevant Debug Output (Logs, manifests, etc)

@jayunit100 jayunit100 added kind/bug PR/Issue related to a bug needs-triage Indicates an issue or PR needs to be triaged labels Jan 10, 2023
@yharish991 yharish991 added area/cli and removed needs-triage Indicates an issue or PR needs to be triaged labels Jan 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/cli kind/bug PR/Issue related to a bug
Projects
None yet
Development

No branches or pull requests

2 participants