Use batching in Heat scheduler

Bug #1439675 reported by Sergey Kraynev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Medium
Peter Razumovsky
6.1.x
Won't Fix
Medium
Peter Razumovsky
7.0.x
Fix Released
High
MOS Heat
8.0.x
Fix Released
Medium
Peter Razumovsky

Bug Description

During processing stacks Heat may DDoS other services (on create, delete, etc.) It happens, because all requests will be sent in the same moment of time.

To avoid this situation we need add hook in scheduler, which allows to process all requests using batches.

Revision history for this message
Sergey Kraynev (skraynev) wrote :
Changed in mos:
status: New → Confirmed
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/heat (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Sergey Kraynev <email address hidden>
Review: https://review.fuel-infra.org/7414

ruhe (ruhe)
tags: added: scale
Revision history for this message
Sergey Kraynev (skraynev) wrote : Re: Change timeout for scheduler in Heat

This bug was re-assigned on Peter , because he currently works on solution for it.

Also Importance was raised, because it's necessary for scaling Heat by Sahara.

Changed in mos:
importance: Wishlist → High
assignee: Sergey Kraynev (skraynev) → Peter Razumovsky (prazumovsky)
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/heat (openstack-ci/fuel-7.0/2015.1.0)

Change abandoned by Sergey Kraynev <email address hidden> on branch: openstack-ci/fuel-7.0/2015.1.0
Review: https://review.fuel-infra.org/7414
Reason: Abandon in favor of

https://review.fuel-infra.org/#/c/9310/

https://review.fuel-infra.org/#/c/9593/

Revision history for this message
Sergey Kraynev (skraynev) wrote : Re: Change timeout for scheduler in Heat

Follow patches were uploaded on review:

https://review.fuel-infra.org/#/c/9310/
https://review.fuel-infra.org/#/c/9593/

They were tested locally.

Revision history for this message
Sergey Kraynev (skraynev) wrote :

Wait answer from Scale team (need deployment for testing these patches on Scale).

ruhe (ruhe)
Changed in mos:
status: Confirmed → In Progress
description: updated
Revision history for this message
Sergey Kraynev (skraynev) wrote :

It can not be ported to 6.1, also Sahara (author of this use case) does not use Heat engine in MOS 6.1

summary: - Change timeout for scheduler in Heat
+ Use batching in Heat scheduler
description: updated
Revision history for this message
Dina Belova (dbelova) wrote :

Adding Murano tag as this issue will also affect Murano huge deployments.

tags: added: murano
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/heat (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/9593
Submitter: mos-infra-ci <>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: 8dcd7fa1c2edc5ca11c8327798f6a9e3f30fb771
Author: Peter Razumovsky <email address hidden>
Date: Wed Aug 19 15:00:57 2015

Batching for tasks in scheduler

Add batch_size config for tasks in scheduler - now
only limited count of tasks can run in one time.

Change-Id: Ie24f16ca9a5b16cb85e976d8a710b0d10d6375c9
Closes-bug: #1439675

Revision history for this message
Sergey Kraynev (skraynev) wrote :

https://review.fuel-infra.org/#/c/9593/ was merged, so move this bug to Fix Committed

Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Evgeny Sikachev (esikachev) wrote :

Not need verification. In MOS 8.0 used direct engine instead Heat engine

Revision history for this message
Evgeny Sikachev (esikachev) wrote :

oops, typo, MOS 7.0 ^^^^^

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Sergey Kraynev (skraynev) wrote :

We need to make sure, that same patch (like for MOS 7.0) will be applied for MOS 8.0 too.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/heat (openstack-ci/fuel-8.0/liberty)

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Peter Razumovsky <email address hidden>
Review: https://review.fuel-infra.org/13387

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Peter Razumovsky <email address hidden>
Review: https://review.fuel-infra.org/13507

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/heat (openstack-ci/fuel-8.0/liberty)

Change abandoned by Sergey Kraynev <email address hidden> on branch: openstack-ci/fuel-8.0/liberty
Review: https://review.fuel-infra.org/13387
Reason: Abandon in favor of https://review.fuel-infra.org/#/c/13507/

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/heat (openstack-ci/fuel-8.0/liberty)

Reviewed: https://review.fuel-infra.org/13507
Submitter: Pkgs Jenkins <email address hidden>
Branch: openstack-ci/fuel-8.0/liberty

Commit: 47b8933ceacca4fabb8f1f3fa667cb86e6899a73
Author: Peter Razumovsky <email address hidden>
Date: Wed Dec 2 11:01:34 2015

Batching for tasks in scheduler

Add batch_size config for tasks in scheduler - now
only limited count of tasks can run in one time.

Change-Id: Ic1534038e4c067e5f72023e0b079f100320a93b2
Closes-bug: #1439675

Revision history for this message
Sergey Kraynev (skraynev) wrote :

The related patch was merged in the 8.0 branch. so we mark it as Fix commited

tags: added: area-heat
removed: heat
Revision history for this message
Evgeny Sikachev (esikachev) wrote :

verified on 482iso. 200 nodes scale

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/heat (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Peter Razumovsky <email address hidden>
Review: https://review.fuel-infra.org/18544

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/heat (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/18544
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 920089340f2c05f0b02f8ecdd035bf3b7f971131
Author: Peter Razumovsky <email address hidden>
Date: Mon Mar 28 11:04:05 2016

Batching for tasks in scheduler

Add batch_size config for tasks in scheduler - now
only limited count of tasks can run in one time.

Change-Id: Ic1534038e4c067e5f72023e0b079f100320a93b2
Closes-bug: #1439675

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/heat (11.0/ocata)

Fix proposed to branch: 11.0/ocata
Change author: Peter Razumovsky <email address hidden>
Review: https://review.fuel-infra.org/34466

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/heat (mcp/ocata)

Fix proposed to branch: mcp/ocata
Change author: Peter Razumovsky <email address hidden>
Review: https://review.fuel-infra.org/34818

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/heat (11.0/ocata)

Change abandoned by Pavlo Shchelokovskyy <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34466
Reason: 11.0/ocata branch is deprecated in favor of mcp/ocata

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/heat (mcp/ocata)

Reviewed: https://review.fuel-infra.org/34818
Submitter: Pkgs Jenkins <email address hidden>
Branch: mcp/ocata

Commit: 4ab11e89d00c7319cdfc376456495c7fa623673b
Author: Peter Razumovsky <email address hidden>
Date: Wed Apr 26 11:39:57 2017

Batching for tasks in scheduler

Add batch_size config for tasks in scheduler - now
only limited count of tasks can run in one time.

Change-Id: Ic1534038e4c067e5f72023e0b079f100320a93b2
Closes-bug: #1439675

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/heat (mcp/newton)

Reviewed: https://review.fuel-infra.org/33484
Submitter: Pkgs Jenkins <email address hidden>
Branch: mcp/newton

Commit: e55d83b7e86ce7a57c2d2a2bc5abb5831d50c400
Author: Peter Razumovsky <email address hidden>
Date: Thu Apr 27 11:18:17 2017

Batching for tasks in scheduler

Add batch_size config for tasks in scheduler - now
only limited count of tasks can run in one time.

Change-Id: Ic1534038e4c067e5f72023e0b079f100320a93b2
Closes-bug: #1439675

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.