Charts: enabled newui in skydive-analyzer chart

skydive-project / skydive

An open source real-time network topology and protocols analyzer

https://skydive.network

Apache License 2.0

2.68k stars 404 forks source link

Charts: enabled newui in skydive-analyzer chart #2333

Closed hunchback closed 3 years ago

hunchback commented 3 years ago

for now there is a bug as the latest webui does not connect to localhost:8082 successfully

to run:

cd charts
make k3d
make skydive-analyzer
make install
make status

attempting to logout and login with:

host: http://localhost:802
user: admin
password: password

fails:

while access localhost:8082 from within the skydive-ui contains succeeds:

hunchback commented 3 years ago

@safchain - please check

other than the bug, I think that from the user perspective it would be best if you would integrate the newui into the skydive binary, as it would be best to just access it via a url such as: http://localhost:8082/newui

safchain commented 3 years ago

Yes the plan is to integrate the new UI, but it still need some work. For the issue can you try to see what is the exact error using the web console (network planel) ? I'll push a fix to report the real error at the UI side.

hunchback commented 3 years ago

Yes the plan is to integrate the new UI, but it still need some work. For the issue can you try to see what is the exact error using the web console (network planel) ? I'll push a fix to report the real error at the UI side.

hunchback commented 3 years ago

Yes the plan is to integrate the new UI, but it still need some work. For the issue can you try to see what is the exact error using the web console (network planel) ? I'll push a fix to report the real error at the UI side.

I think that the fact the newui is outside the core has had an adverse effect on the newui rate of maturing, IMHO the sooner the newui is merged into the core (mapped to a unique prefix such as /newui) the better.

hunchback commented 3 years ago

was only for purposes of reporting a bug