Skip to content

Conversation

@Yashikaone
Copy link
Contributor

@Yashikaone Yashikaone commented Aug 22, 2022

Description

This PR intends to build new functionality

If true safeshutdown will be triggered; if false completion will be awaited and system will be prepared for shutdown with the help of PrepareForShutdown api, this will not only monitor in_progress asynchronous jobs and wait for its completion and prevent addition of new asynchronous jobs.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@github-actions
Copy link

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

@nvazquez
Copy link
Contributor

Hi @Yashikaone thanks for your PR, I can see your changes on the second commit, however the first commit is not needed since there are no changes on it, can you please amend the PR to reflect only your changes?

@Yashikaone
Copy link
Contributor Author

Yashikaone commented Aug 22, 2022

@nvazquez Sure , I'll find a way to fix that. Can you look into the changes and let me know how can I get this correct?
Thanks!
I can't reflect the changes made in QueryManagerImpl.java file, seems strange.

@Yashikaone Yashikaone marked this pull request as draft August 22, 2022 17:58
@Yashikaone Yashikaone marked this pull request as ready for review August 22, 2022 17:58
@DaanHoogland DaanHoogland marked this pull request as draft August 23, 2022 07:05
@DaanHoogland
Copy link
Contributor

@Yashikaone , you should completely remove commit ce70939f3a3aeb9d0c7f0dda10aa5e531c461691 from your PR.
As the current status is you can do that by entering the command
git rebase -i HEAD~5 and then in the editor you get presented remove the line for that commit from the file. Git will remnove the commit. After the complete procedure check the result. You can then git push -- force and the commit with all the empty changes will be gone.

@acs-robot
Copy link

Found UI changes, kicking a new UI QA build
@blueorangutan ui

@blueorangutan
Copy link

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6662 (SL-JID-2231)

@yadvr
Copy link
Member

yadvr commented Apr 19, 2023

This was superseeded by #6755 closing on this remark. Thanks for the PR.

@yadvr yadvr closed this Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants