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

Fix running query favicon becoming blue on reload #52

Merged
merged 2 commits into from
Jun 14, 2024
Merged

Conversation

siddharthvp
Copy link
Collaborator

Orange favicon = query in pending state (running or queued).
Blue favicon = query not in pending state.

Reloading the page always resulted in the default blue favicon. This has been fixed.

Bug: T362101

Orange favicon = query in pending state (running or queued).
Blue favicon = query not in pending state.

Reloading the page always resulted in the default blue favicon. This has been fixed.

Bug: T362101
@siddharthvp siddharthvp marked this pull request as ready for review June 14, 2024 10:12
Copy link
Member

@dhinus dhinus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@siddharthvp siddharthvp merged commit a14ff60 into main Jun 14, 2024
3 checks passed
@vivian-rook
Copy link
Contributor

I tried loading this, but it doesn't seem to work. When I reload the icon still turns to blue if the query is queued or running. I tried opening in a private tab in case I had a cached page, but I saw the same.

@siddharthvp
Copy link
Collaborator Author

It must be due to caching. The view.js file gets http-cached, so restarting the web service isn't enough to invalidate it. You might also need to do a hard reload in the browser.

@vivian-rook
Copy link
Contributor

It's deployed. Let me know if it is working for you.

@siddharthvp
Copy link
Collaborator Author

Deployment doesn't seem to have worked. Although kubectl describe deploy/web shows image with pr-52 tag, contents of the repo in the pods are from pr-51. The view.js file also shows the old code.

quarry@web-559b7c494c-htjdm:/app/helm-quarry$ cat values.yaml
web:
  repository: 'quay.io/wikimedia-quarry/quarry'
  tag: pr-51 # web tag managed by github actions

worker:
  repository: 'quay.io/wikimedia-quarry/quarry'
  tag: pr-51 # worker tag managed by github actions

@vivian-rook
Copy link
Contributor

Sure enough, appears to be building off of main, rather than the pr branch. I've opened T367630 to address the issue.

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

Successfully merging this pull request may close these issues.

3 participants