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

UI rendering issues with Consul v1.15.0 #16429

Closed
heikkilamarko opened this issue Feb 25, 2023 · 19 comments · Fixed by #16444
Closed

UI rendering issues with Consul v1.15.0 #16429

heikkilamarko opened this issue Feb 25, 2023 · 19 comments · Fixed by #16444

Comments

@heikkilamarko
Copy link

heikkilamarko commented Feb 25, 2023

After upgrading to Consul v1.15.0, we started to see rendering issues in UI. For example, the Services page doesn't render all list items when you navigate to the page. After opening the filters (without changing values), the missing items start to come visible. Below are two screenshots. There are 17 services in total, but only some of them are rendered first (screenshot1). After opening the filters, the page start to show all items (screenshot2). As can be seen, there are rendering issues with filters too.

The problem reproduces with latest Chrome, Firefox, and Safari.

screenshot1

screenshot2

@krarey
Copy link
Member

krarey commented Feb 26, 2023

I'm seeing this as well in v1.15.0 across several browsers, with variations of the following stack trace in the browser console:

Uncaught TypeError: (0 , t.isHTMLSafe) is not a function
    at vendor-9b21ddc3d55dbc248b171f3af693427e.js:8257:23
    at vendor-9b21ddc3d55dbc248b171f3af693427e.js:1209:13
    at Object.getValue (vendor-9b21ddc3d55dbc248b171f3af693427e.js:1209:64)
    at vendor-9b21ddc3d55dbc248b171f3af693427e.js:2998:46
    at vendor-9b21ddc3d55dbc248b171f3af693427e.js:3290:41
    at e.track (vendor-9b21ddc3d55dbc248b171f3af693427e.js:3962:5)
    at m (vendor-9b21ddc3d55dbc248b171f3af693427e.js:3290:20)
    at Object.evaluate (vendor-9b21ddc3d55dbc248b171f3af693427e.js:3617:42)
    at Object.evaluate (vendor-9b21ddc3d55dbc248b171f3af693427e.js:3408:13)
    at Lt.evaluateSyscall (vendor-9b21ddc3d55dbc248b171f3af693427e.js:3710:61)

@elsgaard
Copy link

elsgaard commented Feb 26, 2023

I am seeing this as well on Ubuntu 22.04 LTS (Jammy Jellyfish) + Chrome Version 110.0.5481.178. After dongrading to 1.14.4 it works correctly again.

Also "Overview" is not showing anything in the Consul UI.

@sid-6581
Copy link

I am seeing the same thing in Vivaldi after upgrading to 1.15.0.

@marceldegraaf
Copy link

marceldegraaf commented Feb 27, 2023

Same issue here. Adding a screenshot of the empty "Overview" page for completeness:

2023-02-27-08-51-49-O4im38qQ@2x

This is on a fresh install of Consul v1.15.0 (revision 53f65dc3) on a Debian machine (Debian 5.10.162-1 (2023-01-21) x86_64 GNU/Linux) with Consul installed from the official Apt repository.

EDIT: this installation is not running any production workloads yet, so let me know if there's anything I can do to help debug this.

@ashleykleynhans
Copy link

Also seeing the same issue as in the screenshot above after upgrading my Consul cluster from version 1.14.4 to version 1.15.0. I attempted to recover the cluster by creating the raft/peers.json file on each server node in the cluster and restarting Consul on each, but the problem persists.

@david-yu
Copy link
Contributor

david-yu commented Feb 28, 2023

Hi folks, we were able to reproduce and fix the issue via #16444. This will be back ported and released in 1.15.1 as part of our patch release around mid-march timeframe. Thank you for the quick feedback!

@ashleykleynhans
Copy link

  1. Why do you close an issue, when there is no release yet to fix it?
  2. Why must we wait weeks for a fix when this is a pretty severe problem, this is UNACCEPTABLE.
  3. Why was this release clearly NOT tested before releasing to the public.

Your attitude is unacceptable, and I will not be using any further Hashiscorp products because of it.
Get your act together and ensure that appropriate measures are put in place to stop this from happening again!

@sid-6581
Copy link

sid-6581 commented Mar 1, 2023

Having to wait several weeks for a bug that makes the UI completely unusable seems a bit wrong to me also, especially when the fix is simple, only affects the UI, and is ready to go. Why not do a hotfix release instead of forcing people to roll back to a previous version?

@PJDuszynski
Copy link

I don't understand the response when you have a significant component of the program broken in a minor release, have a fix ready, to then decide to wait another 2-3 weeks to release the fix to the users.

@suikast42
Copy link

suikast42 commented Mar 3, 2023

Also seeing the same issue as in the screenshot above after upgrading my Consul cluster from version 1.14.4 to version 1.15.0. I attempted to recover the cluster by creating the raft/peers.json file on each server node in the cluster and restarting Consul on each, but the problem persists.

Dude keep clam.
Hashicrop does a great job. You as a product ownver have to test every release component before going in priduction with it. This inculdes thrid party software like consul or nomad.

If you want have a better support or more stable tested product then make a enterprise subscriptuon on hashicorp.

Your points here are really not appropriate

@blake
Copy link
Contributor

blake commented Mar 3, 2023

Hi folks,

We understand the frustration around this issue. I’d like to apologize on behalf of our team. This was a miss on our part. We acknowledge that these types of regressions are unacceptable and are taking steps to minimize the possibility of such regressions in the future.

We be publishing a patch release next week which will address these issues. The patch releases will be available no later than March 9th.

@blake blake reopened this Mar 3, 2023
@frh-td
Copy link

frh-td commented Mar 8, 2023

1.15.1 fixes the issue on the services and nodes listings, but the cluster overview is still broken with the same error (TypeError: (0 , n.isHTMLSafe) is not a function) showing in the console.

2023-03-08-121819_901x216

@ahjohannessen
Copy link

Experincing the same as @frh-td with cluster overview UI being broken.

@suikast42
Copy link

The same here

@ttys3
Copy link

ttys3 commented Mar 9, 2023

Hi folks,

We understand the frustration around this issue. I’d like to apologize on behalf of our team. This was a miss on our part. We acknowledge that these types of regressions are unacceptable and are taking steps to minimize the possibility of such regressions in the future.

We be publishing a patch release next week which will address these issues. The patch releases will be available no later than March 9th.

1.15.1 still broken.

isHTMLSafe error on "overview", "Intentions" and "Peers" pages.

your guys did not do tests and wait 2 weeks release a broken patched program ?

you need 1.15.2

@sid-6581
Copy link

sid-6581 commented Mar 10, 2023

It's quite impressive that nobody at hashicorp even bothered to click on the pages mentioned in this very issue, for weeks, before releasing another broken build after already having egg on their face. Not sure what kind of QA you have going on over there, but at my company that doesn't fly.

It's not like this is some edge case that only happens to certain people some of the time, this is something that is completely broken for everyone, and was known to be broken before the new version was released.

Makes you wonder how much we can trust the real functionality of the products when totally preventable UI bugs like this aren't even tested and fixed properly.

@andrecp
Copy link
Contributor

andrecp commented Mar 17, 2023

@blake this is still broken in the main page

@nathancoleman nathancoleman linked a pull request Mar 21, 2023 that will close this issue
3 tasks
@ashleykleynhans
Copy link

Confirmed this is fixed in Consul 1.15.2.

@david-yu
Copy link
Contributor

Closing as the issue is now resolved in 1.15.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.