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

Unable to access data in Feast UI when deployed to remote instance #2731

Closed
tkch3n opened this issue May 24, 2022 · 1 comment · Fixed by #2753
Closed

Unable to access data in Feast UI when deployed to remote instance #2731

tkch3n opened this issue May 24, 2022 · 1 comment · Fixed by #2753

Comments

@tkch3n
Copy link

tkch3n commented May 24, 2022

Expected Behavior

Should be able to view registry data when launching UI with feast ui on remote instances (like EC2).

Current Behavior

I’ve tried setting the host to 0.0.0.0 and the static assets get loaded and can accessed via the public IP. But the requests to the registry (http://0.0.0.0:8888/registry) fails, so no data shows up.

I've also tried setting the host to the private IP, but the request to /registry times out.

Steps to reproduce

Run feast ui --host <instance private ip> in EC2 instance.

Specifications

  • Version:0.21.2
  • Platform: EC2
  • Subsystem:

Possible Solution

Potential CORS issue that needs to be fixed?

@creativedutchmen
Copy link
Contributor

creativedutchmen commented Jun 1, 2022

I'm having the same issue, caused by the hardcoded host:port in the projects-list.json file:

{
    "projects": [
        {
            "name": "Project",
            "description": "Test project",
            "id": <YOUR ID>,
            "registryPath": "http://0.0.0.0:8888/registry"
        }
    ]
}

When running the UI in k8s, the host should be set to 0.0.0.0 for routing to work, but of course it's not going to be reachable there. Same for the port, it thinks it's running at 8888, but in reality it's at 443.

I think there are two ways to fix this: make the registryPath a setting in the yaml file, or use a relative path instead of an absolute one.

edit: code responsible is here:

"registryPath": f"http://{host}:{port}/registry",

creativedutchmen added a commit to creativedutchmen/feast that referenced this issue Jun 2, 2022
Signed-off-by: Huib Keemink <huib.keemink@jedlix.com>
adchia pushed a commit that referenced this issue Jun 2, 2022
Signed-off-by: Huib Keemink <huib.keemink@jedlix.com>
adchia pushed a commit that referenced this issue Jun 13, 2022
Signed-off-by: Huib Keemink <huib.keemink@jedlix.com>
adchia pushed a commit that referenced this issue Jun 13, 2022
## [0.21.3](v0.21.2...v0.21.3) (2022-06-13)

### Bug Fixes

* Change online UI schema field to be optional ([#2766](#2766)) ([e6edb58](e6edb58)), closes [#2719](#2719)
* Made URL to registry relative, fixes [#2731](#2731) ([#2753](#2753)) ([a66aea8](a66aea8))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants