*** igor_ has joined #openstack-ironic | 00:00 | |
*** igor_ has quit IRC | 00:04 | |
*** eghobo has joined #openstack-ironic | 00:09 | |
*** romcheg1 has quit IRC | 00:12 | |
*** eguz has quit IRC | 00:12 | |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 00:33 |
---|---|---|
*** newell_ has quit IRC | 00:46 | |
*** hipster has joined #openstack-ironic | 00:47 | |
*** hipster has quit IRC | 00:51 | |
*** igor_ has joined #openstack-ironic | 01:00 | |
*** igor_ has quit IRC | 01:05 | |
*** romcheg has joined #openstack-ironic | 01:14 | |
*** romcheg has quit IRC | 01:32 | |
*** romcheg has joined #openstack-ironic | 01:33 | |
*** eghobo has quit IRC | 01:35 | |
*** nosnos has joined #openstack-ironic | 01:47 | |
*** hipster has joined #openstack-ironic | 01:47 | |
*** foexle_ has joined #openstack-ironic | 01:49 | |
*** hipster has quit IRC | 01:51 | |
*** foexle has quit IRC | 01:52 | |
*** igor_ has joined #openstack-ironic | 02:01 | |
*** igor_ has quit IRC | 02:05 | |
openstackgerrit | ChangBo Guo(gcb) proposed a change to openstack/ironic: Fix wrong usage of mock methods https://review.openstack.org/93532 | 02:06 |
*** hipster has joined #openstack-ironic | 02:39 | |
*** foexle_ has quit IRC | 02:40 | |
*** hipster has quit IRC | 02:44 | |
*** hipster_ has joined #openstack-ironic | 02:46 | |
*** igor_ has joined #openstack-ironic | 03:02 | |
*** nosnos has quit IRC | 03:07 | |
*** igor_ has quit IRC | 03:07 | |
*** ramineni has joined #openstack-ironic | 03:10 | |
*** hipster_ has quit IRC | 03:23 | |
*** hipster has joined #openstack-ironic | 03:32 | |
*** coolsvap|afk is now known as coolsvap | 03:37 | |
openstackgerrit | ChangBo Guo(gcb) proposed a change to openstack/ironic: Don't translate debug level logs https://review.openstack.org/93291 | 03:39 |
*** ccit has joined #openstack-ironic | 03:40 | |
*** killer_prince has quit IRC | 03:44 | |
*** nosnos has joined #openstack-ironic | 03:47 | |
*** romcheg has quit IRC | 03:55 | |
*** romcheg has joined #openstack-ironic | 03:55 | |
*** hipster has quit IRC | 03:57 | |
*** romcheg has quit IRC | 03:59 | |
*** eghobo has joined #openstack-ironic | 04:00 | |
*** nosnos has quit IRC | 04:01 | |
*** killer_prince has joined #openstack-ironic | 04:03 | |
*** rameshg87 has joined #openstack-ironic | 04:11 | |
*** rameshg87 has left #openstack-ironic | 04:12 | |
*** hipster has joined #openstack-ironic | 04:27 | |
*** hipster_ has joined #openstack-ironic | 04:29 | |
*** nosnos has joined #openstack-ironic | 04:30 | |
*** hipster has quit IRC | 04:33 | |
*** hipster_ has quit IRC | 04:33 | |
*** ccit has quit IRC | 04:49 | |
*** igor_ has joined #openstack-ironic | 05:04 | |
*** hipster has joined #openstack-ironic | 05:07 | |
*** igor_ has quit IRC | 05:08 | |
*** hipster has quit IRC | 05:12 | |
*** rameshg87 has joined #openstack-ironic | 05:17 | |
*** Mikhail_D_ltp has joined #openstack-ironic | 05:20 | |
*** radsy has quit IRC | 05:20 | |
*** hipster has joined #openstack-ironic | 05:47 | |
*** hipster has quit IRC | 05:52 | |
*** Mikhail_D_ltp has quit IRC | 06:01 | |
*** igor has joined #openstack-ironic | 06:04 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/88508 | 06:08 |
*** igor has quit IRC | 06:10 | |
*** igor has joined #openstack-ironic | 06:24 | |
*** igor has quit IRC | 06:26 | |
*** harlowja is now known as harlowja_away | 06:28 | |
*** jistr has joined #openstack-ironic | 06:29 | |
*** hipster has joined #openstack-ironic | 06:47 | |
*** jcoufal has joined #openstack-ironic | 06:48 | |
*** hipster has quit IRC | 06:52 | |
*** yuriyz has joined #openstack-ironic | 06:54 | |
*** foexle has joined #openstack-ironic | 07:12 | |
*** eghobo has quit IRC | 07:20 | |
*** killer_prince has quit IRC | 07:21 | |
*** igor has joined #openstack-ironic | 07:28 | |
*** igor has quit IRC | 07:32 | |
Mikhail_D_wk | Good morning Ironic! :) | 07:38 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 2 https://review.openstack.org/73256 | 07:39 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3 https://review.openstack.org/86536 | 07:39 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid of the newline "\" https://review.openstack.org/66793 | 07:39 |
yuriyz | morning Ironic | 07:40 |
*** geertj has joined #openstack-ironic | 07:52 | |
*** derekh_ has joined #openstack-ironic | 08:00 | |
*** pelix has joined #openstack-ironic | 08:03 | |
*** killer_prince has joined #openstack-ironic | 08:17 | |
*** datajerk has joined #openstack-ironic | 08:18 | |
*** uberj has quit IRC | 08:21 | |
*** uberj_ has joined #openstack-ironic | 08:21 | |
*** rwsu has quit IRC | 08:26 | |
*** igor has joined #openstack-ironic | 08:28 | |
*** killer_prince has quit IRC | 08:28 | |
*** killer_prince has joined #openstack-ironic | 08:30 | |
*** coolsvap is now known as coolsvap|afk | 08:32 | |
*** igor has quit IRC | 08:33 | |
*** lazy_prince has joined #openstack-ironic | 08:34 | |
*** rameshg87 has quit IRC | 08:34 | |
*** killer_prince has quit IRC | 08:37 | |
*** lazy_prince is now known as killer_prince | 08:37 | |
*** lucasagomes has joined #openstack-ironic | 08:38 | |
*** rameshg87 has joined #openstack-ironic | 08:52 | |
dtantsur | Late morning, Ironic :) Morning, yuriyz, Mikhail_D_wk! | 08:54 |
Mikhail_D_wk | dtantsur g'morning :) | 08:55 |
*** rameshg871 has joined #openstack-ironic | 09:05 | |
*** rameshg87 has quit IRC | 09:05 | |
*** athomas has joined #openstack-ironic | 09:12 | |
agordeev | morning Mikhail_D_wk dtantsur yuriyz and everyone else! | 09:22 |
Mikhail_D_wk | agordeev morning :) | 09:24 |
*** igor has joined #openstack-ironic | 09:24 | |
*** igor has quit IRC | 09:28 | |
*** jistr has quit IRC | 09:29 | |
*** martyntaylor has joined #openstack-ironic | 09:33 | |
dtantsur | morning, agordeev | 09:43 |
*** jistr has joined #openstack-ironic | 09:45 | |
*** hipster has joined #openstack-ironic | 09:47 | |
*** hipster has quit IRC | 09:51 | |
*** coolsvap|afk is now known as coolsvap | 09:55 | |
*** igor has joined #openstack-ironic | 09:59 | |
*** geertj has quit IRC | 10:04 | |
*** killer_prince has quit IRC | 10:05 | |
*** geertj has joined #openstack-ironic | 10:08 | |
*** jcoufal has quit IRC | 10:09 | |
*** jcoufal has joined #openstack-ironic | 10:09 | |
GheRivero | morning ironic | 10:24 |
*** overlayer has joined #openstack-ironic | 10:31 | |
*** overlayer has quit IRC | 10:34 | |
*** overlayer has joined #openstack-ironic | 10:35 | |
Mikhail_D_wk | GheRivero morning :) | 10:37 |
*** datajerk has quit IRC | 10:44 | |
*** hipster has joined #openstack-ironic | 10:47 | |
*** hipster has quit IRC | 10:52 | |
*** ramineni has quit IRC | 10:56 | |
*** datajerk has joined #openstack-ironic | 10:59 | |
*** rameshg871 has left #openstack-ironic | 11:04 | |
*** rameshg87 has joined #openstack-ironic | 11:04 | |
*** rameshg87 has left #openstack-ironic | 11:04 | |
agordeev | GheRivero: morning | 11:29 |
*** overlayer has quit IRC | 11:35 | |
*** igordcard has joined #openstack-ironic | 11:35 | |
*** GheRivero has quit IRC | 11:39 | |
*** matsuhashi has quit IRC | 11:41 | |
*** igordcard has quit IRC | 11:41 | |
*** igordcard has joined #openstack-ironic | 11:42 | |
*** igordcard has quit IRC | 11:42 | |
*** igordcard has joined #openstack-ironic | 11:43 | |
NobodyCam | Good morning Ironic, says the man making coffee | 11:44 |
dtantsur | morning, NobodyCam | 11:45 |
NobodyCam | morning dtantsur :) | 11:46 |
*** romcheg has joined #openstack-ironic | 11:47 | |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 11:47 |
*** GheRivero has joined #openstack-ironic | 11:50 | |
*** lucasagomes is now known as lucas-hungry | 11:53 | |
agordeev | NobodyCam: morning :) | 11:57 |
NobodyCam | morning agordeev :) | 11:58 |
*** romcheg has quit IRC | 12:01 | |
NobodyCam | love the new spec template!!! | 12:16 |
*** dhellmann_ is now known as dhellmann | 12:16 | |
NobodyCam | brb | 12:22 |
*** coolsvap is now known as coolsvap|afk | 12:22 | |
*** jdob has joined #openstack-ironic | 12:22 | |
*** rloo has joined #openstack-ironic | 12:35 | |
*** geertj has quit IRC | 12:37 | |
*** geertj has joined #openstack-ironic | 12:44 | |
*** tatyana has joined #openstack-ironic | 12:57 | |
*** linggao has joined #openstack-ironic | 12:57 | |
*** jistr has quit IRC | 12:59 | |
*** jgrimm has joined #openstack-ironic | 13:00 | |
*** sseago has quit IRC | 13:00 | |
*** sseago_ has joined #openstack-ironic | 13:00 | |
*** jistr has joined #openstack-ironic | 13:01 | |
*** jbjohnso has joined #openstack-ironic | 13:02 | |
*** nosnos has quit IRC | 13:02 | |
*** lucas-hungry is now known as lucasagomes | 13:07 | |
NobodyCam | morning lucasagomes :) | 13:07 |
lucasagomes | NobodyCam, yo good morning :) | 13:07 |
lucasagomes | how r u? | 13:07 |
NobodyCam | doing ok.. :-p | 13:07 |
NobodyCam | how are you doing ... How was your flight home | 13:08 |
lucasagomes | all is well... the flight was fine, no delays | 13:13 |
lucasagomes | but I still a bit jetlagged | 13:13 |
NobodyCam | :) I'll bet :) | 13:13 |
NobodyCam | thats one good thing about driving... time changes come slowly :) | 13:14 |
NobodyCam | lucasagomes: any chance you have a minute to take a look at 93133 I made some changes based on Deva's comments. | 13:15 |
NobodyCam | not 100 % sure I loke what I did but wanted to get some other eyes on it before I change it again | 13:16 |
NobodyCam | :-p | 13:16 |
lucasagomes | NobodyCam, true driving's cool :) | 13:16 |
lucasagomes | NobodyCam, ok, will just finish one thing quickly and take a look at it | 13:16 |
lucasagomes | NobodyCam, btw, did you consider using the sgdisk thing? | 13:16 |
NobodyCam | I thought about it. but didn't do it. You think it would be better | 13:17 |
NobodyCam | would remove the math bits | 13:18 |
lucasagomes | NobodyCam, yup would remove the math stuff and the get_dev_block_dev as well | 13:19 |
lucasagomes | block_size* | 13:19 |
NobodyCam | ya | 13:19 |
lucasagomes | it maybe should be a method in the DiskPartitioner class, zap_all() or something like that | 13:20 |
NobodyCam | wanta thro a comment up for sgdisk and I'll start working on a patch | 13:22 |
*** sseago_ has quit IRC | 13:23 | |
lucasagomes | NobodyCam, ack | 13:23 |
NobodyCam | https://bugs.launchpad.net/ubuntu/+source/gdisk/+bug/1303903 | 13:23 |
*** rloo has quit IRC | 13:23 | |
*** rloo has joined #openstack-ironic | 13:24 | |
lucasagomes | :/ ouch | 13:24 |
NobodyCam | will prob hold off at least untill thats fixed | 13:27 |
*** dhellmann is now known as dhellmann_ | 13:27 | |
*** rloo has quit IRC | 13:27 | |
*** rloo has joined #openstack-ironic | 13:28 | |
*** hipster has joined #openstack-ironic | 13:29 | |
lucasagomes | NobodyCam, yeah :/ | 13:29 |
NobodyCam | :-p | 13:31 |
*** hipster has quit IRC | 13:32 | |
*** matty_dubs|gone is now known as matty_dubs | 13:32 | |
*** jgrimm has quit IRC | 13:35 | |
*** jgrimm has joined #openstack-ironic | 13:36 | |
*** lazy_prince has joined #openstack-ironic | 13:36 | |
*** sseago_ has joined #openstack-ironic | 13:36 | |
*** lucasagomes has quit IRC | 13:40 | |
openstackgerrit | Dmitry 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/94371 | 13:50 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim https://review.openstack.org/94371 | 13:51 |
*** lucasagomes has joined #openstack-ironic | 13:53 | |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim https://review.openstack.org/94371 | 13:57 |
dtantsur | devananda, ping re my comments to https://review.openstack.org/#/c/79194/ | 13:58 |
*** ifarkas has quit IRC | 13:59 | |
NobodyCam | lucasagomes: (and anyone else) can you still comment on 93133 :) | 14:04 |
dtantsur | NobodyCam, I think reference to parted bug can be in a commit message | 14:05 |
dtantsur | I can hardly believe it's the easiest way to delete labels Oo | 14:08 |
dtantsur | NobodyCam, you use tune2fs to assign a label, why not using it for deleting/setting to smth random? | 14:09 |
*** jistr has quit IRC | 14:10 | |
*** jistr has joined #openstack-ironic | 14:11 | |
*** mkerrin has quit IRC | 14:15 | |
NobodyCam | dtantsur: look at eariler patches.. I did.. I switched to DD per comments | 14:17 |
lucasagomes | NobodyCam, reviewed | 14:20 |
dtantsur | I can't find _the reason_, but ok. Anyway I find it somewhat strange... | 14:21 |
agordeev | dtantsur: 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 |
agordeev | jroll: morning :) | 14:24 |
jroll | morning agordeev | 14:24 |
jroll | and everybody else | 14:24 |
jroll | :) | 14:24 |
GheRivero | morning jroll | 14:24 |
dtantsur | agordeev, well, ok Oo | 14:24 |
dtantsur | jroll, morning | 14:24 |
*** geertj has quit IRC | 14:26 | |
*** jcoufal has quit IRC | 14:33 | |
*** uberj_ is now known as uberj | 14:35 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Set the correct target versions for the RPC methods https://review.openstack.org/94379 | 14:38 |
*** hipster has joined #openstack-ironic | 14:39 | |
NobodyCam | morning jroll GheRivero | 14:39 |
NobodyCam | ty lucasagomes dtantsur :) | 14:40 |
NobodyCam | lucasagomes: my sister just got here.. I will get a new patch up shortly | 14:41 |
dtantsur | lucasagomes, are you taking any actions on https://bugs.launchpad.net/ironic/+bug/1306153 ? | 14:43 |
*** ifarkas has joined #openstack-ironic | 14:43 | |
* dtantsur is looking for something to do :) | 14:43 | |
*** lucasagomes has quit IRC | 14:44 | |
*** geertj has joined #openstack-ironic | 14:44 | |
openstackgerrit | Rohan Kanade proposed a change to openstack/ironic: Partially remove hard-coded ID's from testdata https://review.openstack.org/76990 | 14:47 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim https://review.openstack.org/94371 | 14:50 |
*** datajerk has quit IRC | 14:53 | |
*** derekh_ has quit IRC | 14:54 | |
lazy_prince | can someone please review https://review.openstack.org/#/c/79289/ and https://review.openstack.org/79873/ | 14:55 |
*** lucasagomes has joined #openstack-ironic | 14:56 | |
*** mkerrin has joined #openstack-ironic | 15:01 | |
*** jbjohnso has quit IRC | 15:10 | |
*** zul has quit IRC | 15:11 | |
jroll | devananda: ping | 15:14 |
jroll | devananda: never mind, sorry | 15:17 |
*** foexle has quit IRC | 15:22 | |
*** jbjohnso has joined #openstack-ironic | 15:25 | |
*** jistr has quit IRC | 15:40 | |
*** eghobo has joined #openstack-ironic | 15:46 | |
*** pradipta has joined #openstack-ironic | 15:47 | |
*** rloo has quit IRC | 15:52 | |
*** rloo has joined #openstack-ironic | 15:52 | |
*** rloo has quit IRC | 15:52 | |
*** rloo has joined #openstack-ironic | 15:52 | |
*** rloo has quit IRC | 16:04 | |
*** rloo has joined #openstack-ironic | 16:05 | |
*** rloo has quit IRC | 16:06 | |
*** jbjohnso has quit IRC | 16:06 | |
*** rloo has joined #openstack-ironic | 16:06 | |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: PXE to pass hints to ImageCache on how much space to reclaim https://review.openstack.org/94371 | 16:07 |
*** hipster_ has joined #openstack-ironic | 16:07 | |
*** hipster has quit IRC | 16:11 | |
*** pelix has quit IRC | 16:14 | |
*** jgrimm has quit IRC | 16:14 | |
*** pelix has joined #openstack-ironic | 16:14 | |
*** jbjohnso has joined #openstack-ironic | 16:17 | |
*** rloo has quit IRC | 16:23 | |
*** rloo has joined #openstack-ironic | 16:23 | |
*** mkerrin has quit IRC | 16:23 | |
*** lazy_prince has quit IRC | 16:26 | |
*** matty_dubs is now known as matty_dubs|lunch | 16:28 | |
*** rwsu has joined #openstack-ironic | 16:30 | |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 16:30 |
*** rloo has quit IRC | 16:31 | |
*** rloo has joined #openstack-ironic | 16:31 | |
*** rameshg87 has joined #openstack-ironic | 16:34 | |
*** athomas has quit IRC | 16:35 | |
*** openstackgerrit has quit IRC | 16:35 | |
*** openstackgerrit has joined #openstack-ironic | 16:36 | |
*** newell_ has joined #openstack-ironic | 16:42 | |
*** pelix has quit IRC | 16:43 | |
*** rloo has quit IRC | 16:44 | |
*** rloo has joined #openstack-ironic | 16:44 | |
*** rloo has quit IRC | 16:44 | |
*** rloo has joined #openstack-ironic | 16:45 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Run keepalive in a dedicated thread https://review.openstack.org/93083 | 16:48 |
devananda | mornin, all | 16:48 |
*** rameshg87 has quit IRC | 16:48 | |
lucasagomes | devananda, morning | 16:51 |
*** mkerrin has joined #openstack-ironic | 16:52 | |
matty_dubs|lunch | Howdy devananda | 16:53 |
lucasagomes | I 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-away | 16:56 | |
*** pelix has joined #openstack-ironic | 16:59 | |
devananda | russell_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-ironic | 17:09 | |
jroll | devananda: morning :) I'm working on the spec for utility-ramdisk - where should the existing work be called out? the implementation section? | 17:11 |
devananda | jroll: "implementation - work items" should describe the discrete things you're doing | 17:13 |
jroll | cool, thought so | 17:14 |
devananda | jroll: but i wouldn't put links to the existing code reviews in the spec itself | 17:14 |
jroll | for sure | 17:14 |
*** matty_dubs|lunch is now known as matty_dubs | 17:14 | |
jroll | devananda: also, this should discuss the initial lookup / heartbeats / etc, yes? | 17:14 |
jroll | implementation details like that | 17:14 |
devananda | yep | 17:15 |
jroll | ok | 17:15 |
jroll | trying to find the line between utility-ramdisk and agent-driver | 17:15 |
jroll | I guess it will overlap | 17:15 |
devananda | ah, right | 17:15 |
*** harlowja_away is now known as harlowja | 17:16 | |
devananda | so the agent itself is being developed in its own repo, and for now, that's not necessarily tied to specs | 17:16 |
devananda | but the driver is | 17:16 |
jroll | right | 17:16 |
devananda | and so by way of that, the agent is :) | 17:16 |
jroll | yeah | 17:16 |
jroll | so I should be writing a spec for https://blueprints.launchpad.net/ironic/+spec/utility-ramdisk | 17:16 |
jroll | yes? | 17:16 |
devananda | I would like to see all the functionality that you plan the driver to support (interfaces, call backs, etc) | 17:16 |
jroll | also, that blueprint needs to be edited heavily for scope purposes | 17:17 |
devananda | write a spec for what you guys are working on | 17:17 |
jroll | sure | 17:17 |
devananda | we can easily kill that BP | 17:17 |
devananda | cause, yea, that's massively over scoped with no detail at all | 17:17 |
jroll | oh, ok | 17:17 |
devananda | exactly why we're moving to a formal design process | 17:17 |
jroll | I'm going to just kill that one and make an IPA blueprint | 17:18 |
jroll | and a corresponding driver blueprint | 17:18 |
jroll | unless that should be all in one? | 17:18 |
russell_h | devananda: 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 |
devananda | and wow, I haven't seen that since folks started editing it and adding disk partiktioning, volume management, etc... | 17:18 |
devananda | LP sucks | 17:18 |
devananda | I didn't approve that | 17:18 |
jroll | yep | 17:18 |
russell_h | git ftw | 17:18 |
jroll | well, you did, before people changed it | 17:18 |
jroll | :P | 17:18 |
devananda | well, right | 17:18 |
devananda | and I didn't know it changed | 17:19 |
jroll | which is why I hate the thing where full specs can be in an etherpad | 17:19 |
devananda | russell_h: ah, so an HA section? | 17:19 |
jroll | it has some sort of versioning, but... | 17:19 |
russell_h | devananda: 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 |
devananda | jroll: this is more what you're spec'ing now, IMO - https://blueprints.launchpad.net/ironic/+spec/agent-driver | 17:20 |
jroll | devananda: right. so just kind of outline everything there, is what I'm thinking | 17:21 |
devananda | isn't nodeless vendor passthru done? | 17:21 |
jroll | yes | 17:21 |
russell_h | hmm | 17:21 |
* russell_h marks it as such | 17:21 | |
russell_h | weird thing is | 17:22 |
russell_h | is that supposed to happen automatically when you merge a patch with an "Implements"? | 17:22 |
devananda | that's only for bugs | 17:22 |
devananda | also, it breaks sometimes | 17:22 |
russell_h | ah, my bad | 17:22 |
devananda | BPs have alwyas been manual | 17:23 |
devananda | and i haven't been on top of them | 17:23 |
russell_h | I'll say it again | 17:23 |
russell_h | git ftw | 17:23 |
devananda | also - folks keep filing BPs and writing code AT THE SAME TIME. | 17:23 |
russell_h | yeah, who does that | 17:23 |
*** jgrimm has joined #openstack-ironic | 17:23 | |
russell_h | I much prefer to write code first | 17:23 |
devananda | exactly | 17:24 |
jroll | devananda: 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 blueprint | 17:24 |
jroll | https://wiki.openstack.org/wiki/Blueprints#Nova | 17:24 |
jroll | also, BP goes through the entire process again if target slips | 17:24 |
devananda | yea. that makes a lot of sense | 17:28 |
devananda | covers cases where things slip, which they do | 17:28 |
jroll | right, so code needs to happen to approve BP | 17:28 |
devananda | and forces at least a quick re-evaluation of "does that thing still make sense" when that happens | 17:28 |
jroll | ok | 17:28 |
devananda | not "to approve bp" but to target it to a milestone | 17:29 |
*** eghobo has quit IRC | 17:29 | |
devananda | er, nvm. you're right | 17:30 |
devananda | both code and taret milestone | 17:30 |
*** eghobo has joined #openstack-ironic | 17:30 | |
devananda | also, having more than just me reviewing these would be awesome | 17:30 |
devananda | I'm structuring it so that the spec review team not necessarily the same as code review team | 17:31 |
matty_dubs | I think that'll be much easier when they're up on Gerrit | 17:31 |
devananda | matty_dubs: well - yes :) | 17:31 |
devananda | we're waiting on infra to create the repo at the moment | 17:31 |
devananda | i already have a patch up to do so | 17:32 |
matty_dubs | Yeah. I guess I was kind of just thinking out loud ;) | 17:32 |
*** Penick has joined #openstack-ironic | 17:34 | |
*** igor has quit IRC | 17:34 | |
*** igor_ has joined #openstack-ironic | 17:35 | |
*** tatyana has quit IRC | 17:37 | |
*** martyntaylor has left #openstack-ironic | 17:39 | |
*** krtaylor has quit IRC | 17:42 | |
*** jcoufal has quit IRC | 17:44 | |
*** pelix has quit IRC | 17:49 | |
*** pradipta is now known as pradipta_away | 17:52 | |
devananda | jroll, 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 keystone | 17:55 |
devananda | and the keystone ptl would love to see that added to keystoneclient | 17:55 |
jroll | well | 17:55 |
jroll | we're doing it without keystone, so | 17:55 |
devananda | right | 17:55 |
russell_h | its actually built into swift, but ideally oughtn't be | 17:55 |
devananda | exactly | 17:55 |
devananda | it should be in keystone | 17:56 |
devananda | but swift likes to do its own thing sometimes | 17:56 |
devananda | so really maybe it should be in oslo | 17:56 |
devananda | <shrug> | 17:56 |
jroll | heh | 17:56 |
jroll | devananda: is what you're getting at is that we should implement it in keystoneclient? | 17:57 |
devananda | jroll: that the keystone folks think we should ^ | 17:57 |
jroll | hm | 17:58 |
jroll | as you may know, rax doesn't use keystone (yet) | 17:58 |
devananda | and I think we should avoid implementing the same thing in 3 projects | 17:58 |
devananda | sure. and swift can run without keystone | 17:58 |
russell_h | so like the keystone client would hit the swift API? | 17:58 |
devananda | hence -> oslo | 17:58 |
devananda | ah. no. | 17:58 |
jroll | yeah, I can get behind that | 17:58 |
devananda | the 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.py | 17:59 |
devananda | would instead be either in oslo or ksclient | 17:59 |
russell_h | oh, I gotcha | 17:59 |
jroll | right | 17:59 |
devananda | but that's not actually /hitting/ keystone | 17:59 |
devananda | it's just a library | 17:59 |
devananda | we'll also want to generate signed URLs for ironic | 17:59 |
jroll | right, ok | 17:59 |
devananda | for callbacks from the agent | 17:59 |
devananda | (eventuually) | 18:00 |
jroll | so in oslo, this would just take a url and a key and generate a signed url? | 18:00 |
devananda | i don't have a particular spec in mind | 18:00 |
*** martyntaylor has joined #openstack-ironic | 18:00 | |
jroll | sure | 18:01 |
devananda | just something to think about // coordinate with other projects | 18:01 |
jroll | JoshNang: ^ you should read that blob of conversation | 18:01 |
jroll | yep | 18:01 |
JoshNang | jroll: i am :D | 18:01 |
jroll | thanks deva :) | 18:01 |
jroll | heh, ok | 18:01 |
JoshNang | i 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 blueprint | 18:04 |
jroll | yeah, I vote oslo | 18:04 |
jroll | and maybe not swift-specific? | 18:04 |
JoshNang | devananda: 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 IRC | 18:06 | |
*** rloo has quit IRC | 18:08 | |
*** rloo has joined #openstack-ironic | 18:08 | |
devananda | JoshNang: 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 else | 18:11 |
devananda | JoshNang: but i'm fine landing it in Ironic with the rest of that feature | 18:12 |
JoshNang | works for me! | 18:12 |
*** lucas-away has quit IRC | 18:12 | |
Shrews | devananda: yo yo yo | 18:12 |
Shrews | devananda: 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 |
Shrews | devananda: https://review.openstack.org/94436 and https://review.openstack.org/94439, if interested | 18:13 |
devananda | Shrews: yes, we should | 18:15 |
devananda | it's a feature we support,. thus, it should be tested | 18:15 |
Shrews | k k, cool. interesting, rebuild is not tested by nova | 18:16 |
*** coolsvap|afk has quit IRC | 18:16 | |
*** datajerk has joined #openstack-ironic | 18:16 | |
*** coolsvap|afk has joined #openstack-ironic | 18:19 | |
devananda | heh | 18:20 |
Shrews | hrm, i think i'm going to have to change devstack-gate, first | 18:20 |
devananda | oh? | 18:21 |
Shrews | so the flavor is created with an ephemeral partition | 18:21 |
devananda | that'd be a devstack change, not d-g | 18:21 |
Shrews | devananda: both | 18:21 |
*** rloo has quit IRC | 18:21 | |
Shrews | devananda: already supports the new env var to define the partition | 18:21 |
Shrews | but gate needs updated to use it | 18:22 |
*** rloo has joined #openstack-ironic | 18:22 | |
devananda | ah | 18:22 |
Shrews | devananda: https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n115 | 18:22 |
devananda | may be worth pointing out to the nova team that they raen't testing rebuild --preserve | 18:22 |
Shrews | i can file a bug to them once my changes get into tempest. should be easy for them at that point | 18:24 |
SpamapS | devananda: where are we at with preserve ephemeral patches? | 18:24 |
SpamapS | NobodyCam: ^^ ? | 18:24 |
*** lucas-away has joined #openstack-ironic | 18:25 | |
Shrews | SpamapS: patches? you mean the disk label issue? | 18:26 |
*** rameshg87 has joined #openstack-ironic | 18:26 | |
SpamapS | I don't know | 18:26 |
SpamapS | It was just patches before | 18:26 |
SpamapS | my 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 IRC | 18:28 | |
Shrews | as much as i love watching you dance (particularly jigs), i think the disk label issue NobodyCam is working is still a blocker for devtest, iirc | 18:28 |
Shrews | SpamapS: https://review.openstack.org/93133 | 18:29 |
*** martyntaylor has left #openstack-ironic | 18:30 | |
Shrews | walk time. biab | 18:31 |
*** datajerk has quit IRC | 18:32 | |
*** datajerk has joined #openstack-ironic | 18:32 | |
*** lucas-away has quit IRC | 18:35 | |
devananda | SpamapS: preserve ephemeral itself works. landed nearly a month ago | 18:37 |
devananda | SpamapS: but there is a disklabel issue that lifeless identified last week, which we've been iterating on a fix for | 18:37 |
lifeless | errata, actually NobodyCam found it I think, but yes. | 18:38 |
lifeless | devananda: does rebuild preserve-ephemeral work too ? | 18:38 |
lifeless | devananda: IIRC that was still in gerrit | 18:38 |
devananda | yes | 18:38 |
devananda | https://review.openstack.org/#/c/90429/ | 18:38 |
devananda | gah. wrong timestamp. it landed 2 weeks ago, not a month. | 18:39 |
*** krtaylor has joined #openstack-ironic | 18:40 | |
lifeless | awesome | 18:41 |
*** geertj has quit IRC | 18:43 | |
*** urulama has joined #openstack-ironic | 18:46 | |
*** urulama has left #openstack-ironic | 18:46 | |
*** lucas-away has joined #openstack-ironic | 18:49 | |
*** jistr has joined #openstack-ironic | 19:00 | |
* devananda lunches | 19:00 | |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 19:09 |
NobodyCam | hey hey | 19:10 |
NobodyCam | latest revision pushed up... of course as always looking for reviews ^^^^ | 19:11 |
GheRivero | NobodyCam: why are we wiping out the end of the disk? Maybe we don't want to remove the raid config | 19:31 |
NobodyCam | GheRivero: gpt based labels | 19:32 |
*** hipster has joined #openstack-ironic | 19:33 | |
*** hipster has quit IRC | 19:33 | |
*** hipster has joined #openstack-ironic | 19:34 | |
*** geertj has joined #openstack-ironic | 19:34 | |
*** annegent^ has quit IRC | 19:37 | |
*** annegentle_ has joined #openstack-ironic | 19:40 | |
*** datajerk has quit IRC | 19:40 | |
GheRivero | I'm a bit concern about deleting some raids COD (configuration of disk) Never tried before | 19:44 |
NobodyCam | hummm | 19:47 |
NobodyCam | lifeless: do you have thoughts on GheRivero's concerns ^^^^ | 19:48 |
lifeless | GheRivero: we're repartitioning anyway, no ? | 19:51 |
lifeless | GheRivero: the pre-parted code wrote a new partition table | 19:51 |
devananda | GheRivero: software RAID? Ironic doesn't support that. I'm not aware that HW RAID would be affected (but IMBW) | 19:54 |
lifeless | hardware raid will be fine | 19:55 |
lifeless | GheRivero: 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 erase | 19:56 |
NobodyCam | gah SpamapS didn't see your comments.. Will fix them with next push up | 19:56 |
SpamapS | NobodyCam: I made my comments after you pushed | 19:59 |
NobodyCam | :) hehehe | 19:59 |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 20:02 |
NobodyCam | SpamapS: :) all fixed up | 20:02 |
matty_dubs | So, umm, maybe this is a fixed issue, but it appears that the ironic CLI has _no_ timeout? | 20:04 |
matty_dubs | I logged into a box and ran "ironic node-list", then forgot about it and just came back, like an hour later. | 20:05 |
matty_dubs | It was still going. It turns out that ironic-api isn't running. | 20:05 |
*** Penick has quit IRC | 20:06 | |
SpamapS | matty_dubs: "resiliency" | 20:06 |
NobodyCam | matty_dubs: that sounds like it should be another error not a timeout | 20:07 |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Deregister timeout on broken https://review.openstack.org/94463 | 20:07 |
matty_dubs | Yeah, agreed | 20:07 |
*** Penick has joined #openstack-ironic | 20:08 | |
*** jistr has quit IRC | 20:08 | |
matty_dubs | I 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 one | 20:10 |
matty_dubs | I'll file a bug unless you don't hit it | 20:11 |
*** Penick has quit IRC | 20:11 | |
NobodyCam | spinning up test now... give me a few min | 20:11 |
matty_dubs | Oh, no rush at all | 20:12 |
openstackgerrit | A change was merged to openstack/python-ironicclient: removed py3kcompat module https://review.openstack.org/93517 | 20:14 |
openstackgerrit | A change was merged to stackforge/pyghmi: Deregister timeout on broken https://review.openstack.org/94463 | 20:14 |
*** Penick has joined #openstack-ironic | 20:14 | |
NobodyCam | matty_dubs: easy to test just going to stop the api and issue a node-list | 20:15 |
matty_dubs | If I stop Keystone, that returns an error quickly. But with Keystone up and Ironic down... | 20:18 |
NobodyCam | with just service ironic-api stop I get error 111 connection refused right away | 20:21 |
matty_dubs | Ah, good thing you checked! | 20:22 |
matty_dubs | My setup must be... special. | 20:22 |
NobodyCam | hehehe :-p | 20:22 |
*** Penick has quit IRC | 20:27 | |
*** krtaylor has quit IRC | 20:29 | |
NobodyCam | brb | 20:39 |
*** shakamunyi has joined #openstack-ironic | 20:46 | |
NobodyCam | ok I'm going to head out to go see my dad. I will address any thing on 93133 when I return | 20:54 |
matty_dubs | See ya, NobodyCam | 20:56 |
NobodyCam | :) | 20:58 |
*** jdob has quit IRC | 21:02 | |
*** Penick has joined #openstack-ironic | 21:05 | |
*** jbjohnso has quit IRC | 21:22 | |
*** geertj has quit IRC | 21:22 | |
*** ifarkas has quit IRC | 21:30 | |
*** krtaylor has joined #openstack-ironic | 21:31 | |
*** newell_ has quit IRC | 21:42 | |
*** foexle has joined #openstack-ironic | 21:44 | |
openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Don't begin heartbeating until the API server is listening https://review.openstack.org/94482 | 21:47 |
*** lucas-away has quit IRC | 21:51 | |
*** mrda-summit is now known as mrda | 21:53 | |
mrda | Morning ironic! | 21:53 |
russell_h | mrda: "morning" :) | 21:57 |
mrda | russell_h: I'm not quite back in this TZ anyways :) | 21:58 |
russell_h | heh | 21:58 |
russell_h | at this point I'm not really in any timezone | 21:58 |
*** harlowja is now known as harlowja_away | 21:59 | |
*** lucas-away has joined #openstack-ironic | 22:04 | |
*** Penick has quit IRC | 22:04 | |
*** linggao has quit IRC | 22:06 | |
*** Penick has joined #openstack-ironic | 22:06 | |
*** Penick has quit IRC | 22:08 | |
*** harlowja_away is now known as harlowja | 22:08 | |
*** Penick has joined #openstack-ironic | 22:10 | |
*** Penick has quit IRC | 22:15 | |
*** Penick has joined #openstack-ironic | 22:28 | |
*** radsy has joined #openstack-ironic | 22:29 | |
*** igordcard has quit IRC | 22:59 | |
*** jgrimm has quit IRC | 23:06 | |
*** eghobo has quit IRC | 23:12 | |
*** eghobo has joined #openstack-ironic | 23:22 | |
*** eguz has joined #openstack-ironic | 23:35 | |
*** eguz has quit IRC | 23:37 | |
*** eguz has joined #openstack-ironic | 23:37 | |
*** Penick has quit IRC | 23:38 | |
*** eghobo has quit IRC | 23:39 | |
*** lucas-away has quit IRC | 23:49 | |
*** foexle has quit IRC | 23:51 | |
devananda | catching a bus then getting dinner ... bbl | 23:57 |
devananda | g'night all! | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!