Friday, 2017-07-28

*** thorst has quit IRC00:00
*** dtp has quit IRC00:01
*** yingjun has joined #openstack-nova00:03
*** bkopilov_ has quit IRC00:04
*** kbaegis has joined #openstack-nova00:08
*** ijw has joined #openstack-nova00:10
*** Apoorva_ has quit IRC00:12
*** itlinux has quit IRC00:12
*** Apoorva has joined #openstack-nova00:13
*** yamamoto has quit IRC00:14
*** gbarros has quit IRC00:14
*** gmann has quit IRC00:15
*** itlinux has joined #openstack-nova00:15
*** gmann has joined #openstack-nova00:15
*** yamamoto has joined #openstack-nova00:15
*** annegentle has joined #openstack-nova00:16
*** zz9pzza has joined #openstack-nova00:17
*** bkopilov_ has joined #openstack-nova00:17
*** itlinux has quit IRC00:18
*** yamamoto has quit IRC00:20
*** myteron has joined #openstack-nova00:20
*** myteron has left #openstack-nova00:21
*** zz9pzza has quit IRC00:21
*** rajathagasthya has joined #openstack-nova00:26
openstackgerritDan Smith proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356600:26
openstackgerritDan Smith proposed openstack/nova master: placement: account for move operations in claim  https://review.openstack.org/48758900:26
*** kiennt has joined #openstack-nova00:33
*** marst has quit IRC00:34
*** masber has joined #openstack-nova00:35
openstackgerritMichael Still proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222900:36
openstackgerritMichael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916600:36
openstackgerritMichael Still proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197200:36
openstackgerritMichael Still proposed openstack/nova master: rootwrap.d cleanup mislabeled files  https://review.openstack.org/48683100:36
*** cfriesen_ has joined #openstack-nova00:36
*** psachin has joined #openstack-nova00:36
*** tbachman has quit IRC00:40
*** bkopilov_ has quit IRC00:47
*** yamamoto has joined #openstack-nova00:48
*** licanwei has joined #openstack-nova00:48
*** iceyao has joined #openstack-nova00:52
*** liusheng has quit IRC00:52
*** boris_42 has joined #openstack-nova00:54
*** masber has quit IRC00:54
jgriffithmriedem_afk smcginnis ildikov I guess I'll have to accept defeat :(00:55
*** iceyao has quit IRC00:56
jgriffithmriedem_afk smcginnis ildikov I updated the comments in the review but there's something bigger going on it looks like, inparticular instances going to error state and libvirt failing to clean them up00:56
jgriffithhttp://logs.openstack.org/85/330285/106/check/gate-tempest-dsvm-py35-ubuntu-xenial/40072bb/logs/screen-n-cpu.txt.gz?level=TRACE#_Jul_26_09_34_33_06230500:56
openstackgerritDan Smith proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356600:56
openstackgerritDan Smith proposed openstack/nova master: placement: account for move operations in claim  https://review.openstack.org/48758900:56
*** catintheroof has quit IRC00:57
*** litao__ has joined #openstack-nova00:57
*** liusheng has joined #openstack-nova00:57
jgriffithmriedem_afk smcginnis ildikov the compute host not found error is kinda concerning, wondering if we need a rebase again?00:57
jgriffithanyway00:57
jgriffithI'm heading home, catch ya'll later00:58
*** bkopilov_ has joined #openstack-nova01:00
*** chyka_ has joined #openstack-nova01:00
*** rajathagasthya has quit IRC01:02
*** phuongnh has joined #openstack-nova01:03
*** chyka has quit IRC01:04
* dims peeks01:05
*** chyka_ has quit IRC01:05
*** tbachman has joined #openstack-nova01:06
*** masber has joined #openstack-nova01:08
*** harlowja has quit IRC01:10
*** hongbin has joined #openstack-nova01:11
*** zz9pzza has joined #openstack-nova01:11
*** iceyao has joined #openstack-nova01:12
*** zz9pzza has quit IRC01:16
*** iceyao has quit IRC01:17
*** catintheroof has joined #openstack-nova01:18
*** iceyao has joined #openstack-nova01:24
*** catintheroof has quit IRC01:25
*** edmondsw has joined #openstack-nova01:25
*** baoli has quit IRC01:27
*** baoli has joined #openstack-nova01:28
*** penick has quit IRC01:30
*** edmondsw has quit IRC01:30
*** ijw has quit IRC01:32
*** gouthamr has quit IRC01:33
*** kbaegis has quit IRC01:35
*** cfriesen_ has quit IRC01:38
*** esberglu has quit IRC01:42
*** tbachman has quit IRC01:43
*** iceyao has quit IRC01:44
*** masber has quit IRC01:50
*** saphi has joined #openstack-nova01:51
*** iceyao has joined #openstack-nova01:52
*** masber has joined #openstack-nova01:52
*** larainema has quit IRC01:52
zulmikal: I sent a patch to unbreak nova-lxd btw01:53
*** yamamoto has quit IRC01:53
dimsmriedem_afk : which devstack-gate change reset stuff today? https://review.openstack.org/#/q/project:openstack-infra/devstack-gate+status:merged01:53
dimsthe "Set up stable/pike" one? mriedem_afk01:54
*** iceyao has quit IRC01:56
*** thorst has joined #openstack-nova01:57
dansmithdims: no, just a sec01:59
dansmithdims: it was devstack, and this is the revert: https://review.openstack.org/#/c/488201/01:59
dansmithwhich has a link to the original02:00
dimsgotcha. thanks dansmith02:00
dansmithdims: that merged and we're waiting for the two nova patches that got kicked out to merge before I update the tag02:00
*** iceyao has joined #openstack-nova02:01
dansmithdims: this and the one below: https://review.openstack.org/#/c/48356602:01
*** itlinux has joined #openstack-nova02:01
*** thorst has quit IRC02:02
dimsdansmith : looks like it will take another 2 hours at least02:03
dansmithdims: at least02:03
*** itlinux has quit IRC02:03
dansmithdims: I'm watching it though and will update the tag commit when it lands02:03
dansmitha thrilling thursday evening02:03
dimssounds good dansmith. i'll take care of it when i wake up if ttx hasn't already02:04
dansmithcool, thanks02:04
*** jmlowe has joined #openstack-nova02:04
*** iceyao has quit IRC02:05
*** acormier has quit IRC02:05
*** zz9pzza has joined #openstack-nova02:05
*** acormier has joined #openstack-nova02:06
*** acormier has quit IRC02:07
*** acormier has joined #openstack-nova02:07
*** zz9pzza has quit IRC02:10
*** esberglu has joined #openstack-nova02:11
*** acormier has quit IRC02:12
*** yamahata has quit IRC02:15
*** esberglu has quit IRC02:16
*** chyka has joined #openstack-nova02:16
*** phuongnh has quit IRC02:19
*** chyka has quit IRC02:20
*** Apoorva_ has joined #openstack-nova02:23
*** annegentle has quit IRC02:23
*** penick has joined #openstack-nova02:25
*** saphi has quit IRC02:26
*** itlinux has joined #openstack-nova02:26
*** Apoorva has quit IRC02:27
*** lbragstad has joined #openstack-nova02:27
*** Apoorva_ has quit IRC02:27
*** lbragstad has quit IRC02:29
*** zhurong has joined #openstack-nova02:30
*** penick has quit IRC02:40
*** penick has joined #openstack-nova02:41
*** tbachman has joined #openstack-nova02:44
*** saphi has joined #openstack-nova02:44
*** markvoelker has quit IRC02:50
*** phuongnh has joined #openstack-nova02:58
smcginnisjgriffith: Thanks for digging in to it. Maybe collect notes in an etherpad or something to see if anyone else has more ideas?02:58
*** acormier has joined #openstack-nova02:58
*** zz9pzza has joined #openstack-nova03:00
*** acormier has quit IRC03:02
*** zz9pzza has quit IRC03:04
*** mtanino has joined #openstack-nova03:06
*** iceyao has joined #openstack-nova03:06
*** saphi has quit IRC03:10
*** iceyao has quit IRC03:11
*** masber has quit IRC03:13
*** edmondsw has joined #openstack-nova03:13
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Update URLs in docs, comments and setup.cfg  https://review.openstack.org/48827103:17
*** baoli has quit IRC03:18
*** edmondsw has quit IRC03:18
*** chyka has joined #openstack-nova03:19
jgriffithsmcginnis roger dodger03:19
*** gouthamr has joined #openstack-nova03:22
*** thorst has joined #openstack-nova03:23
*** chyka has quit IRC03:23
*** saphi has joined #openstack-nova03:23
*** armax has quit IRC03:27
*** coreywright has quit IRC03:27
mriedem_afkjgriffith: the compute host not found warnings at the start of the n-cpu logs are 'normal' right now03:29
*** mriedem_afk is now known as mriedem03:29
jgriffithmriedem noted03:29
*** iceyao has joined #openstack-nova03:29
*** zhurong has quit IRC03:31
*** iceyao has quit IRC03:34
*** armax has joined #openstack-nova03:36
*** mtanino has quit IRC03:37
*** thorst has quit IRC03:37
*** udesale has joined #openstack-nova03:38
*** hongbin has quit IRC03:41
mriedemdansmith: #2 in the gate, i can take over if you want03:42
*** boris_42 has quit IRC03:43
dansmithI'm watchin'03:43
*** armax has quit IRC03:43
dansmithyou got to sleep03:43
dansmiths/got/go/03:43
*** armax has joined #openstack-nova03:43
*** armax has quit IRC03:44
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Remove run_tests.sh  https://review.openstack.org/48827603:44
*** armax has joined #openstack-nova03:44
*** armax has quit IRC03:44
*** coreywright has joined #openstack-nova03:46
*** gouthamr has quit IRC03:46
*** yamamoto has joined #openstack-nova03:48
*** masber has joined #openstack-nova03:48
*** tiendc has joined #openstack-nova03:50
*** yamamoto_ has joined #openstack-nova03:51
*** iceyao has joined #openstack-nova03:51
*** mamandle has joined #openstack-nova03:52
*** yamamoto has quit IRC03:53
mriedemi don't sleep03:53
*** zz9pzza has joined #openstack-nova03:54
dansmithhah03:54
*** iceyao has quit IRC03:56
*** yamamoto has joined #openstack-nova03:58
*** yamamoto_ has quit IRC03:58
*** zz9pzza has quit IRC03:59
dansmith10 minutes03:59
*** gouthamr has joined #openstack-nova03:59
dansmithooh, 1 minute03:59
jgriffithmriedem smcginnis https://etherpad.openstack.org/p/cinder-v3-attach-nova-traces04:01
jgriffithI can add more as I go here and maybe even send a request out to the ML tomorrow if there might be anybody interested04:02
jgriffithMy brain has turned a bit mushy after waking through all the bdm stuff all day over and over04:02
jgriffithOne thing I left out... I have a system I reproduced this on that the /dev/sdd device is in fact there and the connection is still valid, but that periodic query against it still fails... I didn't go down that rabbit hole yet though04:03
mriedemjgriffith: ok, adding to our other etherpad tracking blueprints04:03
jgriffithmriedem thanks04:03
jgriffithmriedem and I guess you're call earlier was indeed the right one04:04
jgriffithmriedem sorry if I wasn't quite as easy going about it as I should've been04:04
mriedemnot a problem04:04
*** tbachman has quit IRC04:07
*** itlinux has quit IRC04:07
mriedempy3 job passed tests,04:08
mriedemcollecting logs04:08
*** nicolasbock has joined #openstack-nova04:08
dansmithyep04:08
mriedemyou're also watching over telnet?04:08
dansmithoff and on to make sure it's not stuck04:08
openstackgerritMerged openstack/nova master: placement: account for move operations in claim  https://review.openstack.org/48758904:08
dansmithhmm, I pulled and got the commit but no merge commit yet04:09
dansmithfor the first one04:10
mriedemmuch like the line for the carousel tonight, you must have patience04:11
dansmithheh04:11
mriedemand a high tolerance for carney folk04:11
openstackgerritMerged openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356604:12
mriedemduh duh duh04:13
dansmithgot that one but still no merge commits04:13
mriedemdon't we just need 23c4eb34380bdf3eece11abbe0f6ccb68c060f47 ?04:13
dansmithyou list the merge commit in the release thing right?04:14
dansmithnot the actual commit04:14
mriedemno,04:14
mriedemtip of master04:14
dansmithyour WIP patch is a merge commit04:14
mriedemthat was just tip of master at the time04:14
dansmitherm, okay04:14
dansmithso maybe zuul does a ff merge if it's clean or something? I thought it always did merge commits04:14
mriedemyou want to update the release patch since you've watched this all night?04:15
dansmithjust did04:15
mriedemwell touche04:15
dansmithboth of the other milestones were merge commits04:16
mriedemnot intentionally04:16
*** yamamoto_ has joined #openstack-nova04:16
dansmithhrm, I don't understand, but whatever :)04:17
*** yamamoto has quit IRC04:17
dansmithglad you were here to tell me that04:17
dansmithso I didn't wait all night04:17
mriedemha04:17
mriedemwould have to ask dhellmann tomorrow if there is magic behind the scenes04:17
mriedemnow that that's done, i suppose no one is around to actually merge and tag the damn thing04:18
dansmithseveral I checked from ocata just now were merge commits too04:18
dansmithmriedem: dims said he'd get it in the morning if ttx didn't first04:18
mriedemyeah doug said they'd do it by the 10am CT release meeting04:18
dansmithaye04:19
mriedemalright, well f yeah, i'm going to watch some much deserved garbage tv and then go to bed04:20
dansmithenjoy04:20
mriedemthanks for shepharding04:20
*** links has joined #openstack-nova04:20
mriedem*shepherding?04:20
dansmithnp, g'nite04:20
*** mriedem has quit IRC04:20
*** vishwana_ has joined #openstack-nova04:23
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48828304:25
*** vishwanathj has quit IRC04:26
*** gouthamr has quit IRC04:30
*** harlowja has joined #openstack-nova04:32
*** sridharg has joined #openstack-nova04:38
*** rajathagasthya has joined #openstack-nova04:40
*** yamamoto has joined #openstack-nova04:40
*** yamamoto_ has quit IRC04:41
*** zz9pzza has joined #openstack-nova04:48
*** sree_ has joined #openstack-nova04:49
*** sree_ is now known as Guest9151804:49
*** psachin has quit IRC04:50
*** yamamoto has quit IRC04:51
*** markvoelker has joined #openstack-nova04:51
*** zz9pzza has quit IRC04:52
*** yamamoto has joined #openstack-nova04:53
*** iceyao has joined #openstack-nova04:58
*** yamamoto has quit IRC05:01
*** edmondsw has joined #openstack-nova05:02
*** iceyao has quit IRC05:03
*** edmondsw has quit IRC05:06
*** psachin has joined #openstack-nova05:08
*** hshiina has joined #openstack-nova05:11
*** mamandle has quit IRC05:13
*** gyee has quit IRC05:13
*** vishwana_ has quit IRC05:16
*** vishwanathj has joined #openstack-nova05:17
*** iceyao has joined #openstack-nova05:17
*** ratailor has joined #openstack-nova05:19
*** iceyao has quit IRC05:22
*** iceyao has joined #openstack-nova05:23
*** yassine has quit IRC05:23
*** yassine has joined #openstack-nova05:24
*** iceyao has quit IRC05:24
*** markvoelker has quit IRC05:25
*** _pewp_ has quit IRC05:25
*** iceyao has joined #openstack-nova05:25
*** _pewp_ has joined #openstack-nova05:28
*** harlowja has quit IRC05:28
*** iceyao has quit IRC05:30
*** zhurong has joined #openstack-nova05:30
*** vishwana_ has joined #openstack-nova05:31
*** vishwanathj has quit IRC05:34
*** iceyao has joined #openstack-nova05:39
*** harlowja has joined #openstack-nova05:41
*** zz9pzza has joined #openstack-nova05:42
*** penick has quit IRC05:42
*** kiennt has quit IRC05:44
*** iceyao has quit IRC05:44
*** kiennt has joined #openstack-nova05:44
*** zz9pzza has quit IRC05:47
*** cfriesen_ has joined #openstack-nova05:47
*** kiennt has quit IRC05:50
*** karthiks has joined #openstack-nova05:55
*** rajathagasthya has quit IRC05:56
*** mamandle has joined #openstack-nova05:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unnecesary code  https://review.openstack.org/48829906:00
*** thorst has joined #openstack-nova06:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unnecessary code  https://review.openstack.org/48829906:01
*** trinaths has joined #openstack-nova06:02
*** iceyao has joined #openstack-nova06:02
*** oomichi has quit IRC06:05
*** kiennt has joined #openstack-nova06:05
*** oomichi has joined #openstack-nova06:06
*** thorst has quit IRC06:06
*** iceyao has quit IRC06:06
*** sridharg has quit IRC06:07
*** oomichi has quit IRC06:10
*** oomichi has joined #openstack-nova06:11
*** rcernin has joined #openstack-nova06:15
*** markvoelker has joined #openstack-nova06:22
*** iceyao has joined #openstack-nova06:23
*** jjung has joined #openstack-nova06:28
*** esberglu has joined #openstack-nova06:31
*** phuongnh has quit IRC06:32
*** phuongnh has joined #openstack-nova06:33
*** mamandle has quit IRC06:33
*** esberglu has quit IRC06:36
*** yamamoto has joined #openstack-nova06:36
*** zz9pzza has joined #openstack-nova06:36
*** sridharg has joined #openstack-nova06:37
*** zz9pzza has quit IRC06:41
*** yamamoto has quit IRC06:44
*** yamamoto has joined #openstack-nova06:45
*** harlowja has quit IRC06:48
*** edmondsw has joined #openstack-nova06:50
*** csuttles_ has quit IRC06:50
*** dtantsur|afk is now known as dtantsur06:51
*** edmondsw has quit IRC06:54
*** markvoelker has quit IRC06:55
*** damien_r has quit IRC06:55
*** damien_r has joined #openstack-nova06:56
*** cfriesen_ has quit IRC06:56
*** chyka has joined #openstack-nova06:59
*** vvargaszte has quit IRC07:01
*** chyka has quit IRC07:03
*** oomichi has quit IRC07:05
*** oomichi has joined #openstack-nova07:06
*** adisky__ has quit IRC07:06
*** masber has quit IRC07:13
*** vvargaszte has joined #openstack-nova07:19
*** masber has joined #openstack-nova07:20
*** kaisers has quit IRC07:20
*** bhagyashris has quit IRC07:21
*** phuongnh has quit IRC07:22
*** bhagyashris has joined #openstack-nova07:22
*** zz9pzza has joined #openstack-nova07:24
*** yamamoto has quit IRC07:25
*** yamamoto has joined #openstack-nova07:27
*** kaisers has joined #openstack-nova07:29
openstackgerritTakashi NATSUME proposed openstack/nova master: Amend uuid4 hacking rule  https://review.openstack.org/48831507:33
*** kaisers has quit IRC07:37
*** masber has quit IRC07:38
*** cristicalin has joined #openstack-nova07:41
*** kaisers1 has joined #openstack-nova07:42
*** ralonsoh has joined #openstack-nova07:44
*** markvoelker has joined #openstack-nova07:52
*** yamamoto has quit IRC07:58
*** yamamoto has joined #openstack-nova07:59
*** claudiub has joined #openstack-nova07:59
*** ralonsoh has quit IRC07:59
*** cristicalin has quit IRC08:00
*** ralonsoh has joined #openstack-nova08:00
*** takashin has left #openstack-nova08:01
*** thorst has joined #openstack-nova08:03
*** strigazi_ is now known as strigazi08:04
openstackgerritlongxingmiao proposed openstack/nova master: test review  https://review.openstack.org/48832808:07
*** thorst has quit IRC08:07
*** yamamoto_ has joined #openstack-nova08:13
*** adisky__ has joined #openstack-nova08:14
*** yamamoto has quit IRC08:16
*** zz9pzza has quit IRC08:17
*** openstackgerrit has quit IRC08:18
*** esberglu has joined #openstack-nova08:19
*** derekh has joined #openstack-nova08:20
*** sambetts|afk is now known as sambetts08:23
*** esberglu has quit IRC08:23
*** saphi_ has joined #openstack-nova08:24
*** openstackgerrit has joined #openstack-nova08:24
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with placement api  https://review.openstack.org/48795808:24
*** sridharg has quit IRC08:24
*** lucas-afk is now known as lucasagomes08:25
*** sridharg has joined #openstack-nova08:25
*** alexchadin has joined #openstack-nova08:25
*** rmart04 has joined #openstack-nova08:25
*** markvoelker has quit IRC08:26
*** zz9pzza has joined #openstack-nova08:29
openstackgerritjichenjc proposed openstack/nova master: refactory some code on get_password  https://review.openstack.org/44945808:31
*** efoley has joined #openstack-nova08:33
*** BlackDex_ is now known as BlackDex08:33
openstackgerritjichenjc proposed openstack/nova master: add cert when send keystone request  https://review.openstack.org/48512108:35
*** mvk has quit IRC08:36
openstackgerritjichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix  https://review.openstack.org/48239008:37
*** andreas_s has joined #openstack-nova08:41
*** tuanluong has joined #openstack-nova08:45
*** tinwood has joined #openstack-nova08:52
bhagyashrisgibi: Hi, I have resolved my yesterday's issue there was some problem to restart the nfs server and for locking thing through nfs required statd service to enable.09:04
*** zenoway has joined #openstack-nova09:06
*** rha has quit IRC09:06
gibibhagyashris: good to hear that09:09
bhagyashrisgibi: Now everything is working fine to test the shared resource provider thing. I have just tested one scenario simply created instance as as mentioned in http://paste.openstack.org/show/616724/ but in my observation the root gb of created flavor has been considered in the allocations table against the compute resource provider and CUSTOM_DISK resource against "shared_resource" resource provider.09:11
bhagyashrisgibi: IMO, As i am creating the nfc backed instance so it should take the RAM and VCPU from the computes resource provider and DISK from the "shared_resource" provider09:12
bhagyashriss/nfc/nfs09:13
gibibhagyashris: I don't know how can you tell nova that you want to consume DISK_GB not from the compute provider but from the shared one09:22
gibibhagyashris: this is why I suggested to use CUSTOM_DISK in the shared and then add CUSTOM_DISK to the flavor09:23
gibibhagyashris: that way I sure that the instance will consume from your shared provider09:23
*** SteveR_ has joined #openstack-nova09:23
*** markvoelker has joined #openstack-nova09:23
*** SteveR_ is now known as Guest8047809:23
gibibhagyashris: I hope others can help with this specific issue09:23
*** Guest80478 is now known as SteveRUKC09:23
*** slaweq has joined #openstack-nova09:26
*** cristicalin has joined #openstack-nova09:26
bhagyashrisgibi: ohk. thank you.09:29
*** licanwei has quit IRC09:32
*** cristicalin has quit IRC09:37
SteveRUKChey all, quick question to which im struggling to find an answer. Is disk_allocation_ratio taken into account on live/cold migrations. As we have an allocation ratio of 6 set, but are having issues with migrations failing with a disk instance too large for target message.09:39
SteveRUKCalso, hypervisor_stats is showing a negative number for least_free09:39
SteveRUKCif anyone has any ideas it would be most appreciated09:39
*** saphi_ has quit IRC09:42
*** rha has joined #openstack-nova09:46
openstackgerritBalazs Gibizer proposed openstack/nova master: DNM: trying to turn on periodic tasks  https://review.openstack.org/48835209:48
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276609:53
*** gcb has quit IRC09:53
*** chyka has joined #openstack-nova09:55
*** markvoelker has quit IRC09:57
*** chyka has quit IRC10:00
*** prateek has joined #openstack-nova10:01
*** zenoway has quit IRC10:03
*** zenoway has joined #openstack-nova10:03
*** thorst has joined #openstack-nova10:04
*** slaweq has quit IRC10:04
*** esberglu has joined #openstack-nova10:07
*** thorst has quit IRC10:09
dimsdansmith : looks like ttx took care of it 3 hours ago. w00t10:09
*** damien_r has left #openstack-nova10:10
*** yingjun has quit IRC10:10
*** cdent has joined #openstack-nova10:12
*** esberglu has quit IRC10:12
cdentgibi: I did some similar periodic job fiddling last night and ran into the same confusion with the computes getting weird10:14
cdentI’m guessing the fixture (or the the service) is not happy being in the same process with another version of itself?10:15
*** gszasz has joined #openstack-nova10:15
*** iceyao has quit IRC10:17
*** zenoway has quit IRC10:18
*** zenoway has joined #openstack-nova10:19
*** edmondsw has joined #openstack-nova10:26
*** Guest91518 has quit IRC10:27
*** sree_ has joined #openstack-nova10:28
*** sree_ is now known as Guest5180910:28
*** tiendc has quit IRC10:29
bhagyashriscdent: Hi,  Hi, I have resolved my yesterday's issue there was some problem to restart the nfs server and for locking thing through nfs required statd service to enable.10:29
openstackgerritChris Dent proposed openstack/nova master: [placement] quash unicode warning with shared provider  https://review.openstack.org/48836310:29
cdentbhagyashris: ah, cool, glad you were able to make it work10:30
*** kiennt has quit IRC10:30
bhagyashriscdent: Now everything is working fine to test the shared resource provider thing. I have just tested one scenario simply created instance as as mentioned in http://paste.openstack.org/show/616724/ but in my observation the root gb of created flavor has been considered in the allocations table against the compute resource provider and CUSTOM_DISK resource against "shared_resource" resource provider.10:30
*** edmondsw has quit IRC10:30
bhagyashriscdent:  IMO, As i am creating the nfs backed instance so it should take the RAM and VCPU from the computes resource provider and DISK from the "shared_resource" provider.10:31
kashyapmdbooth: Thanks for the review!10:31
*** tuanluong has quit IRC10:31
kashyapAnd also to dansmith (About: https://review.openstack.org/#/c/485752/, but not for the "summary" remark :P)10:31
cdentbhagyashris: you can address that (at least in part) by using DISK_GB instead of CUSTOM_DISK for the nfs shared resource provider10:32
kashyapjohnthetubaguy: Maybe when you have a minute to ACK this straightforward change: https://review.openstack.org/#/c/485752/10:32
*** zz9pzza has quit IRC10:32
*** Guest51809 has quit IRC10:32
cdentthe thing that will be missing is that there’s nothing currently in place to manage the allocation of disk on the shared resource provider, but that’s in progress with the claim in the scheduler stuff that is being worked on this week10:33
*** iceyao has joined #openstack-nova10:33
bhagyashriscdent: ohk Means instead of creating the "CUSTOM_DISK" resource class and then creating the inventory for that I need to update the inventory against "DISK_GB" resource class right.10:36
*** iceyao has quit IRC10:37
cdentbhagyashris: well basically the general rule is that if you can use a standard resource class (because it maps to the unit of resource a provider is providing) you may as well10:37
bhagyashriscdent: ohk. checking...10:39
*** yamamoto_ has quit IRC10:40
*** kornicameister has quit IRC10:45
*** kornicameister has joined #openstack-nova10:46
*** tbachman has joined #openstack-nova10:46
s-deanHello, could someone point me in the right direction to debug this nova.exception.NeutronAdminCredentialConfigurationInvalid' ?10:48
*** karthiks has quit IRC10:49
s-deannevermind10:50
*** iceyao has joined #openstack-nova10:50
*** sridharg has quit IRC10:51
*** zhurong has quit IRC10:53
bhagyashriscdent: I have updated the inventory using DISK_GB resource class against the nfs shared resource provider but in that case after booting the instance it's not using shared_storage provider.10:53
bhagyashriscdent: It address the disk used against compute node resource provider.10:54
*** iceyao has quit IRC10:54
*** markvoelker has joined #openstack-nova10:55
*** sdague has joined #openstack-nova10:55
*** trungnv has quit IRC10:56
vdrokgood morning everyone!10:58
vdroksdague: small request, if you have some time :) https://review.openstack.org/48062410:58
vdrokthanks!11:00
cdentbhagyashris: ah, right, because the compue is reporting it’s own disk too, so is considered a valid target for all the resources. /me thinks11:01
*** iceyao has joined #openstack-nova11:01
cdentbhagyashris: are you working from today’s master or something earlier?11:01
*** trinaths has left #openstack-nova11:02
bhagyashriscdent: yeah. I am working from todays master commit-id: 23c4eb34380bdf3eece11abbe0f6ccb68c060f47   "claim resources in placement API during schedule()"11:02
*** kornicameister has quit IRC11:03
*** masber has joined #openstack-nova11:04
*** slaweq has joined #openstack-nova11:05
*** iceyao has quit IRC11:05
*** slaweq has quit IRC11:06
cdentbhagyashris: do you have the log of it making the PUT /allocations requests? It might be that the scheduler is claimng one set of allocations, but later (in the periodic update) the resource tracker is changing things. if you’re going to be using the CUSTOM_DISK resource class you need to make sure your flavor doesn’t indicate any requests for disk_gb11:07
cdentI think over the next few days things are likely to be quite unstable with claims, and there will be a few more changes coming11:07
*** saphi has quit IRC11:08
bhagyashriscdent: ohk.11:09
cdentbhagyashris: it’s great that you’re doing this investigating, but right now is going to be a chaotic time11:10
cdentbauzas: if this still alive? https://review.openstack.org/#/c/427200/11:13
bauzascdent: looking11:13
bauzascdent: well, maybe we should discuss with matt11:13
bauzasI can rebase it11:13
bauzasmeanwhile11:14
*** udesale has quit IRC11:14
cdentk, thanks11:14
*** cristicalin has joined #openstack-nova11:15
bhagyashriscdent: ohk. Just to inform you I have checked the allocations table entries and in that I saw the disk_gb reported against the compute node resource provider and not the shared resource provider.11:16
bhagyashriscdent: Thank you for your time :)11:16
cdentyeah, in the case where you have two different provider of DISK_GB it is likely that the non-shared provider will be picked when there are multiple options.11:17
*** iceyao has joined #openstack-nova11:18
*** yamamoto has joined #openstack-nova11:19
*** cristicalin has quit IRC11:20
*** jjung has quit IRC11:21
*** iceyao has quit IRC11:22
bhagyashriscdent: ohk.11:24
*** zenoway has quit IRC11:25
*** zenoway has joined #openstack-nova11:25
openstackgerritSean Dague proposed openstack/nova master: always show urls in list_cells  https://review.openstack.org/48786011:25
*** markvoelker has quit IRC11:27
*** yamamoto has quit IRC11:29
*** zenoway has quit IRC11:29
*** markus_z has quit IRC11:31
*** zenoway has joined #openstack-nova11:34
*** iceyao has joined #openstack-nova11:35
gibicdent: yes, I have the same feeling about the host name confusion11:36
gibicdent: but I cannot spend more time on it today. I will continue looking into it on Monday11:36
cdentgibi: I also left a comment on your test about another thing worth looking into11:36
cdentif I have time today I may experiment some more, but not sure I’ll have time11:36
cdentthanks for messing with it11:37
gibithanks for thelping11:37
*** jjung has joined #openstack-nova11:37
*** prateek_ has joined #openstack-nova11:37
gibiif you make some progress still today then do not hesistate to update any of my reviews11:38
*** thorst has joined #openstack-nova11:40
*** prateek has quit IRC11:40
*** iceyao has quit IRC11:40
*** zenoway has quit IRC11:43
openstackgerritSylvain Bauza proposed openstack/nova master: Add a status check for enabled filters  https://review.openstack.org/42720011:43
bauzascdent: ^11:44
*** thorst has quit IRC11:44
cdent11:44
bauzasafter reviewing it, I think it's important to merge it11:44
bauzasfor Pike11:44
openstackgerritChris Dent proposed openstack/nova master: Add functional test for two-cell scheduler behaviors  https://review.openstack.org/45200611:46
*** catintheroof has joined #openstack-nova11:48
openstackgerritChris Dent proposed openstack/nova master: [placement] gabbi tests for shared custom resource class  https://review.openstack.org/48520911:49
cdentbauzas: there’s quite a lot of small things that either fix minor bugs, cover areas not previously covered, or document things. all of which would be nice for pike. update 30 will be ready in a few minutes. If you and others can get to some of the things in the “other” section that would be very useful.11:50
bauzascdent: sorry, I don't understand you11:51
bauzascdent: you want me to do what ? review ?11:51
*** yamamoto has joined #openstack-nova11:51
cdentbauzas: a) I’m agree with hou on the import to merge 427200 and b) saying there’s lots of similar things, which will be listed in the # Other section rp update 30 coming out soon, which will need review and ought to be merged11:52
bauzasah, yes, agreed on that11:52
bauzaswe need to make sure we use those two weeks correctly, ie. not for features but rather bugs and/or docs11:53
bauzastbc, I'll mostly do bug triage next week11:53
bauzasand the rest of my time will be for reviewing11:53
*** acormier has joined #openstack-nova11:53
*** lucasagomes is now known as lucas-hungry11:54
*** rcernin has quit IRC11:55
*** esberglu has joined #openstack-nova11:55
*** yingjun has joined #openstack-nova11:57
*** yingjun has quit IRC11:58
*** zenoway has joined #openstack-nova11:58
*** litao__ has quit IRC11:59
*** zenoway has quit IRC11:59
*** zenoway has joined #openstack-nova12:00
*** esberglu has quit IRC12:00
*** chyka has joined #openstack-nova12:00
*** Dinesh_Bhor is now known as Dinesh_Bhor|afk12:01
*** Dinesh_Bhor|afk is now known as Dinesh_Bhor12:02
*** iceyao has joined #openstack-nova12:02
sdaguecdent / bauzas / jay - just an FYI of something I closed as won't fix - https://bugs.launchpad.net/nova/+bug/1707085 - in case you have other opinions12:03
openstackLaunchpad bug 1707085 in OpenStack Compute (nova) "Max_unit should account for allocation_ratio" [Undecided,Won't fix]12:03
*** iceyao has quit IRC12:07
*** chyka has quit IRC12:08
cdentsdague: your response makes sense to me, and max_unit is being set as it for the reasons you describe12:09
*** acormier has quit IRC12:10
*** acormier has joined #openstack-nova12:11
*** edmondsw has joined #openstack-nova12:13
*** catintheroof has quit IRC12:14
*** dosaboy has quit IRC12:14
*** rcernin has joined #openstack-nova12:16
*** thorst has joined #openstack-nova12:16
*** ratailor has quit IRC12:19
*** khushbu_ has joined #openstack-nova12:19
*** VAhl_ has joined #openstack-nova12:19
*** VAhl_ has quit IRC12:19
*** strigazi is now known as strigazi_AFK12:20
*** iceyao has joined #openstack-nova12:20
*** jaypipes has joined #openstack-nova12:23
*** jaypipes is now known as leakypipes12:23
cdentedleafe: left a comment on https://review.openstack.org/#/c/487954/ Just trying to make your life harder (as usual)12:23
*** markus_z has joined #openstack-nova12:23
*** vladikr has joined #openstack-nova12:23
*** vladikr has quit IRC12:24
edleafecdent: saw it. I'm not sure that any instance on an ironic node would ever have any other custom resource class. I mean, that's the point, right?12:24
*** markvoelker has joined #openstack-nova12:24
*** vladikr has joined #openstack-nova12:24
cdentedleafe: dunno12:24
cdentit’s the internet. if it is possible to do something, people will12:25
edleafecdent: what I *do* know is that operators can change the CRC of a node12:25
edleafecdent: and like you said, if they can, they will :)12:25
*** markvoelker has quit IRC12:25
*** jangutter has quit IRC12:26
*** jangutter has joined #openstack-nova12:26
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:26
openstackgerritDoug Hellmann proposed openstack/nova master: add a redirect for the old cells landing page  https://review.openstack.org/48793212:27
*** smatzek has joined #openstack-nova12:28
*** markvoelker has joined #openstack-nova12:29
*** pwiltsey has quit IRC12:31
*** liverpooler has joined #openstack-nova12:31
*** pchavva has joined #openstack-nova12:38
*** sree has joined #openstack-nova12:40
*** gbarros has joined #openstack-nova12:40
*** cristicalin has joined #openstack-nova12:42
*** prateek_ has quit IRC12:43
*** sree has quit IRC12:44
*** kfarr has joined #openstack-nova12:47
sdaguecdent: is this still a thing? https://bugs.launchpad.net/nova/+bug/165964712:48
openstackLaunchpad bug 1659647 in OpenStack Compute (nova) "The resource tracker clears the tracked_instances dictionary on every periodic job" [Undecided,New]12:48
* cdent reads12:48
*** lyan has joined #openstack-nova12:49
*** slaweq has joined #openstack-nova12:49
*** gbarros_ has joined #openstack-nova12:49
*** sridharg has joined #openstack-nova12:49
cdentsdague: yeah, that came up in the hangout last night12:49
cdentfixing it as described there would break “healing” but it’s now no longer clear that perfect healing is what we want12:49
cdentit did get brought up in atlanta, but unresolved. will probably need to come up again in denver12:50
* cdent puts it on the etherpad12:50
*** catintheroof has joined #openstack-nova12:50
s-deanHi there, in the nova_cell0 database is the instanaces tables supposed to be populated?12:50
efrieds-dean As I understand it, that table only houses instances that failed to be scheduled.12:51
*** gbarros has quit IRC12:51
efrieds-dean I learned that from https://review.openstack.org/#/c/487183/12:51
s-deanok thanks, im trying to track down why i can list my instances with openstack server list12:52
sdaguecdent: ok, so I'm going to close it as Opinion Wishlist then, as it's one of those notes to self that's getting tracked in other ways (like via the etherpad)12:52
cdentsdague: seems fair12:52
*** cristicalin has quit IRC12:53
*** gbarros_ has quit IRC12:53
efrieds-dean There's a whole thing I don't understand about deleted instances not really being deleted, some kind of "soft" delete.12:53
s-deanNo i haven't deleted the instances yet, the command is hanging and not returning anything however the API server is returning 200 OK12:54
*** annegentle has joined #openstack-nova12:55
*** esberglu has joined #openstack-nova12:55
*** esberglu has quit IRC12:55
efrieds-dean does `nova list` work?12:56
*** esberglu has joined #openstack-nova12:56
*** dosaboy has joined #openstack-nova12:57
s-deanalso in the instances table(nova DB), the cell_name filed is NULL12:57
s-deannope12:57
*** alexchadin has quit IRC12:57
s-deanthat is also hanging, its like it cant access the database;12:57
*** gbarros has joined #openstack-nova12:58
*** acormier_ has joined #openstack-nova12:59
*** annegentle has quit IRC12:59
*** artom has quit IRC12:59
*** artom has joined #openstack-nova12:59
*** artom has quit IRC13:00
s-deanalso there is no response13:00
*** esberglu has quit IRC13:01
*** acormier has quit IRC13:02
*** lucas-hungry is now known as lucasagomes13:03
*** jaosorior has quit IRC13:04
*** lbragstad has joined #openstack-nova13:04
*** yamamoto has quit IRC13:05
*** xyang1 has joined #openstack-nova13:06
bauzassdague: looking up bugs ? I'm about to do that too13:07
sdaguebauzas: yeh, I've been trying to burn the New list down13:08
sdaguegetting close13:08
bauzask13:08
bauzasI'll also look at the in-progress bugs13:09
bauzasto make sure we won't miss any important one13:09
*** sree has joined #openstack-nova13:13
*** acormier_ has quit IRC13:13
*** acormier has joined #openstack-nova13:14
*** eharney has joined #openstack-nova13:14
*** esberglu has joined #openstack-nova13:16
*** links has quit IRC13:16
*** prateek has joined #openstack-nova13:16
*** khushbu_ has quit IRC13:17
*** acormier has quit IRC13:18
*** jangutter has quit IRC13:22
*** jangutter has joined #openstack-nova13:23
*** mriedem has joined #openstack-nova13:28
*** tbachman has quit IRC13:28
mriedempike-3 tag is in https://review.openstack.org/#/c/488218/13:29
*** yamamoto has joined #openstack-nova13:29
*** rajathagasthya has joined #openstack-nova13:33
*** baoli has joined #openstack-nova13:34
*** baoli has quit IRC13:34
*** tbachman has joined #openstack-nova13:35
*** psachin has quit IRC13:35
*** baoli has joined #openstack-nova13:35
*** acormier has joined #openstack-nova13:35
*** tbachman has quit IRC13:35
openstackgerritMatt Riedemann proposed openstack/nova master: Sanity check delete_allocation_for_instance  https://review.openstack.org/48818713:35
*** SteveRUKC has quit IRC13:36
cdentmriedem: can you summarize what you leared while doing that ^?13:36
*** iceyao has quit IRC13:37
*** iceyao has joined #openstack-nova13:37
mriedemcdent: well it does two things13:37
*** prateek has quit IRC13:37
mriedem1. when the RT (source node during a migration) untracks it's instances and attempts to delete allocations, we get the current allocations for the instance for all providers and if the source node rp uuid isn't in the current list of allocations, it noops13:38
*** rcernin has quit IRC13:38
mriedem2. if the source node rp uuid is in the list of current allocations but there is also at least one other VCPU resource provider, it logs a message before deleting the allocations13:38
mriedemi have seen #2 in the live migration CI runs,13:38
mriedemi haven't seen #113:38
mriedemhttp://logs.openstack.org/87/488187/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/107a810/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_27_22_21_14_76622913:39
dansmithyeah so #2 will get reported as a bug and tell us that we need to skip instead of log and continue there13:39
mriedemi really just pushed this to see if things were really bad, like if we hit #1 a lot13:39
*** acormier has quit IRC13:40
dansmithmriedem: you could try setting the heal interval really small so it runs more often during the few migrations we actually do13:40
cdentdo you feel better or worse?13:40
mriedemi feel better13:41
mriedem#1 would be a scarier race imo13:41
mriedemand could still happen, but chances are slim - at least in our CI env which isn't that load intensive13:41
mriedem#1 happens after the source node RT pulls the allocations for it's own uuid,13:41
mriedemso to hit #1, that means the allocations are gone for that node between the time it pulls them and the time it processes it's list of untracked instances13:42
mriedemi think #2 is fixed by making the RT check the vm_state of the instance before trying to delete it's allocations13:42
mriedemwhich is something jay was working on yesterday13:42
leakypipesfor the record, guys, I should be done with my patch shortly.13:42
bauzasmriedem: yup, saw the pike-3 change this morning13:43
leakypipesjust working on unit tests now13:43
bauzasit was already merged13:43
dansmithmriedem: insert some artificial rpc delay into the periodic and see if #1 happens13:43
mriedemi can do that13:43
mriedemcdent: bauzas: also https://bugs.launchpad.net/nova/+bug/1707071 if you haven't seen that yet13:44
openstackLaunchpad bug 1707071 in OpenStack Compute (nova) ocata "Compute nodes will fight over allocations during migration" [Medium,Confirmed]13:44
bauzasnot yet indeed13:44
bauzasmriedem: dansmith: leakypipes: I thought about something this night13:44
bauzascdent: ^13:45
leakypipesdansmith: you need to change your clothes.13:45
*** jistr is now known as jistr|mtg13:45
bauzaswhat if we should just get the current allocations before deleting them by the scheduler, so in case we know we have a problem, we could put them again ?13:45
dansmithheh13:45
*** dansmith is now known as superdan13:45
bauzasoh snap, FF13:45
*** bauzas is now known as bauwser13:45
cdentthanks mriedem13:46
superdanbauwser: get + check is just as safe as get + delete + re-put, except you never have to delete13:46
superdannot sure why we would do the latter13:46
*** artom has joined #openstack-nova13:46
*** jjung has quit IRC13:47
bauwsermriedem: superdan: leakypipes: oh, and me and cdent just discussed about https://review.openstack.org/#/c/427200/ : it could be important for operators13:47
*** andreas_s has quit IRC13:47
*** cfriesen_ has joined #openstack-nova13:47
bauwsersuperdan: sure, was just wondering if it was simplier than just trying to get two allocations for both source and target when moving13:47
bauwserI know it's the consensus13:48
superdanI don't see how it helps13:48
mriedembauwser: yes there is an odd scenario in there which i commented on13:48
bauwserand I totally agree with it13:48
superdanwithout two allocations we're not accounting for the resources used by a moving instance13:48
bauwserbut I do wonder if we could simplify the problem13:48
bauwserat least for the races we know of13:48
superdanwell, the double allocation. there's still only one allocation for the instance13:48
bauwserthe problem here is that we have 3 different services looking at placement when moving : #1 scheduler, #2 source compute, #3 target compute13:49
superdanthat's kindof the point of placement right?13:49
bauwseryeah, I know, but that means we have like shared information13:49
mriedemthat's the point13:49
bauwserbut I think we discussed that yesterday13:49
mriedemthe global view13:49
superdanyeah, that's the whole thing13:50
bauwseranyway, I don't want to nitpick13:50
mriedembut yes that means old code that assumes it's all local and owns everything at any given time has to change13:50
superdanit's not the shared state/view/responsibility that concerns me, it's that the current RT was designed for a different model13:50
bauwserI'm just trying to see how to help with the problems we know about multiple services taking the same allocations13:50
leakypipeshopefully my new patch's code comments explain the situation well enough.13:51
bauwserideally, eventually, I'm not sure we need resouretrackers for computes13:51
bauwserif allocations would only be done by schedulers13:52
superdanbauwser: we don't in the long run13:52
superdanwe still need some of that code on the computes,13:52
superdanbut not the full RT13:52
bauwserI'd see RTs just *deleting* allocations if something goes mad13:52
superdanthat's what I was saying yesterday13:52
bauwsersuperdan: yeah, I know13:52
bauwserI'm just thinking out loud to try to identify how we could simplify13:52
superdanso.... :)13:52
superdanokay :)13:52
bauwserbut I agree with you, using the existing RT means tech deby13:53
*** ratailor has joined #openstack-nova13:53
superdanTech Deby.. that's like the host of a kids show about computers?13:53
bauwser:)13:54
*** gouthamr has joined #openstack-nova13:54
*** weshay is now known as weshay_interview13:54
bauwsermy keyboard is AZERYT :p13:54
*** prateek has joined #openstack-nova13:55
superdanT and Y are together on both13:55
superdanI thought it was AZERTY?13:55
* bauwser is just using mittens when typing /o\13:56
bauwserrather, yting13:56
*** kfarr_ has joined #openstack-nova13:57
*** kfarr_ has quit IRC13:57
*** strigazi_AFK is now known as strigazi_OFF13:58
cdentcan someone merge this https://review.openstack.org/#/c/488363/ looking at that warning is getting tiresome13:59
*** rcernin has joined #openstack-nova14:01
*** tbachman has joined #openstack-nova14:04
*** hshiina has quit IRC14:04
kashyapsuperdan: Thanks for the review on this: https://review.openstack.org/#/c/485752/14:05
kashyapsuperdan: And thanks for the little snark, too :P14:05
superdanheh14:06
kashyapCan anyone +W it?  I can also backport it to the revlevant upstream branches14:06
*** prateek has quit IRC14:08
superdankashyap: sdague loves +Wing patches like that14:09
kashyap:-) I thought of pinging him explicitly, but refrained in the spirit of being a good citizen as I shouldn't specifically nag people14:10
kashyapAnd just ask the generic "ether", that is the channel :-)14:10
superdankashyap: sdague owes us all a beer after yesterday, so I think today it's uniquely okay to ask him directly :P14:10
*** crushil has joined #openstack-nova14:10
*** rmart04 has quit IRC14:10
kashyapsdague: If you are listening in, it's a straight-forward perf issue fixed by calling a simple "please set cache mode" method.  And the reporter has even confirmed the fix works -- https://review.openstack.org/#/c/485752/14:11
superdancdent: fix mriedem's comment and I'll fast approve14:11
kashyapsuperdan: What happened yesterday that bestows this windfall on the rest?14:11
superdankashyap: oh nothing.. just a good-faith-gone-awry sort of deal14:11
kashyap:D14:11
*** yamamoto has quit IRC14:12
*** dfisher has joined #openstack-nova14:12
*** dfisher has left #openstack-nova14:12
*** yamamoto has joined #openstack-nova14:12
*** vishwana_ has quit IRC14:13
*** vishwanathj has joined #openstack-nova14:13
openstackgerritChris Dent proposed openstack/nova master: [placement] quash unicode warning with shared provider  https://review.openstack.org/48836314:13
cdentsuperdan: done ^. tvm14:14
superdancdent: doneski14:14
cdentrawkin or something14:14
*** liverpooler has quit IRC14:15
*** liverpooler has joined #openstack-nova14:15
*** iceyao has quit IRC14:16
*** slaweq has quit IRC14:17
bauwsermriedem: cdent: FYI https://review.openstack.org/#/c/488363/1/nova/objects/resource_provider.py@74114:17
*** iceyao has joined #openstack-nova14:17
bauwserI'm not a SQLA expert, but I know that you need all your strings to be unicode (and UTF-8) if you want to use SQLA14:17
cdentbauwser: I think the relevant point here is that os_traits is _not_ which might be a surprise to some (it was to me)14:18
bauwsercdent: ostraits is a python lib14:18
bauwsercdent: so it really just gives us what python gives us14:18
*** gyee has joined #openstack-nova14:19
bauwserthere is no contract about what it returns14:19
cdentyes, I know that, but no, it has a choice of how it constructs the values14:19
*** AlexeyAbashkin has joined #openstack-nova14:19
bauwserand a shit ton of 3rd-party libs behave the same14:19
cdentit could have don them as unicdoe14:19
bauwserthe point is just, SQLA by default expects unicode, so in general you explicitely convert14:19
bauwsercdent: that reminds me14:20
cdentyes, and?14:20
bauwsercdent: I'm not sure that if we create a trait like CUSTOM_mé14:20
bauwserit will correctly work14:20
bauwserbecause we won't define the charset14:20
*** sbezverk has quit IRC14:22
*** mtanino has joined #openstack-nova14:22
cdent"pattern": "^CUSTOM\_[A-Z0-9_]+$",14:22
*** larainema has joined #openstack-nova14:22
cdentso it is moot on that front14:22
cdentnone of this changes that matt’s right that a comment is helpful14:22
cdentit’s been done, it’s merged.14:22
bauwsercdent: oh correct, we only accept ASCII14:25
bauwsereven less14:25
bauwserso that's not a big deal14:25
*** armax has joined #openstack-nova14:26
*** acormier has joined #openstack-nova14:26
kashyapmriedem: Thanks for the merge!14:31
kashyap(For this https://review.openstack.org/#/c/485752/)14:31
*** baoli has quit IRC14:33
*** marst has joined #openstack-nova14:34
*** pwiltsey has joined #openstack-nova14:36
*** efried is now known as fried_rice14:36
fried_ricesdague You got a minute?14:37
*** edleafe is now known as figleaf14:37
*** hongbin has joined #openstack-nova14:38
openstackgerritAlex Szarka proposed openstack/nova master: fix test_rebuild_server_exc instability  https://review.openstack.org/48738214:39
*** ratailor has quit IRC14:41
kashyapPeople, are simple performance fixes acceptable for "Phase-2" support (Newton)?14:42
*** crushil has quit IRC14:42
*** crushil has joined #openstack-nova14:43
lyarwoodkashyap: your cache change is just a bugfix IMHO, not a perf fix, it just happens to help with the latter.14:43
mriedemomg we have a wild lyarwood sighting14:43
* superdan throws a net over him14:43
* lyarwood goes back into hiding14:43
kashyapLOL14:43
kashyaplyarwood: Noted.  Just in case you're wondering: if I'm masquereding something to "smuggle" a fix in, no, I wasn't :-)14:44
superdanjust yesterday I warned another manager that we were taking lyarwood back by force, if necessary, at the end of his side contract14:44
*** iceyao has quit IRC14:45
*** jistr|mtg is now known as jistr14:46
lyarwoodsuperdan: not helped by my `yeah sure I can help with that` follow up ;)14:46
*** Dinesh_Bhor has quit IRC14:46
kashyaplyarwood: When you craw out of your cave, your comment isn't clear - does that mean, as a stable maint, do you find it acceptable to backport to Newton, or not?14:46
*** masber has quit IRC14:46
superdanlyarwood: yeah :/14:46
mriedemlyarwood: you're the new tonyb14:46
mriedemkashyap: bug fixes to newton are ok14:46
kashyapmriedem: Okido, thanks14:46
lyarwoodmriedem: what did you do to the old tonyb?14:47
* kashyap will be on PTO from 01-Aug, just not want to leave at least "finishable" things hanging in the air14:47
mriedemlyarwood: tonyb signs up to help with everything14:47
sdaguefried_rice: I'm about to lose power here as they move our power to another pole14:48
sdagueso... poke me later14:48
fried_ricesdague Sweet.  I think I may have figured it out.  Thanks.14:48
*** annegentle has joined #openstack-nova14:48
*** sbezverk has joined #openstack-nova14:50
*** harlowja has joined #openstack-nova14:51
*** sdague has quit IRC14:53
*** trinaths has joined #openstack-nova14:53
mriedemsuperdan: looking over my old todo list, i had a note about online data migrations for multiple cells,14:55
*** aarefiev is now known as aarefiev_afk14:55
mriedeme.g. object code looks in the api db first and if not found, looks in the 'cell' db for looking things up14:55
mriedembut with multiple cells, you wouldn't know which cell db to lookup from14:55
mriedemso that probably goes in our list of things you have to do before you can do multi cell14:56
superdandepends on the migration I guess14:56
superdanmriedem: but you could use multiple configs to handle that today14:56
superdanmriedem: i.e. use the config for one of the cell conductors you have if it doesn't hit the api db, and if it does, just add creds for the api db and the one cell you're looking at14:57
*** ijw has joined #openstack-nova14:57
superdanbut yeah I'd have to think about the best way to test those things14:57
*** Drankis has joined #openstack-nova14:58
*** kbaegis has joined #openstack-nova14:59
mriedemok i was just thinking about like migrating quotas, we look in api, then look in cell, but really if we did it like the api code, we'd iterate the cells to find the quota limit - although something like quota limits in multiple cells would be wonky - it could be default in a new cell but custom in an old cell14:59
mriedemprobably just best to say, make sure your online data migrations are done before attempting adding a new cell15:00
superdanyeah15:00
mriedemi should also get this cleaned up https://review.openstack.org/#/c/460377/15:00
*** danpawlik has quit IRC15:01
superdanugh yeah15:02
openstackgerritEd Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration  https://review.openstack.org/48795415:03
*** edmondsw has quit IRC15:06
*** crushil has quit IRC15:07
*** claudiub has quit IRC15:09
*** zenoway has quit IRC15:09
*** baoli has joined #openstack-nova15:11
*** gszasz has quit IRC15:12
mriedembauwser: going to update the reno in https://review.openstack.org/#/c/484828/ per cdent's comment?15:12
*** rcernin has quit IRC15:13
bauwsermriedem: sure, can do15:13
bauwsermriedem: I'm just going to end up my work week prematurely because of corporate network requirements that require major upgrades for me15:14
*** zenoway has joined #openstack-nova15:14
mriedembauwser: you know how when i asked if you were talking to your dog when in boston and i said the dog said 'le woof' and you looked like that doesn't make sense/15:15
mriedem?15:15
*** weshay_interview is now known as weshay15:15
mriedemi said the same thing to my wife, who knows a little french, and she did the same, and said, "that doesn't make sense, it would be la woof"15:15
mriedemi can't win15:16
bauwsermriedem: heh15:16
bauwsermriedem: I was litterally about to show my dog yesterday on the hangout as it was bugging me15:16
*** crushil has joined #openstack-nova15:16
bauwseranyway, my internal network connection is about to blow out in a few, so \o15:17
* cdent is intrigued by bauwser’s network15:19
*** penick has joined #openstack-nova15:19
*** edmondsw has joined #openstack-nova15:19
bauwsercdent: one tuple (openvpn 2.4, fedora 23)15:20
*** awaugama has joined #openstack-nova15:20
*** imacdonn has quit IRC15:22
*** imacdonn has joined #openstack-nova15:22
*** links has joined #openstack-nova15:23
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: simulate rpc delay when cleaning up allocations from source node  https://review.openstack.org/48850015:27
mriedemsuperdan: ^ like this?15:30
superdanmriedem: yeah exactly15:30
*** damien_r has joined #openstack-nova15:31
*** damien_r has quit IRC15:32
mriedemleakypipes: i'm going to close out the placement-claims bp15:32
mriedemthe main bits are done for pike, rest can be something else in queens15:32
leakypipesk15:32
leakypipessuperdan: what was that bug number again? I remember it was a palindrome but my scrollback is too little to grab it..15:37
leakypipes1707071?15:37
superdanyep15:37
leakypipesgot it, thanks15:38
*** tbachman has quit IRC15:39
*** penick has quit IRC15:40
*** itlinux has joined #openstack-nova15:40
*** chyka has joined #openstack-nova15:40
*** Drankis has quit IRC15:43
*** trinaths has left #openstack-nova15:45
*** trinaths has joined #openstack-nova15:45
*** itlinux has quit IRC15:48
*** VAhl has quit IRC15:49
*** mtanin___ has joined #openstack-nova15:50
*** jjung has joined #openstack-nova15:50
*** marst_ has joined #openstack-nova15:50
*** marst has quit IRC15:50
*** harlowja has quit IRC15:51
*** mtanino has quit IRC15:51
*** jmlowe has quit IRC15:51
*** chyka_ has joined #openstack-nova15:52
*** chyka has quit IRC15:55
s-deanHi could somebody tell me what, this command queries15:56
s-deancurl -g -i --cacert "/etc/keystone/tls/ca-cert.pem" -X GET https://10.30.0.2:8774/v2.1/servers/detail -H "User-Agent: python-novaclient" -H "Accept: application/json" -H "X-Auth-Token: {SHA1}3ac08c79ab47c87d8050040c8a18d0774c326933"15:56
s-deanobviously the nova-api15:57
mriedemlist servers for that tenant with details15:57
s-deanbut what does the servers detail part look at15:57
mriedemhttps://developer.openstack.org/api-ref/compute/#list-servers-detailed15:57
s-deanbecause its not returning anything15:57
*** markus_z has quit IRC15:57
s-deani never get a response back15:57
mriedemit hangs?15:58
s-deanyeah15:58
mriedemor you get an empty body?15:58
s-deanno response15:58
mriedemthe response should timeout either way15:58
s-deanmhh15:58
mriedemthere should be a request id you can trace through the nova-api logs15:58
mriedemlook for the GET /servers/detail request in the nova-api debug logs15:59
mriedemthere is a req-<uuid> thing15:59
*** Apoorva has joined #openstack-nova15:59
s-dean nova.osapi_compute.wsgi.server [req-b884b02e-e9c0-4845-9dd8-1f4cd1a6215a 2456405c5d3547e5aeece36828578f99 be5ea0b1031d4949ab94107522f305b4 - default default] 10.30.0.2 "GET /v2.1/servers/detail HTTP/1.1" status: 500 len: 566 time: 600.711502115:59
mriedemyou can follow that through the logs15:59
mriedemreq-b884b02e-e9c0-4845-9dd8-1f4cd1a6215a15:59
*** yamamoto has quit IRC15:59
mriedemyeah15:59
mriedemso something exploded15:59
mriedem500 error15:59
*** yamamoto has joined #openstack-nova15:59
mriedemlook for errors15:59
*** yamamoto has quit IRC15:59
*** kfarr has quit IRC15:59
s-dean DBError: Can't reconnect until invalid transaction is rolled back15:59
mriedemhuh, don't know what that is16:00
s-deanits asking me to report a bug16:00
mriedems-dean: are you hand-rolling all this stuff or using some deployment tooling?16:00
s-deanhand-rolling baby16:00
s-deanhaha16:00
mriedemmisconfiguration of the db wouldn't be a nova bug16:00
s-deanprobably a config error again16:00
mriedemmay i suggest using a deployment tool16:00
*** ralonsoh has quit IRC16:01
s-deanyeah but which oone16:01
mriedemlike openstack-ansible or kolla16:01
mriedem^ are the ones i'd look at16:01
s-deani was looking into kolla16:01
*** saphi has joined #openstack-nova16:01
s-deandeploys it onto docker right16:01
*** thorst_ has joined #openstack-nova16:02
mriedemyeah16:02
mriedemopenstack-ansible deploys into lxc containers16:02
leakypipesbuilds openstack as docker images is what you mean, but yeah.16:02
s-deanive just spent so much time learning how to deploy to bare metal16:03
s-dean3 node setup16:03
s-deani dont even now how all that stuff ties into my backend network16:03
s-deani wanted to deploy a self service network with overlay network provider and management hence why ive been doing it by hand16:04
s-deanbeen one heck of a learning experience i'm glad its the weekend though hahaha16:05
*** thorst has quit IRC16:05
*** lucasagomes is now known as lucas-afk16:06
*** sree has quit IRC16:09
*** sree has joined #openstack-nova16:09
openstackgerritJay Pipes proposed openstack/nova master: remove source provider allocs in confirm_resize()  https://review.openstack.org/48851016:10
*** zenoway has quit IRC16:11
leakypipesmriedem, superdan, bauwser, cdent, figleaf: ^^16:11
* leakypipes wishes we had functional tests for migration...16:12
leakypipesthat didn't mock everything out..16:12
mriedemleakypipes: we do-ish16:12
*** sdague has joined #openstack-nova16:12
leakypipesmriedem: see my qualifying statement above :)16:12
*** jmlowe has joined #openstack-nova16:12
mriedemlike what?16:12
mriedemwe mock out cinder/glance/neutron yeah16:13
mriedembut we don't really need those for testing this16:13
mriedeme.g https://review.openstack.org/#/c/487958/16:13
*** sree has quit IRC16:14
*** david-lyle has quit IRC16:14
openstackgerritMerged openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s)  https://review.openstack.org/48575216:15
leakypipesmriedem: yeah, that's a good start. thanks for the link.16:15
leakypipesmriedem: still no way to test the kinds of timing issues that superdan was bringing up on the call yesterday though... emulating a long MQ interaction on the destination for instance and ensuring that the source host does the right thing...16:16
leakypipesanyway, I'm just bitching. it's Friday, deal with it :P16:16
mriedemleakypipes: you're about as sassy as that gd sax16:16
*** david-lyle has joined #openstack-nova16:16
melwitt*sniff* my precious blueprint16:17
mriedemsorry16:17
melwitts'okay :)16:18
mriedemtell your bp to have a kick grass summer and you'll see it in the fall16:18
mriedemin it's yearbook of course16:18
melwitthaha16:19
mriedemps bobby flynn is soooo cute!16:19
melwitt"Stay cool!"16:19
*** esberglu is now known as esberglu_lunch16:21
*** armax has quit IRC16:21
*** david-lyle has quit IRC16:22
*** gbarros has quit IRC16:23
*** ijw has quit IRC16:23
*** ijw has joined #openstack-nova16:24
*** lyan has quit IRC16:25
leakypipesmriedem: grr, shitbuckets.16:27
leakypipesmriedem: we forgot about resize to same host. :(16:27
mriedemliterally?16:27
*** armax has joined #openstack-nova16:28
leakypipesmriedem: so when doing a resize same host, we'll end up deleting the entire allocation except for the shared providers in the patch I just put up.16:28
*** saphi_ has joined #openstack-nova16:28
cdentyeehaw16:28
mriedemjust check instance.host == CONF.host?16:28
*** saphi_ has quit IRC16:28
leakypipesmriedem: yeah. though... in the thing we added for scheduler claiming a double-up allocation, did we account for resize to same host? :(16:29
*** gbarros has joined #openstack-nova16:29
mriedemyes16:29
mriedemit comares the rp uuid right?16:29
mriedemdoes a set difference16:29
mriedempretty sure i thought about that when reviewing and came to the conclusion it's handled by the set difference on rp uuid16:30
cdentyeah, but isn’t the point that you still need a doubling and since the rp uuid is the same, you need to double within the allocation, not adjacent to16:30
mriedemno16:30
leakypipesmriedem: yeah, but the issue is that we need to create an allocation for the *same provider* but the resource amounts are added together (old and new amount)16:30
mriedemthe double is for the source and target16:30
superdanyes16:30
mriedemso you don't lose the source when putting allocations for the target16:30
superdanbut if resize to same host, you still have two copies16:30
superdanso you need twice the allocation16:30
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48803416:31
mriedemi guess you do need to account for the resize up flavor resource class stuff16:31
leakypipessuperdan: well, not twice the allocation, but the new amount of resources added to the old amount of resources, but on the same resource provider. :(16:31
*** larainema has quit IRC16:31
leakypipesI hate my life.16:31
superdanleakypipes: yes, I mean the sum of the allocations of course :)16:31
mriedemso is it just that we aren't doing the correct 'new' flavor allocation when resize to same host?16:31
superdanleakypipes: I've been using "doubled up" to mean "the sum"16:31
mriedemis it sum?16:31
mriedemfor resize to same host i mean16:32
leakypipessuperdan: understood.16:32
superdanit's definitely sum for disk, but I don't think we should distinguish, we should just sum everything16:32
mriedemok16:32
leakypipesmriedem: yeah, it's sum16:32
mriedemso let's open a bug?16:32
mriedemtag it with pike-rc-potential16:32
superdanif you're resizing to a new flavor with more disk, but fewer cpu/mem, you have to protect the older larger amount of resource16:32
mriedemsuperdan: good point16:32
cdentleakypipes: on the resourcce tracker side, when the resize confirm wants to delete everything on itself, would it be safe/okay to let that  happen, and then immediately call into the _update routines so it would then write a new allocation for the correct size?16:33
leakypipesffs, we're just going to end up porting all the craziness and conditionals from the resource_tracker.py module into the scheduler report client for this stuff :(16:33
leakypipescdent: might be, yes... need to think through it.16:34
mriedemcdent: _update only updates inventory16:35
mriedemif you're talking about ResourceTracker._update16:35
mriedemmaybe we should rename that :)16:35
cdentI don’t mean _update specifically say _update_* perhaps16:35
mriedemheh16:35
cdentwherever the regular allocations happens16:35
*** d9k has quit IRC16:35
mriedem_update_something_for_instance_maybe16:35
cdentanyway, there’s another wrinkle near this that I wanted to ask about: are those regular allocation update shared providers aware yet?16:36
mriedemdon't think so16:36
figleafcdent: no, but they won't delete the shared provider allocation, right?16:36
mriedemhttps://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L91916:36
mriedemhttps://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L102416:37
*** liuyulong has joined #openstack-nova16:37
cdentfigleaf: if we write any allocation at all, it will replace. so it depends on what we’re creating, to write16:38
figleafleakypipes: sorry, but the nuns beat that stuff into me16:38
leakypipesmriedem: right, and the idea was to not have to call put_allocations() once the placement-claims stuff was done.16:38
cdentI’m thinking in terms of these vaunted “heals” we love. Is a heal going to be correct in the face of a shared provider16:38
figleafcdent: OIC what you're getting at16:38
figleafcdent: yeah, that would only work if allocations were consumer/rp specific16:39
mriedemleakypipes: yeah, and my original understanding from 4+ months ago was we'd put code into the computes that wouldn't do anything with allocations if they were already created by the scheduler, and we'd not do claims in the scheduler until all computes had that code16:39
mriedembut then that went away16:39
* figleaf gets this feeling that we will end up persisting the allocation_request with the instance16:40
superdanmriedem: that code _is_ in the computes though16:41
superdanwell, part of it16:41
mriedemsuperdan: the diff thing in the report client you mean right?16:41
mriedemmy_allocations vs current_allocations16:41
mriedem?16:41
superdanyes, but the thing is,16:41
mriedemmy_allocations doesn't account for shared storage16:41
mriedemright?16:41
superdanwe still have to have the healing from the compute node side regardless16:41
*** openstackstatus has quit IRC16:41
*** openstack has joined #openstack-nova16:44
*** links has quit IRC16:44
leakypipesmriedem: if you create a bug, I'll get to work on a fix.16:44
*** sridharg has quit IRC16:44
mriedemwhich bug are we talking about? the fact we don't sum allocations when resize to same host in scheduler?16:45
mriedemsounds like multiple bugs16:45
mriedembecause of the self-heal issue, or is that the one from yesterday?16:45
leakypipesmriedem: no, sorry, I was referring to addressing the TODOs left by cdent around the shared providers (the links you pasted above to the report.py module)16:46
cfriesen_leakypipes: you could always just stop supporting resize-to-same-host and let all the users scream.  /s16:46
leakypipescfriesen_: cool with me.16:46
*** d9k has joined #openstack-nova16:46
melwittyeah really. especially since they complain that it doesn't mean "force resize to same host"16:47
mriedemleakypipes: ok, but we also have a bug for the resize to same host wrt the scheduler double fudge goodness right?16:47
mriedemlet's call it double fudge now16:47
leakypipesmriedem: yeah :)16:47
leakypipestwo different bugs16:47
mriedemok16:47
mriedemwill do in a bit16:47
cdentleakypipes: could you please remind me what the _provider_aggregate_map in in the report client was destined for?16:49
leakypipesmriedem: thanks Matt16:49
leakypipescdent: precisely the case for determining if the compute node was associated to providers of shared resources.16:50
cdentso is it of use in one of these two bugs?16:50
*** jjung has quit IRC16:50
*** abalutoiu has joined #openstack-nova16:51
mriedemleakypipes: bug the first https://bugs.launchpad.net/nova/+bug/170725216:51
openstackLaunchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed]16:51
mriedemsuperdan: ^ make sure i made sense in there16:51
superdanhang on, I'm reviewing leakypipes' other patch16:52
*** efoley has quit IRC16:54
*** randomhack has joined #openstack-nova16:54
*** catinthe_ has joined #openstack-nova16:54
*** lyan has joined #openstack-nova16:54
*** harlowja has joined #openstack-nova16:54
*** crushil has quit IRC16:54
mriedemeven with the fake rpc sleep thing in https://review.openstack.org/#/c/488500/ i'm not seeing the case that the source node is not in the allocations when it goes to delete the allocations for the instance16:55
*** derekh has quit IRC16:55
*** harlowja_ has joined #openstack-nova16:55
superdanmriedem: but we know it can happen16:57
*** Apoorva_ has joined #openstack-nova16:57
superdanso I guess if you want to just leave it, then that's fine, but it'll be super hard to track down if it really happens16:57
*** dtantsur is now known as dtantsur|afk16:57
*** catintheroof has quit IRC16:58
superdanmriedem: yes I think your bug text makes sense16:58
openstackgerritSean Dague proposed openstack/nova master: Add cinder keystone client opts to config reference  https://review.openstack.org/48853016:58
*** harlowja has quit IRC16:59
mriedemsuperdan: leave what? the part of my patch that returns if the source node isn't in the current allocations?16:59
mriedemthe point was really just safe guarding against that and providing logging in case it happens16:59
superdanmriedem: leave it like not apply your patch16:59
mriedemoh16:59
mriedemyeah16:59
mriedemi figured we'd sort that out later16:59
superdanyou said this morning something indicating you might want to not commit this patch17:00
superdanI think we should but...17:00
*** yamamoto has joined #openstack-nova17:00
mriedemi think it would be ok to do it - it's not going to impact performance by pulling the allocations again, because this code only runs from the compute periodic17:00
mriedemi intentionally didn't pass the rp uuid in from the scheduler so the scheduler won't double check17:00
*** Apoorva has quit IRC17:00
superdanokay17:01
*** crushil has joined #openstack-nova17:01
mriedemleakypipes: superdan: cdent: here is the other bug https://bugs.launchpad.net/nova/+bug/170725617:03
openstackLaunchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client is not account for shared resource providers" [High,Confirmed]17:03
*** sambetts is now known as sambetts|afk17:03
mriedemomg me fail english17:04
superdanis not account?17:04
superdanoh17:04
superdanmah17:04
superdangod17:04
superdanone for the record books kids17:04
* cdent is infectious17:04
*** abalutoiu has quit IRC17:04
superdanlet it be known henceforth that mriedem is not perfect17:04
mriedemi was surrounded by a whirlwind of 6 year old in pink17:04
mriedemrattled me17:04
openstackgerritMatt Riedemann proposed openstack/nova master: Remove compatibility code for flavors  https://review.openstack.org/46037717:05
*** randomhack has quit IRC17:05
*** dillaman has quit IRC17:06
*** slaweq has joined #openstack-nova17:06
cdentyou reall do have my disease mriedem : “compute node things it needs”17:08
*** slaweq has quit IRC17:09
sdaguemriedem: for one glorious moment, 0 bugs in New state - https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New17:09
mriedemo.O17:09
mriedemsdague: did you just invalidate everything? :)17:09
cdentleakypipes: [t 3rxI] [t 3edk]17:09
purplerbot<leakypipes> cdent: precisely the case for determining if the compute node was associated to providers of shared resources. [2017-07-28 16:50:19.034601] [n 3rxI]17:09
purplerbot<cdent> so is it of use in one of these two bugs? [2017-07-28 16:50:43.278883] [n 3edk]17:09
sdagueI read every new bug, moved a bunch of them to Incomplete with specific questions, found all the ones that really were going to need specs and linked them to the specs process and put them in Opinion17:10
sdagueduped a few17:10
leakypipescdent: yes, the latter.17:10
sdaguefound some that had been fixed17:10
mriedemsdague: thanks17:10
sdagueso, it's mostly a legit cleaning of the New state17:10
* mriedem goes to lunch and other things for awhile17:10
*** yamamoto has quit IRC17:10
*** mriedem is now known as mriedem_afk17:10
cdentcool, thanks leakypipes, I figured as much but wanted to be sure17:11
sdaguealso... evacuate bugs... zomg17:11
melwittsdague: you are a champion17:11
*** ijw has quit IRC17:12
cdentgerrit hates me17:12
*** esberglu_lunch is now known as esberglu17:12
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline for roughly 5 minutes to perform a database backup and reconfiguration17:13
cdentvoila17:13
superdancdent: hates everyone apparently17:13
superdan*it hates17:13
cdents/://17:13
superdanwell, I suspected, but17:13
melwittyeah, gerrit hates us all. it's not you17:13
*** itlinux has joined #openstack-nova17:14
*** junbo has quit IRC17:15
cfriesen_review.openstack.org is down?17:15
cfriesen_never mind, just saw the note17:15
*** slaweq has joined #openstack-nova17:16
*** yamahata has joined #openstack-nova17:16
*** swebster has quit IRC17:16
*** pwiltsey has quit IRC17:16
*** trinaths has left #openstack-nova17:17
cfriesen_sdague: arguably bug 1669054 is not opinion...if CONF.allow_resize_to_same_host is False and we do a resize then a subsequent rebuild/evacuate, it will continue to ignore the original host.17:19
openstackbug 1669054 in OpenStack Compute (nova) "RequestSpec.ignore_hosts from resize is reused in subsequent evacuate" [Undecided,Opinion] https://launchpad.net/bugs/166905417:19
cfriesen_sdague: the only "might" is around exactly where to do the fix17:20
*** gyee has quit IRC17:23
openstackgerritPeter Hamilton proposed openstack/nova-specs master: Add support for certificate validation  https://review.openstack.org/48854117:23
*** gyee has joined #openstack-nova17:25
openstackgerritmelanie witt proposed openstack/nova master: Detach device from live domain even if not found on persistent  https://review.openstack.org/48854517:26
*** openstackgerrit has quit IRC17:33
*** openstackgerrit has joined #openstack-nova17:34
openstackgerritPeter Hamilton proposed openstack/nova-specs master: Add support for certificate validation  https://review.openstack.org/48854117:34
*** saphi has quit IRC17:36
*** slaweq has quit IRC17:38
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276617:40
*** tbachman has joined #openstack-nova17:41
*** Swami has joined #openstack-nova17:42
openstackgerritDan Smith proposed openstack/nova master: Add some more cellsv2 doc goodness  https://review.openstack.org/48718317:54
*** larainema has joined #openstack-nova17:55
sdaguecfriesen_: Opinion often means bring it back in via the specs process. Did I not do that with this one?17:59
*** acormier has quit IRC17:59
sdaguecfriesen_: if you want it confirmed / medium, that's fine as well, but that ends up being kind of a different black hole.18:00
*** acormier has joined #openstack-nova18:00
sdaguecfriesen_: trying to make every open bug actionable18:00
sdaguemight and may are not really actionable18:01
openstackgerritJay Pipes proposed openstack/nova master: remove source provider allocs in confirm_resize()  https://review.openstack.org/48851018:01
*** crushil has quit IRC18:01
leakypipessuperdan, figleaf: updated &18:01
leakypipes^^18:01
openstackgerritDan Smith proposed openstack/nova master: Add some more cellsv2 doc goodness  https://review.openstack.org/48718318:02
superdanleakypipes: fixed ^18:02
*** gyee has quit IRC18:03
leakypipessuperdan: +218:04
superdantys18:04
*** acormier has quit IRC18:04
*** acormier has joined #openstack-nova18:04
*** gyee has joined #openstack-nova18:06
*** catinthe_ has quit IRC18:08
*** ijw has joined #openstack-nova18:08
*** crushil has joined #openstack-nova18:09
*** acormier has quit IRC18:09
*** catintheroof has joined #openstack-nova18:10
cdentah buckets, dan gets in the revision while I’m reviewing the cellsv2 doc goodness18:10
melwittsuperdan: thanks for writing that up, complete with diagrams! since I've been thinking about consoles, I'm thinking maybe I can add onto that with mention of nova-consoleauth and the websocketproxies similar to "Neutron Metadata API proxy"?18:11
*** esberglu has quit IRC18:11
*** openstackstatus has quit IRC18:11
*** openstack has joined #openstack-nova18:13
*** Apoorva_ has quit IRC18:14
*** Apoorva has joined #openstack-nova18:14
*** esberglu has quit IRC18:15
*** gyee has quit IRC18:15
*** ihrachys has joined #openstack-nova18:15
*** dillaman has quit IRC18:26
*** dillaman has joined #openstack-nova18:27
*** amotoki is now known as amotoki__18:28
*** ijw has quit IRC18:29
*** rajathagasthya has quit IRC18:30
*** jamesdenton has quit IRC18:32
*** jamesdenton has joined #openstack-nova18:32
*** cdent has quit IRC18:33
*** ijw has joined #openstack-nova18:35
*** ijw has quit IRC18:36
*** ijw has joined #openstack-nova18:37
*** dave-mccowan has joined #openstack-nova18:38
*** kbaegis has quit IRC18:39
*** dillaman has quit IRC18:41
*** esberglu has joined #openstack-nova18:41
*** openstackstatus has quit IRC18:42
*** openstack has joined #openstack-nova18:44
*** rajathagasthya has joined #openstack-nova18:46
*** tbachman has quit IRC18:47
*** tbachman has joined #openstack-nova18:51
cfriesen_jaypipes: would https://review.openstack.org/#/c/339715 still be an issue with the new placement/allocations stuff?  (i.e should I bother refreshing it or will it be handled already?)18:54
cfriesen_sdague: in this case the "might" was for a possibly way to fix it.  The bug exists, that is not in question.18:55
*** acormier has joined #openstack-nova18:56
*** rtjure has quit IRC18:58
leakypipescfriesen_: no, I don't think so. different issue.19:00
*** kbaegis has joined #openstack-nova19:00
leakypipescfriesen_: I think you can revamp that patch, address the comments from Matt.19:00
*** yassine has quit IRC19:04
*** dillaman has quit IRC19:05
*** dave-mccowan has quit IRC19:05
*** dillaman has joined #openstack-nova19:07
*** crushil has quit IRC19:11
openstackgerritEd Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration  https://review.openstack.org/48795419:12
*** tbachman has quit IRC19:12
*** nicolasbock has quit IRC19:12
*** pwiltsey has joined #openstack-nova19:13
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_endpoint_data()  https://review.openstack.org/48813719:14
*** acormier has quit IRC19:15
*** acormier has joined #openstack-nova19:16
*** kbaegis has quit IRC19:17
*** rtjure has joined #openstack-nova19:17
*** crushil has joined #openstack-nova19:18
*** rtjure has quit IRC19:19
*** rtjure has joined #openstack-nova19:19
*** rtjure has quit IRC19:19
*** rtjure has joined #openstack-nova19:20
*** rtjure has quit IRC19:21
*** rtjure has joined #openstack-nova19:21
*** rtjure has quit IRC19:23
*** rtjure has joined #openstack-nova19:24
*** rtjure has quit IRC19:26
*** rtjure has joined #openstack-nova19:27
*** rtjure_ has joined #openstack-nova19:27
*** rtjure has quit IRC19:29
*** liverpooler has quit IRC19:29
*** rtjure has joined #openstack-nova19:29
*** rtjure_ has quit IRC19:29
*** rtjure has quit IRC19:29
*** mriedem_afk is now known as mriedem19:33
fried_ricemriedem https://review.openstack.org/#/c/488034/ (g-r bump) is verified now.19:34
*** kfarr has joined #openstack-nova19:35
*** tbachman has joined #openstack-nova19:37
*** gbarros has quit IRC19:38
*** Apoorva has quit IRC19:39
*** Apoorva has joined #openstack-nova19:40
mriedemmgagne: https://bugs.launchpad.net/nova/+bug/170728419:41
openstackLaunchpad bug 1707284 in OpenStack Compute (nova) "Extend attached volume fails with "VolumePathsNotFound: Could not find any paths for the volume." in os-brick iscsi connector" [Medium,Confirmed]19:41
mgagnemriedem: hi19:41
*** lyan has quit IRC19:44
*** Apoorva has quit IRC19:45
*** liusheng has quit IRC19:45
*** mriedem has quit IRC19:46
*** liusheng has joined #openstack-nova19:46
mgagnemriedem: could it be a redheering ? I'm not familiar with os-bricks and friends but right above, you can read "iscsiadm: No active sessions" and u'target_discovered': False in the bdm dump. that's what I found at first glance19:47
*** _pewp_ has quit IRC19:48
*** _pewp_ has joined #openstack-nova19:48
*** mriedem has joined #openstack-nova19:50
hongbinhi folks, i want to confirm that if i create an instance with an existing port and a security group, the security group should be added to the existing port or not?19:53
hongbinor the security group should override the port's security group?19:55
*** lyan has joined #openstack-nova19:56
*** Apoorva has joined #openstack-nova20:00
mgagnemriedem: I'm having a hard time debugging the logic, there are missing debug logs from os-brick, especially around calls to _iterate_all_targets from _get_potential_volume_paths in os_brick/initiator/connectors/iscsi.py20:03
openstackgerritJay Pipes proposed openstack/nova master: placement: don't allocate on compute nodes  https://review.openstack.org/48859520:05
leakypipesmriedem, superdan, figleaf, bauwser: mind blown.... ^20:06
leakypipesmriedem: and that takes care of https://bugs.launchpad.net/nova/+bug/1707256 actually. :)20:06
openstackLaunchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client does not account for shared resource providers" [High,Confirmed] - Assigned to Jay Pipes (jaypipes)20:06
superdanleakypipes: um, what?20:07
superdanleakypipes: who un-doubles the move allocation?20:07
*** randomhack has joined #openstack-nova20:07
leakypipesmriedem: by removing the compute node/resource tracker from all allocation operations, we kind of just remove the whole "is this shared or not" problem on the compute node.20:07
leakypipessuperdan: that's done in the dependent patch.20:07
leakypipessuperdan: https://review.openstack.org/#/c/488510/320:07
* superdan doesn't get it20:08
leakypipessuperdan: so that's the only place on the compute node that will update allocations.20:08
*** catintheroof has quit IRC20:08
superdan...okay20:08
superdanleakypipes: so while we're upgrading,20:09
superdanocata computes will alter/delete the scheduler allocation, and we'll never heal it on pike computes, right? so how do we ensure it heals if it gets damaged?20:09
leakypipessuperdan: "mind blown" was referring to my own mind being blown. :) Basically, the patch makes the resource tracker look like what I wanted it to look like once placement-claims was complete. i.e. don't have it mucking with claims/allocations at all.20:09
superdanyeah, I got it20:10
superdanstill not sure we can go that far yet20:10
superdanmaybe we should hangout to discuss?20:11
leakypipessuperdan: still haven't gotten that far in mind yet :) I could go for a hangout walkthrough of ideas. that last patch was more a "let's see what happens" patch than anything else :)20:11
openstackgerritHongbin Lu proposed openstack/nova master: Handle exception on adding secgroup  https://review.openstack.org/46517320:11
superdanoh okay, I thought you were suggesting that we do that for pike to eliminate these problems20:11
leakypipessuperdan: nah20:11
superdan"let's see what happens in the gate" isn't going to convince me20:11
superdanokay20:11
leakypipessuperdan: in any case, I would be good for a hangout.20:12
leakypipeswould be useful to talk through some of this.20:12
*** randomhack has quit IRC20:12
mgagnemriedem: ok, I think there is a bug in os-brick. When retrieve the iscsi sessions here: https://github.com/openstack/os-brick/blob/1.15.1/os_brick/initiator/connectors/iscsi.py#L24720:15
mgagneThe returned value should look like so: ['10.0.1.24:3260']20:15
mgagneBelow, it tries to do something I don't fully understand. But iscsi_portals_with_sessions will end up with the value ['.']20:15
mgagneSo I'm not sure how this value is useful for the logic below which compares the bdm target_portal against the above value. In the end, you will always end up with an empty host_devices list.20:15
mgagneThat's my hypothesis on what's happening.20:15
mgagneI think it got broken by this refactor despite the comment in the commit "It's worth mentioning that this patch does not touch the extend operation.": https://github.com/openstack/os-brick/commit/56c8665d3d342ce90f5d9433966c0f244063b4c120:18
*** boris_42 has joined #openstack-nova20:20
*** acormier has quit IRC20:25
*** acormier has joined #openstack-nova20:25
*** smatzek has quit IRC20:30
*** acormier has quit IRC20:30
hongbinhi, if i create a vm with an existing neutron port and a security group, the security group is ignored, i wonder why not populating the port with the specified sg?20:32
*** pchavva has quit IRC20:33
*** penick has joined #openstack-nova20:36
*** leakypipes has quit IRC20:36
*** baoli has quit IRC20:37
*** sdague has quit IRC20:37
*** jmlowe has quit IRC20:39
*** dillaman has quit IRC20:41
openstackgerritMerged openstack/nova master: Provide original fault message when BFV fails  https://review.openstack.org/46771520:43
*** kfarr has quit IRC20:44
*** crushil has quit IRC20:44
*** dillaman has joined #openstack-nova20:46
*** weshay is now known as weshay_pto20:53
*** dillaman has quit IRC21:01
*** marst has joined #openstack-nova21:03
openstackgerritOpenStack Release Bot proposed openstack/os-vif master: Update reno for stable/pike  https://review.openstack.org/48867121:04
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/48803421:05
*** marst_ has quit IRC21:05
openstackgerritOpenStack Release Bot proposed openstack/python-novaclient master: Update reno for stable/pike  https://review.openstack.org/48879921:07
*** yamamoto has joined #openstack-nova21:08
*** thorst_ has quit IRC21:09
*** sree_ has joined #openstack-nova21:10
*** sree_ is now known as Guest68821:10
*** yamamoto has quit IRC21:12
*** slaweq has joined #openstack-nova21:13
*** Guest688 has quit IRC21:14
*** vishwanathj has quit IRC21:15
*** yamamoto has joined #openstack-nova21:16
*** yamamoto has quit IRC21:20
*** awaugama has quit IRC21:20
*** edmondsw has quit IRC21:23
*** edmondsw has joined #openstack-nova21:24
*** gouthamr has quit IRC21:24
*** edmondsw has quit IRC21:28
*** david-lyle has joined #openstack-nova21:30
*** esberglu has quit IRC21:34
*** acormier_ has joined #openstack-nova21:35
*** gouthamr has joined #openstack-nova21:36
*** acormier_ has quit IRC21:36
*** dillaman has joined #openstack-nova21:42
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48886521:43
*** david-lyle has quit IRC21:48
*** slaweq has quit IRC21:49
openstackgerritEd Leafe proposed openstack/nova master: Clarifying node_uuid usage in ironic driver.  https://review.openstack.org/48580321:51
*** annegentle has quit IRC21:52
*** yamamoto has joined #openstack-nova21:55
openstackgerritMerged openstack/nova master: [placement] quash unicode warning with shared provider  https://review.openstack.org/48836321:56
*** thorst has joined #openstack-nova22:01
*** slaweq has joined #openstack-nova22:02
*** thorst has quit IRC22:05
*** itlinux has quit IRC22:05
*** esberglu has joined #openstack-nova22:07
*** slaweq has quit IRC22:08
*** esberglu has quit IRC22:11
*** yamamoto has quit IRC22:12
*** jmlowe has joined #openstack-nova22:18
*** slaweq has joined #openstack-nova22:21
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_endpoint_data()  https://review.openstack.org/48813722:25
*** slaweq has quit IRC22:27
*** thorst has joined #openstack-nova22:31
*** lyan has quit IRC22:36
*** fried_rice is now known as efried_zzz22:36
*** figleaf is now known as edleafe22:37
*** marst has quit IRC22:48
*** yamahata has quit IRC22:54
*** pwiltsey has quit IRC23:01
*** rajathagasthya has quit IRC23:02
*** randomhack has joined #openstack-nova23:09
*** yamamoto has joined #openstack-nova23:13
*** randomhack has quit IRC23:13
*** acormier has joined #openstack-nova23:19
*** yamamoto has quit IRC23:21
*** acormier has quit IRC23:23
*** slaweq has joined #openstack-nova23:24
*** hongbin has quit IRC23:30
*** chyka_ has quit IRC23:30
*** rajathagasthya has joined #openstack-nova23:31
*** rtjure has joined #openstack-nova23:32
*** mtanin___ has quit IRC23:39
*** dillaman has quit IRC23:41
*** slaweq has quit IRC23:43
*** acormier has joined #openstack-nova23:47
*** markvoelker has quit IRC23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!