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.

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

[INS-2349] [Bug] Deploy and undeploy local instance first causing "ERROR: instance 'localhost:8080' does not exists" whtn auth login later #601

Closed
1 task done
pinglin opened this issue Oct 14, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@pinglin
Copy link
Member

pinglin commented Oct 14, 2023

Is There an Existing Issue for This?

  • I have searched the existing issues

Project

Instill CLI

Describe the Bug

The current implementation initiates the default_hostname configuration value with api.instill.tech only when running inst auth login and ~~/.config/instill/hosts.yml being yet created.

If the user run inst local deploy and inst local undeploy in the very beginning, the default_hostname configuration value will be set to localhost:8080 and the unregistration function won't delete ~~/.config/instill/hosts.yml and will leave the default_hostname unchanged. This will cause the following inst auth login failing with ERROR: instance 'localhost:8080' does not exists.

Steps to Reproduce

  1. brew install instill-ai/tap/inst
  2. inst local deploy
  3. inst local undeploy
  4. inst auth login

Actual Result

Resulting in ERROR: instance 'localhost:8080' does not exists

Expected Behavior

inst auth login should fallback to user the default api.instlil.tech host configuration.

Technical Details

  • Project version: v0.1.0-alpha
  • OS: macOS

Anything Else?

No

From SyncLinear.com | INS-2349

@pinglin pinglin self-assigned this Oct 14, 2023
@pinglin pinglin changed the title [Bug] Deploy and undeploy local instance first causing "ERROR: instance 'localhost:8080' does not exists" whtn auth login later [INS-2349] [Bug] Deploy and undeploy local instance first causing "ERROR: instance 'localhost:8080' does not exists" whtn auth login later Oct 14, 2023
@pinglin
Copy link
Member Author

pinglin commented Jan 24, 2024

@harsh This issue is still relevant but not critical. I will find time to fix it or have another engineer fix it in the future.

@pinglin pinglin transferred this issue from instill-ai/community May 19, 2024
@pinglin pinglin added the bug Something isn't working label May 19, 2024
@felixcorleone
Copy link

@pinglin Considering that the latest update for this issue was over five weeks ago, we are closing it. If there are any updates in the future, please reopen the issue and provide the latest requirements and status. Thank you.

@felixcorleone felixcorleone closed this as not planned Won't fix, can't repro, duplicate, stale Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants