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

Bug memory tab with dynamic memory missing url #184

Closed
cocoon opened this issue Apr 3, 2020 · 4 comments
Closed

Bug memory tab with dynamic memory missing url #184

cocoon opened this issue Apr 3, 2020 · 4 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@cocoon
Copy link
Collaborator

cocoon commented Apr 3, 2020

Describe the bug
Reported in the forum here:
https://xcp-ng.org/forum/topic/2804/yum-upgrade-8-0-to-8-1-vm-memory-error/

looks like a bug introduced by removing the Citrix-urls:
b2201f6#diff-07eabd7f8066c4f11a4c37b01b68cd39L192

root cause might be here:
https://github.com/xcp-ng/xenadmin/blob/development/XenAdmin/TabPages/BallooningPage.cs#L468

just compiled a build to try to see if commenting this will fix the error.

To Reproduce
Steps to reproduce the behavior:
https://xcp-ng.org/forum/topic/2804/yum-upgrade-8-0-to-8-1-vm-memory-error/

Expected behavior
No error, banner shoudl show that it is deprecated

Screenshots / Logfiles
If applicable, add screenshots / logfiles to help explain your problem.

Hard-/Software involved:
.

Additional context
.

@cocoon
Copy link
Collaborator Author

cocoon commented Apr 3, 2020

Confirmed:
commenting this line:
https://github.com/xcp-ng/xenadmin/blob/development/XenAdmin/TabPages/BallooningPage.cs#L468

removes the error but the "Details"-Link then does nothing.

So I think it would be better to provide an own DEPRECATION_URL or the previous one.

@borzel
Copy link
Member

borzel commented Apr 3, 2020

Hmm.. can you analyse to what that url should lead to? A webpage or some sort of data?

@borzel borzel added the bug Something isn't working label Apr 3, 2020
@borzel borzel added this to the Next Release milestone Apr 3, 2020
@cocoon
Copy link
Collaborator Author

cocoon commented Apr 4, 2020

I "think" it would be this:
https://docs.citrix.com/en-us/citrix-hypervisor/whats-new/removed-features.html

but I can confirm this with the other client.

Update: confirmed, this page opens.

@borzel borzel self-assigned this Apr 12, 2020
@borzel
Copy link
Member

borzel commented Apr 12, 2020

I change the DEPRECATION_URL to https://github.com/xcp-ng/xcp/wiki/XCP-ng-8.1-Release-Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants