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.

Sign up for GitHub

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

Jump to bottom

Re-running PV discovery shows wrong info when navigating back & changing namespaces #781

Closed
ibolton336 opened this issue Mar 27, 2020 · 2 comments · Fixed by #842
Closed

Re-running PV discovery shows wrong info when navigating back & changing namespaces #781

ibolton336 opened this issue Mar 27, 2020 · 2 comments · Fixed by #842
Assignees
@eriknelson
Labels
kind/bug Categorizes issue or PR as related to a bug. release-1.2.0 Required issue for 1.2 release
Projects
Sprint 182

Comments

@ibolton336
Copy link
Contributor

This issue stems from comparing the generation fields on the currentPlan & the response from a getPlan request when polling. There are cases when the generation is stale or not behaving as we expected making it impossible to determine when to rerun pv discovery/when to timeout pv discovery/etc

Related #777
@ibolton336 ibolton336 changed the title Rerunning PV discovery shows wrong info when navigating back & changing namespaces Re-running PV discovery shows wrong info when navigating back & changing namespaces Mar 27, 2020
@eriknelson eriknelson self-assigned this Mar 27, 2020
@eriknelson
Copy link
Contributor

I'll be picking this up when we move to using the new digest of the spec from the controller.

@ibolton336
Copy link
Contributor Author

pv_disc

@eriknelson eriknelson added kind/bug Categorizes issue or PR as related to a bug. release-1.2.0 Required issue for 1.2 release labels Mar 30, 2020
@eriknelson eriknelson added this to To do in Sprint 182 Mar 31, 2020
@ibolton336 ibolton336 mentioned this issue Apr 27, 2020
Merged
Sprint 182 automation moved this from To do to Done Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eriknelson eriknelson

Labels
kind/bug Categorizes issue or PR as related to a bug. release-1.2.0 Required issue for 1.2 release
Projects
No open projects
Sprint 182
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bug 1818021 - Implement observedDigest ibolton336/mig-ui
2 participants
@eriknelson @ibolton336