Page MenuHomePhabricator

cloudvps: capacity -- add cloudvirt1030 to scheduling pool or change documentation
Closed, ResolvedPublic

Description

repository OPUP.git at hieradata/eqiad/profile/openstack/eqiad1/nova.yaml has inconsistent documentation and implementation for cloudvirt1030 and its inclusion in the scheduling pool.

documentation states: # cloudvirt1030: pooled, whereas cloudvirt1030 is not included in profile::openstack::eqiad1::nova::scheduler_pool:

  1. investigate and decide whether cloudvirt1030 should be part of the scheduler pool
  2. update hieradata/eqiad/profile/openstack/eqiad1/nova.yaml to reflect the result of the decision

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 26 2019, 12:01 PM
Techguru.pc added a comment.EditedJun 26 2019, 12:05 PM

Note: this is my first writeup for wikitech and any feedback regarding tags or content is invited and will be appreciated. I am ready and available to submit a patch contribution once a decision on this issue is clear.
@Andrew @aborrero -- I added you as subscribers due to the fact that you are the most recent committers on the referenced puppet hieradata file

aborrero triaged this task as Normal priority.Jun 26 2019, 12:12 PM
aborrero edited projects, added cloud-services-team (Kanban); removed SRE-tools.

You are right, it's inconsistent. cloudvirt1030 is a Debian Stretch server (not affected by the ongoing reimages for Debian Jessie boxes). It has room for running more VMs [0], so I would suggest we simply put the server back into the scheduling pool if I'm not missing any other information bits (like ongoing HW maintenance or somethng).

@Techguru.pc , would you like to send a patch to gerrit addressing it? It should be an one-liner patch.

[0] https://grafana.wikimedia.org/d/aJgffPPmz/wmcs-openstack-eqiad1-hypervisor?orgId=1&var-hypervisor=cloudvirt1030&refresh=30s&from=now-7d&to=now

@aborrero thanks! I will submit a patch to gerrit -- first time for that too, so a bit of reading for me on submission guidelines

Change 519237 had a related patch set uploaded (by Techguru; owner: Techguru):
[operations/puppet@production] Increase capacity of CloudVPS

https://gerrit.wikimedia.org/r/519237

Change 519237 merged by Andrew Bogott:
[operations/puppet@production] Increase capacity of CloudVPS

https://gerrit.wikimedia.org/r/519237

Andrew closed this task as Resolved.Jun 26 2019, 3:05 PM

thanks!