Skip to content

systemvm: document workaround for older ACS env with systemvms#216

Closed
yadvr wants to merge 3 commits intoapache:4.15from
shapeblue:xs-systemvm-eject-fix
Closed

systemvm: document workaround for older ACS env with systemvms#216
yadvr wants to merge 3 commits intoapache:4.15from
shapeblue:xs-systemvm-eject-fix

Conversation

@yadvr
Copy link
Copy Markdown
Member

@yadvr yadvr commented May 31, 2021

No description provided.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr added this to the 4.15 milestone May 31, 2021
@yadvr yadvr requested a review from borisstoyanov May 31, 2021 08:34
@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented May 31, 2021

@blueorangutan docbuild

@blueorangutan
Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/216. (SL-JID 79)

Comment thread source/upgrading/upgrade/_sysvm_restart.rst Outdated
Co-authored-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented May 31, 2021

@blueorangutan docbuild

@blueorangutan
Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@yadvr yadvr requested a review from shwstppr May 31, 2021 08:42
@blueorangutan
Copy link
Copy Markdown

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/216. (SL-JID 80)

Copy link
Copy Markdown
Contributor

@borisstoyanov borisstoyanov left a comment

Choose a reason for hiding this comment

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

LGTM, tested the given workaround, happens with 4.9, 4.11, 4.13

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Jun 1, 2021

This is not needed, let's fix in code https://github.com/apache/cloudstack/pull/5053/files

@yadvr yadvr closed this Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants