| *** Apoorva_ has quit IRC | 00:01 | |
| *** Apoorva has joined #openstack-nova | 00:01 | |
| *** diga has joined #openstack-nova | 00:01 | |
| *** tongli has quit IRC | 00:08 | |
| auggy | stewie925: i am not sure if this is the best way, others might have a better suggestion, but usually you can just update master (git pull) and then switch to your branch and do a "git rebase master" | 00:11 |
|---|---|---|
| *** jwcroppe has joined #openstack-nova | 00:12 | |
| auggy | that works for me 95% of the time when i need to update branches | 00:12 |
| auggy | stewie925: when i have a lot of conflicts, i do a git log -p to see the diff of what i changed in my commit | 00:13 |
| auggy | stewie925: then i create a new branch off the current master and apply that diff | 00:13 |
| stewie925 | hey auggy | 00:14 |
| stewie925 | I know I messed up in that process | 00:14 |
| *** jamielennox is now known as jamielennox|away | 00:14 | |
| auggy | stewie925: it can get a little weird (and others may have more input here) but i've had success just using the same commit msg plus commit id as the original out of date change | 00:14 |
| auggy | when you do git review, as long as that commit id matches up it should post the update there | 00:15 |
| auggy | again it gets weird depending on what your situation is, but that's one route i've taken in the past and it worked ok for me | 00:15 |
| stewie925 | so basically I'd like to ask if - in lieu of doing another git clone - wondering if there is a way to get the latest and greatest version of the remote origin to my local branch | 00:15 |
| auggy | stewie925: oh, so you can checkout master, do agit pull to update and then do git review -d chang# | 00:16 |
| auggy | change# sorry | 00:16 |
| auggy | so on the url for your change it's like review.openstack.com/#/c?1234 | 00:16 |
| auggy | do git review -d 1234 | 00:16 |
| stewie925 | auggy I am planning to quash that latest commit | 00:16 |
| *** jwcroppe has quit IRC | 00:17 | |
| auggy | stewie925: ok so i'm not sure what you are asking | 00:17 |
| stewie925 | yeah I'll do what you suggested | 00:17 |
| *** ducttape_ has joined #openstack-nova | 00:17 | |
| stewie925 | checkout master, then do a git pull | 00:17 |
| auggy | stewie925: if your changes are in a branch you created, not downloaded from remote, then doing remote -d won't overwrite anything | 00:18 |
| auggy | however if you previously did a git review -d then doing it again on the same change may ovewrite, or git will throw up something about there being local changes (not sure) | 00:19 |
| stewie925 | i see | 00:19 |
| auggy | the git remote -d creates a branch that points to your change as the remote | 00:20 |
| auggy | it will be called something like review/your_name/your_branch | 00:20 |
| stewie925 | sorry please bear with me auggy | 00:22 |
| stewie925 | https://github.com/openstack/nova/blob/ceaacaa24fb50d484d4434915f3943fd9d97642a/api-ref/source/parameters.yaml | 00:22 |
| stewie925 | so I am going to change the parameters.yaml file | 00:23 |
| stewie925 | so I just did git pull at the master branch, and now master branch shows "your branch is up to date with origin/master" | 00:23 |
| stewie925 | but when I compare the parameters.yaml file vs that in github, theyre not the same | 00:24 |
| *** amotoki has joined #openstack-nova | 00:24 | |
| *** kaisers1 has joined #openstack-nova | 00:25 | |
| *** kaisers3 has quit IRC | 00:25 | |
| *** kaisers2 has joined #openstack-nova | 00:26 | |
| auggy | stewie925: why are you comparing it to github? | 00:27 |
| *** kaisers has quit IRC | 00:27 | |
| auggy | stewie925: if you're on master and you've pulled master then it's what's in master | 00:27 |
| auggy | unless you have something in your git config that suggests something different | 00:27 |
| *** markvoelker has joined #openstack-nova | 00:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/371112 | 00:29 |
| *** ducttape_ has quit IRC | 00:30 | |
| *** Apoorva_ has joined #openstack-nova | 00:31 | |
| *** Apoorva_ has quit IRC | 00:33 | |
| *** Apoorva has quit IRC | 00:34 | |
| stewie925 | no I have no "special" settings in my git config | 00:37 |
| stewie925 | sorrygot distracted by family stuff | 00:38 |
| stewie925 | oh and youre right auggy about the parameters.yaml.... | 00:41 |
| *** claudiub has joined #openstack-nova | 00:41 | |
| stewie925 | just tried to do a git clone (new repo) and i see the parameter.yaml matching my latest commit | 00:42 |
| stewie925 | (I shouldnt have been comparing with the github then) | 00:42 |
| *** claudiub|2 has quit IRC | 00:44 | |
| *** TravT has quit IRC | 00:44 | |
| *** hoangcx has joined #openstack-nova | 00:46 | |
| *** hoangcx_ has joined #openstack-nova | 00:48 | |
| *** browne has quit IRC | 00:48 | |
| *** hoangcx_ has quit IRC | 00:56 | |
| *** hoangcx has quit IRC | 00:56 | |
| *** hoangcx has joined #openstack-nova | 00:58 | |
| *** claudiub has quit IRC | 00:59 | |
| *** ducttape_ has joined #openstack-nova | 01:03 | |
| *** thorst_ has joined #openstack-nova | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:05 | |
| *** thorst_ has quit IRC | 01:07 | |
| *** jwcroppe has joined #openstack-nova | 01:07 | |
| *** jwcroppe has quit IRC | 01:14 | |
| *** warp_ has joined #openstack-nova | 01:15 | |
| *** haukebruno has quit IRC | 01:24 | |
| *** jwcroppe has joined #openstack-nova | 01:24 | |
| *** nic has quit IRC | 01:26 | |
| *** haukebruno has joined #openstack-nova | 01:26 | |
| *** ijw has quit IRC | 01:28 | |
| *** zz_dimtruck is now known as dimtruck | 01:30 | |
| *** EinstCrazy has joined #openstack-nova | 01:33 | |
| *** ducttape_ has quit IRC | 01:33 | |
| *** ducttape_ has joined #openstack-nova | 01:36 | |
| *** EinstCrazy has quit IRC | 01:37 | |
| *** diga has quit IRC | 01:40 | |
| *** dimtruck is now known as zz_dimtruck | 01:40 | |
| *** yuntongjin has joined #openstack-nova | 01:41 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add quota related tables to the api database. https://review.openstack.org/331810 | 01:41 |
| *** jwcroppe has quit IRC | 01:42 | |
| *** salv-orl_ has joined #openstack-nova | 01:42 | |
| *** tbachman has quit IRC | 01:42 | |
| *** ijw has joined #openstack-nova | 01:45 | |
| *** jwcroppe has joined #openstack-nova | 01:45 | |
| *** stewie925 has quit IRC | 01:45 | |
| *** salv-orlando has quit IRC | 01:45 | |
| *** dikonoor has joined #openstack-nova | 01:47 | |
| openstackgerrit | Alex Xu proposed openstack/nova: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051 | 01:51 |
| *** ljxiash has joined #openstack-nova | 01:52 | |
| *** liuyulong_ is now known as liuyulong | 01:53 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: devname tag will expose PFs when used with pci_passthrough_whitelist https://review.openstack.org/372728 | 01:54 |
| *** ljxiash has quit IRC | 01:56 | |
| *** jwcroppe has quit IRC | 01:57 | |
| *** tuanla_ has joined #openstack-nova | 01:58 | |
| *** baoli has joined #openstack-nova | 01:58 | |
| *** tuanla_ has quit IRC | 01:58 | |
| *** tuanla has joined #openstack-nova | 01:59 | |
| *** jwcroppe has joined #openstack-nova | 01:59 | |
| *** ljxiash has joined #openstack-nova | 02:00 | |
| *** ljxiash has quit IRC | 02:00 | |
| *** markvoelker has quit IRC | 02:02 | |
| *** Sukhdev has joined #openstack-nova | 02:02 | |
| *** markvoelker has joined #openstack-nova | 02:02 | |
| *** ducttape_ has quit IRC | 02:03 | |
| *** gstepanov__ has quit IRC | 02:03 | |
| *** ducttape_ has joined #openstack-nova | 02:03 | |
| *** Sukhdev has quit IRC | 02:04 | |
| *** unicell has quit IRC | 02:05 | |
| *** gstepanov__ has joined #openstack-nova | 02:05 | |
| *** tjones has joined #openstack-nova | 02:05 | |
| *** tjones has left #openstack-nova | 02:05 | |
| *** sdake has quit IRC | 02:06 | |
| *** sdake has joined #openstack-nova | 02:06 | |
| *** yuntongjin has quit IRC | 02:09 | |
| *** jwcroppe has quit IRC | 02:10 | |
| *** namnh has joined #openstack-nova | 02:12 | |
| *** yuntongjin has joined #openstack-nova | 02:12 | |
| *** tbachman has joined #openstack-nova | 02:12 | |
| *** ducttape_ has quit IRC | 02:17 | |
| *** tbachman has quit IRC | 02:17 | |
| *** sdake has quit IRC | 02:18 | |
| *** jamielennox|away is now known as jamielennox | 02:20 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs: CellsV2 - Move quota tables to the API db https://review.openstack.org/372839 | 02:23 |
| *** zz_dimtruck is now known as dimtruck | 02:24 | |
| *** jwcroppe has joined #openstack-nova | 02:24 | |
| *** ijw has quit IRC | 02:28 | |
| *** ducttape_ has joined #openstack-nova | 02:28 | |
| *** Kevin_Zheng has joined #openstack-nova | 02:32 | |
| *** ijw has joined #openstack-nova | 02:37 | |
| *** Sukhdev has joined #openstack-nova | 02:38 | |
| *** baoli has quit IRC | 02:38 | |
| *** baoli has joined #openstack-nova | 02:39 | |
| *** kevinz has joined #openstack-nova | 02:40 | |
| *** ijw has quit IRC | 02:41 | |
| *** ljxiash has joined #openstack-nova | 02:42 | |
| *** jwcroppe has quit IRC | 02:44 | |
| *** jwcroppe has joined #openstack-nova | 02:45 | |
| *** ducttape_ has quit IRC | 02:50 | |
| *** zul has joined #openstack-nova | 02:52 | |
| *** yuntongjin has quit IRC | 02:54 | |
| *** adisky has joined #openstack-nova | 02:58 | |
| *** woodster_ has quit IRC | 03:00 | |
| *** markvoelker has quit IRC | 03:02 | |
| *** markvoelker has joined #openstack-nova | 03:02 | |
| *** david-lyle has quit IRC | 03:03 | |
| *** baoli has quit IRC | 03:04 | |
| *** jwcroppe has quit IRC | 03:10 | |
| *** prateek_ has joined #openstack-nova | 03:16 | |
| *** ljxiash has quit IRC | 03:17 | |
| *** ljxiash has joined #openstack-nova | 03:18 | |
| *** _coolsvap_ has joined #openstack-nova | 03:22 | |
| *** _coolsvap_ is now known as coolsvap | 03:22 | |
| *** ljxiash has quit IRC | 03:22 | |
| *** gcb has joined #openstack-nova | 03:27 | |
| *** sudipto has joined #openstack-nova | 03:27 | |
| *** sudipto_ has joined #openstack-nova | 03:28 | |
| *** yuntongjin has joined #openstack-nova | 03:30 | |
| *** ljxiash has joined #openstack-nova | 03:32 | |
| *** jwcroppe has joined #openstack-nova | 03:33 | |
| *** unicell has joined #openstack-nova | 03:39 | |
| *** yuntongjin has quit IRC | 03:42 | |
| *** jwcroppe has quit IRC | 03:43 | |
| *** jwcroppe has joined #openstack-nova | 03:46 | |
| *** dimtruck is now known as zz_dimtruck | 03:47 | |
| *** ducttape_ has joined #openstack-nova | 03:52 | |
| *** garyk has joined #openstack-nova | 03:56 | |
| *** artom has quit IRC | 03:56 | |
| *** artom has joined #openstack-nova | 03:57 | |
| *** artom has quit IRC | 03:58 | |
| *** ducttape_ has quit IRC | 03:58 | |
| *** artom has joined #openstack-nova | 03:58 | |
| openstackgerrit | Ha Van Tu proposed openstack/nova: Remove unused parameters https://review.openstack.org/365929 | 03:59 |
| *** ducttape_ has joined #openstack-nova | 04:04 | |
| *** ducttape_ has quit IRC | 04:09 | |
| *** ducttape_ has joined #openstack-nova | 04:10 | |
| *** jwcroppe has quit IRC | 04:11 | |
| *** jwcroppe has joined #openstack-nova | 04:17 | |
| *** ducttape_ has quit IRC | 04:18 | |
| openstackgerrit | Hussain Chachuliya proposed openstack/nova: Remove duplicate key from dictionary https://review.openstack.org/372878 | 04:18 |
| *** ducttape_ has joined #openstack-nova | 04:18 | |
| *** yamahata has joined #openstack-nova | 04:21 | |
| *** amotoki has quit IRC | 04:21 | |
| *** artom has quit IRC | 04:25 | |
| *** artom has joined #openstack-nova | 04:27 | |
| *** artom has quit IRC | 04:27 | |
| *** artom has joined #openstack-nova | 04:28 | |
| *** amotoki has joined #openstack-nova | 04:30 | |
| *** Jeffrey4l_ is now known as Jeffrey4l | 04:36 | |
| *** yuntongjin has joined #openstack-nova | 04:38 | |
| *** parora has joined #openstack-nova | 04:39 | |
| *** psachin_ has joined #openstack-nova | 04:40 | |
| *** prateek_ has quit IRC | 04:41 | |
| *** artom has quit IRC | 04:41 | |
| *** artom has joined #openstack-nova | 04:42 | |
| *** artom has quit IRC | 04:42 | |
| *** dikonoor has quit IRC | 04:43 | |
| *** ratailor has joined #openstack-nova | 04:43 | |
| *** artom has joined #openstack-nova | 04:43 | |
| *** ducttape_ has quit IRC | 04:43 | |
| *** jwcroppe has quit IRC | 04:43 | |
| *** amotoki has quit IRC | 04:44 | |
| *** diga has joined #openstack-nova | 04:45 | |
| *** kevinz has quit IRC | 04:46 | |
| *** jwcroppe has joined #openstack-nova | 04:46 | |
| *** sdake has joined #openstack-nova | 04:46 | |
| *** markvoelker has quit IRC | 04:46 | |
| *** dikonoor has joined #openstack-nova | 04:48 | |
| *** kevin__ has joined #openstack-nova | 04:50 | |
| *** kevin__ is now known as kevinz | 04:50 | |
| *** kevinz has quit IRC | 04:53 | |
| *** jaosorior has joined #openstack-nova | 04:55 | |
| *** GB21 has joined #openstack-nova | 04:56 | |
| *** kevin__ has joined #openstack-nova | 04:59 | |
| *** kevin__ is now known as kevinz | 04:59 | |
| *** dikonoo has joined #openstack-nova | 05:02 | |
| *** jwcroppe has quit IRC | 05:02 | |
| *** armax has quit IRC | 05:04 | |
| *** dikonoor has quit IRC | 05:05 | |
| *** thorst_ has joined #openstack-nova | 05:05 | |
| *** pbandark has joined #openstack-nova | 05:05 | |
| *** jroll has quit IRC | 05:09 | |
| *** jroll has joined #openstack-nova | 05:09 | |
| *** jroll has quit IRC | 05:09 | |
| *** jroll has joined #openstack-nova | 05:09 | |
| *** jaosorior has quit IRC | 05:09 | |
| *** moshele has joined #openstack-nova | 05:09 | |
| *** jaosorior has joined #openstack-nova | 05:09 | |
| *** parora has quit IRC | 05:10 | |
| *** sudipto has quit IRC | 05:10 | |
| *** sudipto_ has quit IRC | 05:10 | |
| *** jwcroppe has joined #openstack-nova | 05:10 | |
| *** akshai has joined #openstack-nova | 05:11 | |
| *** yuntongjin has quit IRC | 05:12 | |
| *** amotoki has joined #openstack-nova | 05:14 | |
| *** thorst_ has quit IRC | 05:14 | |
| *** jwcroppe has quit IRC | 05:15 | |
| *** prateek has joined #openstack-nova | 05:16 | |
| *** dikonoor has joined #openstack-nova | 05:16 | |
| *** dikonoo has quit IRC | 05:19 | |
| *** akshai has quit IRC | 05:19 | |
| *** ecelik has joined #openstack-nova | 05:20 | |
| *** Sukhdev has quit IRC | 05:22 | |
| *** bnemec has joined #openstack-nova | 05:27 | |
| *** aswadr_ has joined #openstack-nova | 05:29 | |
| *** ekuris has joined #openstack-nova | 05:29 | |
| *** 7GHAAB7BF has joined #openstack-nova | 05:31 | |
| *** 64MAAYZON has joined #openstack-nova | 05:31 | |
| *** jwcroppe has joined #openstack-nova | 05:34 | |
| *** hieulq has quit IRC | 05:36 | |
| *** tuanla_ has joined #openstack-nova | 05:36 | |
| *** hoangcx_ has joined #openstack-nova | 05:36 | |
| *** phuongnh has quit IRC | 05:37 | |
| *** namnh has quit IRC | 05:38 | |
| *** hoangcx has quit IRC | 05:38 | |
| *** tuanla has quit IRC | 05:38 | |
| *** phuongnh has joined #openstack-nova | 05:38 | |
| *** namnh has joined #openstack-nova | 05:38 | |
| *** hshiina has joined #openstack-nova | 05:43 | |
| *** ducttape_ has joined #openstack-nova | 05:44 | |
| *** jwcroppe has quit IRC | 05:47 | |
| *** markvoelker has joined #openstack-nova | 05:47 | |
| *** mjura has joined #openstack-nova | 05:47 | |
| *** ducttape_ has quit IRC | 05:49 | |
| openstackgerrit | edan david proposed openstack/nova: Add missing slash to dir path https://review.openstack.org/372558 | 05:49 |
| *** amotoki_ has joined #openstack-nova | 05:51 | |
| *** hieulq has joined #openstack-nova | 05:51 | |
| *** markvoelker has quit IRC | 05:52 | |
| *** mjura has quit IRC | 05:54 | |
| *** mjura has joined #openstack-nova | 05:54 | |
| *** amotoki has quit IRC | 05:54 | |
| *** dikonoor has quit IRC | 05:55 | |
| *** slaweq_ has joined #openstack-nova | 05:56 | |
| *** edand has joined #openstack-nova | 05:58 | |
| ratailor | johnthetubaguy, I have replied to your comment on https://review.openstack.org/#/c/294491/, I can't call the newly added method as we don't have context object in reset method. | 06:02 |
| *** jwcroppe has joined #openstack-nova | 06:04 | |
| *** lpetrut has joined #openstack-nova | 06:04 | |
| *** diga has quit IRC | 06:06 | |
| openstackgerrit | Rong Han proposed openstack/nova: Reset migration.dest_node after finish_revert_resize successfully https://review.openstack.org/335270 | 06:09 |
| *** claudiub has joined #openstack-nova | 06:09 | |
| *** amotoki has joined #openstack-nova | 06:10 | |
| *** dikonoor has joined #openstack-nova | 06:12 | |
| *** amotoki_ has quit IRC | 06:13 | |
| *** jwcroppe has quit IRC | 06:14 | |
| *** oanson has joined #openstack-nova | 06:15 | |
| *** slaweq_ has quit IRC | 06:16 | |
| *** sandanar has joined #openstack-nova | 06:18 | |
| *** sandanar_ has joined #openstack-nova | 06:18 | |
| *** EinstCrazy has joined #openstack-nova | 06:18 | |
| *** digambar_patil has joined #openstack-nova | 06:19 | |
| *** irenab has joined #openstack-nova | 06:20 | |
| *** hoangcx_ has quit IRC | 06:22 | |
| *** hoangcx has joined #openstack-nova | 06:22 | |
| *** irenab_ has joined #openstack-nova | 06:22 | |
| *** irenab has quit IRC | 06:24 | |
| *** irenab_ is now known as irenab | 06:24 | |
| *** irenab has quit IRC | 06:29 | |
| *** irenab has joined #openstack-nova | 06:29 | |
| *** andreas_s has joined #openstack-nova | 06:30 | |
| *** amotoki has quit IRC | 06:31 | |
| *** slaweq_ has joined #openstack-nova | 06:31 | |
| *** digambar_patil has quit IRC | 06:32 | |
| *** diga has joined #openstack-nova | 06:32 | |
| *** ociuhandu has joined #openstack-nova | 06:32 | |
| *** jwcroppe has joined #openstack-nova | 06:33 | |
| *** amotoki has joined #openstack-nova | 06:35 | |
| *** sahid has joined #openstack-nova | 06:39 | |
| openstackgerrit | zhufl proposed openstack/nova: Remove unnecessary setUp and tearDown https://review.openstack.org/372941 | 06:39 |
| *** unicell1 has joined #openstack-nova | 06:43 | |
| *** unicell has quit IRC | 06:43 | |
| *** paul-carlton2 has joined #openstack-nova | 06:44 | |
| *** sridharg has joined #openstack-nova | 06:47 | |
| *** jwcroppe has quit IRC | 06:47 | |
| *** markvoelker has joined #openstack-nova | 06:48 | |
| *** pcaruana has joined #openstack-nova | 06:50 | |
| *** markvoelker has quit IRC | 06:52 | |
| *** ociuhandu has quit IRC | 06:53 | |
| *** hussain_c has joined #openstack-nova | 06:54 | |
| hussain_c | @alaski - Hello Andrew, can you please review this patch? https://review.openstack.org/#/c/372878/ | 06:58 |
| *** hshiina has quit IRC | 06:59 | |
| *** slaweq_ has quit IRC | 07:01 | |
| *** slaweq_ has joined #openstack-nova | 07:02 | |
| *** jwcroppe has joined #openstack-nova | 07:09 | |
| *** matrohon has joined #openstack-nova | 07:15 | |
| *** jwcroppe has quit IRC | 07:15 | |
| *** hussain_c has quit IRC | 07:15 | |
| *** namnh has quit IRC | 07:16 | |
| *** ashok has joined #openstack-nova | 07:16 | |
| *** hussain_c has joined #openstack-nova | 07:16 | |
| *** lpetrut has quit IRC | 07:17 | |
| *** slaweq_ has quit IRC | 07:18 | |
| *** jwcroppe has joined #openstack-nova | 07:18 | |
| *** rcernin has joined #openstack-nova | 07:19 | |
| *** yamahata has quit IRC | 07:27 | |
| *** jlanoux has joined #openstack-nova | 07:27 | |
| *** jwcroppe has quit IRC | 07:30 | |
| *** slaweq_ has joined #openstack-nova | 07:32 | |
| *** liuyulong_ has joined #openstack-nova | 07:34 | |
| *** liuyulong has quit IRC | 07:36 | |
| dulek | johnthetubaguy: Hi, quick question on https://review.openstack.org/#/c/289449 - how does this strategy go with continuous deployments from master. | 07:37 |
| dulek | johnthetubaguy: I mean - ready-to-be-backported migration on master has same number as in backport (one of the placeholders)? | 07:38 |
| bauzas | dulek: we tend to avoid DB migration backports unless it's a security bugfix | 07:39 |
| dulek | johnthetubaguy: I've started to wonder, as continuous deployments will not apply a placeholder, because they're past it already. | 07:39 |
| bauzas | dulek: but in case we need to backport a critical bugfix, then we have the placeholders for making sure we can still update the stable branch | 07:40 |
| dulek | bauzas: Sure thing, so if <x;y> are placeholders, then the bugfixing migration on master is also placed in this range? | 07:41 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix incorrect host cpus giving to emulator threads when RT https://review.openstack.org/356383 | 07:41 |
| *** salv-orlando has joined #openstack-nova | 07:42 | |
| dulek | bauzas: Or on master you're placing it in (y;inf) and cherry-pick to stable uses <x;y>? | 07:42 |
| bauzas | dulek: the latter | 07:44 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Remove default=None for config options https://review.openstack.org/373001 | 07:44 |
| *** salv-orl_ has quit IRC | 07:44 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix incorrect host cpus giving to emulator threads when RT https://review.openstack.org/356383 | 07:45 |
| dulek | bauzas: Okay, so an important assumption here is that migrations are idempotent. Am I thinking correctly? I think we're not enforcing this rule in Cinder, so we should change that. | 07:46 |
| bauzas | ooops, I made a translation error | 07:46 |
| bauzas | dulek: the former I meant | 07:46 |
| bauzas | dulek: ie. we keep the same number | 07:46 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix incorrect host cpus giving to emulator threads when RT https://review.openstack.org/356383 | 07:47 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622 | 07:48 |
| *** markvoelker has joined #openstack-nova | 07:49 | |
| bauzas | dulek: okay, I found the only example | 07:49 |
| bauzas | dulek: I remember we merged one in the past | 07:49 |
| dulek | bauzas: Okay, so now guys continuously deploying from trunk need to make sure the backported migration is applied manually, as their DBs are past the placeholders. Works for me, just wanted to make sure I understand this one. | 07:49 |
| bauzas | dulek: https://review.openstack.org/#/c/112107/ | 07:50 |
| bauzas | dulek: so, AFAICS, we made the initial bugfix on a non-placeholder migraiton number https://review.openstack.org/#/c/109660/ | 07:50 |
| *** ociuhandu has joined #openstack-nova | 07:51 | |
| bauzas | dulek: but given we wanted to backport that migration to the old release, we also made the same migration on master on an earlier DB migration number (ie. the placeholder) and then we just cherry-picked the commit to the stable branch we wanted to fix | 07:51 |
| *** liuyulong has joined #openstack-nova | 07:52 | |
| bauzas | from a DB perspective, I think what's really important is that we now only allow additive modifications past-Kilo | 07:52 |
| bauzas | which means that we're sure we're idempotent | 07:52 |
| *** EinstCrazy has quit IRC | 07:53 | |
| bauzas | dulek: ^ | 07:53 |
| *** markvoelker has quit IRC | 07:53 | |
| dulek | bauzas: Cool, now I get that! Thank you! | 07:54 |
| *** ociuhandu has quit IRC | 07:55 | |
| *** paul-carlton2 has quit IRC | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** ihrachys has joined #openstack-nova | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** speller has joined #openstack-nova | 08:07 | |
| *** jlanoux has quit IRC | 08:08 | |
| *** yassine has joined #openstack-nova | 08:08 | |
| *** yassine is now known as Guest19246 | 08:09 | |
| *** ociuhandu has joined #openstack-nova | 08:09 | |
| *** jlanoux has joined #openstack-nova | 08:09 | |
| *** ociuhandu has quit IRC | 08:14 | |
| *** ftersin has joined #openstack-nova | 08:19 | |
| *** mvk has quit IRC | 08:25 | |
| *** paul-carlton2 has joined #openstack-nova | 08:25 | |
| *** jwcroppe has joined #openstack-nova | 08:25 | |
| *** slaweq_ has quit IRC | 08:31 | |
| *** bradjones has quit IRC | 08:33 | |
| *** derekh has joined #openstack-nova | 08:34 | |
| *** bradjones has joined #openstack-nova | 08:36 | |
| *** bradjones has quit IRC | 08:36 | |
| *** bradjones has joined #openstack-nova | 08:36 | |
| openstackgerrit | Moshe Levi proposed openstack/nova-specs: User-controlled SR-IOV ports allocation https://review.openstack.org/182242 | 08:40 |
| *** slaweq_ has joined #openstack-nova | 08:43 | |
| *** jwcroppe has quit IRC | 08:45 | |
| johnthetubaguy | dulek: yeah, what bauzas said | 08:45 |
| *** yuntongjin has joined #openstack-nova | 08:46 | |
| *** claudiub has quit IRC | 08:47 | |
| *** zenoway has joined #openstack-nova | 08:47 | |
| *** ljxiash has quit IRC | 08:48 | |
| *** markvoelker has joined #openstack-nova | 08:50 | |
| *** hoangcx has quit IRC | 08:52 | |
| *** hoangcx has joined #openstack-nova | 08:52 | |
| openstackgerrit | Tom Patzig proposed openstack/nova-specs: Add "disable local disks" option to flavors https://review.openstack.org/373054 | 08:54 |
| *** danpb has joined #openstack-nova | 08:54 | |
| *** markvoelker has quit IRC | 08:54 | |
| *** 64MAAYZON has quit IRC | 08:59 | |
| *** 7GHAAB7BF has quit IRC | 08:59 | |
| *** I has joined #openstack-nova | 08:59 | |
| *** I has quit IRC | 08:59 | |
| *** Guest55745 has joined #openstack-nova | 09:00 | |
| *** sudipto_ has joined #openstack-nova | 09:00 | |
| *** jinli has quit IRC | 09:02 | |
| *** mvk has joined #openstack-nova | 09:02 | |
| *** syjulian has quit IRC | 09:03 | |
| *** cdent has joined #openstack-nova | 09:03 | |
| *** stew925 has quit IRC | 09:03 | |
| *** jwcroppe has joined #openstack-nova | 09:04 | |
| *** xenogear has quit IRC | 09:04 | |
| *** mirrorbo1 has quit IRC | 09:04 | |
| *** xenogear has joined #openstack-nova | 09:04 | |
| *** MikeG451 has quit IRC | 09:04 | |
| *** ociuhandu has joined #openstack-nova | 09:05 | |
| openstackgerrit | Tom Patzig proposed openstack/nova-specs: Add "disable local disks" option to flavors https://review.openstack.org/373054 | 09:07 |
| *** efoley has joined #openstack-nova | 09:08 | |
| *** jwcroppe has quit IRC | 09:11 | |
| *** efoley_ has joined #openstack-nova | 09:11 | |
| *** xenogear has quit IRC | 09:12 | |
| *** jwcroppe has joined #openstack-nova | 09:13 | |
| *** jinli has joined #openstack-nova | 09:14 | |
| *** efoley has quit IRC | 09:14 | |
| *** code-R has joined #openstack-nova | 09:15 | |
| *** gszasz has joined #openstack-nova | 09:17 | |
| *** MikeG451 has joined #openstack-nova | 09:19 | |
| *** edand has quit IRC | 09:19 | |
| *** yuntongjin has quit IRC | 09:21 | |
| bauzas | johnthetubaguy: so I was doing a bit of stable reviews when I found https://review.openstack.org/#/c/369116/1 | 09:21 |
| bauzas | johnthetubaguy: given you reviewed the master code, you could probably shed some light here :) | 09:21 |
| sfinucan | macsz: edleafe: I'd like to make some changes to these patches https://review.openstack.org/#/c/349666/ https://review.openstack.org/#/c/339736/ | 09:21 |
| sfinucan | So I can rebase this onto them https://review.openstack.org/#/c/349959/ | 09:22 |
| sfinucan | Is that OK?> | 09:22 |
| johnthetubaguy | bauzas: that sounds like a reasonable assumption, I can take a peak | 09:22 |
| macsz | sfinucan: what changes? :) | 09:22 |
| bauzas | johnthetubaguy: I can easily be confused and just thinking wrong, but I'd appreciate some other thoughts :) | 09:23 |
| sfinucan | macsz: Addressing the nits for edleafe's patch | 09:23 |
| bauzas | johnthetubaguy: I don't get why Cinder is returning the old volume ID as the connection info tl;dr | 09:23 |
| johnthetubaguy | bauzas: I did some digging around in the cinder code, the API is crazy, we get a new volume id that gets switch over back to the old one magically in some cases | 09:23 |
| *** code-R_ has joined #openstack-nova | 09:23 | |
| macsz | sfinucan: i still have UT to fix, but other than that I can hold on | 09:23 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: cleanup never used migratable flag checking https://review.openstack.org/310352 | 09:23 |
| bauzas | johnthetubaguy: that's what I fear | 09:24 |
| macsz | sfinucan: so you can do what you want to do | 09:24 |
| johnthetubaguy | bauzas: I am just reviewing the spec about CInder's API right now | 09:24 |
| sfinucan | macsz: Fire away with the UT - it's nothing drastic that would break rebase | 09:24 |
| johnthetubaguy | bauzas: its a little scary for a backport I guess | 09:24 |
| *** amotoki has quit IRC | 09:24 | |
| bauzas | johnthetubaguy: I tend to be pragmatic and accept the backport to have the same bugix | 09:25 |
| bauzas | johnthetubaguy: but I'd love to discuss why our master is so crappy | 09:25 |
| bauzas | johnthetubaguy: so that we could make sure we have a proper resolution | 09:25 |
| bauzas | that whole conditional seems just some clonk | 09:26 |
| macsz | finucan: go ahead and do what you want :) | 09:26 |
| *** code-R has quit IRC | 09:26 | |
| johnthetubaguy | bauzas: swap volume seems totally nuts, this is the first step in the correct direction: https://review.openstack.org/#/c/361512/1/specs/newton/add-new-attach-apis.rst | 09:26 |
| * bauzas looking | 09:27 | |
| bauzas | johnthetubaguy: woah, that's a fairly big change | 09:28 |
| johnthetubaguy | bauzas: yep, similar things cooking for neutron | 09:28 |
| *** xenogear has joined #openstack-nova | 09:29 | |
| *** lpetrut has joined #openstack-nova | 09:30 | |
| bauzas | mmm, not approved for Newton | 09:30 |
| *** mirrorbo1 has joined #openstack-nova | 09:31 | |
| bauzas | that looks promising, but for the moment, we deal with our tech debt | 09:31 |
| *** ircuser-1 has quit IRC | 09:32 | |
| bauzas | johnthetubaguy: looking at the migrate_volume_completion API doc | 09:32 |
| *** PaulMurray has joined #openstack-nova | 09:33 | |
| openstackgerrit | Ha Van Tu proposed openstack/nova: [api-ref] Correct parameter type https://review.openstack.org/367117 | 09:33 |
| johnthetubaguy | bauzas: its something we need to debate at the summit | 09:34 |
| bauzas | well, sure | 09:35 |
| bauzas | johnthetubaguy: sorry about that dumb question, but are using the Cinder v1 API ? | 09:37 |
| bauzas | are we* | 09:37 |
| *** syjulian has joined #openstack-nova | 09:37 | |
| johnthetubaguy | bauzas: good question, I assumed we were using v2, but I am not sure actually | 09:38 |
| bauzas | me too | 09:38 |
| bauzas | but can't find that call for v2 https://github.com/openstack/python-cinderclient/search?utf8=%E2%9C%93&q=migrate_volume_completion | 09:38 |
| bauzas | johnthetubaguy: ^ | 09:38 |
| *** stewie925 has joined #openstack-nova | 09:38 | |
| bauzas | I'd assume that v2 would use the v1 code | 09:39 |
| bauzas | (on the client I mean) | 09:39 |
| bauzas | I'm like total newbie on the cinder code :) | 09:39 |
| bauzas | except I played with it as operator :) | 09:39 |
| *** rkmrhj has joined #openstack-nova | 09:42 | |
| *** namnh has joined #openstack-nova | 09:43 | |
| rkmrhj | Hi all, sorry to bother you so late at night... | 09:45 |
| rkmrhj | I pushed a bugfix: Nova shelve creates duplicated images in cells https://review.openstack.org/#/c/327001/ | 09:46 |
| rkmrhj | This patch got one "+1" and one "+2" . I want another reviewer to merge. | 09:46 |
| rkmrhj | Andrew Laski already put "+2". This patch is small. | 09:46 |
| rkmrhj | If anyone can review my patch, please do it. | 09:46 |
| *** houming has quit IRC | 09:46 | |
| rkmrhj | P.S. The bug that is fixed by my patch happenes in normal pattern in cells. I think that the bug happening in normal parrtern should be fixed earlier. | 09:46 |
| *** oanson has quit IRC | 09:48 | |
| *** oanson has joined #openstack-nova | 09:48 | |
| bauzas | johnthetubaguy: okay, I guess I need to reproduce the bug in my local devstack... | 09:49 |
| *** Oku_OS is now known as Oku_OS-away | 09:49 | |
| *** Oku_OS-away is now known as Oku_OS | 09:50 | |
| *** jwcroppe has quit IRC | 09:50 | |
| *** ociuhandu has quit IRC | 09:50 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix incorrect host cpus giving to emulator threads when RT https://review.openstack.org/356383 | 09:53 |
| *** ociuhandu has joined #openstack-nova | 09:54 | |
| *** slaweq_ has quit IRC | 09:57 | |
| *** psachin_ has quit IRC | 09:58 | |
| *** artom has quit IRC | 10:02 | |
| *** artom has joined #openstack-nova | 10:03 | |
| *** phuongnh has quit IRC | 10:05 | |
| *** amotoki has joined #openstack-nova | 10:09 | |
| *** vipul has quit IRC | 10:11 | |
| *** hoangcx has quit IRC | 10:11 | |
| *** vipul has joined #openstack-nova | 10:11 | |
| *** jwcroppe has joined #openstack-nova | 10:12 | |
| *** abalutoiu has joined #openstack-nova | 10:12 | |
| *** jvgrant__ has joined #openstack-nova | 10:13 | |
| *** pkoniszewski has quit IRC | 10:13 | |
| *** slaweq_ has joined #openstack-nova | 10:13 | |
| *** jvgrant_ has quit IRC | 10:14 | |
| *** pkoniszewski has joined #openstack-nova | 10:16 | |
| sfinucan | johnthetubaguy: Can we add 'min' to config options without warning people the cycle before? | 10:17 |
| johnthetubaguy | sfinucan: not really, unless it would never have worked before | 10:18 |
| sfinucan | johnthetubaguy: that's what I though. Cool cool | 10:18 |
| johnthetubaguy | sfinucan: for all those period ones, I think letting people set -42 or whatever is halmless, so probably not worth worrying about | 10:18 |
| *** jwcroppe has quit IRC | 10:19 | |
| *** amotoki has quit IRC | 10:21 | |
| *** jwcroppe has joined #openstack-nova | 10:21 | |
| *** tyrola has joined #openstack-nova | 10:23 | |
| *** dave-johnston has joined #openstack-nova | 10:30 | |
| *** sdague has joined #openstack-nova | 10:31 | |
| *** veena has joined #openstack-nova | 10:32 | |
| *** mvk has quit IRC | 10:33 | |
| *** prateek has quit IRC | 10:34 | |
| *** claudiub has joined #openstack-nova | 10:34 | |
| *** mvk has joined #openstack-nova | 10:35 | |
| *** prateek has joined #openstack-nova | 10:35 | |
| *** claudiub|2 has joined #openstack-nova | 10:36 | |
| *** tyrola has quit IRC | 10:37 | |
| *** jaosorior is now known as jaosorior_lunch | 10:37 | |
| *** claudiub has quit IRC | 10:39 | |
| *** dikonoor has quit IRC | 10:39 | |
| *** rodrigods has quit IRC | 10:41 | |
| *** rodrigods has joined #openstack-nova | 10:41 | |
| *** amotoki has joined #openstack-nova | 10:42 | |
| *** jwcroppe has quit IRC | 10:46 | |
| *** tyrola has joined #openstack-nova | 10:47 | |
| *** rtheis has joined #openstack-nova | 10:48 | |
| *** slaweq_ has quit IRC | 10:48 | |
| *** haukebruno has quit IRC | 10:49 | |
| *** tr3buchet has quit IRC | 10:49 | |
| *** tr3buchet has joined #openstack-nova | 10:50 | |
| *** haukebruno has joined #openstack-nova | 10:50 | |
| *** psachin_ has joined #openstack-nova | 10:51 | |
| *** slaweq_ has joined #openstack-nova | 10:51 | |
| *** smatzek has joined #openstack-nova | 10:52 | |
| *** dikonoor has joined #openstack-nova | 10:53 | |
| *** tyrola has quit IRC | 10:54 | |
| *** amotoki has quit IRC | 10:57 | |
| *** jaosorior_lunch is now known as jaosorior | 11:03 | |
| *** jwcroppe has joined #openstack-nova | 11:05 | |
| *** code-R_ has quit IRC | 11:06 | |
| *** EinstCrazy has joined #openstack-nova | 11:09 | |
| *** Guest55745 has quit IRC | 11:11 | |
| *** sudipto_ has quit IRC | 11:11 | |
| *** code-R has joined #openstack-nova | 11:11 | |
| *** houming has joined #openstack-nova | 11:14 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: POC: Resize ephemerals https://review.openstack.org/372721 | 11:16 |
| *** jwcroppe_ has joined #openstack-nova | 11:20 | |
| *** tyrola has joined #openstack-nova | 11:20 | |
| *** jwcroppe has quit IRC | 11:20 | |
| *** CristinaPauna has quit IRC | 11:21 | |
| *** dikonoo has joined #openstack-nova | 11:21 | |
| *** dikonoor has quit IRC | 11:22 | |
| openstackgerrit | liuxiuli proposed openstack/nova: Modify request numa_topology when resize https://review.openstack.org/340270 | 11:22 |
| *** jaypipes has joined #openstack-nova | 11:22 | |
| openstackgerrit | liuxiuli proposed openstack/nova: Modify workflow of rebuild_instance when schedule fails https://review.openstack.org/348111 | 11:23 |
| *** ochalups has joined #openstack-nova | 11:23 | |
| *** bvanhav has joined #openstack-nova | 11:23 | |
| *** dikonoo has quit IRC | 11:26 | |
| *** psachin_ has quit IRC | 11:27 | |
| *** speller has quit IRC | 11:27 | |
| *** karthiks has quit IRC | 11:28 | |
| *** purplerbot has quit IRC | 11:30 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Remove duplicated code blocks in format_dom https://review.openstack.org/232145 | 11:30 |
| *** jwcroppe_ has quit IRC | 11:31 | |
| *** code-R has quit IRC | 11:31 | |
| *** psachin has joined #openstack-nova | 11:32 | |
| *** tyrola has quit IRC | 11:32 | |
| *** tyrola has joined #openstack-nova | 11:33 | |
| *** tyrola has quit IRC | 11:33 | |
| jaypipes | *yawn* morning folks | 11:33 |
| *** dikonoo has joined #openstack-nova | 11:33 | |
| *** jwcroppe has joined #openstack-nova | 11:38 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203 | 11:39 |
| *** code-R has joined #openstack-nova | 11:40 | |
| *** purplerbot has joined #openstack-nova | 11:43 | |
| *** smatzek has quit IRC | 11:43 | |
| *** lucas-afk is now known as lucasagomes | 11:44 | |
| *** jwcroppe has quit IRC | 11:46 | |
| *** amotoki has joined #openstack-nova | 11:47 | |
| *** smatzek has joined #openstack-nova | 11:49 | |
| *** shz has quit IRC | 11:49 | |
| *** ekuris has quit IRC | 11:49 | |
| *** ducttape_ has joined #openstack-nova | 11:49 | |
| *** shihanzhang has joined #openstack-nova | 11:49 | |
| *** psachin_ has joined #openstack-nova | 11:52 | |
| *** psachin_ has quit IRC | 11:52 | |
| *** ducttape_ has quit IRC | 11:53 | |
| *** psachin_ has joined #openstack-nova | 11:53 | |
| *** coolsvap is now known as coolsvap_ | 11:55 | |
| *** coolsvap_ has quit IRC | 11:55 | |
| *** coolsvap_ has joined #openstack-nova | 11:55 | |
| *** GB21 has quit IRC | 11:55 | |
| *** xiaoyang has joined #openstack-nova | 11:57 | |
| *** ratailor has quit IRC | 11:59 | |
| *** psilvad has joined #openstack-nova | 11:59 | |
| *** psilvad has joined #openstack-nova | 11:59 | |
| *** namnh has quit IRC | 11:59 | |
| *** mvk has quit IRC | 12:00 | |
| *** mvk has joined #openstack-nova | 12:04 | |
| *** jwcroppe has joined #openstack-nova | 12:05 | |
| *** nkrinner_afk has quit IRC | 12:06 | |
| *** rfolco has joined #openstack-nova | 12:07 | |
| *** tuanla_ has quit IRC | 12:07 | |
| *** namnh has joined #openstack-nova | 12:08 | |
| *** amotoki has quit IRC | 12:08 | |
| *** CristinaPauna has joined #openstack-nova | 12:09 | |
| *** amotoki has joined #openstack-nova | 12:09 | |
| *** baoli has joined #openstack-nova | 12:10 | |
| *** jwcroppe has quit IRC | 12:11 | |
| *** baoli_ has joined #openstack-nova | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:12 | |
| *** jwcroppe has joined #openstack-nova | 12:13 | |
| *** baoli has quit IRC | 12:15 | |
| *** ducttape_ has joined #openstack-nova | 12:16 | |
| *** hshiina has joined #openstack-nova | 12:16 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: pci: Clarify SR-IOV ports vs direct passthrough ports https://review.openstack.org/300890 | 12:17 |
| *** nkrinner_afk has joined #openstack-nova | 12:18 | |
| *** maticue has joined #openstack-nova | 12:21 | |
| *** ecelik has quit IRC | 12:21 | |
| *** matrohon has quit IRC | 12:23 | |
| *** markvoelker has joined #openstack-nova | 12:24 | |
| *** psachin_ has quit IRC | 12:24 | |
| *** psachin_ has joined #openstack-nova | 12:25 | |
| *** psachin_ has quit IRC | 12:25 | |
| *** cdent has quit IRC | 12:26 | |
| *** diga has quit IRC | 12:28 | |
| *** yassine has joined #openstack-nova | 12:28 | |
| *** yassine is now known as Guest54237 | 12:29 | |
| *** Guest19246 has quit IRC | 12:29 | |
| *** cdent has joined #openstack-nova | 12:30 | |
| *** ayoung has quit IRC | 12:30 | |
| *** ducttape_ has quit IRC | 12:31 | |
| *** jwcroppe has quit IRC | 12:32 | |
| sfinucan | jaypipes: o/ | 12:33 |
| edleafe | sfinucan: Just saw your comment. Please feel free to take those over. I got discouraged with the whole process when reviewers couldn't even bother to read the commit message. | 12:33 |
| sfinucan | (an hour late) | 12:33 |
| sfinucan | edleafe: Yeah. I was one of those reviewers, sadly | 12:33 |
| sfinucan | ...but I've learnt the error of my ways, I hope | 12:33 |
| *** sudipto_ has joined #openstack-nova | 12:34 | |
| *** sudswas__ has joined #openstack-nova | 12:34 | |
| edleafe | sfinucan: After so many "oh, the scheduler options are too big to do in one patch" (I think I had to break them into 7), to then get dinged for not doing everything in one patch just made me not care anymore. | 12:34 |
| *** annegentle has joined #openstack-nova | 12:35 | |
| *** krtaylor has joined #openstack-nova | 12:36 | |
| *** kylek3h has joined #openstack-nova | 12:36 | |
| *** mdrabe has joined #openstack-nova | 12:37 | |
| *** matrohon has joined #openstack-nova | 12:37 | |
| *** kylek3h_ has joined #openstack-nova | 12:38 | |
| *** jwcroppe has joined #openstack-nova | 12:39 | |
| *** psachin_ has joined #openstack-nova | 12:39 | |
| *** psachin_ has quit IRC | 12:40 | |
| *** kylek3h has quit IRC | 12:41 | |
| sfinucan | Anyone know if there's an equivalent of unittest's assertRaises context manager ('with assertRaises(X):') in testtools? | 12:41 |
| namnh | johnthetubaguy, are you free?. Could you please give me few minutes? | 12:42 |
| openstackgerrit | jichenjc proposed openstack/nova: Remove RateLimitFault class https://review.openstack.org/373254 | 12:42 |
| *** psachin_ has joined #openstack-nova | 12:43 | |
| jaypipes | sfinucan: with testtools.ExpectedException()... | 12:43 |
| sfinucan | jaypipes: Yup, that's the one | 12:44 |
| *** mvk has quit IRC | 12:45 | |
| johnthetubaguy | namnh: I should be able to help quickly, hows things? | 12:46 |
| *** mvk has joined #openstack-nova | 12:46 | |
| namnh | johnthetubaguy, Thanks for your reply, Mr.Garbutt | 12:47 |
| *** jwcroppe has quit IRC | 12:47 | |
| namnh | johnthetubaguy, Accroding to your comment. We need to turn off all three controllers to upgrade as link: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2016-09-12.log.html#t2016-09-12T11:11:06 | 12:47 |
| namnh | johnthetubaguy, It's mean Nova does not support rolling-upgrade feature, right? | 12:47 |
| namnh | johnthetubaguy, But when I try to test this feature. I just shut off one controller node and upgrade from L to M. | 12:47 |
| openstackgerrit | jichenjc proposed openstack/nova: Rate limit is removed , update doc accordingly https://review.openstack.org/373261 | 12:47 |
| namnh | johnthetubaguy, On the controller, I ran two commands to upgrade nova-database. After that, I tried to create the VM on other controller. Everything worked well. | 12:47 |
| *** burgerk has joined #openstack-nova | 12:47 | |
| namnh | johnthetubaguy, It means that old-source-code (Librety) can affect to new database (Mitaka). So I think that nova supported rolling-upgrade feature. How do you think about this? Did I have any mistake here? | 12:47 |
| johnthetubaguy | namnh: Nova supports rolling upgrade, but that doesn't mean what you think it means | 12:47 |
| *** kylek3h_ has quit IRC | 12:47 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix opt indentation for scheduler.py https://review.openstack.org/339736 | 12:47 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve consistency in scheduler opts https://review.openstack.org/349666 | 12:47 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Reorder scheduler opts https://review.openstack.org/373262 | 12:47 |
| johnthetubaguy | namnh: we support running older nova-compute nodes, with newer control plane nodes, the way we have architected things, nova-compute never writes directly to the database | 12:48 |
| johnthetubaguy | namnh: its possible running an old and new controller side by side, might work in some situations, but I wouldn't recommend doing it, as I don't believe we test that | 12:49 |
| *** kylek3h has joined #openstack-nova | 12:50 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Refactor console checks in live migration process https://review.openstack.org/373264 | 12:51 |
| namnh | johnthetubaguy, One more thing, I see that Nova-conductor (Librety) can affact to nova-db(Mitaka). Because during upgrading, there is only expand phase (I means there is only adding new table, column,... to DB). So that why nova-conductor can do that. right? | 12:53 |
| *** cdent has quit IRC | 12:53 | |
| namnh | johnthetubaguy, so regarding to your option, currenlty, nova does not support completely.right? | 12:54 |
| *** ekuris has joined #openstack-nova | 12:54 | |
| *** psilvad has quit IRC | 12:55 | |
| openstackgerrit | jichenjc proposed openstack/nova: Fix a typo from ID to Id https://review.openstack.org/373270 | 12:55 |
| *** psachin_ has quit IRC | 12:55 | |
| *** kaisers has joined #openstack-nova | 12:56 | |
| johnthetubaguy | namnh: I don't quite understand your question, we have made it work so you can do "nova db sync" with the old system running, i.e. its expand only | 12:56 |
| *** david-lyle has joined #openstack-nova | 12:56 | |
| *** psachin has quit IRC | 12:56 | |
| *** psachin has joined #openstack-nova | 12:56 | |
| *** vishwanathj has joined #openstack-nova | 12:57 | |
| *** psachin has quit IRC | 12:57 | |
| *** psachin has joined #openstack-nova | 12:57 | |
| *** psachin has quit IRC | 12:57 | |
| johnthetubaguy | namnh: ah, maybe I do get your question, you don't want mitaka and liberty controllers both running, because you will most likely get DB corruption | 12:58 |
| *** psachin has joined #openstack-nova | 12:58 | |
| *** psachin has joined #openstack-nova | 12:58 | |
| *** sridharg has quit IRC | 12:58 | |
| *** wasmum has quit IRC | 12:58 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add migrate_uri for invoking the migration https://review.openstack.org/356558 | 12:59 |
| *** rlrossit has joined #openstack-nova | 13:00 | |
| *** prateek has quit IRC | 13:01 | |
| *** prateek has joined #openstack-nova | 13:01 | |
| namnh | johnthetubaguy, Sorry for this. I means when I run "nova-manage db sync" and "nova-manage db api_db", at this time, we will have nova-database (Mitaka). After that, I still can continue to create new VMs on other controller (running Liberty). | 13:02 |
| *** zenoway has quit IRC | 13:02 | |
| prateek | ping ratailor | 13:02 |
| openstackgerrit | jichenjc proposed openstack/nova: Rate limit is removed , update doc accordingly https://review.openstack.org/373261 | 13:02 |
| *** zenoway has joined #openstack-nova | 13:02 | |
| namnh | johnthetubaguy, It means nova-conductor (Liberty) can affect to database (Mitaka). | 13:03 |
| *** acabot has joined #openstack-nova | 13:03 | |
| namnh | johnthetubaguy, I wondering why it can do that? | 13:03 |
| namnh | johnthetubaguy, so I did not have downtime when I upgrade Nova on controllers. | 13:04 |
| *** cdent has joined #openstack-nova | 13:05 | |
| namnh | johnthetubaguy, so I would like to confirm with you whether Nova support rolling upgrade or not? | 13:05 |
| *** jwcroppe has joined #openstack-nova | 13:06 | |
| johnthetubaguy | namnh: we support rolling upgrade, but its all about old and new nova-computes being side by side, see: https://governance.openstack.org/reference/tags/assert_supports-rolling-upgrade.html | 13:06 |
| johnthetubaguy | namnh: we don't support zero downtime upgrades, where you have old and new controllers both running (yet) | 13:06 |
| *** zenoway has quit IRC | 13:07 | |
| *** markd__ has joined #openstack-nova | 13:07 | |
| *** pilgrimstack has quit IRC | 13:07 | |
| *** markd__ has quit IRC | 13:08 | |
| *** lpetrut1 has joined #openstack-nova | 13:08 | |
| *** liverpooler has quit IRC | 13:08 | |
| *** liverpooler has joined #openstack-nova | 13:09 | |
| *** jaosorior has quit IRC | 13:09 | |
| *** jaosorior has joined #openstack-nova | 13:10 | |
| *** houming has quit IRC | 13:10 | |
| *** lpetrut has quit IRC | 13:10 | |
| *** lpetrut1 is now known as lpetrut | 13:10 | |
| *** sridharg has joined #openstack-nova | 13:11 | |
| *** jwcroppe has quit IRC | 13:11 | |
| *** tblakes has joined #openstack-nova | 13:11 | |
| *** mriedem has joined #openstack-nova | 13:11 | |
| namnh | johnthetubaguy, I think I need to investigate your commnet and rolling-upgrade in nova more deeply. I realy appreciate for your help. | 13:12 |
| namnh | johnthetubaguy, thanks a lot. | 13:12 |
| *** mgoddard__ has joined #openstack-nova | 13:13 | |
| *** zenoway has joined #openstack-nova | 13:16 | |
| *** mgoddard_ has quit IRC | 13:17 | |
| *** sdake has quit IRC | 13:17 | |
| *** namnh has quit IRC | 13:17 | |
| *** jwcroppe has joined #openstack-nova | 13:18 | |
| *** swamireddy has quit IRC | 13:19 | |
| *** ljxiash has joined #openstack-nova | 13:20 | |
| *** READ10 has joined #openstack-nova | 13:22 | |
| *** zenoway has quit IRC | 13:22 | |
| *** swamireddy has joined #openstack-nova | 13:23 | |
| *** zenoway has joined #openstack-nova | 13:23 | |
| *** cmartin has joined #openstack-nova | 13:23 | |
| *** mlavalle has joined #openstack-nova | 13:26 | |
| *** gouthamr has joined #openstack-nova | 13:27 | |
| cmartin | avolkov: I am trying to address your comments on this change https://review.openstack.org/#/c/370861/3/nova/compute/manager.py I have already addressed all of your comments but one on my local changes. What other instance_states do we need to cover in a function that catches a rollback on a resize exception? | 13:27 |
| *** smatzek has quit IRC | 13:27 | |
| *** psachin has quit IRC | 13:28 | |
| *** psachin has joined #openstack-nova | 13:28 | |
| *** tanvirt has joined #openstack-nova | 13:30 | |
| avolkov | cmartin: I thought about SHELVED, SHELVED_OFFLOADED, but I'd prefer don't resize for instance this such state at all | 13:31 |
| *** ducttape_ has joined #openstack-nova | 13:31 | |
| *** ducttape_ has quit IRC | 13:33 | |
| avolkov | paused and suspended also can, possibly, influence on this case | 13:33 |
| bauzas | johnthetubaguy: I'm confused | 13:34 |
| *** baoli_ has quit IRC | 13:34 | |
| *** ducttape_ has joined #openstack-nova | 13:34 | |
| bauzas | johnthetubaguy: I looked at my local devstack, and the serial info is related to the current volume | 13:34 |
| bauzas | so, it's fine | 13:34 |
| johnthetubaguy | bauzas: there are two cases, when you call nova, and when cinder calls nova | 13:35 |
| cmartin | avolkov: ok give me a bit to look into it | 13:35 |
| johnthetubaguy | bauzas: I can't remember which case has the problem now | 13:35 |
| bauzas | johnthetubaguy: the problem was when using the nova call | 13:35 |
| johnthetubaguy | bauzas: you are testing with stable that doesn't have the patch right? | 13:36 |
| bauzas | johnthetubaguy: so, I checked that with nova volume-update | 13:36 |
| bauzas | johnthetubaguy: nope | 13:36 |
| bauzas | using master | 13:36 |
| bauzas | but looking at the code, I don't understand why we use the previous volume ID | 13:36 |
| *** openstackstatus has joined #openstack-nova | 13:36 | |
| *** ChanServ sets mode: +v openstackstatus | 13:36 | |
| bauzas | anyway... | 13:36 |
| bauzas | I'll pdb it | 13:36 |
| johnthetubaguy | I think the need for the previous volume id is for the cinder case, but I am properly confused right now | 13:37 |
| mriedem | bauzas: johnthetubaguy: i'd try to get a hold of takashi natsume, he has been in the swap volume code the most recently | 13:37 |
| mriedem | e.g. https://github.com/openstack/nova/commit/be553fb15591c6fc212ef3a07c1dd1cbc43d6866 | 13:37 |
| prateek | johnthetubaguy, you have given a -2 on the patch https://review.openstack.org/#/c/201019/ , if I work on it, is this something that we can allow in the master branch ? | 13:37 |
| mriedem | at least for the nova (non-cinder volume migration) case, i had recreated the bug for ^ and his fixed resolved it | 13:38 |
| mriedem | s/fixed/fix/ | 13:38 |
| bauzas | mriedem: yup, I'm discussing with johnthetubaguy about https://review.openstack.org/#/c/369116/1 which is confusing me | 13:38 |
| *** r-daneel has joined #openstack-nova | 13:38 | |
| prateek | mriedem, are we willing to allow this to land on the master https://review.openstack.org/#/c/201019/ ? | 13:39 |
| *** belliott has quit IRC | 13:39 | |
| *** belliott has joined #openstack-nova | 13:39 | |
| *** belliott has quit IRC | 13:40 | |
| mriedem | prateek: as johnthetubaguy pointed out in that review, it looks like a feature, there is a blueprint, but i don't see a spec | 13:40 |
| *** belliott has joined #openstack-nova | 13:40 | |
| mriedem | prateek: i think that needs a spec for ocata | 13:41 |
| mriedem | the commit message isn't very clear to me on the use case or what it's trying to solve | 13:41 |
| prateek | mriedem, ok, so the way to get this patch merged is write a spec then a bp, and then take the code forward, also there is a bp attached to it | 13:41 |
| mriedem | prateek: there is already a blueprint yes https://blueprints.launchpad.net/nova/+spec/non-ephemeral-storage-quota-assign | 13:42 |
| prateek | mriedem, yeah , that is pretty vague, the patch addresses that when we boot an instance from volume | 13:42 |
| mriedem | but you'd need an ocata spec for that blueprint | 13:42 |
| prateek | then it does not pick the iotune parameter from the flavor | 13:42 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/readme.html | 13:42 |
| bauzas | mriedem: super weirdo, I just put an pdb entrypoint, but when using nova volume-update, it doesn't go thru that section | 13:42 |
| prateek | *nova flavor | 13:42 |
| *** salv-orl_ has joined #openstack-nova | 13:42 | |
| *** psachin has quit IRC | 13:42 | |
| bauzas | mriedem: which call were you using for testing the problem above ? | 13:42 |
| mriedem | bauzas: nova volume-update | 13:42 |
| *** psachin has joined #openstack-nova | 13:42 | |
| bauzas | weirdo again | 13:43 |
| mriedem | since i didn't have a setup for volume migration | 13:43 |
| bauzas | that's the one I'm using | 13:43 |
| bauzas | wtf... | 13:43 |
| -openstackstatus- NOTICE: OpenStack Infra now has a Twitter bot, follow it at https://twitter.com/openstackinfra | 13:44 | |
| bauzas | ah-ah ! | 13:45 |
| *** salv-orlando has quit IRC | 13:45 | |
| *** eharney has joined #openstack-nova | 13:46 | |
| prateek | mridem , thanks i would try writing a spec for the bp | 13:46 |
| *** annegentle has quit IRC | 13:46 | |
| *** hamzy has quit IRC | 13:47 | |
| johnthetubaguy | prateek: once we get the spec merged, and the blueprint approved for ocata, happy to remove the -2 | 13:47 |
| *** ducttape_ has quit IRC | 13:48 | |
| prateek | johnthetubaguy, do you consider that spec worthy to be merged ? | 13:48 |
| *** tbachman has joined #openstack-nova | 13:49 | |
| prateek | because the feature the spec addresses can be done through cinder | 13:49 |
| *** sdake has joined #openstack-nova | 13:49 | |
| *** yamahata has joined #openstack-nova | 13:50 | |
| bauzas | mriedem: something messed up with my local devstack, fun | 13:50 |
| *** belliott has quit IRC | 13:50 | |
| *** jwcroppe has quit IRC | 13:50 | |
| *** smatzek has joined #openstack-nova | 13:51 | |
| johnthetubaguy | prateek: for me, cinder volume quota should be handled by cinder, but I haven't read the spec or blueprint at all closely | 13:51 |
| *** gouthamr has quit IRC | 13:52 | |
| *** sandanar has quit IRC | 13:53 | |
| mriedem | garyk: see comment in https://review.openstack.org/#/c/253666/ | 13:53 |
| *** sandanar_ has quit IRC | 13:53 | |
| mriedem | garyk: looks like browne isn't around | 13:53 |
| mriedem | but i don't think we can just make that jump w/o a warning period | 13:53 |
| prateek | johnthetubaguy, ok , that patch addresses disk iops parameter , what the patch says is that when we BFV the nova flavor specs are not applied particularly for the iops parameter | 13:53 |
| prateek | so having 100 instances with the same iops , i have to go everytime to cinder and apply on every VM | 13:53 |
| mriedem | prateek: so this is the case that nova creates the volume? | 13:53 |
| mriedem | prateek: the qos for cinder is in the volume type right? | 13:54 |
| prateek | mriedem, yes | 13:54 |
| *** sridharg has quit IRC | 13:54 | |
| *** jwcroppe has joined #openstack-nova | 13:54 | |
| mriedem | prateek: there is already agreement to allow passing a volume type when booting from volume and nova creates the volume, | 13:54 |
| mriedem | that should be the solution, not nova creating the volume using qos from the nova flavor | 13:54 |
| mriedem | prateek: so i'd be -1 on the spec | 13:54 |
| prateek | mriedem, ok, thanks | 13:55 |
| mriedem | prateek: fyi http://lists.openstack.org/pipermail/openstack-dev/2016-August/102401.html | 13:56 |
| *** mgoddard__ has quit IRC | 13:56 | |
| *** belliott has joined #openstack-nova | 13:57 | |
| *** thorst_ has joined #openstack-nova | 13:57 | |
| *** mgoddard has joined #openstack-nova | 13:57 | |
| prateek | mriedem, ok, that is exactly what i needed | 13:57 |
| *** ducttape_ has joined #openstack-nova | 13:57 | |
| *** cleong has joined #openstack-nova | 13:58 | |
| mriedem | cool | 13:58 |
| bauzas | mriedem: johnthetubaguy: ah-ah, found a bug with the volume-update call related to my comments | 13:59 |
| bauzas | mriedem: johnthetubaguy: nova messes up the volume ID when we update the volume | 13:59 |
| *** TravT has joined #openstack-nova | 13:59 | |
| *** garyk has quit IRC | 14:00 | |
| *** psilvad has joined #openstack-nova | 14:00 | |
| *** tongli has joined #openstack-nova | 14:01 | |
| mdbooth | bauzas: Is this related to volume migration, btw? | 14:01 |
| PaulMurray | live migration on #openstack-meeting-3 PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk mrhillsman raj_singh | 14:01 |
| bauzas | mriedem: johnthetubaguy: see http://paste.openstack.org/show/582248/ | 14:01 |
| bauzas | mriedem: too ^ | 14:01 |
| bauzas | oops mdbooth rather ^ | 14:02 |
| *** ducttape_ has quit IRC | 14:02 | |
| *** markvoelker has quit IRC | 14:02 | |
| mdbooth | volume-update is weird :/ | 14:02 |
| bauzas | it's terrible | 14:03 |
| *** TravT has quit IRC | 14:03 | |
| gibi | mriedem: hi! Does this bp needs a spec? https://blueprints.launchpad.net/nova/+spec/json-schema-for-versioned-notifications | 14:03 |
| bauzas | I'm trying to revert https://github.com/openstack/nova/commit/be553fb15591c6fc212ef3a07c1dd1cbc43d6866 to see if the regression is here | 14:03 |
| *** amotoki has quit IRC | 14:03 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Virtual guest device role tagging https://review.openstack.org/373354 | 14:04 |
| bauzas | mdbooth: like I said earlier, my concerns are related to https://review.openstack.org/#/c/369116/1 | 14:04 |
| bauzas | which is proposed to mitaka | 14:05 |
| *** belliott has quit IRC | 14:05 | |
| *** belliott has joined #openstack-nova | 14:05 | |
| mriedem | gibi: please throw that into the nova meeting agenda, under open discussion | 14:05 |
| mriedem | so it can be discussed in the weekly meeting | 14:05 |
| gibi | mriedem: OK, I will do that | 14:05 |
| mdbooth | bauzas: Live migration meeting just started, but I'll look propertly in a bit. I have poked this code a couple of times in the past, but don't claim deep expertise. | 14:06 |
| bauzas | mdbooth: I'm continuing to investigate on my local devstack | 14:06 |
| *** TravT has joined #openstack-nova | 14:06 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix serial console not correctly defined after live-migration https://review.openstack.org/356335 | 14:06 |
| bauzas | but something seems definitely broken :/ | 14:06 |
| *** lamt has joined #openstack-nova | 14:07 | |
| *** markvoelker has joined #openstack-nova | 14:09 | |
| *** sdake_ has joined #openstack-nova | 14:09 | |
| *** burgerk has quit IRC | 14:11 | |
| *** sdake has quit IRC | 14:11 | |
| *** woodster_ has joined #openstack-nova | 14:12 | |
| jroll | jaypipes: did the code that uses this resource_class thing end up in newton? https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L329 | 14:13 |
| *** marst has quit IRC | 14:14 | |
| jaypipes | jroll: :( no | 14:14 |
| *** prateek has quit IRC | 14:14 | |
| jaypipes | jroll: I had a chat with dansmith yesterday about that. going to be working on it this week, but of cxourse, that is ocata | 14:15 |
| jroll | hrmmm | 14:15 |
| *** ayoung has joined #openstack-nova | 14:15 | |
| jroll | jaypipes: so we can't schedule on it until pike? | 14:15 |
| *** ociuhandu has quit IRC | 14:16 | |
| jaypipes | jroll: not necessarily. | 14:16 |
| jroll | jaypipes: well, we could use it in ocata but there'll be a short outage in scheduling upon upgrade | 14:17 |
| bauzas | mriedem: johnthetubaguy: mdbooth: regression confirmed, I just reverted https://review.openstack.org/#/c/257135/46 and I can now detach the volume after swapping the bdm | 14:17 |
| JayF | jaypipes: doesn't this go back to the "downtime vs no downtime" upgrade .... ^ what jroll said | 14:17 |
| bauzas | mriedem: johnthetubaguy: mdbooth: that's a very impacting bug, compared to the previous one | 14:17 |
| *** jwcroppe has quit IRC | 14:17 | |
| jaypipes | one sec guys on phone | 14:17 |
| jroll | no rush | 14:17 |
| bauzas | because now, we break our ability to detach a volume but by resetting the cinder state | 14:17 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make nova-manage online migrations more verbose https://review.openstack.org/373361 | 14:17 |
| *** ekuris has quit IRC | 14:18 | |
| mriedem | bauzas: please write up the bug report and tag it for newton-rc-potential and i can look after morning meetings | 14:18 |
| mriedem | bauzas: i knew about that patch of takashi's for a long time, but because we have 0 integration testing of swap volume i was loath to get it in | 14:19 |
| mriedem | but it did fix the bug he reported, but apparently now introduces a new bug | 14:19 |
| bauzas | mriedem: I'm just filing the bug report | 14:19 |
| mriedem | so basically i think we need a tempest test for swap volume before working on this code ever again | 14:19 |
| bauzas | mriedem: what's terrible is that we introduced a user-impacting regression for an admin api call | 14:19 |
| *** annegentle has joined #openstack-nova | 14:20 | |
| mriedem | scottda is working on a tempest test for volume retype but that doesn't hit the swap_volume api in nova | 14:20 |
| bauzas | mriedem: while we were having a workaround, by just detaching/reattaching the volume before swappign it back | 14:20 |
| mriedem | gmann was also working on a swap-volume test in tempest at one point, but ran into issues i htink | 14:20 |
| mriedem | *think | 14:20 |
| bauzas | mriedem: so, tbh, I did -1 the backport proposal, and I'm gonna propose a revert change | 14:20 |
| mriedem | gmann: do you remember what the issue was with testing swap-volume in tempest? | 14:20 |
| scottda | mriedem: Yes, I need to get that running on cinder multi-node tests | 14:20 |
| scottda | mriedem: In order to exercise swap_volume call | 14:21 |
| bauzas | scottda: mriedem: tbc, I only tested the nova call, not the cinder path | 14:21 |
| bauzas | ie. nova volume-update, which mimics the cinder behaviour | 14:21 |
| tojuvone | mriedem: This was discussed in Austin OPS meeting with some Nova cores, but spec didn't make it in N. How would it be for Ocata: https://review.openstack.org/310510 ? | 14:21 |
| bauzas | given our agreement to stop proxying the volume API, I think we should just deprecate the volume-update call TBH | 14:22 |
| mriedem | bauzas: i didn't test the cinder-initiated swap volume either b/c i didn't have a local setup for it | 14:23 |
| mriedem | bauzas: it's not a proxy | 14:23 |
| mriedem | it's a callback for certain cinder backends | 14:23 |
| bauzas | okay, then we need to fix that call properly | 14:23 |
| mriedem | although i might be confusing that with guest-assisted snapshots for glusterfs | 14:24 |
| mriedem | swap-volume is only implemented by the libvirt driver too | 14:24 |
| bauzas | but for Newton, I'm just proposing to revert given the workaround (detach/attach before swapping back) | 14:24 |
| mriedem | the api should probably be more like the events stuff we have for neutron and vif plugging | 14:24 |
| bauzas | mriedem: I trust you on that, I'm fairly rookie on the neutron side | 14:25 |
| mriedem | i have to run to a meeting | 14:25 |
| *** moshele has quit IRC | 14:25 | |
| *** Guest14517 is now known as med_ | 14:26 | |
| *** med_ has quit IRC | 14:26 | |
| *** med_ has joined #openstack-nova | 14:26 | |
| *** marst has joined #openstack-nova | 14:26 | |
| *** zz_dimtruck is now known as dimtruck | 14:28 | |
| *** mriedem has quit IRC | 14:29 | |
| *** mriedem has joined #openstack-nova | 14:29 | |
| *** jamesden_ has joined #openstack-nova | 14:31 | |
| *** rlrossit has quit IRC | 14:31 | |
| macsz | mriedem: can you remove -2's from https://review.openstack.org/#/q/topic:bp/centralize-config-options-ocata ? | 14:33 |
| jaypipes | jroll, JayF: k, off call now. so, no, there should not be any downtime involved in upgrading. the way we have stuff working (in Newton) is that inventory and allocation records get populated smoothly over time from the resource tracker in the nova-compute node. We will be adding code in Ocata that enables the scheduler to analyze this new inventory and allocation information. If Ironic (or the | 14:33 |
| *** BjoernT has joined #openstack-nova | 14:33 | |
| jaypipes | Ironic virt driver in Nova) in Ocata starts saving the node_class information for Ironic nodes in the new placement database, we should be able to seamlessly move to using the new placement engine for scheduling decisions of Ironic nodes. | 14:33 |
| *** hamzy has joined #openstack-nova | 14:33 | |
| *** jamesde__ has joined #openstack-nova | 14:34 | |
| mriedem | macsz: i guess, the bp isn't approved yet, but i see people are approving those anyway https://review.openstack.org/#/c/356572/ | 14:34 |
| *** gouthamr has joined #openstack-nova | 14:34 | |
| mriedem | sdague: johnthetubaguy: so i'm assuming we're good to just approve some of these low-hanging-fruit efforts for ocata now, like config options, py3, mox, etc | 14:34 |
| mriedem | dansmith: alaski: ^ | 14:35 |
| johnthetubaguy | I am good with doing that | 14:35 |
| *** jwcroppe has joined #openstack-nova | 14:35 | |
| macsz | mriedem: thanks for attention :) | 14:35 |
| johnthetubaguy | I mean the only worry is making a backport harder | 14:35 |
| *** jamesden_ has quit IRC | 14:36 | |
| *** baoli has joined #openstack-nova | 14:36 | |
| mriedem | fyi btw https://etherpad.openstack.org/p/ocata-nova-priorities-tracking | 14:36 |
| mriedem | could use some scrubbing | 14:36 |
| dansmith | mriedem: yeah, I guess, aside from backport concerns | 14:37 |
| *** panbalag has joined #openstack-nova | 14:37 | |
| alaski | mriedem: yep, approval seems fine at this point. And as reviewers we just take care in approving patches that might make backports difficult | 14:37 |
| mriedem | tojuvone: sorry what was the question re: https://review.openstack.org/#/c/310510/ ? | 14:38 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: WIP: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203 | 14:38 |
| sdague | mriedem: yeh, I'm fine with that | 14:38 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] Stringify class and provider uuid in error https://review.openstack.org/373380 | 14:40 |
| *** ducttape_ has joined #openstack-nova | 14:40 | |
| *** tblakes has quit IRC | 14:41 | |
| tojuvone | mriedem: I have POC code for it, would need some Nova people to review and get it further accepted for Ocata. Should I bring it up still in some meeting or so? | 14:41 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Group scheduler options https://review.openstack.org/349959 | 14:41 |
| *** tblakes has joined #openstack-nova | 14:42 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Group scheduler options https://review.openstack.org/349959 | 14:42 |
| mriedem | tojuvone: if it's not blocked for some reason then i think we just let the nova specs repo gods work it out as usual | 14:45 |
| mriedem | i don't think ocata specs reviews have started really | 14:45 |
| tojuvone | mriedem: Thanks, the last one is the answer really :) Do not rush. | 14:46 |
| mriedem | i plan to send an email out later about ocata | 14:46 |
| jroll | jaypipes: so I think what you're saying is it will fall back to old methods if the node_class isn't there? | 14:47 |
| tojuvone | ok :) Then there is a thing that for Telco use cases fault management needs to be lightning fast, but servers APIs are instance by instance | 14:48 |
| tojuvone | Could chat a little about the topic | 14:49 |
| *** rmcall_ has joined #openstack-nova | 14:49 | |
| mriedem | i think jaypipes is our telco CPL | 14:49 |
| tojuvone | Or can bring it in API meeting | 14:50 |
| *** ashok has quit IRC | 14:50 | |
| tojuvone | Thanks | 14:50 |
| *** rmcall has quit IRC | 14:51 | |
| *** rmcall_ is now known as rmcall | 14:51 | |
| *** jwcroppe has quit IRC | 14:51 | |
| tojuvone | Anyhow I think the thing is more how change could land on API level. Not the origin as Telco. | 14:52 |
| jaypipes | jroll: that's the idea, yes. | 14:52 |
| *** jwcroppe has joined #openstack-nova | 14:52 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Revert "Set 'serial' to new volume ID in swap volumes" https://review.openstack.org/373390 | 14:53 |
| bauzas | mriedem: johnthetubaguy: ^ I'd appreciate some review :) | 14:53 |
| sfinucan | lxsli: I thought you weren't working on https://review.openstack.org/#/c/349959/ for now? https://review.openstack.org/#/c/349959/1 | 14:53 |
| sfinucan | Just finished rebasing that onto the other changes around that file (~3 hours) :/ | 14:54 |
| sfinucan | Can I push anyway? | 14:54 |
| jroll | jaypipes: ok, cool | 14:54 |
| jroll | thanks! | 14:54 |
| *** armax has joined #openstack-nova | 14:55 | |
| lxsli | sfinucan: sure my changes were minor | 14:56 |
| *** thorst_ has quit IRC | 14:56 | |
| *** burgerk has joined #openstack-nova | 14:56 | |
| lxsli | basically s/placement/scheduler/ | 14:56 |
| tojuvone | jaypipes: mriedem gave you are the Telco link | 14:57 |
| mriedem | tojuvone: i was just joking about that | 14:57 |
| mriedem | sorry | 14:57 |
| tojuvone | :D | 14:57 |
| mriedem | tojuvone: i think we've pushed back in the past on passing lists of things to the apis | 14:58 |
| mriedem | especially lists of instances | 14:58 |
| mriedem | but i think the migration/resize APIs are now or will be taking lists of hosts | 14:58 |
| mriedem | which alaski doesn't like | 14:58 |
| *** dikonoo has quit IRC | 14:59 | |
| mriedem | rather than a list of instances, you could pass a server group, which some people don't like server groups either | 14:59 |
| mriedem | and server groups aren't mutable | 14:59 |
| tojuvone | ok, this is now for reset server state that we need to do with force down host | 14:59 |
| tojuvone | need to call it for each instance | 15:00 |
| alaski | I think I'm okay with passing lists of instances. it's the list of hosts I don't like | 15:00 |
| jaypipes | tojuvone: mriedem was being "tongue in cheek" when he referred to me as the cross-project liaison for Telco. :) In any case, I've got your spec open and am commenting on it (note: I'm not a nova-drivers member) | 15:00 |
| *** gouthamr has quit IRC | 15:01 | |
| tojuvone | so trough force down host, it could optionally reset server state for all VMs | 15:01 |
| tojuvone | or hypervoros API could do that for VMs on host | 15:01 |
| *** mgoddard has quit IRC | 15:01 | |
| tojuvone | or the servers API coudl somehow take host(s | 15:02 |
| *** mgoddard has joined #openstack-nova | 15:02 | |
| *** mjura has quit IRC | 15:02 | |
| sdague | alaski: https://review.openstack.org/#/c/347651/3 I think this is just a missed bit in the refactor for the context (arg name doesn't line up with docs) | 15:02 |
| tojuvone | I should have typo fixing in a fly while I type :D | 15:02 |
| *** burgerk_ has joined #openstack-nova | 15:03 | |
| siva_krishnan | johnthetubaguy: https://bugs.launchpad.net/nova/+bug/1516536. this is the bug that I mentioned | 15:03 |
| openstack | Launchpad bug 1516536 in OpenStack Compute (nova) "image-list should check filter correctness" [Low,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade) | 15:03 |
| *** ochalups has quit IRC | 15:04 | |
| alaski | sdague: yep. got it | 15:04 |
| *** burgerk has quit IRC | 15:04 | |
| tojuvone | mriedem: Ok, so if migration/resize is already having some hosts, then could take a look how that is done | 15:05 |
| *** panbalag has quit IRC | 15:05 | |
| johnthetubaguy | siva_krishnan: thats a proxy API we are thinking of removing, I don't think its something we will want to create a new microversion for | 15:05 |
| johnthetubaguy | siva_krishnan: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/images.py#L79 | 15:06 |
| tojuvone | I mean if exposing host to something like reset server state, have similar approach. Not to go to hypervisors or force down API. | 15:06 |
| johnthetubaguy | siva_krishnan: I would just close the bug as won't fix, since we are killing that API | 15:06 |
| *** yamahata has quit IRC | 15:07 | |
| tojuvone | jaypipes: Thanks | 15:08 |
| siva_krishnan | johnthetubaguy: ah ! you are right, just now realised it we would deprecation message when you use this command. didn't strike me | 15:08 |
| mriedem | tojuvone: so if i'm following, you want to modify the os-services API such that when you update a service and pass forced_down=True, you want to get the instances on that host and call reset-state on them? | 15:08 |
| *** takedakn has joined #openstack-nova | 15:08 | |
| mriedem | tojuvone: and what state would the vm's be reset to? active or error? | 15:08 |
| tojuvone | mriedem: that would be the esiest approach | 15:08 |
| jaypipes | tojuvone: so, the main issue I have with that proposal is that cloud users don't/shouldn't know what physical host they are on. So, specifying particular physical host information in the servers API is not cloudy. Instead, what I believe should be done is have a notification event that is sent out to a queue where something could be listening that can trigger some action on potentially-affected | 15:09 |
| jaypipes | servers on a host. | 15:09 |
| johnthetubaguy | siva_krishnan: no worries, good to clear out the bug queue, either way | 15:09 |
| tojuvone | mriedem: When you externally force down host, you might put error | 15:09 |
| tojuvone | mriedem: and back to active when unset the force down | 15:09 |
| mriedem | jaypipes: forced_down on the service and reset-state on the servers is all admin-api stuff | 15:09 |
| alaski | mriedem: tojuvone jaypipes why not just have the force_host call set the power_state of any instances on that host to off | 15:10 |
| johnthetubaguy | I thought we reported the host state in the server list operation now, so you can tell | 15:10 |
| jaypipes | mriedem: yes, I know. I'm referring to the "integration" with the server instance... | 15:10 |
| *** jwcroppe has quit IRC | 15:11 | |
| mriedem | johnthetubaguy: that's only for admins i think | 15:11 |
| mriedem | to see the host in the server detail | 15:11 |
| tojuvone | alaski jaypipes, mriedem: setting on error state will give nive notification that translates to alarm that user gets | 15:11 |
| tojuvone | s/nive/nice/ | 15:11 |
| alaski | tojuvone: it's not really an error state though, it's an off state | 15:12 |
| *** cmartin has quit IRC | 15:12 | |
| johnthetubaguy | mriedem: ah, possibly, well depends on your policy | 15:12 |
| jaypipes | mriedem: in other words, the spec is describing how the server instance should be notified that a maintenance event is or will be occurring. and I'm saying that the server instance doesn't have the access to physical host information but could potentially get notified by an external agent. | 15:12 |
| mriedem | extended_server_attributes_policies = [ | 15:12 |
| mriedem | policy.RuleDefault( | 15:12 |
| mriedem | name=BASE_POLICY_NAME, | 15:12 |
| mriedem | check_str=base.RULE_ADMIN_API), | 15:12 |
| johnthetubaguy | host_state in http://developer.openstack.org/api-ref/compute/?expanded=list-servers-detailed-detail was what I was thiking about | 15:12 |
| mriedem | johnthetubaguy: yeah totally policy driven, but admin api by default | 15:12 |
| johnthetubaguy | mriedem: yeah | 15:13 |
| mriedem | johnthetubaguy: i think that's the old api-ref site | 15:13 |
| tojuvone | I have 2 things here. The spec was one and this reset error state is another that does not have a spec now | 15:13 |
| mriedem | oh maybe not | 15:14 |
| *** rrecio has joined #openstack-nova | 15:14 | |
| tojuvone | So now just talking about the reset state that would be for all Vms on host | 15:14 |
| *** rama_y has joined #openstack-nova | 15:15 | |
| tojuvone | 1. admin calls force down API for host with failure | 15:16 |
| *** thorst_ has joined #openstack-nova | 15:17 | |
| johnthetubaguy | mriedem: thats the new one I thought | 15:17 |
| tojuvone | 2. admin would like to make single API call to reset server state for each instance on host | 15:17 |
| johnthetubaguy | mriedem: did it move again? | 15:17 |
| *** bdeschenes has joined #openstack-nova | 15:17 | |
| *** brian_price has joined #openstack-nova | 15:17 | |
| mriedem | johnthetubaguy: it is, it took awhile to load for me | 15:17 |
| tojuvone | 3. user that need to act fast should get alarm as of instance.update | 15:18 |
| johnthetubaguy | ah, cools | 15:18 |
| mriedem | tojuvone: users are listening on the nova rpc notification queue? | 15:18 |
| *** moshele has joined #openstack-nova | 15:18 | |
| *** tblakes has quit IRC | 15:19 | |
| tojuvone | mriedem: adoh alarm based on notification | 15:19 |
| mriedem | but is the user the end user of the cloud? | 15:19 |
| mriedem | or some internal thing? | 15:19 |
| *** oanson has quit IRC | 15:20 | |
| ayoung | what is the command to autogenerate the policy.json file from the defaults? | 15:20 |
| *** lucasagomes is now known as lucas-hungry | 15:21 | |
| tojuvone | VNFM / tenant should get alarm about his instances | 15:21 |
| johnthetubaguy | ayoung: oslopolicy-policy-generator –namespace nova –output-file policy-merged.yaml and oslopolicy-list-redundant –namespace nova | 15:21 |
| johnthetubaguy | ayoung: I was looking in here: http://docs.openstack.org/releasenotes/nova/unreleased.html | 15:22 |
| ayoung | johnthetubaguy, thanks | 15:22 |
| tojuvone | and orchestration / cloud should get at least the host level alarm. | 15:22 |
| *** pbandark has quit IRC | 15:22 | |
| siva_krishnan | artom: I didn't understand your comment on this patch https://review.openstack.org/#/c/369176/1. test case is related to change that has been made. Should I approach this in different way ? | 15:23 |
| *** takedakn has quit IRC | 15:23 | |
| artom | siva_krishnan, I was just explaining what I reviewed and why I gave the +1, sorry it wasn't clear :) | 15:24 |
| *** hemnafk is now known as hemna | 15:24 | |
| artom | I meant that I checked out the change, removed the extra conditional from your 'if ...' change and checked that the test does indeed fail | 15:24 |
| *** zenoway has quit IRC | 15:25 | |
| siva_krishnan | artom actually I see -1 in my patch so i was confused | 15:25 |
| *** lpetrut has quit IRC | 15:25 | |
| artom | siva_krishnan, doh, my bad | 15:25 |
| tojuvone | The whole thing is to have all API calls as fast as possible, to get everything/everybody the "force-down" as fast as possible. | 15:25 |
| mriedem | johnthetubaguy: you're on the hook to remove the -2 on this https://review.openstack.org/#/c/338553/ | 15:25 |
| bauzas | johnthetubaguy: ayoung: or tox -egenpolicy :p | 15:25 |
| artom | siva_krishnan, I did that this morning pre-coffee is my excuse | 15:25 |
| mriedem | tojuvone: sure, but nova could send a notification that a service/host is forced down with the list of instances on that host, i'm not sure we need to put the instances into ERROR state | 15:26 |
| ayoung | bauzas, johnthetubaguy thanks | 15:26 |
| siva_krishnan | artom happens coffee always makes a difference ;-) | 15:26 |
| *** psachin has quit IRC | 15:27 | |
| mriedem | tojuvone: i'm assuming some internal service is listening for that notification and would trigger an (internal) alarm, which would then notify users that their vm is on a downed host | 15:27 |
| mriedem | tojuvone: anyway, the use case is a bit unclear wrt who the actors are, so probably needs a spec | 15:27 |
| mriedem | if there is a proposed api change, then it requires a spec regardless, | 15:28 |
| mriedem | if it's just sending a new notification, that might not need a spec | 15:28 |
| *** paul-carlton2 has quit IRC | 15:29 | |
| mriedem | macsz: i've removed my -2s on https://review.openstack.org/#/q/topic:bp/centralize-config-options-ocata | 15:30 |
| mriedem | thanks for updating several of those | 15:30 |
| *** _ducttape_ has joined #openstack-nova | 15:30 | |
| *** rlrossit has joined #openstack-nova | 15:30 | |
| macsz | mriedem: thanks :) | 15:30 |
| tojuvone | mriedem: Thanks, yes better come back to this with spec to go forward. Have to discuss this in OPNFV Doctor and see what would be the most important thing here. Resetting the state or the notification. | 15:31 |
| tojuvone | mriedem: Thanks for your time :) | 15:32 |
| mriedem | yw | 15:32 |
| *** tblakes has joined #openstack-nova | 15:32 | |
| mriedem | mikal: looks like you have a bunch of -2s on these which we can lift now https://review.openstack.org/#/q/topic:bp/remove-mox-ocata,n,z | 15:33 |
| *** akshai has joined #openstack-nova | 15:33 | |
| *** ducttape_ has quit IRC | 15:34 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Change disable_live_snapshot workaround https://review.openstack.org/373430 | 15:35 |
| *** Hieu has joined #openstack-nova | 15:37 | |
| *** jwcroppe has joined #openstack-nova | 15:37 | |
| *** panbalag has joined #openstack-nova | 15:37 | |
| *** panbalag has left #openstack-nova | 15:38 | |
| *** krtaylor has quit IRC | 15:40 | |
| stevemar | mriedem: you should idle in -keystone so i can ask you questions in there | 15:40 |
| *** jaosorior has quit IRC | 15:40 | |
| stevemar | mriedem: oh i guess there is alerady context here | 15:41 |
| stevemar | mriedem: should we add a comment in the for of a json key-value pair to the empty policy file? | 15:41 |
| stevemar | ayoung: ^ | 15:41 |
| *** lpetrut has joined #openstack-nova | 15:41 | |
| mriedem | form? | 15:43 |
| mriedem | i wasn't following the earlier conversation | 15:43 |
| *** Swami has joined #openstack-nova | 15:44 | |
| *** danpb has quit IRC | 15:44 | |
| ayoung | mriedem, I was suggesting to stevemar that we do | 15:44 |
| jroll | reminder that comments aren't valid json :) | 15:45 |
| ayoung | "comment" : "to generate the rulesfile run the command oslopolicy-policy-generator --namespace nova --output-file policy.json" | 15:45 |
| ayoung | or something in the blank default policy file | 15:45 |
| jroll | ah | 15:45 |
| ayoung | jroll, it just needs to sneak past the parser. | 15:45 |
| jroll | ayoung: right, I see what you mean by comment now :) | 15:45 |
| mriedem | how about something like this like what we have for config options? https://github.com/openstack/nova/blob/master/etc/nova/README-nova.conf.txt | 15:46 |
| *** lpetrut has quit IRC | 15:46 | |
| *** matrohon has quit IRC | 15:46 | |
| ayoung | jroll, that would be friendly, too | 15:46 |
| mriedem | README-policy.json.txt | 15:46 |
| mriedem | i'd prefer ^ | 15:46 |
| ayoung | mriedem, I'd prefer that it all go into a central database and be handed out automatically based on endpoint ID, but we can't always get what we want | 15:47 |
| *** andreas_s has quit IRC | 15:47 | |
| mriedem | one of my least favorite rolling stones songs | 15:48 |
| *** danpb has joined #openstack-nova | 15:48 | |
| *** jwcroppe has quit IRC | 15:48 | |
| *** derekh has quit IRC | 15:49 | |
| *** zenoway has joined #openstack-nova | 15:49 | |
| pumaranikar | sdague, can you please check my comment on os-vif logging https://review.openstack.org/#/c/371785/ | 15:50 |
| *** acabot has quit IRC | 15:50 | |
| *** EinstCrazy has quit IRC | 15:50 | |
| stevemar | mriedem: you strike me as a 'paint it black' sort of guy | 15:51 |
| *** jwcroppe has joined #openstack-nova | 15:52 | |
| mriedem | no | 15:53 |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 15:53 |
| mdbooth | bauzas: That nova<->cinder interaction is absurdly complex. | 15:53 |
| bauzas | mdbooth: sssshhht | 15:54 |
| bauzas | :p | 15:54 |
| mriedem | stevemar: think sticky fingers | 15:54 |
| bauzas | mdbooth: johnthetubaguy pointed me out some Cinder spec trying to improve the API actions | 15:54 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: POC: Resize ephemerals https://review.openstack.org/372721 | 15:54 |
| bauzas | mdbooth: https://review.openstack.org/#/c/373203/ is the nova side of that spec | 15:55 |
| *** mvk has quit IRC | 15:55 | |
| johnthetubaguy | mdbooth: +1 on absurdly complex | 15:55 |
| *** diga has joined #openstack-nova | 15:55 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606 | 15:56 |
| mdbooth | bauzas: No shame on anybody who introduces bugs trying to update that, imho. | 15:57 |
| bauzas | mdbooth: well, the problem is when you want to update some admin API and then you trample a detach action... | 15:58 |
| *** ircuser-1 has joined #openstack-nova | 15:58 | |
| *** thorst_ has quit IRC | 15:58 | |
| *** abalutoiu has quit IRC | 15:59 | |
| mdbooth | johnthetubaguy: "it has become clear Cinder and Nova interaction is not well understood" :) | 15:59 |
| *** Swami has quit IRC | 16:00 | |
| *** Swami has joined #openstack-nova | 16:01 | |
| *** eharney has quit IRC | 16:01 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606 | 16:01 |
| *** jheroux has joined #openstack-nova | 16:02 | |
| sdague | pumaranikar: the point isn't to change the default logger levels, it's to eliminate os-vif log messages that aren't useful to anyone | 16:02 |
| *** ljxiash has quit IRC | 16:02 | |
| *** _ducttape_ has quit IRC | 16:03 | |
| mriedem | gibi: fyi i've re-approved https://blueprints.launchpad.net/nova/+spec/versioned-notification-transformation-ocata | 16:03 |
| mriedem | s/re-approved/approved/ | 16:03 |
| *** ducttape_ has joined #openstack-nova | 16:04 | |
| mriedem | mdbooth: please move any libvirt-instance-storage patches over to https://blueprints.launchpad.net/nova/+spec/libvirt-instance-storage-ocata now | 16:04 |
| mdbooth | mriedem: Will do. I should start working on those patches again some time soon. | 16:05 |
| *** jlanoux has quit IRC | 16:05 | |
| *** code-R has quit IRC | 16:05 | |
| *** zenoway has quit IRC | 16:05 | |
| *** zenoway has joined #openstack-nova | 16:06 | |
| *** code-R has joined #openstack-nova | 16:06 | |
| mriedem | claudiub|2: if you're tracking py3 conversion changes please get those moved over to https://blueprints.launchpad.net/nova/+spec/goal-python35 | 16:06 |
| mriedem | for ocata | 16:06 |
| *** Hieu has quit IRC | 16:06 | |
| *** code-R has quit IRC | 16:07 | |
| pumaranikar | sdague, yes. i tried to eliminate through standardlogging fixture https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L87 | 16:07 |
| *** hieulq__ has joined #openstack-nova | 16:07 | |
| *** hieulq_ has joined #openstack-nova | 16:07 | |
| sdague | pumaranikar: ok, but that's not actually what I'm saying | 16:08 |
| kashyap | johnthetubaguy: Shouldn't the asignee be you, as you've submitted the patch for this? https://bugs.launchpad.net/nova/+bug/1603909 | 16:08 |
| openstack | Launchpad bug 1603909 in OpenStack Compute (nova) "Nova unable to delete dangling ports on timeout/failure; leads to multiple ports per VM instead of one" [High,In progress] - Assigned to Anush Krishnamurthy (anush-krishnamurthy) | 16:08 |
| *** lpetrut has joined #openstack-nova | 16:08 | |
| mriedem | mikal: you can drop the -2s on these https://review.openstack.org/#/q/topic:bp/nova-python3-newton+status:open | 16:08 |
| sdague | pumaranikar: what are the log messages which are coming out that have no value? | 16:08 |
| sdague | and can we just delete them entirely? | 16:08 |
| mdbooth | johnthetubaguy: Is Ironic still using nodes? | 16:08 |
| *** paul-carlton2 has joined #openstack-nova | 16:08 | |
| * mdbooth hasn't kept up with that | 16:08 | |
| *** browne has joined #openstack-nova | 16:08 | |
| claudiub|2 | mriedem: will do, ty | 16:09 |
| mdbooth | browne: Does VMware still have multiple nodes per compute? I know there was a move to get rid of it, but it never seemed to land. | 16:09 |
| *** zenoway has quit IRC | 16:10 | |
| pumaranikar | sdague, log messages are from https://github.com/openstack/os-vif/blob/master/os_vif/__init__.py#L46 | 16:10 |
| *** jwcroppe has quit IRC | 16:11 | |
| browne | mdbooth: no that support was removed | 16:11 |
| mdbooth | browne: Cool, thanks. | 16:11 |
| *** ociuhandu has joined #openstack-nova | 16:12 | |
| sdague | pumaranikar: ok, that seems like it could be downgrade to debug | 16:12 |
| sdague | info is definitely pretty high | 16:12 |
| *** rama_y has quit IRC | 16:13 | |
| openstackgerrit | jichenjc proposed openstack/nova: Remove RateLimitFault class https://review.openstack.org/373254 | 16:13 |
| pumaranikar | sdague, makes sense. I thought we can set loglevel in unit test module for os-vif without altering that. | 16:15 |
| *** ociuhandu has quit IRC | 16:16 | |
| *** mriedem has quit IRC | 16:16 | |
| *** nic has joined #openstack-nova | 16:17 | |
| *** gszasz has quit IRC | 16:17 | |
| *** mriedem has joined #openstack-nova | 16:20 | |
| *** krtaylor has joined #openstack-nova | 16:21 | |
| *** eharney has joined #openstack-nova | 16:21 | |
| *** tblakes has quit IRC | 16:22 | |
| *** tjones has joined #openstack-nova | 16:22 | |
| *** lucas-hungry is now known as lucasagomes | 16:22 | |
| *** hieulq__ has quit IRC | 16:24 | |
| *** hieulq__ has joined #openstack-nova | 16:24 | |
| mdbooth | lyarwood: I just added you to a spec I think you may be interested in. | 16:26 |
| *** hieulq__ has quit IRC | 16:31 | |
| *** rlrossit has quit IRC | 16:33 | |
| *** mtanino has joined #openstack-nova | 16:34 | |
| *** paul-carlton2 has quit IRC | 16:34 | |
| *** Apoorva has joined #openstack-nova | 16:35 | |
| *** Apoorva has quit IRC | 16:35 | |
| *** code-R has joined #openstack-nova | 16:35 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Expand feature classification matrix with gate checks https://review.openstack.org/362456 | 16:35 |
| gibi | mriedem: thanks for the bp approve! | 16:35 |
| *** jwcroppe has joined #openstack-nova | 16:36 | |
| *** code-R_ has joined #openstack-nova | 16:36 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for interval_opts https://review.openstack.org/331761 | 16:37 |
| *** Apoorva has joined #openstack-nova | 16:38 | |
| *** code-R has quit IRC | 16:40 | |
| *** moshele has quit IRC | 16:40 | |
| *** unicell1 has quit IRC | 16:42 | |
| *** sahid has quit IRC | 16:45 | |
| *** unicell has joined #openstack-nova | 16:50 | |
| *** jvgrant__ has quit IRC | 16:50 | |
| *** jvgrant__ has joined #openstack-nova | 16:51 | |
| *** jwcroppe has quit IRC | 16:51 | |
| *** baoli_ has joined #openstack-nova | 16:55 | |
| *** baoli has quit IRC | 16:55 | |
| *** mvk has joined #openstack-nova | 16:55 | |
| *** rcernin has quit IRC | 16:55 | |
| *** ducttape_ has quit IRC | 16:56 | |
| *** EinstCrazy has joined #openstack-nova | 16:56 | |
| *** mgoddard has quit IRC | 16:56 | |
| *** mgoddard_ has joined #openstack-nova | 16:56 | |
| *** ociuhandu has joined #openstack-nova | 16:57 | |
| *** jwcroppe has joined #openstack-nova | 16:59 | |
| *** danpb has quit IRC | 16:59 | |
| auggy | Reminder: Nova Bugs Team Meeting in one hour (1800 UTC) in #openstack-meeting-4 | 17:00 |
| *** ociuhandu has quit IRC | 17:01 | |
| *** BjoernT has quit IRC | 17:02 | |
| *** jwcroppe has quit IRC | 17:03 | |
| *** tblakes has joined #openstack-nova | 17:06 | |
| *** anush__ has quit IRC | 17:07 | |
| *** sai is now known as sai|mtg | 17:08 | |
| *** rama_y has joined #openstack-nova | 17:11 | |
| *** unicell has quit IRC | 17:11 | |
| *** jwcroppe has joined #openstack-nova | 17:12 | |
| *** rama_y has quit IRC | 17:12 | |
| *** rama_y has joined #openstack-nova | 17:13 | |
| *** pcaruana has quit IRC | 17:14 | |
| claudiub|2 | mriedem:hi. if you have some time, can you take a look at some specless blueprints for ocata? https://blueprints.launchpad.net/nova/+spec/hyperv-use-os-brick https://blueprints.launchpad.net/nova/+spec/hyper-v-ovs-vif https://blueprints.launchpad.net/nova/+spec/hyper-v-block-device-mapping-support-ocata | 17:15 |
| *** lpetrut1 has joined #openstack-nova | 17:17 | |
| *** lpetrut has quit IRC | 17:17 | |
| *** thorst_ has joined #openstack-nova | 17:18 | |
| *** jwcroppe has quit IRC | 17:18 | |
| *** thorst_ has quit IRC | 17:20 | |
| *** liverpooler has quit IRC | 17:20 | |
| mriedem | claudiub|2: yeah | 17:20 |
| mriedem | can you also get the hyper-v + uefi spec re-proposed for ocata? | 17:20 |
| mriedem | this one https://review.openstack.org/#/c/190997/ | 17:21 |
| *** lpetrut1 has quit IRC | 17:21 | |
| mriedem | oh nvm i guess that's mitaka | 17:21 |
| mriedem | this one https://review.openstack.org/#/c/279045/ | 17:22 |
| *** hieulq_ has quit IRC | 17:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Move implemented specs for newton https://review.openstack.org/372798 | 17:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Fix spec names to match blueprint names https://review.openstack.org/372793 | 17:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Add a script to move completed specs to implemented directory https://review.openstack.org/372790 | 17:23 |
| *** ihrachys has quit IRC | 17:23 | |
| *** liverpooler has joined #openstack-nova | 17:24 | |
| claudiub|2 | yep, they were on my to-do shortlist. :) ty | 17:24 |
| *** hshiina has quit IRC | 17:24 | |
| claudiub|2 | same with the hyper-v numa stuff | 17:24 |
| *** efoley_ has quit IRC | 17:24 | |
| *** liverpooler has quit IRC | 17:27 | |
| *** bkopilov has quit IRC | 17:28 | |
| mriedem | claudiub|2: also, is someone working on an ocata spec to use the (v3) diagnostics virt driver code in the api with a microversion, per https://blueprints.launchpad.net/nova/+spec/hyperv-vm-diagnostics ? | 17:29 |
| *** bkopilov has joined #openstack-nova | 17:30 | |
| claudiub|2 | my coleague abalutoiu was supposed to work on it, he got caught up with other tasks. will see he's still up for it | 17:31 |
| *** yamahata has joined #openstack-nova | 17:31 | |
| mriedem | ok, otherwise i'll throw it in the api subteam meeting agenda for someone to work on it | 17:31 |
| *** jwcroppe has joined #openstack-nova | 17:32 | |
| *** PaulMurray has quit IRC | 17:34 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Reproposes the Hyper-V NUMA instance spec to Ocata https://review.openstack.org/373483 | 17:34 |
| *** code-R_ has quit IRC | 17:36 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Reproposes the Hyper-V UEFI Secure Boot spec to Ocata https://review.openstack.org/373484 | 17:39 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 17:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Move implemented specs for newton https://review.openstack.org/372798 | 17:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Fix spec names to match blueprint names https://review.openstack.org/372793 | 17:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Add a script to move completed specs to implemented directory https://review.openstack.org/372790 | 17:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 17:43 |
| *** akshai has quit IRC | 17:44 | |
| *** akshai has joined #openstack-nova | 17:44 | |
| *** unicell has joined #openstack-nova | 17:44 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 17:45 |
| *** akshai has quit IRC | 17:45 | |
| *** jwcroppe has quit IRC | 17:47 | |
| *** slaweq_ has quit IRC | 17:47 | |
| *** jwcroppe has joined #openstack-nova | 17:48 | |
| *** eglynn has joined #openstack-nova | 17:49 | |
| *** eglynn is now known as eglynn-pto | 17:49 | |
| *** anush has joined #openstack-nova | 17:50 | |
| *** ducttape_ has joined #openstack-nova | 17:52 | |
| *** kaisers has quit IRC | 17:54 | |
| *** sudipto_ has quit IRC | 17:55 | |
| *** sudswas__ has quit IRC | 17:55 | |
| *** hemna has quit IRC | 17:55 | |
| *** ociuhandu has joined #openstack-nova | 17:56 | |
| *** sai|mtg is now known as sai | 17:57 | |
| *** akshai has joined #openstack-nova | 17:57 | |
| auggy | Reminder: Nova Bugs Team Meeting in a few minutes (1800 UTC) in #openstack-meeting-4 | 17:58 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Reproposes the Hyper-V UEFI Secure Boot spec to Ocata https://review.openstack.org/373484 | 17:59 |
| *** ociuhandu has quit IRC | 18:01 | |
| mriedem | sdague: did you have issues with this at the midcycle? https://review.openstack.org/#/c/339232/ | 18:01 |
| *** rama_y has quit IRC | 18:04 | |
| *** rmcall has quit IRC | 18:05 | |
| *** rmcall has joined #openstack-nova | 18:05 | |
| *** rama_y has joined #openstack-nova | 18:05 | |
| *** lpetrut has joined #openstack-nova | 18:06 | |
| *** slaweq_ has joined #openstack-nova | 18:10 | |
| mriedem | cdent: what was the trick again for running only the placement tests in the functional tree? tox -e functional -- placement? | 18:11 |
| cdent | yeah, that will work | 18:12 |
| cdent | odd coincidence, I'm currently working on https://review.openstack.org/#/c/371370/1 | 18:12 |
| cdent | mriedem: you don't have to have the -- but it's also okay if you do | 18:12 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 18:13 |
| *** code-R has joined #openstack-nova | 18:13 | |
| sdague | mriedem: I think the issue that was raised was that autospecing was increasing test run time by an order of magnitude | 18:14 |
| *** code-R has quit IRC | 18:14 | |
| *** thorst_ has joined #openstack-nova | 18:14 | |
| mriedem | sdague: yeah, i think claudiub|2 said he found a way to resolve that via lazy mocking or something | 18:14 |
| mriedem | i'd have to dig in again | 18:14 |
| claudiub|2 | ye | 18:14 |
| *** tblakes has quit IRC | 18:14 | |
| mriedem | i.e. only autospec when needed? | 18:14 |
| claudiub|2 | let me dig that patch | 18:14 |
| claudiub|2 | https://review.openstack.org/#/c/339232/ | 18:15 |
| *** tanvirt has quit IRC | 18:15 | |
| claudiub|2 | yeah, basically, only autospec when a property / method is accessed | 18:15 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: POC: Resize ephemerals https://review.openstack.org/372721 | 18:16 |
| mriedem | ok, i added lifeless to that one in case there are crazy traps we don't know about with that approach | 18:16 |
| *** anush has quit IRC | 18:17 | |
| claudiub|2 | cool | 18:17 |
| *** thorst_ has quit IRC | 18:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Move wsgi-intercept to test-requirements.txt https://review.openstack.org/373498 | 18:19 |
| mriedem | easy ^ | 18:19 |
| cdent | oop, sorry about that | 18:20 |
| sdague | +2 | 18:20 |
| mriedem | np | 18:20 |
| claudiub|2 | mriedem: seems I've left some comments on that patch in the past. full autospec of 301 hyper-v unit tests was taking 4000+ seconds, autospecing only the public methods was taking 1000+ seconds. with lazy autospec, it was taking something like 60-70 seconds | 18:20 |
| mriedem | yeah | 18:20 |
| mriedem | seems like something that should actually be part of mock itself | 18:21 |
| claudiub|2 | yeah | 18:21 |
| mriedem | like, i'm surprised that doesn't already exist | 18:21 |
| mriedem | which makes me think there are dragons | 18:21 |
| sdague | ug... apparently live_snapshot is completely broken | 18:21 |
| claudiub|2 | well, the code is not easy to read at a first glance | 18:21 |
| *** aswadr_ has quit IRC | 18:22 | |
| mriedem | if not dragons, at least Căpcăun | 18:22 |
| mriedem | :) | 18:22 |
| claudiub|2 | cãpcãun? :)) | 18:22 |
| *** purplerbot has quit IRC | 18:22 | |
| *** purplerbot has joined #openstack-nova | 18:22 | |
| mriedem | https://en.wikipedia.org/wiki/C%C4%83pc%C4%83un | 18:23 |
| mriedem | was just looking up romanian folklore monsters | 18:23 |
| claudiub|2 | anyways, i'll try to send a patch there, or at least raise this problem to the mock people. | 18:23 |
| *** liuyulong_ has quit IRC | 18:23 | |
| claudiub|2 | gotcha. :D well, the most famous romanian monster would be the "zmeu" :D | 18:24 |
| *** liuyulong_ has joined #openstack-nova | 18:24 | |
| claudiub|2 | they're the main antagonists of romanian folklore | 18:24 |
| *** baoli_ has quit IRC | 18:25 | |
| *** baoli has joined #openstack-nova | 18:25 | |
| sdague | oh, it's a bad mock | 18:27 |
| *** hemanthm is now known as hemanthm|lunch | 18:27 | |
| *** ducttape_ has quit IRC | 18:27 | |
| *** stewie925_ has joined #openstack-nova | 18:28 | |
| mriedem | claudiub|2: but dragons are some blasé | 18:29 |
| mriedem | *so | 18:29 |
| mriedem | there does seem to be a common thread with these monsters and stealing things | 18:29 |
| *** hemna has joined #openstack-nova | 18:29 | |
| mriedem | children, young women, etc | 18:29 |
| mriedem | they really need to just get a solid job and hit up a dating site, like the rest of us | 18:30 |
| *** mdrabe has quit IRC | 18:30 | |
| claudiub|2 | yeah, the only dating sites at that time were taverns. :) | 18:31 |
| *** hemna has quit IRC | 18:32 | |
| claudiub|2 | also, other antagonists in romanian folklore are bald people, for some reason. | 18:32 |
| artom | Well, would *you* trust a bald person? ;) | 18:34 |
| *** nic has quit IRC | 18:36 | |
| *** tri2sing has joined #openstack-nova | 18:37 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Add support for setting boot order in Hyper-V https://review.openstack.org/248029 | 18:38 |
| claudiub|2 | ehm, maybe? :D | 18:38 |
| *** hemna has joined #openstack-nova | 18:42 | |
| *** thorst_ has joined #openstack-nova | 18:48 | |
| *** sdake_ is now known as sdake | 18:50 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Change disable_live_snapshot workaround https://review.openstack.org/373430 | 18:50 |
| *** ducttape_ has joined #openstack-nova | 18:50 | |
| *** yamahata has quit IRC | 18:50 | |
| sdague | ^^^ still doesn't pass unit tests | 18:50 |
| sdague | the libvirt unit tests use mocks on fakes, and end up really highly tuned to particular configs | 18:51 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Add Hyper-V storage QoS support https://review.openstack.org/175479 | 18:51 |
| *** kaisers has joined #openstack-nova | 18:53 | |
| mriedem | sdague: dansmith: johnthetubaguy: alaski: mikal: in newton we didn't approve specs for *new* things before the summit, in order to try and get re-approvals and reviews on backlog items that missed mitaka, | 18:53 |
| mriedem | i'm thinking we should do the same for ocata, but wanted to bring it up first | 18:53 |
| mriedem | bauzas: melwitt: oomichi: alex_xu: ^ | 18:54 |
| dansmith | I had assumed we'd do the same, yeah | 18:54 |
| bauzas | the only point I see is that Ocata is a bit smaller than the other cycles | 18:55 |
| mriedem | bauzas: which to me means let's not overload it with new things when we have a backlog | 18:55 |
| bauzas | so I just actually wonder if we should accept new non-prio features for this cycle, given all the backlog + priorities | 18:56 |
| bauzas | mriedem: yeah | 18:56 |
| mriedem | bauzas: i think that could be decided at the summit | 18:56 |
| mriedem | but for example we told takashi at the midcycle that he could get some api changes into ocata | 18:56 |
| mriedem | for passing hosts to the cold migrate api i think | 18:56 |
| *** Apoorva has quit IRC | 18:57 | |
| bauzas | tbh, I'm okay with accepting that, but I just want to make sure we would have time for that | 18:57 |
| bauzas | should we maybe ask people to write specs before the summit, and then if not approved, then not merging the implementations until Pike ? | 18:57 |
| *** tblakes has joined #openstack-nova | 18:58 | |
| bauzas | mriedem: ^ | 18:58 |
| mriedem | people should have specs written as early as possible imo | 18:58 |
| bauzas | yeah, I know | 18:58 |
| mriedem | but i don't really want to say that you can't write a spec during or after the summit and get it approved before some deadline | 18:58 |
| mriedem | because people bring things up at the summit in person | 18:59 |
| *** akshai has quit IRC | 18:59 | |
| * bauzas looking at the Ocata schedule | 18:59 | |
| mriedem | i also selfishly don't want to be stuck only reviewing specs for the next 4 weeks | 18:59 |
| bauzas | soooo, ocata-1 is litterally 2.5 weeks after the summit | 19:00 |
| mriedem | i think that's fine | 19:00 |
| mriedem | non-priority spec approval freeze at o-1 | 19:00 |
| mriedem | that's what we did with newton-1 | 19:00 |
| bauzas | but then, only 4 weeks for merging all the accepted ones ? | 19:01 |
| bauzas | until ocata-2 ? | 19:01 |
| bauzas | if we say we block approvals for non-already approved specs before the summit, it means 2 weeks for reviewing non-prio specs and then 4 weeks for merging the proposals | 19:01 |
| bauzas | hard call IMHO | 19:01 |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Reproposes Hyper-V Storage QOS spec to Ocata https://review.openstack.org/373501 | 19:02 |
| *** akshai has joined #openstack-nova | 19:02 | |
| mriedem | yeah it's tricky because there are some new things that i think we need to get in for ocata | 19:03 |
| *** markvoelker has quit IRC | 19:03 | |
| mriedem | so alternatively we just have a spec freeze on o-1 | 19:04 |
| *** hemanthm|lunch is now known as hemanthm | 19:04 | |
| mriedem | that's for anything, old/new/priority/non-priority | 19:04 |
| mriedem | o-2 is non-priority FF | 19:05 |
| mriedem | o-3 is FF | 19:05 |
| mriedem | another twist on that could be we don't have priorities for ocata | 19:05 |
| mriedem | spec freeze is o-1, FF is o-3 | 19:06 |
| mriedem | that's it | 19:06 |
| bauzas | I think we need to look at how many specs we had for newton | 19:06 |
| mriedem | we had 100 approved blueprints for newton | 19:06 |
| mriedem | we completed or partially completed 64 | 19:06 |
| bauzas | oh, that's not that bad | 19:06 |
| mriedem | we had ~75 approved specs for newton | 19:06 |
| *** lpetrut has quit IRC | 19:07 | |
| *** markvoelker has joined #openstack-nova | 19:07 | |
| mriedem | we had 45 implemented specs for newton | 19:08 |
| bauzas | still the same ratio | 19:08 |
| mriedem | yeah roughly | 19:08 |
| alaski | not having a non-priority FF earlier than FF seems reasonable given the short cycle. I'd still like to have priorities for the cycle though | 19:08 |
| mriedem | i think we'll have priorities either way, since we have to communicate that | 19:09 |
| mriedem | s/we/i/ | 19:09 |
| mriedem | they have these marketing things | 19:10 |
| bauzas | <3 | 19:10 |
| mriedem | so now i'm thinking spec freeze on o-1, any and all specs, FF on o-3 | 19:10 |
| *** READ10 has quit IRC | 19:10 | |
| mriedem | i will say, w/o a non-priority FF i don't think some things would have made it in newton, because we had about a 2 week push before that FF to get about 10 things in | 19:11 |
| *** dave-johnston has quit IRC | 19:11 | |
| mriedem | we had the same 2 week push before the final FF for priority things | 19:11 |
| mriedem | if they both buckle up to o-3 for FF, priority is going to get priority | 19:11 |
| mriedem | brb | 19:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: refactor instance translate function https://review.openstack.org/373505 | 19:12 |
| alaski | yep. Perhaps we send something to the ML around o-2 saying if you want your non-priority thing to get in finish it up before it has to contend with the priority push | 19:13 |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 19:14 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] Adjust the name of the gabbi tests https://review.openstack.org/371370 | 19:16 |
| *** efried1 has quit IRC | 19:24 | |
| *** haypo has joined #openstack-nova | 19:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Fix opt indentation for scheduler.py https://review.openstack.org/339736 | 19:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Reorder scheduler opts https://review.openstack.org/373262 | 19:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Improve consistency in scheduler opts https://review.openstack.org/349666 | 19:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Group scheduler options https://review.openstack.org/349959 | 19:25 |
| *** akshai has quit IRC | 19:28 | |
| *** efried has joined #openstack-nova | 19:28 | |
| *** thorst_ has quit IRC | 19:28 | |
| *** ociuhandu has joined #openstack-nova | 19:30 | |
| *** xyang1 has joined #openstack-nova | 19:30 | |
| claudiub|2 | mriedem: there are some obsolete specs on launchpad. I've added a comment on them. Can you close them? https://blueprints.launchpad.net/nova/+spec/hyperv-resource-pool-enablment https://blueprints.launchpad.net/nova/+spec/customization-of-vm-in-hyper-v-and-esx https://blueprints.launchpad.net/nova/+spec/openstack-scvmm-integration | 19:31 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Revert "Set 'serial' to new volume ID in swap volumes" https://review.openstack.org/373390 | 19:31 |
| bauzas | mriedem: hope my comment makes it clearer now ^ | 19:31 |
| mriedem | claudiub|2: yeah | 19:31 |
| *** mfisch has quit IRC | 19:32 | |
| *** mdrabe has joined #openstack-nova | 19:32 | |
| *** abalutoiu has joined #openstack-nova | 19:32 | |
| *** bvanhav has quit IRC | 19:33 | |
| *** mfisch has joined #openstack-nova | 19:33 | |
| *** mfisch has quit IRC | 19:33 | |
| *** mfisch has joined #openstack-nova | 19:33 | |
| *** akshai has joined #openstack-nova | 19:34 | |
| *** thorst_ has joined #openstack-nova | 19:34 | |
| *** baoli has quit IRC | 19:37 | |
| *** Apoorva has joined #openstack-nova | 19:37 | |
| *** thorst_ has quit IRC | 19:37 | |
| *** baoli has joined #openstack-nova | 19:38 | |
| *** salv-orlando has joined #openstack-nova | 19:42 | |
| *** slaweq_ has quit IRC | 19:42 | |
| *** haypo has quit IRC | 19:45 | |
| *** salv-orl_ has quit IRC | 19:45 | |
| *** haypo has joined #openstack-nova | 19:45 | |
| *** haypo has joined #openstack-nova | 19:45 | |
| *** haypo has joined #openstack-nova | 19:45 | |
| mriedem | macsz: on https://bugs.launchpad.net/nova/+bug/1625644 i have a duplicate for that, but slightly different | 19:46 |
| openstack | Launchpad bug 1625644 in OpenStack Compute (nova) "Policy config is misplaced" [Undecided,New] - Assigned to Maciej Szankin (mszankin) | 19:46 |
| mriedem | those aren't normal config options, they are CLI options | 19:46 |
| *** eglynn-pto has quit IRC | 19:46 | |
| mriedem | which makes them a bit janky | 19:46 |
| *** haypo has left #openstack-nova | 19:47 | |
| mriedem | macsz: this https://bugs.launchpad.net/nova/+bug/1619382 | 19:47 |
| openstack | Launchpad bug 1619382 in OpenStack Compute (nova) "nova-policy policy check command should allow CLI args for user/tenant/role" [Low,Confirmed] | 19:47 |
| macsz | i talked with johnthetubaguy today, and he agreed that CLI options should be allowed to be outside the conf dir | 19:48 |
| mriedem | i'm saying, they shouldn't even by cli config options | 19:48 |
| mriedem | they should be options on the CLI itself | 19:48 |
| mriedem | like argparse | 19:48 |
| macsz | mriedem: so i think that i will just modify the fix for hacking check to treat CLI opts as exception from "configs in config dir" rule | 19:48 |
| mriedem | i would do that as a minimum yeah | 19:49 |
| *** eharney has quit IRC | 19:50 | |
| *** EinstCrazy has quit IRC | 19:56 | |
| *** kylek3h has quit IRC | 19:57 | |
| *** lpetrut has joined #openstack-nova | 19:57 | |
| *** baoli has quit IRC | 19:59 | |
| *** baoli has joined #openstack-nova | 19:59 | |
| *** eharney has joined #openstack-nova | 20:05 | |
| *** ducttape_ has quit IRC | 20:08 | |
| *** lucasagomes is now known as lucas-afk | 20:15 | |
| *** lpetrut has quit IRC | 20:16 | |
| *** lpetrut has joined #openstack-nova | 20:16 | |
| *** matrohon has joined #openstack-nova | 20:17 | |
| *** eglynn has joined #openstack-nova | 20:17 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Make nova-manage online migrations more verbose https://review.openstack.org/373361 | 20:18 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove PCI parent_addr online migration https://review.openstack.org/373519 | 20:18 |
| *** liuyulong_ has quit IRC | 20:19 | |
| *** liuyulong_ has joined #openstack-nova | 20:19 | |
| *** lpetrut has quit IRC | 20:21 | |
| *** marst has quit IRC | 20:25 | |
| *** diga has quit IRC | 20:25 | |
| *** _alastor_ has left #openstack-nova | 20:25 | |
| dansmith | alaski: I don't disagree, except that I do | 20:25 |
| dansmith | alaski: we often don't really know when we can drop a thing until well after we add it | 20:25 |
| alaski | that's fair | 20:26 |
| dansmith | alaski: adding when it was added, and a prospective removal date seems like a good idea though, since I had to do some git archaeology to find this one | 20:26 |
| alaski | though we expect that this is run each cycel right? | 20:26 |
| alaski | yeah, a tentative removal is all I would expect we could add | 20:26 |
| *** tblakes has quit IRC | 20:26 | |
| alaski | maybe even a "can't be removed before" type note | 20:27 |
| dansmith | alaski: we do, but the first time this was added, there was nothing to really force them to do this, and we couldn't have a blocking migration, so ... | 20:27 |
| alaski | gotcha | 20:28 |
| *** smatzek has quit IRC | 20:30 | |
| *** ducttape_ has joined #openstack-nova | 20:31 | |
| *** Jeffrey4l has quit IRC | 20:35 | |
| *** thorst_ has joined #openstack-nova | 20:38 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Add Hyper-V storage QoS support https://review.openstack.org/175479 | 20:39 |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips https://review.openstack.org/329704 | 20:46 |
| *** hemanthm is now known as hemanthm|afk | 20:48 | |
| *** jamesde__ has quit IRC | 20:50 | |
| *** jamesden_ has joined #openstack-nova | 20:53 | |
| *** thorst_ has quit IRC | 20:54 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Add support for setting boot order in Hyper-V https://review.openstack.org/248029 | 20:57 |
| *** priteau has joined #openstack-nova | 20:58 | |
| *** pwnall1 has joined #openstack-nova | 20:58 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 20:59 |
| *** gyee has joined #openstack-nova | 21:04 | |
| *** psilvad has quit IRC | 21:04 | |
| *** xinliang_ has quit IRC | 21:05 | |
| *** lamt has quit IRC | 21:05 | |
| *** lpetrut has joined #openstack-nova | 21:06 | |
| *** browne has quit IRC | 21:09 | |
| *** rtheis has quit IRC | 21:10 | |
| *** krtaylor has quit IRC | 21:11 | |
| openstackgerrit | Sujitha proposed openstack/nova: Improve help text for libvirt options https://review.openstack.org/372722 | 21:12 |
| *** annegentle has quit IRC | 21:13 | |
| *** matrohon has quit IRC | 21:15 | |
| *** ijw has joined #openstack-nova | 21:16 | |
| *** ijw has quit IRC | 21:16 | |
| *** ijw has joined #openstack-nova | 21:16 | |
| *** xinliang has joined #openstack-nova | 21:17 | |
| *** jheroux has quit IRC | 21:20 | |
| *** browne has joined #openstack-nova | 21:23 | |
| *** tjones has left #openstack-nova | 21:23 | |
| *** lpetrut has quit IRC | 21:23 | |
| *** browne has quit IRC | 21:24 | |
| *** salv-orlando has quit IRC | 21:26 | |
| *** salv-orlando has joined #openstack-nova | 21:26 | |
| *** jamesden_ has quit IRC | 21:26 | |
| *** burgerk__ has joined #openstack-nova | 21:27 | |
| *** annegentle has joined #openstack-nova | 21:27 | |
| *** umbSublime has joined #openstack-nova | 21:28 | |
| *** mriedem has quit IRC | 21:28 | |
| *** yamahata has joined #openstack-nova | 21:28 | |
| *** umbSublime has left #openstack-nova | 21:28 | |
| *** burgerk_ has quit IRC | 21:30 | |
| *** burgerk__ has quit IRC | 21:32 | |
| *** annegentle has quit IRC | 21:33 | |
| *** cleong has quit IRC | 21:37 | |
| *** akshai has quit IRC | 21:39 | |
| *** tbachman has quit IRC | 21:41 | |
| *** edmondsw has quit IRC | 21:46 | |
| *** wznoinsk has quit IRC | 21:50 | |
| *** cdent has quit IRC | 21:52 | |
| *** Matti_ has joined #openstack-nova | 21:52 | |
| *** eglynn has quit IRC | 21:55 | |
| *** kaisers has quit IRC | 21:55 | |
| *** lamt has joined #openstack-nova | 21:56 | |
| *** ijw_ has joined #openstack-nova | 22:01 | |
| *** wznoinsk has joined #openstack-nova | 22:02 | |
| *** tblakes has joined #openstack-nova | 22:02 | |
| *** ijw has quit IRC | 22:03 | |
| *** tblakes has quit IRC | 22:05 | |
| *** eharney has quit IRC | 22:06 | |
| *** Guest54237 has quit IRC | 22:07 | |
| *** claudiub|2 has quit IRC | 22:08 | |
| *** xyang1 has quit IRC | 22:09 | |
| *** ducttape_ has quit IRC | 22:22 | |
| *** artom has quit IRC | 22:22 | |
| *** hoonetorg has quit IRC | 22:24 | |
| *** annegentle has joined #openstack-nova | 22:27 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Update rolling upgrade steps from upgrades documentation https://review.openstack.org/373549 | 22:31 |
| *** bdeschenes has quit IRC | 22:33 | |
| *** r-daneel has quit IRC | 22:34 | |
| *** rama_y has quit IRC | 22:37 | |
| *** marst has joined #openstack-nova | 22:41 | |
| *** gyee has quit IRC | 22:47 | |
| *** baoli has quit IRC | 22:52 | |
| *** priteau has quit IRC | 22:53 | |
| *** yuntongjin has joined #openstack-nova | 22:56 | |
| *** hoonetorg has joined #openstack-nova | 22:56 | |
| *** tonytan4ever has quit IRC | 22:58 | |
| *** hoonetorg has quit IRC | 23:02 | |
| *** markvoelker has quit IRC | 23:03 | |
| *** rama_y has joined #openstack-nova | 23:03 | |
| *** markvoelker has joined #openstack-nova | 23:03 | |
| *** tri2sing has quit IRC | 23:03 | |
| *** tonytan4ever has joined #openstack-nova | 23:06 | |
| *** Swami has quit IRC | 23:08 | |
| *** rrecio has quit IRC | 23:09 | |
| *** yuntongjin has quit IRC | 23:15 | |
| *** hoonetorg has joined #openstack-nova | 23:19 | |
| *** sdague has quit IRC | 23:22 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (2) https://review.openstack.org/316550 | 23:25 |
| *** hoonetorg has quit IRC | 23:29 | |
| *** takedakn has joined #openstack-nova | 23:29 | |
| *** ijw_ has quit IRC | 23:30 | |
| *** hoonetorg has joined #openstack-nova | 23:31 | |
| *** takedakn has quit IRC | 23:32 | |
| *** devananda is now known as devananda|dinner | 23:33 | |
| *** ijw has joined #openstack-nova | 23:33 | |
| *** Matti_ has quit IRC | 23:36 | |
| *** maticue has quit IRC | 23:37 | |
| *** BjoernT has joined #openstack-nova | 23:37 | |
| *** bdeschenes has joined #openstack-nova | 23:37 | |
| *** rmcall_ has joined #openstack-nova | 23:40 | |
| *** annegentle has quit IRC | 23:41 | |
| *** rmcall has quit IRC | 23:42 | |
| *** rmcall_ is now known as rmcall | 23:42 | |
| *** yamahata has quit IRC | 23:42 | |
| *** tbachman has joined #openstack-nova | 23:42 | |
| *** kaisers has joined #openstack-nova | 23:45 | |
| *** tonytan4ever has quit IRC | 23:47 | |
| *** kaisers has quit IRC | 23:49 | |
| *** jwcroppe has quit IRC | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!