-
Notifications
You must be signed in to change notification settings - Fork 266
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
Cluster Tools: 'Install' button displays after installing chart when running Rancher locally #9940
Comments
This issue blocks incomming e2e tests #9749 |
* cluster tools tests * skip edit & delete tests until resolution of #9940
* cluster tools tests * skip edit & delete tests until resolution of rancher#9940
* cluster tools tests * skip edit & delete tests until resolution of rancher#9940
* cluster tools tests * skip edit & delete tests until resolution of rancher#9940
The is rarely replicable, but I noticed that, just before to install a new App from catalog, we receive some errors from In this case, the UI loses the new messages containing the new app list. I pushed a draft to re-sync those resource having failed ws messages, ONLY to try the e2e tests and analyze the issue, but I would be very careful in touching the ws utils. @richard-cox do you have any idea on this? |
Those are frustrating warnings sent when the BE closes a steve socket (there's an issue somewhere for them to fix). So that would track what you're seeing. When a socket goes down we should immediately re-subscribe. Any resource that was previously being watched should re-watch with a Does the socket always re-connect ok, and does the new connection have any messages containing the resource in question? |
For what i've seen the UI doesn't receive the lost updates for dashboard/shell/plugins/steve/subscribe.js Line 850 in e800261
fixed the issue and the e2e tests, since we do a re-sync for this type of ws errors, but I'm not sure if this is the right way to fix it until we don't have a fix back-end side.
|
That works around the issue and would cause a LOT of http requests to fetch all resources. If the BE isn't sending the correct updates after re-watching on the new socket connection with an old revision then you'll need to create an issue |
Exactly; I'll collect more info to send a t the backend team. Thanks |
Setup
Describe the bug
The 'Install' button is displayed after installing a chart when running Rancher locally. Edit and delete buttons should be displayed instead. It seems as though Rancher does not recognize that the installation occurred when running locally.
To Reproduce
Result
'Install' button is displayed after installing a chart when using localhost.
Expected Result
Edit and delete buttons display after installing a chart when using localhost.
Screenshots
...show what is displayed in localhost vs. remote setup after chart installation
Additional context
The text was updated successfully, but these errors were encountered: