Skip to content

Conversation

@pritisarap12
Copy link

No description provided.

@asfbot
Copy link

asfbot commented Jun 18, 2015

cloudstack-pull-requests #486 SUCCESS
This pull request looks good

Choose a reason for hiding this comment

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

Does this test require any special storage pool? or with any tags? If not, there is no reason to add this check as it is expected that storage pool is present in setup.

Copy link
Author

Choose a reason for hiding this comment

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

This test does not require any special storage pool or with any tags. This check was put since it did not find any storage pool in UP state.i.e storage pool was present but was not in UP state.
So should I remove the check for skipTest?

Choose a reason for hiding this comment

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

There is no need for these checks. Instead the tests which leave the storage pool in disabled state should be looked at. Hold it for now.

@asfbot
Copy link

asfbot commented Jun 19, 2015

Sanjeev N on [email protected] replies:
Agree with Gaurav,
We should look at the tests which are changing the storage pool status. We
should make sure that those tests turn the status back to UP.

@gauravaradhye
Copy link

Please close this review request.

yadvr pushed a commit that referenced this pull request Jan 20, 2021
RodrigoDLopez pushed a commit to RodrigoDLopez/cloudstack that referenced this pull request Aug 23, 2022
…clouds'

Fix quota summary to removed domains

Closes apache#480

See merge request scclouds/scclouds!213
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Jan 27, 2025
* Safely shutdown feature (ref: apache#6755)

* Updated version and some improvements

* Management Server Maintenance - Prepare and Cancel Maintenance changes

This is supported for the Cloudstack deployments with multiple management servers.
- During preparing for maintenance, MS waits for pending jobs to finish, and then Transfer/Migrate the agents to other available MS
- New APIs: prepareForMaintenance, cancelMaintenance
- New MS States: PreparingToMaintenance, Maintenance

* check for single active management server

* refactoring plugin name

* updated version, and cleanup

* code improvements

* support list hosts by management server id

* update ui with ms maintenance apis

* code improvements

* ui changes

* ui icons update

* ui fixes

* cond checks for maintenance and shutdown

* fix for management server not down issue on service stop

* continue with other components on error

* agent transfer fixes

* maintenance window timeout and fixes

* ui changes - added connected agents tab, and updated hosts & management servers fields

* marvin test update

* keep maintenance after shutdown/restart, do not update last_updated time in cluster heartbeat during maintenance (notifies node inactive/down after heartbeat threshold)

* listener for ms maintenance updates

* cleanup

* keep last msid in host table

* review comments

* allow only one mgmt server to prepare for maintenance

* added ms uuid in logs

* minor code improvements

* ui fields update

* fix systemvm navigation in connected agents

* algorithm check and input from ui

* check for active ms from host setting

* agent migration code improvements

* minor ui label fix

* fixes & code improvements

* agent reconnect fixes, consider avoid list

* ui fixes

* direct agents transfer and pending jobs timer task fixes

* close unclosed socket channels if any

* Updated pending jobs check timer task with ScheduledExecutorService

* fixes

* keep maintenance state on trigger shutdown call when ms is in maintenance

* direct agent transfer fixes

* add pending jobs count to ms response

* during ms heartbeat, update state to up only when it's down

* allow vm work jobs of async job created before prepare for maintenance

* Revert "keep maintenance state on trigger shutdown call when ms is in maintenance"

This reverts commit 4ebbea71ef20a65286bed41a517f03e253a8fe90.

* removed duplicate schema changes from schema-41800to41810.sql (already defined at schema-41811to41812.sql)
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.

3 participants