Tuesday, 2014-05-20

*** igor_ has joined #openstack-ironic00:00
*** igor_ has quit IRC00:04
*** eghobo has joined #openstack-ironic00:09
*** romcheg1 has quit IRC00:12
*** eguz has quit IRC00:12
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent  https://review.openstack.org/8479500:33
*** newell_ has quit IRC00:46
*** hipster has joined #openstack-ironic00:47
*** hipster has quit IRC00:51
*** igor_ has joined #openstack-ironic01:00
*** igor_ has quit IRC01:05
*** romcheg has joined #openstack-ironic01:14
*** romcheg has quit IRC01:32
*** romcheg has joined #openstack-ironic01:33
*** eghobo has quit IRC01:35
*** nosnos has joined #openstack-ironic01:47
*** hipster has joined #openstack-ironic01:47
*** foexle_ has joined #openstack-ironic01:49
*** hipster has quit IRC01:51
*** foexle has quit IRC01:52
*** igor_ has joined #openstack-ironic02:01
*** igor_ has quit IRC02:05
openstackgerritChangBo Guo(gcb) proposed a change to openstack/ironic: Fix wrong usage of mock methods  https://review.openstack.org/9353202:06
*** hipster has joined #openstack-ironic02:39
*** foexle_ has quit IRC02:40
*** hipster has quit IRC02:44
*** hipster_ has joined #openstack-ironic02:46
*** igor_ has joined #openstack-ironic03:02
*** nosnos has quit IRC03:07
*** igor_ has quit IRC03:07
*** ramineni has joined #openstack-ironic03:10
*** hipster_ has quit IRC03:23
*** hipster has joined #openstack-ironic03:32
*** coolsvap|afk is now known as coolsvap03:37
openstackgerritChangBo Guo(gcb) proposed a change to openstack/ironic: Don't translate debug level logs  https://review.openstack.org/9329103:39
*** ccit has joined #openstack-ironic03:40
*** killer_prince has quit IRC03:44
*** nosnos has joined #openstack-ironic03:47
*** romcheg has quit IRC03:55
*** romcheg has joined #openstack-ironic03:55
*** hipster has quit IRC03:57
*** romcheg has quit IRC03:59
*** eghobo has joined #openstack-ironic04:00
*** nosnos has quit IRC04:01
*** killer_prince has joined #openstack-ironic04:03
*** rameshg87 has joined #openstack-ironic04:11
*** rameshg87 has left #openstack-ironic04:12
*** hipster has joined #openstack-ironic04:27
*** hipster_ has joined #openstack-ironic04:29
*** nosnos has joined #openstack-ironic04:30
*** hipster has quit IRC04:33
*** hipster_ has quit IRC04:33
*** ccit has quit IRC04:49
*** igor_ has joined #openstack-ironic05:04
*** hipster has joined #openstack-ironic05:07
*** igor_ has quit IRC05:08
*** hipster has quit IRC05:12
*** rameshg87 has joined #openstack-ironic05:17
*** Mikhail_D_ltp has joined #openstack-ironic05:20
*** radsy has quit IRC05:20
*** hipster has joined #openstack-ironic05:47
*** hipster has quit IRC05:52
*** Mikhail_D_ltp has quit IRC06:01
*** igor has joined #openstack-ironic06:04
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/8850806:08
*** igor has quit IRC06:10
*** igor has joined #openstack-ironic06:24
*** igor has quit IRC06:26
*** harlowja is now known as harlowja_away06:28
*** jistr has joined #openstack-ironic06:29
*** hipster has joined #openstack-ironic06:47
*** jcoufal has joined #openstack-ironic06:48
*** hipster has quit IRC06:52
*** yuriyz has joined #openstack-ironic06:54
*** foexle has joined #openstack-ironic07:12
*** eghobo has quit IRC07:20
*** killer_prince has quit IRC07:21
*** igor has joined #openstack-ironic07:28
*** igor has quit IRC07:32
Mikhail_D_wkGood morning Ironic! :)07:38
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 2  https://review.openstack.org/7325607:39
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3  https://review.openstack.org/8653607:39
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid of the newline "\"  https://review.openstack.org/6679307:39
yuriyzmorning Ironic07:40
*** geertj has joined #openstack-ironic07:52
*** derekh_ has joined #openstack-ironic08:00
*** pelix has joined #openstack-ironic08:03
*** killer_prince has joined #openstack-ironic08:17
*** datajerk has joined #openstack-ironic08:18
*** uberj has quit IRC08:21
*** uberj_ has joined #openstack-ironic08:21
*** rwsu has quit IRC08:26
*** igor has joined #openstack-ironic08:28
*** killer_prince has quit IRC08:28
*** killer_prince has joined #openstack-ironic08:30
*** coolsvap is now known as coolsvap|afk08:32
*** igor has quit IRC08:33
*** lazy_prince has joined #openstack-ironic08:34
*** rameshg87 has quit IRC08:34
*** killer_prince has quit IRC08:37
*** lazy_prince is now known as killer_prince08:37
*** lucasagomes has joined #openstack-ironic08:38
*** rameshg87 has joined #openstack-ironic08:52
dtantsurLate morning, Ironic :) Morning, yuriyz, Mikhail_D_wk!08:54
Mikhail_D_wkdtantsur g'morning :)08:55
*** rameshg871 has joined #openstack-ironic09:05
*** rameshg87 has quit IRC09:05
*** athomas has joined #openstack-ironic09:12
agordeevmorning Mikhail_D_wk dtantsur yuriyz and everyone else!09:22
Mikhail_D_wkagordeev morning :)09:24
*** igor has joined #openstack-ironic09:24
*** igor has quit IRC09:28
*** jistr has quit IRC09:29
*** martyntaylor has joined #openstack-ironic09:33
dtantsurmorning, agordeev09:43
*** jistr has joined #openstack-ironic09:45
*** hipster has joined #openstack-ironic09:47
*** hipster has quit IRC09:51
*** coolsvap|afk is now known as coolsvap09:55
*** igor has joined #openstack-ironic09:59
*** geertj has quit IRC10:04
*** killer_prince has quit IRC10:05
*** geertj has joined #openstack-ironic10:08
*** jcoufal has quit IRC10:09
*** jcoufal has joined #openstack-ironic10:09
GheRiveromorning ironic10:24
*** overlayer has joined #openstack-ironic10:31
*** overlayer has quit IRC10:34
*** overlayer has joined #openstack-ironic10:35
Mikhail_D_wk GheRivero morning :)10:37
*** datajerk has quit IRC10:44
*** hipster has joined #openstack-ironic10:47
*** hipster has quit IRC10:52
*** ramineni has quit IRC10:56
*** datajerk has joined #openstack-ironic10:59
*** rameshg871 has left #openstack-ironic11:04
*** rameshg87 has joined #openstack-ironic11:04
*** rameshg87 has left #openstack-ironic11:04
agordeevGheRivero: morning11:29
*** overlayer has quit IRC11:35
*** igordcard has joined #openstack-ironic11:35
*** GheRivero has quit IRC11:39
*** matsuhashi has quit IRC11:41
*** igordcard has quit IRC11:41
*** igordcard has joined #openstack-ironic11:42
*** igordcard has quit IRC11:42
*** igordcard has joined #openstack-ironic11:43
NobodyCamGood morning Ironic, says the man making coffee11:44
dtantsurmorning, NobodyCam11:45
NobodyCammorning dtantsur :)11:46
*** romcheg has joined #openstack-ironic11:47
openstackgerritChris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels  https://review.openstack.org/9313311:47
*** GheRivero has joined #openstack-ironic11:50
*** lucasagomes is now known as lucas-hungry11:53
agordeevNobodyCam: morning :)11:57
NobodyCammorning agordeev :)11:58
*** romcheg has quit IRC12:01
NobodyCamlove the new spec template!!!12:16
*** dhellmann_ is now known as dhellmann12:16
NobodyCambrb12:22
*** coolsvap is now known as coolsvap|afk12:22
*** jdob has joined #openstack-ironic12:22
*** rloo has joined #openstack-ironic12:35
*** geertj has quit IRC12:37
*** geertj has joined #openstack-ironic12:44
*** tatyana has joined #openstack-ironic12:57
*** linggao has joined #openstack-ironic12:57
*** jistr has quit IRC12:59
*** jgrimm has joined #openstack-ironic13:00
*** sseago has quit IRC13:00
*** sseago_ has joined #openstack-ironic13:00
*** jistr has joined #openstack-ironic13:01
*** jbjohnso has joined #openstack-ironic13:02
*** nosnos has quit IRC13:02
*** lucas-hungry is now known as lucasagomes13:07
NobodyCammorning lucasagomes :)13:07
lucasagomesNobodyCam, yo good morning :)13:07
lucasagomeshow r u?13:07
NobodyCamdoing ok.. :-p13:07
NobodyCamhow are you doing ... How was your flight home13:08
lucasagomesall is well... the flight was fine, no delays13:13
lucasagomesbut I still a bit jetlagged13:13
NobodyCam:) I'll bet :)13:13
NobodyCamthats one good thing about driving... time changes come slowly :)13:14
NobodyCamlucasagomes: any chance you have a minute to take a look at 93133 I made some changes based on Deva's comments.13:15
NobodyCamnot 100 % sure I loke what I did but wanted to get some other eyes on it before I change it again13:16
NobodyCam:-p13:16
lucasagomesNobodyCam, true driving's cool :)13:16
lucasagomesNobodyCam, ok, will just finish one thing quickly and take a look at it13:16
lucasagomesNobodyCam, btw, did you consider using the sgdisk thing?13:16
NobodyCamI thought about it. but didn't do it. You think it would be better13:17
NobodyCamwould remove the math bits13:18
lucasagomesNobodyCam, yup would remove the math stuff and the get_dev_block_dev as well13:19
lucasagomesblock_size*13:19
NobodyCamya13:19
lucasagomesit maybe should be a method in the DiskPartitioner class, zap_all() or something like that13:20
NobodyCamwanta thro a comment up for sgdisk and I'll start working on a patch13:22
*** sseago_ has quit IRC13:23
lucasagomesNobodyCam, ack13:23
NobodyCamhttps://bugs.launchpad.net/ubuntu/+source/gdisk/+bug/130390313:23
*** rloo has quit IRC13:23
*** rloo has joined #openstack-ironic13:24
lucasagomes:/ ouch13:24
NobodyCamwill prob hold off at least untill thats fixed13:27
*** dhellmann is now known as dhellmann_13:27
*** rloo has quit IRC13:27
*** rloo has joined #openstack-ironic13:28
*** hipster has joined #openstack-ironic13:29
lucasagomesNobodyCam, yeah :/13:29
NobodyCam:-p13:31
*** hipster has quit IRC13:32
*** matty_dubs|gone is now known as matty_dubs13:32
*** jgrimm has quit IRC13:35
*** jgrimm has joined #openstack-ironic13:36
*** lazy_prince has joined #openstack-ironic13:36
*** sseago_ has joined #openstack-ironic13:36
*** lucasagomes has quit IRC13:40
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space is required to reclaim  https://review.openstack.org/9437113:50
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim  https://review.openstack.org/9437113:51
*** lucasagomes has joined #openstack-ironic13:53
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim  https://review.openstack.org/9437113:57
dtantsurdevananda, ping re my comments to https://review.openstack.org/#/c/79194/13:58
*** ifarkas has quit IRC13:59
NobodyCamlucasagomes: (and anyone else) can you still comment on 93133 :)14:04
dtantsurNobodyCam, I think reference to parted bug can be in a commit message14:05
dtantsurI can hardly believe it's the easiest way to delete labels Oo14:08
dtantsurNobodyCam, you use tune2fs to assign a label, why not using it for deleting/setting to smth random?14:09
*** jistr has quit IRC14:10
*** jistr has joined #openstack-ironic14:11
*** mkerrin has quit IRC14:15
NobodyCamdtantsur: look at eariler patches.. I did.. I switched to DD per comments14:17
lucasagomesNobodyCam, reviewed14:20
dtantsurI can't find _the reason_, but ok. Anyway I find it somewhat strange...14:21
agordeevdtantsur: i think that dd'ing the beginning and the end of the disk is the one of the most bullet-proof methods to completely wipe off the partitioning information at all.14:23
jroll^14:23
agordeevjroll: morning :)14:24
jrollmorning agordeev14:24
jrolland everybody else14:24
jroll:)14:24
GheRiveromorning jroll14:24
dtantsuragordeev, well, ok Oo14:24
dtantsurjroll, morning14:24
*** geertj has quit IRC14:26
*** jcoufal has quit IRC14:33
*** uberj_ is now known as uberj14:35
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Set the correct target versions for the RPC methods  https://review.openstack.org/9437914:38
*** hipster has joined #openstack-ironic14:39
NobodyCammorning jroll GheRivero14:39
NobodyCamty lucasagomes dtantsur :)14:40
NobodyCamlucasagomes: my sister just got here.. I will get a new patch up shortly14:41
dtantsurlucasagomes, are you taking any actions on https://bugs.launchpad.net/ironic/+bug/1306153 ?14:43
*** ifarkas has joined #openstack-ironic14:43
* dtantsur is looking for something to do :)14:43
*** lucasagomes has quit IRC14:44
*** geertj has joined #openstack-ironic14:44
openstackgerritRohan Kanade proposed a change to openstack/ironic: Partially remove hard-coded ID's from testdata  https://review.openstack.org/7699014:47
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim  https://review.openstack.org/9437114:50
*** datajerk has quit IRC14:53
*** derekh_ has quit IRC14:54
lazy_princecan someone please review https://review.openstack.org/#/c/79289/ and https://review.openstack.org/79873/14:55
*** lucasagomes has joined #openstack-ironic14:56
*** mkerrin has joined #openstack-ironic15:01
*** jbjohnso has quit IRC15:10
*** zul has quit IRC15:11
jrolldevananda: ping15:14
jrolldevananda: never mind, sorry15:17
*** foexle has quit IRC15:22
*** jbjohnso has joined #openstack-ironic15:25
*** jistr has quit IRC15:40
*** eghobo has joined #openstack-ironic15:46
*** pradipta has joined #openstack-ironic15:47
*** rloo has quit IRC15:52
*** rloo has joined #openstack-ironic15:52
*** rloo has quit IRC15:52
*** rloo has joined #openstack-ironic15:52
*** rloo has quit IRC16:04
*** rloo has joined #openstack-ironic16:05
*** rloo has quit IRC16:06
*** jbjohnso has quit IRC16:06
*** rloo has joined #openstack-ironic16:06
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim  https://review.openstack.org/9437116:07
*** hipster_ has joined #openstack-ironic16:07
*** hipster has quit IRC16:11
*** pelix has quit IRC16:14
*** jgrimm has quit IRC16:14
*** pelix has joined #openstack-ironic16:14
*** jbjohnso has joined #openstack-ironic16:17
*** rloo has quit IRC16:23
*** rloo has joined #openstack-ironic16:23
*** mkerrin has quit IRC16:23
*** lazy_prince has quit IRC16:26
*** matty_dubs is now known as matty_dubs|lunch16:28
*** rwsu has joined #openstack-ironic16:30
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent  https://review.openstack.org/8479516:30
*** rloo has quit IRC16:31
*** rloo has joined #openstack-ironic16:31
*** rameshg87 has joined #openstack-ironic16:34
*** athomas has quit IRC16:35
*** openstackgerrit has quit IRC16:35
*** openstackgerrit has joined #openstack-ironic16:36
*** newell_ has joined #openstack-ironic16:42
*** pelix has quit IRC16:43
*** rloo has quit IRC16:44
*** rloo has joined #openstack-ironic16:44
*** rloo has quit IRC16:44
*** rloo has joined #openstack-ironic16:45
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Run keepalive in a dedicated thread  https://review.openstack.org/9308316:48
devanandamornin, all16:48
*** rameshg87 has quit IRC16:48
lucasagomesdevananda, morning16:51
*** mkerrin has joined #openstack-ironic16:52
matty_dubs|lunchHowdy devananda16:53
lucasagomesI gotta run, will try to go to http://www.meetup.com/OpenStack-Ireland/events/178774492/ , idk if there's someone else from dublin in the channel that might be interested in going there, there's a lot of people in the channel but feel talks so who knows :)16:55
*** lucasagomes is now known as lucas-away16:56
*** pelix has joined #openstack-ironic16:59
devanandarussell_h: hi! I'm not sure what your message tothe ML means by failure scenarios that shouldn't block the spec. can you clarify?17:02
*** jcoufal has joined #openstack-ironic17:09
jrolldevananda: morning :) I'm working on the spec for utility-ramdisk - where should the existing work be called out? the implementation section?17:11
devanandajroll: "implementation - work items" should describe the discrete things you're doing17:13
jrollcool, thought so17:14
devanandajroll: but i wouldn't put links to the existing code reviews in the spec itself17:14
jrollfor sure17:14
*** matty_dubs|lunch is now known as matty_dubs17:14
jrolldevananda: also, this should discuss the initial lookup / heartbeats / etc, yes?17:14
jrollimplementation details like that17:14
devanandayep17:15
jrollok17:15
jrolltrying to find the line between utility-ramdisk and agent-driver17:15
jrollI guess it will overlap17:15
devanandaah, right17:15
*** harlowja_away is now known as harlowja17:16
devanandaso the agent itself is being developed in its own repo, and for now, that's not necessarily tied to specs17:16
devanandabut the driver is17:16
jrollright17:16
devanandaand so by way of that, the agent is :)17:16
jrollyeah17:16
jrollso I should be writing a spec for https://blueprints.launchpad.net/ironic/+spec/utility-ramdisk17:16
jrollyes?17:16
devanandaI would like to see all the functionality that you plan the driver to support (interfaces, call backs, etc)17:16
jrollalso, that blueprint needs to be edited heavily for scope purposes17:17
devanandawrite a spec for what you guys are working on17:17
jrollsure17:17
devanandawe can easily kill that BP17:17
devanandacause, yea, that's massively over scoped with no detail at all17:17
jrolloh, ok17:17
devanandaexactly why we're moving to a formal design process17:17
jrollI'm going to just kill that one and make an IPA blueprint17:18
jrolland a corresponding driver blueprint17:18
jrollunless that should be all in one?17:18
russell_hdevananda: as an example, when the hash ring implementation landed, if we were doing specs I would have wanted to see something saying "If a conductor dies, others won't automatically take over for it. This will be resolved by implementing spec <some-other-spec>."17:18
devanandaand wow, I haven't seen that since folks started editing it and adding disk partiktioning, volume management, etc...17:18
devanandaLP sucks17:18
devanandaI didn't approve that17:18
jrollyep17:18
russell_hgit ftw17:18
jrollwell, you did, before people changed it17:18
jroll:P17:18
devanandawell, right17:18
devanandaand I didn't know it changed17:19
jrollwhich is why I hate the thing where full specs can be in an etherpad17:19
devanandarussell_h: ah, so an HA section?17:19
jrollit has some sort of versioning, but...17:19
russell_hdevananda: well, I just mean more generally that it is sometimes OK to implement something that has known limitations. I'd like a section to document any of those limitations.17:20
devanandajroll: this is more what you're spec'ing now, IMO - https://blueprints.launchpad.net/ironic/+spec/agent-driver17:20
jrolldevananda: right. so just kind of outline everything there, is what I'm thinking17:21
devanandaisn't nodeless vendor passthru done?17:21
jrollyes17:21
russell_hhmm17:21
* russell_h marks it as such17:21
russell_hweird thing is17:22
russell_his that supposed to happen automatically when you merge a patch with an "Implements"?17:22
devanandathat's only for bugs17:22
devanandaalso, it breaks sometimes17:22
russell_hah, my bad17:22
devanandaBPs have alwyas been manual17:23
devanandaand i haven't been on top of them17:23
russell_hI'll say it again17:23
russell_hgit ftw17:23
devanandaalso - folks keep filing BPs and writing code AT THE SAME TIME.17:23
russell_hyeah, who does that17:23
*** jgrimm has joined #openstack-ironic17:23
russell_hI much prefer to write code first17:23
devanandaexactly17:24
jrolldevananda: one more thing on specs - do you plan to follow nova's method fully? merge to ironic-specs -> update blueprint spec URL -> propose blueprint -> write code -> is cycle target achievable? yes -> approve blueprint17:24
jrollhttps://wiki.openstack.org/wiki/Blueprints#Nova17:24
jrollalso, BP goes through the entire process again if target slips17:24
devanandayea. that makes a lot of sense17:28
devanandacovers cases where things slip, which they do17:28
jrollright, so code needs to happen to approve BP17:28
devanandaand forces at least a quick re-evaluation of "does that thing still make sense" when that happens17:28
jrollok17:28
devanandanot "to approve bp" but to target it to a milestone17:29
*** eghobo has quit IRC17:29
devanandaer, nvm. you're right17:30
devanandaboth code and taret milestone17:30
*** eghobo has joined #openstack-ironic17:30
devanandaalso, having more than just me reviewing these would be awesome17:30
devanandaI'm structuring it so that the spec review team not necessarily the same as code review team17:31
matty_dubsI think that'll be much easier when they're up on Gerrit17:31
devanandamatty_dubs: well - yes :)17:31
devanandawe're waiting on infra to create the repo at the moment17:31
devanandai already have a patch up to do so17:32
matty_dubsYeah. I guess I was kind of just thinking out loud ;)17:32
*** Penick has joined #openstack-ironic17:34
*** igor has quit IRC17:34
*** igor_ has joined #openstack-ironic17:35
*** tatyana has quit IRC17:37
*** martyntaylor has left #openstack-ironic17:39
*** krtaylor has quit IRC17:42
*** jcoufal has quit IRC17:44
*** pelix has quit IRC17:49
*** pradipta is now known as pradipta_away17:52
devanandajroll, russell_h: i talked a bit with folks about swift-temp-urls last week. tldr, it's not really a function of swift -- it's really keystone17:55
devanandaand the keystone ptl would love to see that added to keystoneclient17:55
jrollwell17:55
jrollwe're doing it without keystone, so17:55
devanandaright17:55
russell_hits actually built into swift, but ideally oughtn't be17:55
devanandaexactly17:55
devanandait should be in keystone17:56
devanandabut swift likes to do its own thing sometimes17:56
devanandaso really maybe it should be in oslo17:56
devananda<shrug>17:56
jrollheh17:56
jrolldevananda: is what you're getting at is that we should implement it in keystoneclient?17:57
devanandajroll: that the keystone folks think we should ^17:57
jrollhm17:58
jrollas you may know, rax doesn't use keystone (yet)17:58
devanandaand I think we should avoid implementing the same thing in 3 projects17:58
devanandasure. and swift can run without keystone17:58
russell_hso like the keystone client would hit the swift API?17:58
devanandahence -> oslo17:58
devanandaah. no.17:58
jrollyeah, I can get behind that17:58
devanandathe chunk of code that's generating the URLs, which you guys put into https://review.openstack.org/#/c/81391/21/ironic/common/glance_service/v2/image_service.py17:59
devanandawould instead be either in oslo or ksclient17:59
russell_hoh, I gotcha17:59
jrollright17:59
devanandabut that's not actually /hitting/ keystone17:59
devanandait's just a library17:59
devanandawe'll also want to generate signed URLs for ironic17:59
jrollright, ok17:59
devanandafor callbacks from the agent17:59
devananda(eventuually)18:00
jrollso in oslo, this would just take a url and a key and generate a signed url?18:00
devanandai don't have a particular spec in mind18:00
*** martyntaylor has joined #openstack-ironic18:00
jrollsure18:01
devanandajust something to think about // coordinate with other projects18:01
jrollJoshNang: ^ you should read that blob of conversation18:01
jrollyep18:01
JoshNangjroll: i am :D18:01
jrollthanks deva :)18:01
jrollheh, ok18:01
JoshNangi think it's reasonable. i've had it on my todo list, because i found it odd to not be in glance/swift/somewhere. either oslo or ksclient makes sense and i'll add a blueprint18:04
jrollyeah, I vote oslo18:04
jrolland maybe not swift-specific?18:04
JoshNangdevananda: do you want me to land that code somewhere else before landing the swift patch (which still needs a spec, etc)?18:05
*** hipster_ has quit IRC18:06
*** rloo has quit IRC18:08
*** rloo has joined #openstack-ironic18:08
devanandaJoshNang: yes, swift one needs a spec. I'd prefer to at least modularize it in Ironic so that it's easier to rip out, when it lands somewhere else18:11
devanandaJoshNang: but i'm fine landing it in Ironic with the rest of that feature18:12
JoshNangworks for me!18:12
*** lucas-away has quit IRC18:12
Shrewsdevananda: yo yo yo18:12
Shrewsdevananda: so, i have the tempest rebuild test working. it doesn't test ephemeral preservation yet, though. is that something you think we should do?18:13
Shrewsdevananda: https://review.openstack.org/94436 and https://review.openstack.org/94439, if interested18:13
devanandaShrews: yes, we should18:15
devanandait's a feature we support,. thus, it should be tested18:15
Shrewsk k, cool. interesting, rebuild is not tested by nova18:16
*** coolsvap|afk has quit IRC18:16
*** datajerk has joined #openstack-ironic18:16
*** coolsvap|afk has joined #openstack-ironic18:19
devanandaheh18:20
Shrewshrm, i think i'm going to have to change devstack-gate, first18:20
devanandaoh?18:21
Shrewsso the flavor is created with an ephemeral partition18:21
devanandathat'd be a devstack change, not d-g18:21
Shrewsdevananda: both18:21
*** rloo has quit IRC18:21
Shrewsdevananda: already supports the new env var to define the partition18:21
Shrewsbut gate needs updated to use it18:22
*** rloo has joined #openstack-ironic18:22
devanandaah18:22
Shrewsdevananda: https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n11518:22
devanandamay be worth pointing out to the nova team that they raen't testing rebuild --preserve18:22
Shrewsi can file a bug to them once my changes get into tempest. should be easy for them at that point18:24
SpamapSdevananda: where are we at with preserve ephemeral patches?18:24
SpamapSNobodyCam: ^^ ?18:24
*** lucas-away has joined #openstack-ironic18:25
ShrewsSpamapS: patches? you mean the disk label issue?18:26
*** rameshg87 has joined #openstack-ironic18:26
SpamapSI don't know18:26
SpamapSIt was just patches before18:26
SpamapSmy understanding was we cannot use preserve ephemeral from ironic trunk. Tell me I'm wrong and I'll do an irish jig (gig?).18:27
*** rameshg87 has quit IRC18:28
Shrewsas much as i love watching you dance (particularly jigs), i think the disk label issue NobodyCam is working is still a blocker for devtest, iirc18:28
ShrewsSpamapS: https://review.openstack.org/9313318:29
*** martyntaylor has left #openstack-ironic18:30
Shrewswalk time. biab18:31
*** datajerk has quit IRC18:32
*** datajerk has joined #openstack-ironic18:32
*** lucas-away has quit IRC18:35
devanandaSpamapS: preserve ephemeral itself works. landed nearly a month ago18:37
devanandaSpamapS: but there is a disklabel issue that lifeless identified last week, which we've been iterating on a fix for18:37
lifelesserrata, actually NobodyCam found it I think, but yes.18:38
lifelessdevananda: does rebuild preserve-ephemeral work too ?18:38
lifelessdevananda: IIRC that was still in gerrit18:38
devanandayes18:38
devanandahttps://review.openstack.org/#/c/90429/18:38
devanandagah. wrong timestamp. it landed 2 weeks ago, not a month.18:39
*** krtaylor has joined #openstack-ironic18:40
lifelessawesome18:41
*** geertj has quit IRC18:43
*** urulama has joined #openstack-ironic18:46
*** urulama has left #openstack-ironic18:46
*** lucas-away has joined #openstack-ironic18:49
*** jistr has joined #openstack-ironic19:00
* devananda lunches19:00
openstackgerritChris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels  https://review.openstack.org/9313319:09
NobodyCamhey hey19:10
NobodyCamlatest revision pushed up... of course as always looking for reviews ^^^^19:11
GheRiveroNobodyCam: why are we wiping out the end of the disk? Maybe we don't want to remove the raid config19:31
NobodyCamGheRivero: gpt based labels19:32
*** hipster has joined #openstack-ironic19:33
*** hipster has quit IRC19:33
*** hipster has joined #openstack-ironic19:34
*** geertj has joined #openstack-ironic19:34
*** annegent^ has quit IRC19:37
*** annegentle_ has joined #openstack-ironic19:40
*** datajerk has quit IRC19:40
GheRiveroI'm a bit concern about deleting some raids COD (configuration of disk) Never tried before19:44
NobodyCamhummm19:47
NobodyCamlifeless: do you have thoughts on GheRivero's concerns ^^^^19:48
lifelessGheRivero: we're repartitioning anyway, no ?19:51
lifelessGheRivero: the pre-parted code wrote a new partition table19:51
devanandaGheRivero: software RAID? Ironic doesn't support that. I'm not aware that HW RAID would be affected (but IMBW)19:54
lifelesshardware raid will be fine19:55
lifelessGheRivero: so wiping the partition table and not wiping the end of the disk is invalid *because* of software raid/lvm - its basically a cheap full disk erase19:56
NobodyCamgah SpamapS didn't see your comments.. Will fix them  with next push up19:56
SpamapSNobodyCam: I made my comments after you pushed19:59
NobodyCam:) hehehe19:59
openstackgerritChris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels  https://review.openstack.org/9313320:02
NobodyCamSpamapS: :) all fixed up20:02
matty_dubsSo, umm, maybe this is a fixed issue, but it appears that the ironic CLI has _no_ timeout?20:04
matty_dubsI logged into a box and ran "ironic node-list", then forgot about it and just came back, like an hour later.20:05
matty_dubsIt was still going. It turns out that ironic-api isn't running.20:05
*** Penick has quit IRC20:06
SpamapSmatty_dubs: "resiliency"20:06
NobodyCammatty_dubs: that sounds like it should be another error not a timeout20:07
openstackgerritJarrod Johnson proposed a change to stackforge/pyghmi: Deregister timeout on broken  https://review.openstack.org/9446320:07
matty_dubsYeah, agreed20:07
*** Penick has joined #openstack-ironic20:08
*** jistr has quit IRC20:08
matty_dubsI haven't updated this in a while, so I'm not positive it's a current issue.20:08
NobodyCam:) i'll have to test that one20:10
matty_dubsI'll file a bug unless you don't hit it20:11
*** Penick has quit IRC20:11
NobodyCamspinning up test now... give me a few min20:11
matty_dubsOh, no rush at all20:12
openstackgerritA change was merged to openstack/python-ironicclient: removed py3kcompat module  https://review.openstack.org/9351720:14
openstackgerritA change was merged to stackforge/pyghmi: Deregister timeout on broken  https://review.openstack.org/9446320:14
*** Penick has joined #openstack-ironic20:14
NobodyCammatty_dubs: easy to test just going to stop the api and issue a node-list20:15
matty_dubsIf I stop Keystone, that returns an error quickly. But with Keystone up and Ironic down...20:18
NobodyCamwith just service ironic-api stop I get error 111 connection refused right away20:21
matty_dubsAh, good thing you checked!20:22
matty_dubsMy setup must be... special.20:22
NobodyCamhehehe :-p20:22
*** Penick has quit IRC20:27
*** krtaylor has quit IRC20:29
NobodyCambrb20:39
*** shakamunyi has joined #openstack-ironic20:46
NobodyCamok I'm going to head out to go see my dad. I will address any thing on 93133 when I return20:54
matty_dubsSee ya, NobodyCam20:56
NobodyCam:)20:58
*** jdob has quit IRC21:02
*** Penick has joined #openstack-ironic21:05
*** jbjohnso has quit IRC21:22
*** geertj has quit IRC21:22
*** ifarkas has quit IRC21:30
*** krtaylor has joined #openstack-ironic21:31
*** newell_ has quit IRC21:42
*** foexle has joined #openstack-ironic21:44
openstackgerritRussell Haering proposed a change to openstack/ironic-python-agent: Don't begin heartbeating until the API server is listening  https://review.openstack.org/9448221:47
*** lucas-away has quit IRC21:51
*** mrda-summit is now known as mrda21:53
mrdaMorning ironic!21:53
russell_hmrda: "morning" :)21:57
mrdarussell_h: I'm not quite back in this TZ anyways :)21:58
russell_hheh21:58
russell_hat this point I'm not really in any timezone21:58
*** harlowja is now known as harlowja_away21:59
*** lucas-away has joined #openstack-ironic22:04
*** Penick has quit IRC22:04
*** linggao has quit IRC22:06
*** Penick has joined #openstack-ironic22:06
*** Penick has quit IRC22:08
*** harlowja_away is now known as harlowja22:08
*** Penick has joined #openstack-ironic22:10
*** Penick has quit IRC22:15
*** Penick has joined #openstack-ironic22:28
*** radsy has joined #openstack-ironic22:29
*** igordcard has quit IRC22:59
*** jgrimm has quit IRC23:06
*** eghobo has quit IRC23:12
*** eghobo has joined #openstack-ironic23:22
*** eguz has joined #openstack-ironic23:35
*** eguz has quit IRC23:37
*** eguz has joined #openstack-ironic23:37
*** Penick has quit IRC23:38
*** eghobo has quit IRC23:39
*** lucas-away has quit IRC23:49
*** foexle has quit IRC23:51
devanandacatching a bus then getting dinner ... bbl23:57
devanandag'night all!23:57

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