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

CA-381212, CA-381664, CA-381215, CP-44618 #3204

Merged
merged 4 commits into from
Aug 18, 2023
Merged

Conversation

kc284
Copy link
Contributor

@kc284 kc284 commented Aug 18, 2023

No description provided.

@kc284 kc284 added the ASAP PR should be reviewed as soon as possible label Aug 18, 2023
@kc284 kc284 self-assigned this Aug 18, 2023
@kc284
Copy link
Contributor Author

kc284 commented Aug 18, 2023

I added one more commit. It's irrelevant to updates from CDN, but it was needed because the space requiremetns for the HA hearbeat SR have now increased and the info was outdated. I removed it completely, so we don't have to update it again. The user can always find more details in the docs.

@danilo-delbusso danilo-delbusso added the needs updating A reviewer has requested changes label Aug 18, 2023
…ates:

- Try waiting on get_updates if it is not immediately available.
- Log instead of failing when a known (404/500) BadServerResponseException occurs.
- Ensure the UI is refreshed after fetching the list of updates.

Signed-off-by: Konstantina Chremmou <[email protected]>
Signed-off-by: Konstantina Chremmou <[email protected]>
@kc284 kc284 added updated Changes completed, please review and removed needs updating A reviewer has requested changes labels Aug 18, 2023
@danilo-delbusso danilo-delbusso added 2 approvals PR has been approved by two reviewers and removed updated Changes completed, please review labels Aug 18, 2023
@kc284 kc284 merged commit 002bcbf into xenserver:master Aug 18, 2023
1 check passed
@kc284 kc284 deleted the cdn-fixes branch August 18, 2023 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 approvals PR has been approved by two reviewers ASAP PR should be reviewed as soon as possible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants