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

Can we hide "CheckControllerStatus is disabled for this session, not going to retry" when checking Avi connectivity? #1634

Closed
cormachogan opened this issue Sep 9, 2021 · 2 comments
Labels
good-first-issue Good for newcomers help-wanted Looking for contributors to help kind/enhancement An enhancement to an existing capability kind/feature A request for a new feature owner/framework Work executed in vmware-tanzu/tanzu-framework
Milestone

Comments

@cormachogan
Copy link

Feature Request

When we do the Avi connectivity check in the UI, the following messages are thrown in the CLI:

% tanzu management-cluster create --ui

Validating the pre-requisites...
Serving kickstart UI at http://127.0.0.1:8080
E0909 09:04:31.721860    2053 avisession.go:714] CheckControllerStatus is disabled for this session, not going to retry.
E0909 09:04:31.751117    2053 avisession.go:714] CheckControllerStatus is disabled for this session, not going to retry.
E0909 09:04:31.840045    2053 avisession.go:714] CheckControllerStatus is disabled for this session, not going to retry.

Could we hide these messages from the user to provide a better UX?

Describe alternatives you've considered

N/A

Environment Details

  • Build version (tanzu version):

    version: v1.4.0-pre-alpha-2
    buildDate: 2021-08-19
    sha: 75cfa0e

  • Operating System (client):

    macOS (Big Sur) v11.5.2.
    vSphere 7.0U2c
    NSX ALB (Avi Vantage) v2.0.1.5.

@cormachogan cormachogan added kind/feature A request for a new feature triage/needs-triage Needs triage by TCE maintainers labels Sep 9, 2021
@joshrosso joshrosso added kind/enhancement An enhancement to an existing capability owner/framework Work executed in vmware-tanzu/tanzu-framework help-wanted Looking for contributors to help good-first-issue Good for newcomers and removed triage/needs-triage Needs triage by TCE maintainers kind/feature A request for a new feature labels Sep 12, 2021
@joshrosso
Copy link
Contributor

Most definitely! We should fix this in framework.

@joshrosso joshrosso added this to the icebox milestone Sep 12, 2021
@joshrosso joshrosso added the kind/feature A request for a new feature label Oct 12, 2021
@mritunjaysharma394
Copy link
Contributor

I think this behaviour is due to the below line of code?

LogOptions: tkgctl.LoggingOptions{Verbosity: logLevel, File: logFile},

Can we think of removing it if it is not required or there's a better alternative to fix it? Thanks a lot!

@jdumars jdumars closed this as completed Oct 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good-first-issue Good for newcomers help-wanted Looking for contributors to help kind/enhancement An enhancement to an existing capability kind/feature A request for a new feature owner/framework Work executed in vmware-tanzu/tanzu-framework
Projects
None yet
Development

No branches or pull requests

4 participants