Friday, 2014-06-06

*** matsuhashi has joined #openstack-ironic00:03
*** stevebaker has quit IRC00:04
*** stevebaker has joined #openstack-ironic00:05
devanandalifeless: ran out of time today00:05
lifelessdevananda: ack00:06
devanandawill try to bbiafh will you be around?00:06
lifelessyes, for 4.5 hoursthen its C time for a bit,then back around your midnight00:07
devanandak, will ping you when i get back00:07
lifelesskk00:07
*** rloo has quit IRC00:19
*** rloo has joined #openstack-ironic00:19
*** rloo has quit IRC00:21
*** rloo has joined #openstack-ironic00:21
*** ellenh has joined #openstack-ironic00:22
jrolldevananda / lifeless: I know it's probably a bad time for this, but would you check this out when you have time? https://review.openstack.org/#/c/97361/00:22
jrollI'm fine with it landing or being blocked, just want it off of my list :)00:23
*** bigjools has quit IRC00:24
*** bigjools has joined #openstack-ironic00:46
*** igordcard has quit IRC00:53
*** ellenh has quit IRC01:09
openstackgerritYongli He proposed a change to openstack/ironic: Rewrite ironic policy to use the new changes of common policy  https://review.openstack.org/9773101:25
*** rloo_ has joined #openstack-ironic01:35
*** rloo has quit IRC01:35
*** rloo_ has quit IRC01:42
*** rloo has joined #openstack-ironic01:43
*** nosnos has joined #openstack-ironic01:46
lifelessjroll: wait, what01:48
*** rloo has quit IRC01:49
*** eghobo has quit IRC01:49
*** rloo has joined #openstack-ironic01:49
*** rloo has quit IRC01:50
*** rloo has joined #openstack-ironic01:51
*** rloo has quit IRC01:52
*** rloo has joined #openstack-ironic01:52
*** rloo has quit IRC01:56
*** rloo has joined #openstack-ironic01:56
*** rloo has quit IRC01:57
*** rloo has joined #openstack-ironic01:58
*** Haomeng has quit IRC01:58
lifelessjroll: you have your wish (blocked :))01:59
lifeless(I hope that came across as  humourous)02:01
*** rloo has quit IRC02:18
*** rloo has joined #openstack-ironic02:19
*** rloo has quit IRC02:19
*** rloo has joined #openstack-ironic02:20
*** rloo has quit IRC02:28
*** ellenh has joined #openstack-ironic02:32
*** ellenh has quit IRC02:41
*** harlowja is now known as harlowja_away02:46
*** ellenh has joined #openstack-ironic02:51
*** ellenh has quit IRC02:52
openstackgerritYongli He proposed a change to openstack/ironic: Rewrite ironic policy to use the new changes of common policy  https://review.openstack.org/9773103:03
*** matsuhashi has quit IRC03:13
*** nosnos has quit IRC03:13
*** nosnos has joined #openstack-ironic03:13
*** matsuhashi has joined #openstack-ironic03:14
*** matsuhashi has quit IRC03:14
*** vinbs has joined #openstack-ironic03:14
*** nosnos has quit IRC03:17
*** rainya has quit IRC03:27
*** nosnos has joined #openstack-ironic03:31
*** matsuhashi has joined #openstack-ironic03:38
*** nosnos has quit IRC03:39
*** nosnos has joined #openstack-ironic03:40
*** rainya has joined #openstack-ironic03:45
*** sabah has joined #openstack-ironic03:46
*** matsuhashi has quit IRC03:48
*** rameshg87 has joined #openstack-ironic03:50
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Add a HardwareManager method to erase devices  https://review.openstack.org/9787103:53
*** matsuhashi has joined #openstack-ironic03:56
*** matsuhashi has quit IRC03:56
*** godp1301 has joined #openstack-ironic03:57
*** nosnos has quit IRC04:00
*** Poornima has joined #openstack-ironic04:01
*** matsuhashi has joined #openstack-ironic04:04
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Add a HardwareManager method to erase devices  https://review.openstack.org/9787104:13
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Add a HardwareManager method to erase devices  https://review.openstack.org/9787104:17
*** nosnos has joined #openstack-ironic04:21
*** eghobo has joined #openstack-ironic04:26
*** k4n0 has joined #openstack-ironic04:32
*** godp1301 has quit IRC04:43
*** godp1301 has joined #openstack-ironic04:44
devanandalifeless: hi! have a few minutes04:46
*** godp1301 has quit IRC04:50
*** ellenh has joined #openstack-ironic04:53
*** coolsvap|afk is now known as coolsvap04:56
*** wendar has quit IRC05:04
*** wendar has joined #openstack-ironic05:05
*** datajerk has quit IRC05:15
*** devananda has quit IRC05:15
*** r0j4z0 has quit IRC05:15
*** JayF has quit IRC05:15
*** Mikhail_D_wk has quit IRC05:15
*** sseago has quit IRC05:15
*** agordeev has quit IRC05:15
*** pcrews has quit IRC05:15
*** ellenh has quit IRC05:15
*** sabah has quit IRC05:15
*** lsmola_ has quit IRC05:15
*** NobodyCam has quit IRC05:15
*** ekarlso has quit IRC05:15
*** soren has quit IRC05:15
*** lifeless has quit IRC05:15
*** pradipta_away has quit IRC05:15
*** Shrews has quit IRC05:15
*** mikal has quit IRC05:15
*** pleia2 has quit IRC05:15
*** aignatov has quit IRC05:15
*** Ng has quit IRC05:15
*** sbadia has quit IRC05:15
*** boris-42 has quit IRC05:15
*** Madasi has quit IRC05:15
*** lucas-dinner has quit IRC05:15
*** yuriyz has quit IRC05:15
*** tteggel has quit IRC05:15
*** gilliard has quit IRC05:15
*** bigjools has quit IRC05:15
*** BadCub has quit IRC05:15
*** jrist has quit IRC05:15
*** dguerri has quit IRC05:15
*** beecee has quit IRC05:15
*** ryanpetrello has quit IRC05:15
*** vinbs has quit IRC05:15
*** zul has quit IRC05:15
*** dshulyak has quit IRC05:15
*** comstud has quit IRC05:15
*** matty_dubs|gone has quit IRC05:15
*** adam_g has quit IRC05:15
*** matsuhashi has quit IRC05:15
*** openstackgerrit has quit IRC05:15
*** christopheraedo has quit IRC05:15
*** rushiagr has quit IRC05:15
*** kevinbenton has quit IRC05:15
*** lynxman has quit IRC05:15
*** SpamapS has quit IRC05:21
*** SpamapS has joined #openstack-ironic05:27
*** lazy_prince has joined #openstack-ironic05:27
*** ellenh has joined #openstack-ironic05:27
*** matsuhashi has joined #openstack-ironic05:27
*** sabah has joined #openstack-ironic05:27
*** vinbs has joined #openstack-ironic05:27
*** bigjools has joined #openstack-ironic05:27
*** devananda has joined #openstack-ironic05:27
*** r0j4z0 has joined #openstack-ironic05:27
*** openstackgerrit has joined #openstack-ironic05:27
*** christopheraedo has joined #openstack-ironic05:27
*** pcrews has joined #openstack-ironic05:27
*** lucas-dinner has joined #openstack-ironic05:27
*** lsmola_ has joined #openstack-ironic05:27
*** rushiagr has joined #openstack-ironic05:27
*** yuriyz has joined #openstack-ironic05:27
*** kevinbenton has joined #openstack-ironic05:27
*** JayF has joined #openstack-ironic05:27
*** ryanpetrello has joined #openstack-ironic05:27
*** NobodyCam has joined #openstack-ironic05:27
*** BadCub has joined #openstack-ironic05:27
*** zul has joined #openstack-ironic05:27
*** Mikhail_D_wk has joined #openstack-ironic05:27
*** sseago has joined #openstack-ironic05:27
*** agordeev has joined #openstack-ironic05:27
*** tteggel has joined #openstack-ironic05:27
*** datajerk has joined #openstack-ironic05:27
*** jrist has joined #openstack-ironic05:27
*** dguerri has joined #openstack-ironic05:27
*** lynxman has joined #openstack-ironic05:27
*** ekarlso has joined #openstack-ironic05:27
*** boris-42 has joined #openstack-ironic05:27
*** pleia2 has joined #openstack-ironic05:27
*** dshulyak has joined #openstack-ironic05:27
*** gilliard has joined #openstack-ironic05:27
*** pradipta_away has joined #openstack-ironic05:27
*** soren has joined #openstack-ironic05:27
*** lifeless has joined #openstack-ironic05:27
*** Ng has joined #openstack-ironic05:27
*** comstud has joined #openstack-ironic05:27
*** beecee has joined #openstack-ironic05:27
*** aignatov has joined #openstack-ironic05:27
*** Madasi has joined #openstack-ironic05:27
*** mikal has joined #openstack-ironic05:27
*** Shrews has joined #openstack-ironic05:27
*** sbadia has joined #openstack-ironic05:27
*** matty_dubs|gone has joined #openstack-ironic05:27
*** adam_g has joined #openstack-ironic05:27
*** datajerk has quit IRC05:28
*** SpamapS has quit IRC05:28
*** devananda has quit IRC05:28
*** r0j4z0 has quit IRC05:28
*** JayF has quit IRC05:28
*** Mikhail_D_wk has quit IRC05:28
*** sseago has quit IRC05:28
*** agordeev has quit IRC05:28
*** pcrews has quit IRC05:28
*** ellenh has quit IRC05:28
*** sabah has quit IRC05:28
*** lsmola_ has quit IRC05:28
*** NobodyCam has quit IRC05:28
*** ekarlso has quit IRC05:28
*** soren has quit IRC05:28
*** lifeless has quit IRC05:28
*** pradipta_away has quit IRC05:28
*** Shrews has quit IRC05:28
*** mikal has quit IRC05:28
*** pleia2 has quit IRC05:28
*** aignatov has quit IRC05:28
*** Ng has quit IRC05:28
*** sbadia has quit IRC05:28
*** boris-42 has quit IRC05:28
*** Madasi has quit IRC05:28
*** lucas-dinner has quit IRC05:28
*** yuriyz has quit IRC05:28
*** tteggel has quit IRC05:28
*** gilliard has quit IRC05:28
*** bigjools has quit IRC05:28
*** BadCub has quit IRC05:28
*** jrist has quit IRC05:28
*** dguerri has quit IRC05:28
*** beecee has quit IRC05:28
*** ryanpetrello has quit IRC05:28
*** lazy_prince has quit IRC05:28
*** vinbs has quit IRC05:28
*** zul has quit IRC05:28
*** dshulyak has quit IRC05:28
*** comstud has quit IRC05:28
*** matty_dubs|gone has quit IRC05:28
*** adam_g has quit IRC05:28
*** matsuhashi has quit IRC05:28
*** openstackgerrit has quit IRC05:28
*** christopheraedo has quit IRC05:28
*** rushiagr has quit IRC05:28
*** kevinbenton has quit IRC05:28
*** lynxman has quit IRC05:28
*** rakesh_hs has joined #openstack-ironic05:33
*** SpamapS has joined #openstack-ironic05:33
*** lazy_prince has joined #openstack-ironic05:33
*** matsuhashi has joined #openstack-ironic05:33
*** sabah has joined #openstack-ironic05:33
*** vinbs has joined #openstack-ironic05:33
*** bigjools has joined #openstack-ironic05:33
*** devananda has joined #openstack-ironic05:33
*** r0j4z0 has joined #openstack-ironic05:33
*** openstackgerrit has joined #openstack-ironic05:33
*** christopheraedo has joined #openstack-ironic05:33
*** pcrews has joined #openstack-ironic05:33
*** lucas-dinner has joined #openstack-ironic05:33
*** lsmola_ has joined #openstack-ironic05:33
*** rushiagr has joined #openstack-ironic05:33
*** yuriyz has joined #openstack-ironic05:33
*** kevinbenton has joined #openstack-ironic05:33
*** JayF has joined #openstack-ironic05:33
*** ryanpetrello has joined #openstack-ironic05:33
*** NobodyCam has joined #openstack-ironic05:33
*** BadCub has joined #openstack-ironic05:33
*** zul has joined #openstack-ironic05:33
*** Mikhail_D_wk has joined #openstack-ironic05:33
*** sseago has joined #openstack-ironic05:33
*** agordeev has joined #openstack-ironic05:33
*** tteggel has joined #openstack-ironic05:33
*** datajerk has joined #openstack-ironic05:33
*** jrist has joined #openstack-ironic05:33
*** dguerri has joined #openstack-ironic05:33
*** lynxman has joined #openstack-ironic05:33
*** ekarlso has joined #openstack-ironic05:33
*** boris-42 has joined #openstack-ironic05:33
*** pleia2 has joined #openstack-ironic05:33
*** dshulyak has joined #openstack-ironic05:33
*** gilliard has joined #openstack-ironic05:33
*** pradipta_away has joined #openstack-ironic05:33
*** soren has joined #openstack-ironic05:33
*** lifeless has joined #openstack-ironic05:33
*** Ng has joined #openstack-ironic05:33
*** comstud has joined #openstack-ironic05:33
*** beecee has joined #openstack-ironic05:33
*** aignatov has joined #openstack-ironic05:33
*** Madasi has joined #openstack-ironic05:33
*** mikal has joined #openstack-ironic05:33
*** Shrews has joined #openstack-ironic05:33
*** sbadia has joined #openstack-ironic05:33
*** matty_dubs|gone has joined #openstack-ironic05:33
*** adam_g has joined #openstack-ironic05:33
*** lucas-dinner has quit IRC05:34
*** lucas-dinner has joined #openstack-ironic05:46
*** pradipta_away is now known as pradipta05:57
k4n0Good morning ironic05:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/9606306:02
*** sysexit has joined #openstack-ironic06:21
*** sysexit has quit IRC06:29
mrdak4n0: \o06:30
k4n0mrda, Hi :)06:31
lifelessdevananda: hi07:07
lifelessdevananda: I suspect you've halt()ed now, but lets sync monday or possibly if I have time slices tomorrow07:07
*** romcheg has joined #openstack-ironic07:10
*** max_lobur has joined #openstack-ironic07:19
*** rameshg871 has joined #openstack-ironic07:24
*** rameshg87 has quit IRC07:26
*** mkerrin1 has quit IRC07:31
Mikhail_D_wkGood morning folks! :)07:31
*** mkerrin has joined #openstack-ironic07:38
*** mrda is now known as mrda-longweekend07:43
mrda-longweekendNight folks! :)07:43
romchegMorning Mikhail_D_wk!07:44
romchegGood night mrda-longweekend!07:44
romchegHave a nice weekend!07:44
openstackgerritYuiko Takada proposed a change to openstack/ironic: Fix the logging message to identify node by uuid  https://review.openstack.org/9831907:45
*** eghobo has quit IRC07:51
*** takadayuiko has joined #openstack-ironic07:51
*** ndipanov has joined #openstack-ironic07:52
*** r0j4z0 has quit IRC08:01
*** sabah has quit IRC08:02
openstackgerritYuiko Takada proposed a change to openstack/ironic: Fix the logging message to identify node by uuid  https://review.openstack.org/9831908:05
*** jistr has joined #openstack-ironic08:07
*** pelix has joined #openstack-ironic08:08
*** derekh_ has joined #openstack-ironic08:08
*** max_lobur has quit IRC08:19
*** romcheg has quit IRC08:24
*** Kai14 has joined #openstack-ironic08:25
*** romcheg has joined #openstack-ironic08:39
*** takadayuiko has quit IRC08:41
*** lucas-dinner is now known as lucasagomes08:42
lucasagomesmorning all08:43
lucasagomeslifeless, pong, sorry missed ur ping08:43
* lucasagomes reads the scrollback08:43
*** sysexit has joined #openstack-ironic08:46
Mikhail_D_wklucasagomes:  g'morning :)08:51
romchegMorning lucasagomes!08:51
*** foexle has joined #openstack-ironic09:06
*** igordcard has joined #openstack-ironic09:07
lifelesslucasagomes: hi09:17
lucasagomeslifeless, hey there, so the api... I think I can understand that spec better after the last talk with devananda09:20
*** martyntaylor has joined #openstack-ironic09:20
lucasagomeslifeless, https://restful-api-design.readthedocs.org/en/latest/methods.html#asynchronous-requests < that's a good example and something we can copy to ironic09:20
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Use temporary dir for image conversion  https://review.openstack.org/9833809:21
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported  https://review.openstack.org/9810209:25
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Sync processutils from oslo code  https://review.openstack.org/9834209:25
*** rameshg871 has quit IRC09:27
lifelesslucasagomes: so I think thats a bit of an antipattern actually09:28
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Use temporary dir for image conversion  https://review.openstack.org/9833809:28
lifelesslucasagomes: it plays badly with multiple clients, concurrency and requires significant api side complexty09:29
lifelesslucasagomes: if instead of thinking of the API as a 'do X for me' API and instead a 'make it like X for me' API, I think its clearer09:32
lucasagomeslifeless, right, do we have any etherpad for the API work? I might start one so we can put/discuss the ideas there before we put it in the spec review (might be quicker)09:32
lucasagomeslifeless, like the target state we have?09:32
lifelessbecause - for instance, 'do X' requires locking other people out of changing the operation until it completes09:32
lucasagomesI like that, I think it's a clear path between the current and desided state of the resources09:32
lifelessbut 'make it like X' can be changed at any point09:32
lifelessand the server just takes the next opportunity to move in the right direction09:33
lifelessyes, there's an ether pad09:33
*** GheRivero has quit IRC09:33
lifelesshttps://etherpad.openstack.org/p/ironic-and-fragile-hardware09:33
* lucasagomes clicks09:33
*** rameshg87 has joined #openstack-ironic09:34
lucasagomeslifeless, sounds good so far, I pointed to devananda last time as well... we might want to work the periodic tasks a bit as part of this work, right now they all run in a single greenthread one after the other09:37
lucasagomeslifeless, this might be a problem if we are going to depend a lot on them to maintain everything sync'd09:37
*** igordcard has quit IRC09:42
Kai14hi Ironic. I'm have a controller vm with ironic. I try to deploy another vm on my host using pxe_ssh driver. Ironic powers on this vm. But unfortunately there is not reply to the DHCP/Boot.09:45
Kai14I checked with tcpdump and the DHCP/Boot request comes through the network interfaces of my controller vm. Also dnsmasq is running09:46
lifelesslucasagomes: we'll want to tune that for sure09:49
lucasagomeslifeless, and you see it's going directly to oslo, or something local in Ironic for now? ( I was thinking about having a multi greenthreaded periodic-task like in Ironic after the heartbeat problem #93083)09:50
lifelessI think this is ironic specific to strt with09:51
*** max_lobur has joined #openstack-ironic09:51
*** Mikhail_D_wk has quit IRC10:11
*** Mikhail_D_wk has joined #openstack-ironic10:15
rameshg87lucasagomes, dtantsur, since you had done the review before, request you to have a look at that patch for ilodriver https://review.openstack.org/#/c/89500/ :-)10:18
*** GheRivero has joined #openstack-ironic10:19
*** vinbs has quit IRC10:19
*** athomas has joined #openstack-ironic10:24
*** pradipta is now known as pradipta_away10:32
*** romcheg1 has joined #openstack-ironic10:37
*** romcheg has quit IRC10:39
*** coolsvap is now known as coolsvap|afk10:39
dtantsur|afkg'afternoon, Ironic, TGIF!10:43
*** dtantsur|afk is now known as dtantsur10:43
*** nosnos has quit IRC10:45
*** rameshg87 has quit IRC10:52
Mikhail_D_wkGood afternoon dtantsur!11:01
dtantsurMikhail_D_wk, morning :)11:01
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Reuse validate_instance_and_node() Nova ironic Driver  https://review.openstack.org/9837211:03
lucasagomes:( 97757 still hanging on the queue, probably won't get merged today11:08
lucasagomesdamn11:09
dtantsurlucasagomes, that's insane. Hope when I get back from PTO, it will be solved :)11:12
lucasagomesdtantsur, it will!11:12
*** matsuhashi has quit IRC11:16
*** Alexei_9871 has joined #openstack-ironic11:17
*** matsuhashi has joined #openstack-ironic11:21
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported  https://review.openstack.org/9810211:29
*** lucasagomes is now known as lucas-hungry11:33
*** matsuhashi has quit IRC11:36
*** matsuhashi has joined #openstack-ironic11:36
*** matsuhashi has quit IRC11:40
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported  https://review.openstack.org/9810211:43
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported  https://review.openstack.org/9810211:46
*** max_lobur has quit IRC11:52
*** romcheg has joined #openstack-ironic11:59
*** romcheg1 has quit IRC12:01
dtantsurFolks, want your opinion on this: I sometimes close bugs, because it's a feature requiring a spec. I do it, because from just a description of bug, it's not clear at all, what and how should be fixed12:04
dtantsurHaving a bug around will make people assume, they can just work on it like they want12:04
dtantsurWhat do you think?12:04
*** athomas has quit IRC12:04
dtantsur(of course I link the author to a spec process page)12:04
Shrewsmorning all12:10
openstackgerritStig Telfer proposed a change to openstack/ironic-specs: Power driver for SNMP-enabled smart PDUs  https://review.openstack.org/9683812:10
*** athomas has joined #openstack-ironic12:15
*** Poornima has quit IRC12:16
dtantsurmorning, Shrews12:26
Shrewso/12:26
*** lucas-hungry is now known as lucasagomes12:30
lucasagomesdtantsur, hey, so it's a bug or a new feature being proposed?12:31
lucasagomesdtantsur, we have the "whislist" from the bug which might be the right status for it12:31
*** godp1301 has joined #openstack-ironic12:31
lucasagomesdtantsur, I mean, the way I see the specs is for big features that would affect many parts of the project (like a blueprint) where a whilist could be something smaller12:34
dtantsurlucasagomes, my issue that I can't set it to confirmed, because, well, one can't confirm it based on short description12:34
lucasagomesdtantsur, right, so you can mark as "incomplete"12:35
dtantsurlucasagomes, ok, let's have an example: https://bugs.launchpad.net/ironic/+bug/132718712:35
dtantsurlucasagomes, by marking it Incomplete I'll force a guy to put the whole spec to a bug12:36
dtantsurlucasagomes, and again will have to set it to triaged, actually approving it without a formal approval process12:36
*** Shrews has quit IRC12:37
lucasagomesdtantsur, right I see it... I agree that one would be better to have in a spec12:39
*** Shrews has joined #openstack-ironic12:39
lucasagomesbecause that would include changes in the driver, in the scheduler12:39
lucasagomeswill impact a couple of areas12:39
romchegNobodyCam: Morning, around?12:39
lucasagomesprobably ironic as well by making ironic able to filter nodes based on those extra attributes12:40
dtantsuryeah, it definitely requires proper rationale and approval12:40
lucasagomesdtantsur, https://bugs.launchpad.net/ironic/+bug/1326656 << nice whislit12:41
lucasagomesdtantsur, but yeah we might go towards having all the features to have to have a spec or something12:42
lucasagomesbut it might be overkill12:42
dtantsuryeah...12:43
dtantsurI'll also talk to devananda about it, of course12:43
lucasagomesdtantsur, cool, yeah we need a consensus on that for sure12:44
*** zul has quit IRC12:45
openstackgerritRakesh H S proposed a change to openstack/ironic: ipmi double bridging functionality  https://review.openstack.org/9577512:46
*** lazy_prince has quit IRC12:48
*** zul has joined #openstack-ironic12:49
*** jbjohnso has joined #openstack-ironic12:58
*** k4n0 has quit IRC13:02
*** rakesh_hs has quit IRC13:02
*** jbjohnso has quit IRC13:09
*** matty_dubs|gone is now known as matty_dubs13:24
*** pradipta_away is now known as pradipta13:25
*** ifarkas has quit IRC13:26
*** ifarkas has joined #openstack-ironic13:27
NobodyCamgood Friday Morning iRonic, says the man making coffee13:28
NobodyCamromcheg: slept in late this morning13:28
NobodyCamwoo hoo 97757 is at hour marker 1413:29
NobodyCambad news is the entire queue is over 40 hours13:31
dtantsurmorning, NobodyCam!13:31
dtantsuryes, hopefully this weekend would be enough to 97757 to reach the top and fail again :)13:31
* dtantsur is optimistic 13:31
NobodyCamgood morning dtantsur13:32
NobodyCamdont say that13:33
NobodyCamhehehe13:33
romchegMorning NobodyCam!13:34
romchegI'm concerned about https://review.openstack.org/#/c/97590/7/ironic/drivers/modules/deploy_utils.py13:34
romchegIt's not checking for the sizes of the partitions to detect whether they were changed13:35
romchegNobodyCam: Is it going to be implemented in an upper layer?13:35
lucasagomesmorning NobodyCam13:35
NobodyCammorning lucasagomes13:37
NobodyCamromcheg: see lifeless's commmetents on 93133 @ Jun 3 1:21 AM13:39
NobodyCamcoffee ready brb13:40
* romcheg is looking13:40
romchegNobodyCam: So as far as I understand we have to detect changes in the disk layout later, right?13:49
NobodyCamromcheg: yep in another patch13:49
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Use temporary dir for image conversion  https://review.openstack.org/9833813:57
*** linggao has joined #openstack-ironic13:59
*** pradipta is now known as pradipta_away14:04
jrolllifeless: heh :)14:12
jrollmornin ironic14:12
*** jistr has quit IRC14:12
Mikhail_D_wk jroll: g'morning! :)14:12
romchegMorning jroll!14:13
*** jgrimm has joined #openstack-ironic14:13
romchegGood %whatever_you_have_there% lifeless :)14:13
*** jistr has joined #openstack-ironic14:14
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported  https://review.openstack.org/9810214:15
NobodyCammorning jroll14:16
NobodyCammorning Mikhail_D_wk14:16
Mikhail_D_wkNobodyCam: morning :)14:16
*** derekh_ is now known as derekh_underscor14:17
*** Lingo__ has joined #openstack-ironic14:19
*** athomas has quit IRC14:21
*** Lingo__ has quit IRC14:23
*** foexle has quit IRC14:23
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Fix AttributeError when calling create_engine()  https://review.openstack.org/9842514:23
jrolllucasagomes: are we actually running those tests somewhere? :/14:28
lucasagomesjroll, just opened a bug about it14:28
lucasagomesjroll, they are not being executed at all14:28
lucasagomesI'm trying to investigate14:28
jrollfun14:28
lucasagomesjroll, https://bugs.launchpad.net/ironic/+bug/132722514:28
lucasagomesyeah14:28
lucasagomes:(14:29
jrollI saw that, just didn't click yet :)14:29
lucasagomesseems the list of dbs is empty14:29
lucasagomesso when I added sqlite to ir manully I got that attributeerror14:29
jrollsee, I thought testr didn't run them by default, and they were just run in the gate14:29
lucasagomesI c, hmm14:29
jrollmmm14:29
*** gilliard has quit IRC14:29
lucasagomesI don't think they would run with that attributeerror there tho14:30
jrollI remember when I put up a migration, I had to do that14:30
jrolland I got them to run14:30
jrollbut didn't get an error14:30
* jroll looks in history14:30
lucasagomesjroll, right, I thought we could run it against sqlite locally14:30
lucasagomesafaicr we did run it before we moved to alembic14:30
lucasagomesromcheg, am I right? ^14:30
jrollahhh14:31
jrollhttps://github.com/openstack/ironic/blob/7735ad404bb2849f5be738c2f8a536b1f817c6b7/ironic/tests/db/sqlalchemy/test_migrations.py14:31
jrollso at that point, it was correct14:31
jrollit must have been changed since14:31
jrollerr, looking at the wrong line14:32
romcheglucasagomes: +14:32
lucasagomesjroll, right, this still broken: https://github.com/openstack/ironic/blob/7735ad404bb2849f5be738c2f8a536b1f817c6b7/ironic/tests/db/sqlalchemy/test_migrations.py#L16814:32
jrollright14:32
lucasagomesromcheg, and it should run against sqllite locally right?14:32
jrollit was introduced here: https://github.com/openstack/ironic/commit/9bbea3bf0040eb1138f9e8a8e719039f79c647de14:33
jrollso yeah, when we moved to oslo.db14:33
romcheglucasagomes: it should14:33
lucasagomesjroll, thanks for the update, so at that point it would run normally with testr? or u tweaked it somehow14:33
lucasagomesromcheg, ack, thank u14:33
jrolllucasagomes: I think I added sqlite to the list of dbs14:33
jrolland I thought it ran14:34
jrollI remember having a hard time figuring out how to make it run14:34
romcheglucasagomes: By "it should" I don't mean "I'm sure it works now" :)14:34
lucasagomesromcheg, heh right yeah I dunno as well14:35
lucasagomesjroll, lol +1, I'm trying still to make it run14:35
jrollidk what happened14:35
lucasagomesjroll, well if I add the sqlite and the address sqlite:// to that list of dbs14:35
lucasagomesit does run locally here now14:35
lucasagomes:/14:35
lucasagomesurgh will look other projects, I think neutron is using alembic14:36
jrollyeah, I think there's a config file you add it to14:36
jrolltests/db/sqlalchemy/test_migrations.conf14:36
lucasagomesjroll, yeah14:36
lucasagomesironic/tests/db/sqlalchemy/test_migrations.conf14:36
lucasagomesexactly14:36
jrollI wonder how long those take to run14:37
jrollprobably pretty quick, yeah? there's no data in the db14:37
lucasagomesyeah seems to be quick enough14:37
jrollcool14:38
jrollso maybe we default that to sqlite14:38
jrollfor local testing14:39
jrollbut would be nice to also run against mysql/postgres in the gate14:39
lucasagomes+1, it's pretty hard to add the migration tests without running it locally14:39
lucasagomeswell now it's too easy because it seems to never run :P so it never fails14:39
lucasagomeshehe14:39
*** athomas has joined #openstack-ironic14:40
jrollheh14:41
*** derekh_underscor is now known as derekh_14:43
*** romcheg1 has joined #openstack-ironic14:53
lucasagomesjroll, https://github.com/openstack/ironic/blob/7735ad404bb2849f5be738c2f8a536b1f817c6b7/ironic/tests/db/sqlalchemy/test_migrations.conf#L414:56
lucasagomesew :/14:56
*** romcheg has quit IRC14:56
jroll:(14:57
lucasagomeswill google more about it14:57
lucasagomescause it seems to run fine when I add sqllite=sqlite:// to that file14:58
romcheg1May we summon viktors here?14:59
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Add a HardwareManager method to erase devices  https://review.openstack.org/9787114:59
*** romcheg1 is now known as romcheg14:59
lucasagomesromcheg, heh can u ask him about the situation with alembic + sqlite?15:00
romchegHe told me that everything he works on works perfectly :D15:00
lucasagomeslol15:00
romchegThis requires some investigation15:00
lucasagomesok15:00
lucasagomesyeah the tests seems to run fine after I added sqlite there15:00
romchegHe's about to enter the room and ask some questions15:01
lucasagomesheh ok thanks15:01
*** viktors has joined #openstack-ironic15:01
romchegviktors: ping15:01
viktorsromcheg: hi!15:01
lucasagomesviktors, hi there15:01
romchegjroll, lucasagomes: could you please describe the problem briefly to viktors?15:01
lucasagomesis this still true? https://github.com/openstack/ironic/blob/7735ad404bb2849f5be738c2f8a536b1f817c6b7/ironic/tests/db/sqlalchemy/test_migrations.conf#L415:02
viktorslucasagomes: yes15:02
lucasagomesviktors, basically we are not running the migration tests locally against sqlite15:02
lucasagomesviktors, right15:02
lucasagomesso that's explain why we are not doing it :(15:02
lucasagomesviktors, I enabled the sqlite at ironic/tests/db/sqlalchemy/test_migrations.conf15:02
lucasagomesand it seems to run fine15:02
lucasagomesbut our migration tests right now is way too simple15:02
lucasagomesjust adding columns to some tables15:03
lucasagomesno unique constraints or anything like that15:03
lucasagomeswhich may cause problems with alembic + sqlite I suppose15:03
viktorslucasagomes: sorry, I'll, be back in a 15 minutes. I'll look at Ironic migrations and tests and will answer to you.15:05
lucasagomesviktors, cool thank you, no hurry. I opened a bug about it and was taking a look https://bugs.launchpad.net/ironic/+bug/132722515:05
lucasagomesviktors, feel free to add some feedback there15:05
romchegNobodyCam: is a bug filed for verifying the disk layout on re-provisioning?15:06
*** lsmola_ has quit IRC15:15
NobodyCamromcheg: I have not done that15:20
romchegI will do that15:20
romchegIf you don't mind15:21
NobodyCamawesome TY...NOT AT ALL15:21
*** openstackgerrit has quit IRC15:22
NobodyCambrb .. morning quick walkies15:26
*** openstackgerrit has joined #openstack-ironic15:29
*** ifarkas has quit IRC15:35
romchegNobodyCam: https://bugs.launchpad.net/ironic/+bug/132726015:37
NobodyCamsweet TY15:38
lucasagomesjroll, forgot to ask before, did the keepalive in a dedicated greenthread worked for u?15:44
jrolllucasagomes: I haven't hit our systems exceptionally hard, but we haven't seen conductors dying, so15:44
lucasagomesjroll, ack15:45
jrollI'll issue a ton of power commands today or early next week15:45
jrollI would guess it helps15:46
lucasagomesthat will15:46
lucasagomescheers for that15:46
jrollnp15:46
jrollI meant I would guess the dedicated greenthread helps, I know testing helps :P15:47
*** Kai14 has quit IRC15:48
*** romcheg has quit IRC15:48
*** romcheg1 has joined #openstack-ironic15:48
lucasagomes:)15:49
*** openstackgerrit has quit IRC15:53
*** jistr has quit IRC15:54
NobodyCamdid the patch bot die, or just really slow today15:54
*** jistr has joined #openstack-ironic15:54
jroll08:53:07              <-- | openstackgerrit (~openstack@review.openstack.org) has quit (Ping timeout: 245 seconds)15:55
NobodyCamdoh15:55
*** jistr has quit IRC16:02
lucasagomeshah16:02
*** jistr has joined #openstack-ironic16:02
*** matty_dubs is now known as matty_dubs|lunch16:04
*** martyntaylor has quit IRC16:09
dtantsurok folks, pto starts for me now :) have a nice weekend!16:09
*** dtantsur is now known as dtantsur|pto16:10
jrollhave fun dtantsur|pto :)16:10
*** zdiN0bot has joined #openstack-ironic16:12
*** ellenh has joined #openstack-ironic16:16
devanandag'morning, all16:18
NobodyCamhey hey morning devananda16:18
devanandadtantsur|pto: ciao!16:18
jrollheya devananda16:18
NobodyCamI just push up a review for 9690216:18
* devananda reads all the scrollback16:19
*** rwsu has joined #openstack-ironic16:21
lucasagomesdevananda, morning16:22
NobodyCamromcheg1: have time to look at the new and improved 93133?16:22
NobodyCammorning lucasagomes (not sure I said hi this morning)16:22
lucasagomesNobodyCam, heh u did :)16:23
lucasagomesNobodyCam, morning again :D16:23
NobodyCamlol I was flooded with corp chat this morning so its all kinda fuzzy16:23
lucasagomes:D16:25
*** mikal has quit IRC16:26
*** mikal has joined #openstack-ironic16:27
*** openstackgerrit has joined #openstack-ironic16:36
*** godp1301 has quit IRC16:37
*** ndipanov has quit IRC16:40
devanandalucasagomes: ooh - we're not running migration tests? :(16:45
lucasagomesdevananda, doesn't seem so :(16:45
lucasagomesdevananda, I found out yesterday when I was adding the unique constraint to the instance_uuid16:45
lucasagomesI had a very clear error there but the tests didn't fail, so I was looking and yeah, it doesn't seem to be running16:46
lucasagomesdevananda, it may be running in the gate tho, with mysql and postgree (will take a look soon)16:46
lucasagomespostgres*16:47
openstackgerritEllen Hui proposed a change to openstack/ironic-python-agent: Tries to advertise valid default IP  https://review.openstack.org/9698016:47
devanandaromcheg1: rebuild should recreate partitions. no need to verify. rebuild--preserve-ephemeral should not touch partitions at all, so again, no need to verify them :)16:48
lucasagomesdevananda, it's a bit long, but u might enjoy the reading: http://fish2.com/ipmi/river.pdf16:50
devanandalucasagomes: yep16:52
lucasagomesdevananda, IPMI is really insecure, like _really_16:52
devanandalucasagomes: i'm gad to see that dan finally published that16:53
devanandalucasagomes: I KNOW ....16:53
devanandaawesome statement: today a complete internet search of a UDP port may be done16:53
devanandain a couple of hours or less16:54
*** matty_dubs|lunch is now known as matty_dubs16:56
lucasagomesheh at the end u'll think "the internet is f*cked" :)16:57
devanandalucasagomes: do you have a summary somewhere of how you're running the migratin tests?16:59
lucasagomesdevananda, locally? u can edit this file here16:59
lucasagomesironic/tests/db/sqlalchemy/test_migrations.conf16:59
lucasagomesand add17:00
lucasagomessqlite=sqlite://17:00
*** derekh_ is now known as derekh_afk17:00
devanandaack17:00
*** comstud is now known as bearhands17:00
* devananda tests with mysql17:01
devanandaIndexError: list index out of range17:04
devananda  File "ironic/tests/db/sqlalchemy/test_migrations.py", line 112, in get_db_connection_info17:05
devanandausing this line in test_migrations.conf: mysql=mysql://ironic/cinori@localhost/test_migrations17:06
devanandabrb17:06
openstackgerritChris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk  https://review.openstack.org/9313317:06
lucasagomesdevananda, :( will try with mysql... I found this as well https://review.openstack.org/#/c/98425/17:06
NobodyCamoh bot back :)17:06
jrolldevananda: shoudln't that be ironic:cinori?17:07
jrollrather than ironic/cinori17:07
bearhandsyes17:08
* lucasagomes lucas-dinner17:12
lucasagomeshave a good night everyone, gotta go out to get some food17:12
*** lucasagomes is now known as lucas-dinner17:13
*** harlowja_away is now known as harlowja17:13
jrollnight lucas17:16
*** godp1301 has joined #openstack-ironic17:20
NobodyCamnight lucas-dinner17:21
*** jistr has quit IRC17:24
*** romcheg1 has quit IRC17:26
*** ellenh has quit IRC17:33
*** Shrews has quit IRC17:41
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Add a HardwareManager method to erase devices  https://review.openstack.org/9787117:41
*** Shrews has joined #openstack-ironic17:41
*** Alexei_9871 has quit IRC17:44
*** viktors is now known as viktors|afk17:44
*** athomas has quit IRC17:45
*** linggao has quit IRC17:51
*** romcheg has joined #openstack-ironic17:55
*** pelix has quit IRC17:57
NobodyCam97757 now at hour marker 1917:57
*** godp13011 has joined #openstack-ironic18:01
devanandajroll: d'oh18:02
devanandathanks18:02
*** godp1301 has quit IRC18:03
jroll:)18:03
*** godp1301 has joined #openstack-ironic18:05
*** godp13011 has quit IRC18:05
*** eghobo has joined #openstack-ironic18:06
devanandawoot. random conflicts when testing with mysql now18:07
NobodyCam:(18:07
devanandayep! awesome18:08
devanandalucas-dinner: your patch works to enable tests - and test_walk_versions passes when tests run in a single thread18:09
devanandalucas-dinner: and when run with >1 thread, tests conflict and blow up18:09
jrollheh18:09
*** Mifta has joined #openstack-ironic18:10
*** romcheg has quit IRC18:10
*** martyntaylor has joined #openstack-ironic18:13
*** Mifta has left #openstack-ironic18:15
*** zdiN0bot has quit IRC18:19
*** zdiN0bot has joined #openstack-ironic18:20
jrollshould the 'rest api impact' section of a spec include methods within vendor_passthru?18:24
devanandajroll: that'd be great18:26
devanandaas yes, that IS a rest api18:26
jrollright, but it's not impacting the existing api18:27
jrollI'll add it, thuogh18:27
jrollthough18:27
devanandaif you're adding to vendor_passthu, you're adding an API18:27
devanandait's a "designated unstandardized" section18:27
jrollsure18:27
devanandaas opposed to an extension, which clearly, if you were adding an extension, you'd document18:27
devanandabut it fills the same functional role18:27
devanandaso yes18:27
*** zdiN0bot has quit IRC18:28
*** zdiN0bot has joined #openstack-ironic18:28
jrollcool18:29
*** romcheg has joined #openstack-ironic18:37
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update Nova driver's list_instance_uuids()  https://review.openstack.org/9826818:42
lucas-dinnerdevananda, :( thanks for the update, yeah I will take a deeper look at the problems there18:49
*** igordcard has joined #openstack-ironic19:10
Shrewshrm, ironic mid-cycle sprint dates might be a bit tough for me19:15
NobodyCam:(19:15
*** ellenh has joined #openstack-ironic19:16
devanandaShrews: that sucks19:20
devanandaShrews: hopefully the tripleo sprint will be in early august and non-overlapping19:21
*** zdiN0bot has quit IRC19:22
Shrewsdevananda: well, i have a family reunion on the 27th in WV. i'd have to drive back that night and spend Monday traveling to OR. so it is possible, just shortened (and a bit exhausting)19:22
*** zdiN0bot has joined #openstack-ironic19:27
devanandalucas-dinner: migration test passes when it doesn't run19:38
devanandaheh19:38
NobodyCamnow that is just not right19:41
NobodyCamhumm anyone seen this error before? https://bugs.launchpad.net/ironic/+bug/132739219:43
*** zdiN0bot has quit IRC19:43
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Skip migrations test_walk_versions instead of pass  https://review.openstack.org/9850419:45
devanandaNobodyCam: I think that's a duplicate bug, but need to step away now.19:49
* devananda lunches19:51
NobodyCamok19:52
NobodyCamenjoy :)19:53
NobodyCam97757 now at 21 hour markr19:57
lifelessmorning19:57
NobodyCamgood morning lifeless19:58
NobodyCamisn't it saturday for you19:58
*** coolsvap|afk has quit IRC20:01
*** JoshNang has quit IRC20:01
*** dividebin has quit IRC20:01
*** coolsvap|afk has joined #openstack-ironic20:01
*** dividehex has joined #openstack-ironic20:02
*** JoshNang has joined #openstack-ironic20:02
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-specs: Add deploy driver for ironic-python-agent  https://review.openstack.org/9850620:05
*** zdiN0bot has joined #openstack-ironic20:11
*** igordcard has quit IRC20:12
*** ellenh has quit IRC20:12
NobodyCamhumm wounder if something like https://review.openstack.org/#/c/96577 would help with that bug20:15
jrollhey y'all, feel free to tear this apart: https://review.openstack.org/9850620:16
jrollI could also use some help there to fix the build bug for 'Unexpected indentation' in the middle of a code block :(20:17
*** zdiN0bot has quit IRC20:18
*** zdiN0bot has joined #openstack-ironic20:26
lifelessNobodyCam: it is20:32
NobodyCam:)20:32
* devananda returns from lunch20:34
devanandalifeless: hi there!20:34
*** zdiN0bot has quit IRC20:36
*** igordcard has joined #openstack-ironic20:36
openstackgerritChris Krelle proposed a change to openstack/ironic: Add retry logic to iscsiadm commands  https://review.openstack.org/9851020:37
lifelessdevananda: hey20:47
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-specs: Add deploy driver for ironic-python-agent  https://review.openstack.org/9850620:48
jrolldevananda: you should check that spec out :)20:50
devanandajroll: \o/20:50
jroll:)20:50
*** godp1301 has quit IRC20:58
*** jgrimm has quit IRC20:58
NobodyCambearhands: around?21:00
*** zdiN0bot has joined #openstack-ironic21:07
NobodyCambbiab21:08
*** ellenh has joined #openstack-ironic21:08
*** matty_dubs is now known as matty_dubs|gone21:09
bearhandsNobodyCam: yeah21:22
*** eghobo has quit IRC21:34
*** eghobo has joined #openstack-ironic21:34
lifelessdevananda: ok so21:39
*** r-daneel has joined #openstack-ironic21:48
*** zdiN0bot has quit IRC21:56
NobodyCamhey hey bearhands upto rebasing 96912 to remove 9744721:56
bearhandsok21:56
bearhandser21:57
bearhandsah right21:57
bearhandssec21:57
*** zdiN0bot has joined #openstack-ironic21:57
NobodyCamawesome Ty21:58
bearhandsoh gosh21:58
bearhandsgerrit is stupid and thinks nothing changed with it21:58
bearhandsso it won't rebase it21:58
bearhandsi mean21:58
bearhandsaccept the rebased patch21:58
bearhandsi'll make a slight change to the commit msg or something21:59
openstackgerritChris Behrens proposed a change to openstack/ironic: Add __init__.py for nova scheduler filters  https://review.openstack.org/9691221:59
bearhandsok.21:59
NobodyCamTY bearhands22:00
bearhandsnp22:01
openstackgerritChris Behrens proposed a change to openstack/ironic: Virt driver logging improvements  https://review.openstack.org/9704722:04
openstackgerritChris Behrens proposed a change to openstack/ironic: Allow overriding the log level for ironicclient  https://review.openstack.org/9704822:04
NobodyCamMikhail_D_wk: your not still here are you?22:07
NobodyCam97757 update hour marker 23.522:17
*** romcheg has quit IRC22:20
*** lucas-dinner has quit IRC22:32
*** romcheg has joined #openstack-ironic22:35
NobodyCambrb22:39
openstackgerritEllen Hui proposed a change to openstack/ironic-python-agent: Tries to advertise valid default IP  https://review.openstack.org/9698022:40
*** lucas-dinner has joined #openstack-ironic22:44
*** sysexit has quit IRC22:47
*** romcheg has quit IRC22:49
*** igordcard has quit IRC22:53
*** r-daneel has quit IRC22:58
*** viktors|afk has quit IRC23:04
*** igordcard has joined #openstack-ironic23:06
*** viktors has joined #openstack-ironic23:06
NobodyCamok heading out23:33
NobodyCamgoing to make long haul driver Sat and Sun so we hit home base Sunday night23:34
NobodyCamgah s/driver/drive/23:34
NobodyCamlol23:34
devanandaNobodyCam: drive safe! lemme know when you're in town :-D23:35
NobodyCam:) sunday night we should be up in Sequim23:36
NobodyCamthey turned on the cable internet today23:36
NobodyCamand we're two minutes from hour marker 25 on 9775723:37
*** zdiN0bot has quit IRC23:53
*** zdiN0bot has joined #openstack-ironic23:59

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