| harlowja | devananda interesting little open discussion there at the end | 00:04 |
|---|---|---|
| harlowja | peoples brains thinking about what to do, thats a plus | 00:05 |
| harlowja | its a challenging one thats for sure to figure out the best way to go from here to somewhere else (and will the somewhere else be any better) | 00:06 |
| *** Haomeng|2 has joined #openstack-ironic | 00:14 | |
| *** Penick has quit IRC | 00:15 | |
| *** Haomeng has quit IRC | 00:15 | |
| *** max_lobur has quit IRC | 00:46 | |
| *** dkehnx1 has quit IRC | 00:48 | |
| openstackgerrit | Michael Davies proposed a change to openstack/python-ironicclient: Expose auth_ref in ironicclient client object https://review.openstack.org/104467 | 00:50 |
| jerryz | could anyone help on a booting problem? the machine hung at booting from hard disk, btw, i was using heat to deploy undercloud. | 01:26 |
| *** eghobo has quit IRC | 01:38 | |
| *** killer_prince has quit IRC | 01:41 | |
| *** nosnos has joined #openstack-ironic | 01:44 | |
| *** eghobo has joined #openstack-ironic | 02:16 | |
| *** killer_prince has joined #openstack-ironic | 02:20 | |
| *** killer_prince is now known as lazy_prince | 02:20 | |
| *** harlowja is now known as harlowja_away | 02:50 | |
| *** ramineni has joined #openstack-ironic | 03:00 | |
| *** shardy has quit IRC | 03:01 | |
| *** vinbs has joined #openstack-ironic | 03:02 | |
| *** vinbs_ has joined #openstack-ironic | 03:04 | |
| *** vinbs has quit IRC | 03:07 | |
| *** vinbs_ is now known as vinbs | 03:07 | |
| *** lazy_prince is now known as killer_prince | 03:09 | |
| *** eghobo has quit IRC | 03:17 | |
| *** nosnos has quit IRC | 03:20 | |
| *** killer_prince is now known as lazy_prince | 03:21 | |
| *** stendulker has joined #openstack-ironic | 03:47 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/ironic: Use my_ip for neutron URL https://review.openstack.org/104738 | 04:10 |
| *** nosnos has joined #openstack-ironic | 04:13 | |
| *** nosnos has quit IRC | 04:24 | |
| *** sabah has joined #openstack-ironic | 04:25 | |
| *** amitpp has joined #openstack-ironic | 04:32 | |
| *** stendulker has quit IRC | 04:33 | |
| *** amitpp has quit IRC | 04:34 | |
| *** amitpp has joined #openstack-ironic | 04:38 | |
| *** sseago__ has quit IRC | 04:40 | |
| *** eghobo has joined #openstack-ironic | 04:42 | |
| *** max_lobur has joined #openstack-ironic | 04:46 | |
| *** bmahalakshmi has joined #openstack-ironic | 05:15 | |
| *** bmahalakshmi has quit IRC | 05:29 | |
| *** bmahalakshmi has joined #openstack-ironic | 05:30 | |
| *** bvivek has joined #openstack-ironic | 05:31 | |
| *** lazy_prince is now known as killer_prince | 05:32 | |
| *** romcheg has joined #openstack-ironic | 05:34 | |
| *** romcheg has quit IRC | 05:35 | |
| *** romcheg has joined #openstack-ironic | 05:35 | |
| *** rakesh_hs has joined #openstack-ironic | 05:36 | |
| *** romcheg has quit IRC | 05:36 | |
| *** sabah has quit IRC | 05:36 | |
| *** sabah has joined #openstack-ironic | 05:36 | |
| *** vinbs has quit IRC | 05:37 | |
| *** sabah has quit IRC | 05:37 | |
| *** sabah has joined #openstack-ironic | 05:38 | |
| *** vinbs has joined #openstack-ironic | 05:38 | |
| *** subah has joined #openstack-ironic | 05:39 | |
| *** sabah has quit IRC | 05:39 | |
| *** chuckC has quit IRC | 05:41 | |
| *** max_lobur has quit IRC | 05:47 | |
| *** vinbs has quit IRC | 05:49 | |
| *** vinbs has joined #openstack-ironic | 05:49 | |
| *** rameshg87_afk is now known as rameshg87 | 05:50 | |
| *** sabah has joined #openstack-ironic | 05:50 | |
| *** subah has quit IRC | 05:50 | |
| *** coolsvap|afk is now known as coolsvap | 05:51 | |
| *** Nisha has joined #openstack-ironic | 05:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/105359 | 06:02 |
| *** geekyogi has joined #openstack-ironic | 06:04 | |
| *** bmahalakshmi has quit IRC | 06:07 | |
| *** sabah has quit IRC | 06:07 | |
| *** sabah has joined #openstack-ironic | 06:07 | |
| *** bmahalakshmi has joined #openstack-ironic | 06:08 | |
| *** subah has joined #openstack-ironic | 06:10 | |
| *** sabah has quit IRC | 06:10 | |
| *** Isotopp has quit IRC | 06:10 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-update https://review.openstack.org/100951 | 06:10 |
| *** Isotopp has joined #openstack-ironic | 06:11 | |
| *** Isotopp has quit IRC | 06:11 | |
| *** Isotopp has joined #openstack-ironic | 06:11 | |
| *** bmahalakshmi has quit IRC | 06:15 | |
| *** bmahalakshmi has joined #openstack-ironic | 06:15 | |
| *** sabah has joined #openstack-ironic | 06:17 | |
| *** subah has quit IRC | 06:17 | |
| *** romcheg has joined #openstack-ironic | 06:18 | |
| *** shausy has joined #openstack-ironic | 06:36 | |
| *** shausy has quit IRC | 06:41 | |
| *** shausy has joined #openstack-ironic | 06:44 | |
| *** mkerrin has joined #openstack-ironic | 06:49 | |
| *** jcoufal has joined #openstack-ironic | 06:55 | |
| *** geekyogi has quit IRC | 06:56 | |
| *** jistr has joined #openstack-ironic | 07:01 | |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Implement the Conductor side of ring rebalancing https://review.openstack.org/79194 | 07:05 |
| *** eghobo has quit IRC | 07:17 | |
| *** eghobo has joined #openstack-ironic | 07:18 | |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Implement the Conductor side of ring rebalancing https://review.openstack.org/79194 | 07:20 |
| mrda | 'night ironic | 07:28 |
| *** mrda is now known as mrda-away | 07:28 | |
| *** eguz has joined #openstack-ironic | 07:41 | |
| *** eguz has quit IRC | 07:43 | |
| *** bvivek has quit IRC | 07:44 | |
| *** sabah has quit IRC | 07:44 | |
| *** sabah has joined #openstack-ironic | 07:44 | |
| *** bvivek has joined #openstack-ironic | 07:44 | |
| *** eghobo has quit IRC | 07:45 | |
| *** geekyogi has joined #openstack-ironic | 07:46 | |
| *** rakesh_hs has quit IRC | 07:47 | |
| *** sabah has quit IRC | 07:47 | |
| *** sabah has joined #openstack-ironic | 07:48 | |
| *** sabah has quit IRC | 07:49 | |
| *** sabah has joined #openstack-ironic | 07:49 | |
| *** rakesh_hs has joined #openstack-ironic | 07:50 | |
| *** bvivek has quit IRC | 07:51 | |
| *** overlayer has joined #openstack-ironic | 07:58 | |
| *** rakesh_hs has quit IRC | 07:59 | |
| *** sabah has quit IRC | 07:59 | |
| *** sabah has joined #openstack-ironic | 07:59 | |
| *** rakesh_hs has joined #openstack-ironic | 08:00 | |
| *** pcrews has quit IRC | 08:04 | |
| *** martyntaylor has joined #openstack-ironic | 08:05 | |
| *** lazy_prince has joined #openstack-ironic | 08:09 | |
| *** lazy_prince has quit IRC | 08:10 | |
| *** shausy has quit IRC | 08:10 | |
| *** killer_prince is now known as lazy_prince | 08:10 | |
| *** subah has joined #openstack-ironic | 08:11 | |
| *** derekh_ has joined #openstack-ironic | 08:11 | |
| *** sabah has quit IRC | 08:14 | |
| *** lucasagomes has joined #openstack-ironic | 08:28 | |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Remove aliases `arg` and `env` from utils https://review.openstack.org/104553 | 08:30 |
| *** overlayer has quit IRC | 08:37 | |
| *** lazy_prince is now known as killer_prince | 08:56 | |
| *** romcheg has quit IRC | 09:03 | |
| *** shausy has joined #openstack-ironic | 09:10 | |
| *** romcheg has joined #openstack-ironic | 09:12 | |
| *** killer_prince is now known as lazy_prince | 09:12 | |
| *** bvivek has joined #openstack-ironic | 09:12 | |
| *** pelix has joined #openstack-ironic | 09:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix nodes left in an incosistent state if no workers https://review.openstack.org/100958 | 09:20 |
| rameshg87 | lucasagomes: good morning | 09:23 |
| rameshg87 | lucasagomes: request you to take a look at the ilo power code for which spec was approved :-) - https://review.openstack.org/#/c/89500/ | 09:24 |
| openstackgerrit | A change was merged to openstack/ironic-specs: Swift Temporary URLs Spec https://review.openstack.org/98904 | 09:28 |
| *** romcheg has quit IRC | 09:28 | |
| *** martyntaylor has quit IRC | 09:37 | |
| *** rakesh_hs has quit IRC | 09:40 | |
| *** rakesh_hs has joined #openstack-ironic | 09:40 | |
| *** aswadr has joined #openstack-ironic | 09:43 | |
| *** lazy_prince has quit IRC | 09:44 | |
| *** martyntaylor has joined #openstack-ironic | 09:50 | |
| *** Nisha has quit IRC | 09:57 | |
| *** overlayer has joined #openstack-ironic | 10:01 | |
| *** killer_prince has joined #openstack-ironic | 10:03 | |
| *** killer_prince is now known as lazy_prince | 10:03 | |
| *** jistr has quit IRC | 10:06 | |
| *** jerryz has quit IRC | 10:21 | |
| *** Haomeng has joined #openstack-ironic | 10:26 | |
| *** Haomeng|2 has quit IRC | 10:27 | |
| *** jistr has joined #openstack-ironic | 10:27 | |
| *** bvivek has quit IRC | 10:29 | |
| *** bvivek has joined #openstack-ironic | 10:33 | |
| *** bvivek has quit IRC | 10:41 | |
| *** Shrews has quit IRC | 10:43 | |
| *** romcheg has joined #openstack-ironic | 10:47 | |
| *** Shrews has joined #openstack-ironic | 10:47 | |
| *** bmahalakshmi has quit IRC | 11:01 | |
| *** ramineni has quit IRC | 11:03 | |
| *** subah has quit IRC | 11:03 | |
| *** lazy_prince has quit IRC | 11:08 | |
| *** romcheg has quit IRC | 11:18 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:19 | |
| *** Haomeng has quit IRC | 11:21 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic-specs: New driver ManagementInterface https://review.openstack.org/100218 | 11:30 |
| *** killer_prince has joined #openstack-ironic | 11:41 | |
| *** killer_prince is now known as lazy_prince | 11:41 | |
| *** Haomeng has joined #openstack-ironic | 11:50 | |
| *** Haomeng|2 has quit IRC | 11:51 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:55 | |
| *** Haomeng has quit IRC | 11:55 | |
| *** jistr is now known as jistr|english | 11:56 | |
| *** shausy has quit IRC | 11:59 | |
| *** Poornima has joined #openstack-ironic | 12:04 | |
| *** vinbs has quit IRC | 12:06 | |
| *** sseago has joined #openstack-ironic | 12:08 | |
| *** jdob has joined #openstack-ironic | 12:09 | |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| *** jistr|english is now known as jistr | 12:11 | |
| *** rakesh_hs has quit IRC | 12:17 | |
| *** romcheg has joined #openstack-ironic | 12:26 | |
| romcheg | Morning Ironic! | 12:30 |
| romcheg | lucas-hungry: what was that yesterday? :( | 12:30 |
| *** jdob has quit IRC | 12:37 | |
| *** jdob has joined #openstack-ironic | 12:38 | |
| Shrews | morning romcheg, everyone | 12:43 |
| romcheg | Morning Shrews! | 12:44 |
| *** chuckC has joined #openstack-ironic | 12:49 | |
| *** lucas-hungry is now known as lucasagomes | 12:55 | |
| lucasagomes | romcheg, lol man | 12:59 |
| lucasagomes | I don't know what to say | 12:59 |
| *** d0ugal has joined #openstack-ironic | 13:05 | |
| *** Alexei_987 has quit IRC | 13:05 | |
| Shrews | grr... it should be a gerrit-enforced rule that if you vote -1 on a review that your IRC nick must be posted in your comment | 13:10 |
| Shrews | so i can hunt you down and tell you how wrong you are | 13:10 |
| *** Alexei_987 has joined #openstack-ironic | 13:10 | |
| * Shrews might have had too much caffeine already | 13:11 | |
| *** zul has quit IRC | 13:11 | |
| * NobodyCam wanders in and thinks is there such a thing as "Too Much Caffeine" | 13:12 | |
| NobodyCam | Good morning iRonic | 13:12 |
| *** ndipanov has quit IRC | 13:12 | |
| Shrews | ooh, we've gone all Apple-ish... i like it | 13:12 |
| NobodyCam | lol | 13:13 |
| Shrews | NobodyCam: clearly, for some people, such as myself, there IS such a thing :) | 13:13 |
| NobodyCam | :) | 13:13 |
| Shrews | i walk a tight rope between sanity and insanity | 13:13 |
| *** rloo has joined #openstack-ironic | 13:14 | |
| * NobodyCam has decided that sanity is not all its cracked up to be :-p | 13:15 | |
| *** ndipanov has joined #openstack-ironic | 13:16 | |
| lucasagomes | NobodyCam, when you get some time can you take a look at the mgmt interface spec? | 13:16 |
| NobodyCam | sure, | 13:17 |
| lucasagomes | ta much | 13:17 |
| lucasagomes | ah ops, morning NobodyCam Shrews | 13:17 |
| NobodyCam | hehehe mornign lucasagomes | 13:17 |
| NobodyCam | morning even | 13:18 |
| romcheg | Morning NobodyCam! | 13:18 |
| Shrews | lucasagomes: morning, and my condolences | 13:18 |
| NobodyCam | morning romcheg | 13:18 |
| *** zul has joined #openstack-ironic | 13:18 | |
| lucasagomes | Shrews, :( thanks man | 13:18 |
| lucasagomes | that was harsh | 13:18 |
| lucasagomes | now I hope netherlands to beat argentina | 13:19 |
| lucasagomes | will beat* | 13:19 |
| NobodyCam | oh coolness your going to implement set boot device in the ssh driver! that going to be a bit of work | 13:21 |
| *** matty_dubs|gone is now known as matty_dubs | 13:22 | |
| NobodyCam | morning matty_dubs | 13:22 |
| lucasagomes | NobodyCam, already did :) | 13:22 |
| lucasagomes | NobodyCam, will submit the patch soon | 13:22 |
| NobodyCam | for all ssh "types" vbox, vmware, and virsh? | 13:23 |
| lucasagomes | NobodyCam, that's the problem, I did only for virsh :( | 13:23 |
| lucasagomes | NobodyCam, I can work on the vbox but vmware I don't have how to test | 13:24 |
| NobodyCam | vbox can be be done with a single cli command | 13:24 |
| NobodyCam | not sure about vmware | 13:24 |
| lucasagomes | yeah "VBoxManage modifyvm <uuid|name> [--boot<1-4> none|floppy|dvd|disk|net>]" | 13:24 |
| lucasagomes | yeah vmware i've no clue as well | 13:25 |
| NobodyCam | question: The "persistent" flag is that per instance deployed, or forever and ever | 13:25 |
| lucasagomes | NobodyCam, so persistent is when setting the boot device, you can say whether it should live accross multiple reboots | 13:27 |
| lucasagomes | or if False (default behavior) it's going to persist until the next boot only | 13:27 |
| NobodyCam | yes but we will have a way of resetting it once the instance is removed from the node? | 13:28 |
| lucasagomes | for the PXE driver for example, we set it as persistent=True, so all the boots will be network boots | 13:28 |
| NobodyCam | ie. go back to pxe booting | 13:28 |
| lucasagomes | NobodyCam, oh yeah definitely | 13:28 |
| NobodyCam | :) | 13:28 |
| NobodyCam | so that setting is per instance | 13:28 |
| *** jgrimm has joined #openstack-ironic | 13:28 | |
| NobodyCam | :) | 13:28 |
| NobodyCam | ...per depolyed instance | 13:29 |
| lucasagomes | the PXE driver sets it when deploying the node | 13:29 |
| NobodyCam | :) | 13:29 |
| lucasagomes | I think, lemme check it out | 13:29 |
| lucasagomes | https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L510 | 13:30 |
| lucasagomes | as part of the deploy | 13:30 |
| NobodyCam | thats perfect...:) | 13:30 |
| NobodyCam | Add a new boot_device field to the nodes database table. / Data model impact: None???? | 13:32 |
| NobodyCam | I like it | 13:33 |
| lucasagomes | oh | 13:35 |
| lucasagomes | hah gotta clean that up, won't be added any boot_device | 13:35 |
| lucasagomes | NobodyCam, heh will clean that up thanks for pointing it | 13:35 |
| *** foexle has joined #openstack-ironic | 13:36 | |
| *** pcrews has joined #openstack-ironic | 13:40 | |
| *** davidlenwell_ has joined #openstack-ironic | 13:41 | |
| *** geekyogi1 has joined #openstack-ironic | 13:41 | |
| *** rloo has quit IRC | 13:42 | |
| *** lifeless1 has joined #openstack-ironic | 13:43 | |
| *** GheRiver1 has joined #openstack-ironic | 13:43 | |
| *** jistr has quit IRC | 13:43 | |
| *** SpamapS_ has joined #openstack-ironic | 13:44 | |
| *** ellenh_ has joined #openstack-ironic | 13:46 | |
| *** _sirushti has joined #openstack-ironic | 13:46 | |
| *** mitz- has joined #openstack-ironic | 13:47 | |
| *** mordred_ has joined #openstack-ironic | 13:47 | |
| *** foexle has quit IRC | 13:48 | |
| *** Poornima has quit IRC | 13:48 | |
| *** geekyogi has quit IRC | 13:48 | |
| *** coolsvap has quit IRC | 13:48 | |
| *** yuriyz has quit IRC | 13:48 | |
| *** mitz has quit IRC | 13:48 | |
| *** krtaylor has quit IRC | 13:48 | |
| *** mordred has quit IRC | 13:48 | |
| *** NobodyCam has quit IRC | 13:48 | |
| *** mordred_ is now known as mordred | 13:48 | |
| *** GheRivero has quit IRC | 13:48 | |
| *** pradipta_away has quit IRC | 13:48 | |
| *** davidlenwell has quit IRC | 13:48 | |
| *** SpamapS has quit IRC | 13:48 | |
| *** sirushti has quit IRC | 13:48 | |
| *** ellenh has quit IRC | 13:48 | |
| *** lifeless has quit IRC | 13:48 | |
| *** keekz has quit IRC | 13:48 | |
| *** anteaya has quit IRC | 13:48 | |
| *** _sirushti is now known as sirushti | 13:48 | |
| *** Poornima has joined #openstack-ironic | 13:49 | |
| *** foexle has joined #openstack-ironic | 13:50 | |
| *** eghobo has joined #openstack-ironic | 13:52 | |
| *** anteaya has joined #openstack-ironic | 13:55 | |
| jroll | lucasagomes: got a sec to chat about https://review.openstack.org/#/c/103595 and https://review.openstack.org/#/c/100734/ ? | 13:56 |
| jroll | uh oh, we lost nobodycam | 13:57 |
| lucasagomes | jroll, hey, sure | 13:57 |
| jroll | lucasagomes: so, one of us should depend on the other here... | 13:57 |
| jroll | because they're going to conflict hard | 13:57 |
| lucasagomes | jroll, right, I think it would be better to me depend on that one | 13:58 |
| jroll | and rloo has -2'd you because you mentioned the blueprint :/ | 13:58 |
| lucasagomes | otherwise you will get a bit blocked by the -2 | 13:58 |
| lucasagomes | exactly | 13:58 |
| jroll | but it's just a refactoring, shouldn't need a -2 | 13:58 |
| jroll | ok | 13:58 |
| *** NobodyCam has joined #openstack-ironic | 13:58 | |
| lucasagomes | yeah that's true | 13:58 |
| lucasagomes | that's not actually related to the iPXE | 13:58 |
| lucasagomes | it's a plumbing work | 13:58 |
| lucasagomes | jroll, I will set a dependency on ur patch | 13:58 |
| jroll | NobodyCam: I remembered to ask lucas about https://review.openstack.org/#/c/103595 \o/ | 13:58 |
| lucasagomes | jroll, but anyway, do you agree in moving it to pxe_utils? | 13:59 |
| jroll | lucasagomes: awesome, thanks. wondering if I should shuffle things around now so the ipmitool things don't block you, or just hurry up and get that patch fixed up | 13:59 |
| jroll | lucasagomes: kind of... I'd say ironic/common/pxe_utils.py | 13:59 |
| jroll | the agent driver also depends on this stuff | 13:59 |
| *** eghobo has quit IRC | 13:59 | |
| NobodyCam | ahh seems I got disconnected | 13:59 |
| jroll | so I don't think it's so related to the pxe driver | 14:00 |
| lucasagomes | jroll, right, if u want to add a comment about it I can address that too as part of the rebase | 14:00 |
| jroll | sure thing | 14:00 |
| *** jistr has joined #openstack-ironic | 14:00 | |
| lucasagomes | ta much! | 14:00 |
| *** keekz has joined #openstack-ironic | 14:01 | |
| NobodyCam | brb | 14:01 |
| *** coolsvap has joined #openstack-ironic | 14:01 | |
| *** pradipta_away has joined #openstack-ironic | 14:01 | |
| *** mordred has quit IRC | 14:02 | |
| *** mordred has joined #openstack-ironic | 14:02 | |
| jroll | lucasagomes: no, thank you! commented | 14:02 |
| *** jistr has quit IRC | 14:03 | |
| *** romcheg has left #openstack-ironic | 14:04 | |
| lucasagomes | jroll, btw, u good to go with the swift temp url | 14:04 |
| lucasagomes | I approved the spec this morning | 14:04 |
| *** rloo has joined #openstack-ironic | 14:04 | |
| *** jistr has joined #openstack-ironic | 14:04 | |
| jroll | lucasagomes: awesommme, thanks! | 14:05 |
| jroll | JoshNang: ^^ | 14:05 |
| *** overlayer has quit IRC | 14:05 | |
| jroll | rloo: when you have time can you remove your -2 from https://review.openstack.org/#/c/81391 ? spec was approved | 14:06 |
| *** jcoufal has quit IRC | 14:06 | |
| jroll | JoshNang: wrt your last comment here... this should still be a separate patch https://review.openstack.org/#/c/86744/20 | 14:09 |
| *** krtaylor has joined #openstack-ironic | 14:09 | |
| NobodyCam | anyone know why the spec repo is running gate-ironic-specs-python27 check? | 14:09 |
| *** jcoufal has joined #openstack-ironic | 14:10 | |
| rloo | jroll: I was just about to do that. Although technically, the blueprint itself hasn't yet been approved ;) | 14:10 |
| *** ndipanov has quit IRC | 14:11 | |
| Shrews | NobodyCam: to run this test? tests.test_titles.TestTitles.test_template | 14:11 |
| Shrews | looks like it checks that all sections are present | 14:11 |
| *** amitpp has quit IRC | 14:12 | |
| jroll | rloo: can we just burn launchpad? :) | 14:13 |
| NobodyCam | ahh ok then I wont remove it | 14:13 |
| rloo | jroll: I'm sorry, I cannot answer that question ;) | 14:15 |
| Shrews | anybody happen to have the link to the spreadsheet w/ approved specs? | 14:17 |
| jroll | Shrews: this guy? https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit#gid=1357128775 | 14:18 |
| Shrews | jroll: thx | 14:18 |
| jroll | np | 14:18 |
| *** ndipanov has joined #openstack-ironic | 14:20 | |
| jroll | morgabra: oh hey, that neutron spec got merged: https://review.openstack.org/#/c/87825/ | 14:21 |
| jroll | ^ this is awesome for ironic, y'all | 14:21 |
| NobodyCam | lucasagomes: ManageIq looks neat-o | 14:23 |
| lucasagomes | NobodyCam, ta much! I will fix that point on the spec, I'm trying to adapt the code to the new revision | 14:23 |
| matty_dubs | NobodyCam: I worked on that for about a month after RH first acquired them. It's very neat stuff, and I'm psyched that they've now open-sourced it. | 14:25 |
| NobodyCam | ya :) | 14:29 |
| jroll | devananda: finally updated https://etherpad.openstack.org/p/ipa-todos :) | 14:30 |
| *** Poornima has left #openstack-ironic | 14:32 | |
| *** lazy_prince is now known as killer_prince | 14:33 | |
| NobodyCam | for thoses that care to know /me very much dislikes the Cirtrix client | 14:39 |
| NobodyCam | Citrix even | 14:41 |
| *** dkehnx1 has joined #openstack-ironic | 14:41 | |
| *** max_lobur has joined #openstack-ironic | 14:44 | |
| rloo | folks, quick question. Are we OK adding CLI support for vendor-passthru? Doesn't look like anyone has reviewed this: https://review.openstack.org/#/c/101297/ | 14:45 |
| JayF | Seems like something that'd be potentially useful to have | 14:46 |
| JayF | if the public api exists, why shouldn't the official cli client be able to cal lit | 14:47 |
| jroll | idk, what's the use case? | 14:47 |
| rloo | JayF: yeah. So why hasn't anyone reviewed it? I don't know that I want to be the first ;) | 14:47 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Allow Ironic URL from config file https://review.openstack.org/105590 | 14:48 |
| rloo | JayF: I suppose if someone is manually testing and don't want to use the API? | 14:48 |
| jroll | maybe | 14:49 |
| jroll | I guess it can't hurt | 14:49 |
| NobodyCam | rloo: I looking at that too | 14:52 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic-specs: New driver ManagementInterface https://review.openstack.org/100218 | 14:52 |
| rloo | NobodyCam: ok thx. If you're looking at it, I won't (for now). | 14:52 |
| lucasagomes | rloo, I think there's no problem adding it to the CLI, as JayF said it's alreayd in the public api | 14:54 |
| lucasagomes | the CLI should just relfect that | 14:54 |
| lucasagomes | reflect* | 14:54 |
| rloo | thx lucasagomes | 14:54 |
| lucasagomes | rloo, btw, on #91585 you mean to use the same value as the api.max_limit | 14:56 |
| lucasagomes | rloo, but not use the config option itself right? (cause client and ironic are 2 diff projects) | 14:56 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Expose {set,get}_boot_device in the API https://review.openstack.org/90151 | 14:56 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: SSH virsh to use the new ManagementInterface https://review.openstack.org/89884 | 14:56 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface https://review.openstack.org/86092 | 14:56 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: SeaMicro to use the new ManagementInterface https://review.openstack.org/86328 | 14:57 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMINative to use the new ManagementInterface https://review.openstack.org/86588 | 14:57 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: ManagementInterface {set, get}_boot_device() to support 'persistent' https://review.openstack.org/105757 | 14:57 |
| rloo | lucasagomes: no. if you hardcode it to 1000, it may not be 1000 if someone changes that value in api.max_limit. | 14:58 |
| NobodyCam | oh he's using MagicMock | 14:58 |
| rloo | lucasagomes: I mean, don't code a default value for limit in the CLI. If you don't code it, it should default to using ironic's default, right? | 14:58 |
| lucasagomes | rloo, no | 14:58 |
| lucasagomes | rloo, if no value is specified by it's going to return everything | 14:59 |
| Shrews | devananda: adam_g: http://lists.openstack.org/pipermail/openstack-dev/2014-July/039751.html | 14:59 |
| rloo | lucasagomes: no, it'll return max 1000 (ie, it'll return max whateve is specified in api.max_limit. | 14:59 |
| lucasagomes | rloo, that's what the patch is solving, the list() function on the CLI lib will follow the pagination and returning everything if no limit is specified | 15:00 |
| rloo | lucasagomes: why do you want to do this? the REST api doesn't do that? | 15:00 |
| lucasagomes | rloo, no the API returning X per request (via max_limit), if you want to get the next batch you have to submit a new request to the link that the API provides (the 'next' link) | 15:01 |
| lucasagomes | but when you use the CLI, for e.g for the Nova driver | 15:01 |
| lucasagomes | you want the list function to return all the items registered in ironic | 15:02 |
| lucasagomes | so we need the add the ability for the cli to follow those links | 15:02 |
| lucasagomes | and return everything to it | 15:02 |
| rloo | lucasagomes: so that is fine, you can do that with limit=0 I think based on your description. | 15:02 |
| lucasagomes | rloo, right, so what I can do on the CLI is | 15:02 |
| lucasagomes | if it's None it will just return whatever number of items the API have returned (based on max_limit) | 15:03 |
| rloo | lucasagomes: yes, that's what I'd like. if limit in CLI is not specified, what you just said. | 15:03 |
| lucasagomes | rloo, ack | 15:03 |
| lucasagomes | rloo, cheers, will do it | 15:04 |
| rloo | lucasagomes: btw, have you tested with limit=0? | 15:04 |
| lucasagomes | rloo, yeah, right now there's no difference between None or 0 for the lib | 15:04 |
| lucasagomes | if not limit: ... | 15:04 |
| lucasagomes | 0 and None will enter that condition | 15:04 |
| rloo | lucasagomes: cuz i actually had a question for you. in ironic, there is code to take the min(limit, config-value) as the limit. in some util.py file. i was wondering why. | 15:04 |
| *** ndipanov has quit IRC | 15:05 | |
| lucasagomes | rloo, because if the config file says max_limit is 1000 | 15:05 |
| lucasagomes | and someone in the api inputs ?limit=8192898219821982192 | 15:05 |
| rloo | lucasagomes: so they want more than 1000. | 15:05 |
| lucasagomes | rloo, then we wouldn't need to cap it in a config option | 15:06 |
| *** ndipanov has joined #openstack-ironic | 15:06 | |
| rloo | lucasagomes: i was trying to understand your changes yest so I set api.max_limit=1. When I then set limit=2 in the CLI, I kept getting 1 :-( | 15:06 |
| lucasagomes | the idea is that the config is what prevent people from flooding everything by misuing the public API | 15:06 |
| rloo | lucasagomes: oh. I didn't realize that was the intent of the config. I thought it was for pagination via the REST api. | 15:07 |
| rloo | lucasagomes: anyway, so it seems to me (I didn't test) that if you set limit=0 it won't work unless you do something 'under the hood' first. | 15:08 |
| rloo | lucasagomes: and I'm not sure I would agree for the CLI to treat limit=0 differently from the API. | 15:08 |
| lucasagomes | rloo, right... maybe we should document the --limit that the maximum that can be specified is related to the max_limit? | 15:09 |
| lucasagomes | rloo, or you think that limit=0 on the api should also return everything? | 15:10 |
| lucasagomes | it can get messy because http server might also limit the size of the response | 15:10 |
| lucasagomes | the idea of the max_limit is that, prevent it from flooding | 15:10 |
| rloo | lucasagomes: hmm, I hadn't thought about the http server/response size issue. How do other applications deal with it? [Ie, I have no answer cuz I don't know enough about this.] | 15:12 |
| rloo | lucasagomes: any idea how nova deals with this? or keystone? | 15:13 |
| lucasagomes | rloo, https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L541-L546 | 15:15 |
| lucasagomes | rloo, yes, we pretty much copied the same behavior from other openstack apis | 15:15 |
| lucasagomes | I think that's the normal way pagination works in general | 15:15 |
| rloo | lucasagomes: ok. what does novaclient do, do you know? | 15:16 |
| lucasagomes | admins needs a way to set a config with a sane value to prevent people from wasting bandwidth/misuing the api | 15:16 |
| lucasagomes | rloo, lemme check | 15:16 |
| *** max_lobur has quit IRC | 15:16 | |
| rloo | jroll, JoshNang : the swift-temp-url spec was approved, and it mentioned something about a change in swiftclient. So is https://review.openstack.org/#/c/81391 ready to be reviewed, or does it need to be updated to reflect the spec? | 15:20 |
| NobodyCam | rameshg87: you around | 15:20 |
| *** SpamapS_ is now known as SpamapS | 15:21 | |
| NobodyCam | morning SpamapS | 15:21 |
| jroll | rloo: it's not :/ a bunch of that is going into swiftclient instead... https://review.openstack.org/#/c/102632/ | 15:21 |
| *** SpamapS has quit IRC | 15:21 | |
| *** SpamapS has joined #openstack-ironic | 15:21 | |
| rloo | jroll: could you do me a favour and put wip or something on 81391 then, so people don't spend time looking at it just yet? | 15:21 |
| SpamapS | NobodyCam: yo yo how's life in rolling thunder these days? | 15:21 |
| rloo | jroll: people == me :-) | 15:22 |
| jroll | rloo: only JoshNang can do that, I can add a comment if that helps? | 15:22 |
| rloo | jroll: a comment is better than nothing. is JoshNang out? | 15:22 |
| NobodyCam | SpamapS: doing good :) | 15:22 |
| jroll | rloo: done | 15:23 |
| rloo | thx jroll! | 15:23 |
| jroll | rloo: it's still earlyish out here :) | 15:23 |
| rloo | jroll: ha ha. sorry, I forgot. it wasn't that urgent, I'm fine if JoshNang does it later today. I thought maybe he was away for a few days or something. My memory is good enough that I'll remember for at least today ;) | 15:24 |
| lucasagomes | rloo, so the nova client doesn't seem to implement a way to follow the pagination :( | 15:24 |
| jroll | rloo: :) | 15:24 |
| lucasagomes | rloo, it does have a way to set the limit, but all it does is to construct the URL | 15:24 |
| lucasagomes | ?limit=<limit> | 15:24 |
| lucasagomes | and then the API would limit it in case it's bigger than the max_limit there | 15:24 |
| rloo | lucasagomes: ooo, not good on nova's part. | 15:24 |
| * NobodyCam is not a mock expert, anyone know one MagicMock a dict? | 15:25 | |
| jroll | NobodyCam: why would you mock a dict? got an example? | 15:25 |
| lucasagomes | rloo, yeah, usually the clients are second citizens heh people don't love them that much heh | 15:25 |
| rloo | lucasagomes: ok, so for our needs, I think setting limit=0 may be the only reasonable way to deal with it. | 15:25 |
| rloo | lucasagomes: why is our nova ironic driver using the CLI and not the API? | 15:25 |
| NobodyCam | jroll: line 37 of https://review.openstack.org/#/c/101297/1/ironicclient/tests/v1/test_driver_shell.py | 15:26 |
| BadCub_ | Morning World | 15:26 |
| NobodyCam | good morning BadCub_ | 15:26 |
| lucasagomes | rloo, cause the library is meant to encapuslate the CLI and make things easier? | 15:26 |
| lucasagomes | rloo, instead of sending raw curl commands you would just do node.list() | 15:26 |
| lucasagomes | encapuslate the API* | 15:27 |
| rloo | lucasagomes: ok, seems like a reasonable reason. Cuz it is easier. ha ha. So then, I'm fine with limit=0 then. | 15:27 |
| *** killer_prince is now known as lazy_prince | 15:27 | |
| lucasagomes | rloo, ok so... limit=0 to return everything? | 15:27 |
| rloo | lucasagomes: not sure how/if we need to document the diff between API and CLI so I'm going to ignore that part. No one will ask... | 15:27 |
| lucasagomes | limit = None to return what's in the max_limit? | 15:27 |
| rloo | lucasagomes: yeah, limit=0 for everything since that's what you/driver needs. | 15:28 |
| jroll | NobodyCam: I don't see a problem with that... seems do_driver_vendor_passthru takes an object, not a dict | 15:28 |
| JoshNang | rloo: jroll done! | 15:28 |
| lucasagomes | rloo, right | 15:28 |
| lucasagomes | rloo, yeah it's hmm hard | 15:29 |
| lucasagomes | rloo, if u think it's better we can even not expose the limit=0 on the CLI | 15:29 |
| jroll | JoshNang: whoa, you're here early :) | 15:29 |
| *** max_lobur has joined #openstack-ironic | 15:29 | |
| lucasagomes | so that if people want to get the next batch they have to use the marker parameter manually | 15:29 |
| rloo | lucasagomes: although I still am not sure about api.max_limits, but since nova does it etc. It just seems ... inefficient... when ironic can just spew it all out in one request. | 15:29 |
| rloo | thx JoshNang ! | 15:30 |
| NobodyCam | jroll: I was just about to say that it looks like obj vs dict | 15:30 |
| NobodyCam | :-p | 15:30 |
| lucasagomes | rloo, well as a public api I think that admins should have a way to limit the size of the request | 15:30 |
| NobodyCam | jroll: Ty | 15:30 |
| lucasagomes | rloo, if well documented it may not be a big problem | 15:30 |
| rloo | lucasagomes: yeah, I understand about admins. BUT you realize that the list could get out-of-sync between requests. | 15:31 |
| rloo | lucasagomes: todate, only admins can use the CLI too. | 15:31 |
| lucasagomes | rloo, yea about the out-of-sync, I don't think there's anything we can do for it | 15:33 |
| jroll | we don't translate for log.debug, right? | 15:33 |
| lucasagomes | any pagination mechanism will have that problem | 15:33 |
| rloo | jroll: right. | 15:33 |
| jroll | thanks | 15:33 |
| rloo | lucasagomes: yes, any pagination mech will have that problem. | 15:34 |
| rloo | lucasagomes: nova cli gives special 'privileges' if the user is an admin. | 15:34 |
| rloo | lucasagomes: just thinking. maybe ironic cli can do a similar thing. if admin, they can override limit. | 15:34 |
| jroll | do we see the ironic api ever being used by a non-admin? | 15:35 |
| lucasagomes | rloo, nova does that? | 15:35 |
| lucasagomes | and also we don't have policies in the ironic api | 15:35 |
| lucasagomes | it's always admin | 15:35 |
| rloo | lucasagomes: eg, if you're admin, you can get VMs for all tenants. | 15:35 |
| lucasagomes | ah right, but that's because the nova api can be used by normal tenants | 15:35 |
| lucasagomes | where ironic is always admins | 15:36 |
| rloo | lucasagomes: having said that about nova, they don't give special admin priv for 'limit' so I dunno. | 15:36 |
| lucasagomes | rloo, yeah they don't seem to do anything for the limit, if more than max_limit is specified they use max_limit | 15:36 |
| rloo | lucasagomes: yeah, so at least for now, if to use ironic you have to be admin, why can't an admin override the max_limits value. | 15:36 |
| lucasagomes | rloo, so maybe we shouldn't have max_limit at all | 15:37 |
| rloo | lucasagomes: so really, I may just be lazy. I don't want to have/review code in our CLI that loops on the list request, just to get the entire list :-) | 15:37 |
| lucasagomes | rloo, heh I think it's also have to do with the configuration of the http server as I mentioned | 15:37 |
| rloo | lucasagomes: I would say get rid of max_limit, except for your point about http response sizes. | 15:37 |
| lucasagomes | if the size of the response is limit | 15:37 |
| lucasagomes | limited* | 15:38 |
| lucasagomes | rloo, yeah well what you can do in this case is set the max_limit to a really big number heh | 15:38 |
| lucasagomes | rloo, or maybe even create a flag to it to kinda "disable" max_limit | 15:39 |
| lucasagomes | idk -1 or something | 15:39 |
| rloo | lucasagomes: if say an ironic conductor can handle 10000 baremetal nodes, and the nova-ironic-driver wants that list, what happens if the request fails cuz the response is too large. | 15:39 |
| lucasagomes | rloo, that's why we have pagination, so none of the requests wil be that big to actually reach the http server limit | 15:40 |
| lucasagomes | but if it does hit it for whatever misconfiguration problem | 15:40 |
| lucasagomes | it may return a 500 (server error) | 15:40 |
| *** romcheg has joined #openstack-ironic | 15:40 | |
| NobodyCam | rloo: I like that patch. | 15:41 |
| * NobodyCam would check on devananda's thoughts before landing thou | 15:41 | |
| rloo | lucasagomes: but the max_limits value is something an admin puts (or 1000). For one thing, maybe we ought to update the descriptor for that (or put in some admin doc) about it. | 15:41 |
| rloo | NobodyCam: thx. that's the vendor passthru CLI thing right? | 15:42 |
| NobodyCam | ya | 15:42 |
| rloo | NobodyCam: ok, I'll put it on my list to look today. | 15:42 |
| lucasagomes | rloo, but that's the thing... how the CLI get's all items registered even if the max limit per request is 1000? by following the 'next' link and doing more than one request to retrieve everything | 15:42 |
| lucasagomes | that's the limitation that we are trying to solve with that cli patch | 15:43 |
| lucasagomes | having a way to retrieve all the items by issuing multiple requests (respecting the limit set by the API) | 15:43 |
| rloo | lucasagomes: so for the CLI change, I'm fine with limit=0 meaning to get all. | 15:43 |
| rloo | lucasagomes: it is the implementation. but I think the max_limits is needed cuz of http response size limit. | 15:43 |
| rloo | lucasagomes: so yeah, there's no way to avoid it. | 15:44 |
| lucasagomes | yeah it's a bit hairy | 15:44 |
| lucasagomes | rloo, ack, thanks for bringing that up tho, it's good make us think about better ways to solve something | 15:44 |
| *** eghobo has joined #openstack-ironic | 15:45 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add methods to ipmitool driver https://review.openstack.org/100364 | 15:45 |
| rloo | lucasagomes: at least for me, it clarified some things. Thx for being patient ;) | 15:45 |
| lucasagomes | rloo, no worries at all | 15:45 |
| lucasagomes | it's all good | 15:45 |
| jroll | https://review.openstack.org/100364 that should be an easy review, had tons of feedback | 15:45 |
| *** Nisha has joined #openstack-ironic | 15:54 | |
| *** martyntaylor has left #openstack-ironic | 15:55 | |
| *** matty_dubs is now known as matty_dubs|lunch | 15:55 | |
| *** ndipanov has quit IRC | 16:02 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Factor out TFTPImageCache https://review.openstack.org/100734 | 16:02 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Factor out deploy info from PXE driver https://review.openstack.org/100735 | 16:03 |
| *** martyntaylor has joined #openstack-ironic | 16:03 | |
| jroll | ^ more easy reviews :) | 16:04 |
| *** ndipanov has joined #openstack-ironic | 16:05 | |
| *** martyntaylor has quit IRC | 16:12 | |
| NobodyCam | brb | 16:16 |
| Shrews | lucasagomes: for 100958, do you think we should also clear out the node instance_uuid on failure too? | 16:16 |
| Shrews | it's removed if you 'nova delete' the failed instance | 16:16 |
| Shrews | not sure if there is some sort of race condition there where another process may be trying to boot an instance and grab that node and see instance_uuid is already set | 16:17 |
| lucasagomes | Shrews, right... hmm thinking | 16:18 |
| *** coolsvap is now known as coolsvap|afk | 16:18 | |
| lucasagomes | Shrews, because right now it's possible to PATCH the instance uuid without triggering deploy | 16:18 |
| lucasagomes | so if we have a race condition we may want to prevent that too | 16:19 |
| jroll | we already remove instance_uuid on boot failure | 16:19 |
| lucasagomes | being only possible to associate the instance with the node when actually deploying that node | 16:19 |
| jroll | and it breaks things | 16:19 |
| Shrews | jroll: this is due to a bug where the node is left in an inconsistent state if no worker threads are available | 16:20 |
| jroll | errr, that's a different issue, ignore me | 16:20 |
| Shrews | jroll: https://bugs.launchpad.net/ironic/+bug/1331494 | 16:20 |
| jroll | yeah | 16:20 |
| lucasagomes | Shrews, thinking about the flow here, if the node failed because there's no workers available | 16:20 |
| lucasagomes | Shrews, there's a way to re-trigger the deployment of that failed instance? or we need to delete it before? | 16:21 |
| *** ellenh has joined #openstack-ironic | 16:21 | |
| Shrews | lucasagomes: looks to me like nova repeatedly makes calls to verify progress, times out, and puts the instance in ERROR state. No way that I know of to say to nova "try again" when it's in ERROR | 16:22 |
| *** martyntaylor has joined #openstack-ironic | 16:23 | |
| Shrews | When instance goes to ERROR, not sure if that makes the ironic node available for new instances or not. | 16:23 |
| Shrews | i guess we can test by using only 1 ironic node and try | 16:23 |
| lucasagomes | Shrews, yeah I will give it a try, see if by removing the instance_uuid we make the node free for other instances to be deployed on it | 16:24 |
| lucasagomes | if so I think it would makes sense to remove the instance_uuid yes | 16:24 |
| *** ndipanov has quit IRC | 16:24 | |
| Shrews | lucasagomes: ++ | 16:24 |
| Shrews | will leave a comment on the review | 16:24 |
| lucasagomes | Shrews, right, thanks! | 16:24 |
| NobodyCam | lucasagomes: matty_dubs|lunch: nice... re: http://people.redhat.com/~jcoufal/openstack/juno/2014-07-09_nodes-ui_juno.pdf | 16:27 |
| rameshg87 | hello NobodyCam | 16:27 |
| lucasagomes | NobodyCam, yup! nice wireframes | 16:28 |
| NobodyCam | morning rameshg87 | 16:30 |
| Shrews | oh lookie there... lunch time. bbs | 16:30 |
| NobodyCam | rameshg87: my ping was going to ask about the use of magicmock | 16:30 |
| NobodyCam | jroll helped me with the answer | 16:31 |
| rameshg87 | good morning NobodyCam | 16:31 |
| *** martyntaylor has left #openstack-ironic | 16:35 | |
| rameshg87 | NobodyCam: | 16:37 |
| *** jistr has quit IRC | 16:37 | |
| rameshg87 | NobodyCam: was this question regarding the review that i had raised | 16:37 |
| *** rameshg87 has quit IRC | 16:37 | |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 16:37 |
| NobodyCam | yes. it was on the add vender pass through to cli tests | 16:38 |
| *** rameshg87 has joined #openstack-ironic | 16:38 | |
| *** jcoufal has quit IRC | 16:39 | |
| *** jgrimm has quit IRC | 16:42 | |
| *** aswadr has quit IRC | 16:42 | |
| *** GheRiver1 is now known as GheRivero | 16:43 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Use suitable assert https://review.openstack.org/105512 | 16:45 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 16:48 |
| *** derekh_ has quit IRC | 16:56 | |
| *** amitpp has joined #openstack-ironic | 16:59 | |
| *** harlowja_away is now known as harlowja | 17:03 | |
| *** matty_dubs|lunch is now known as matty_dubs | 17:04 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add support for interacing with swift https://review.openstack.org/105795 | 17:04 |
| *** romcheg has quit IRC | 17:05 | |
| *** Alexei_987 has quit IRC | 17:07 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/python-ironicclient: Add bash completion support for ironic cli https://review.openstack.org/101765 | 17:20 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Make ComputeCapabilitiesFilter work with Ironic https://review.openstack.org/105802 | 17:23 |
| devananda | morning, all | 17:23 |
| NobodyCam | good morning devananda | 17:24 |
| jroll | mornin devananda | 17:26 |
| *** Penick has joined #openstack-ironic | 17:26 | |
| lucasagomes | devananda, morning | 17:26 |
| lucasagomes | and I will have to go, have a good night everybody! | 17:27 |
| *** lucasagomes is now known as lucas-dinner | 17:27 | |
| NobodyCam | have a good night lucas-dinner | 17:27 |
| rameshg87 | NobodyCam: you wanted to check regarding something :-) | 17:33 |
| NobodyCam | rameshg87: I found my answer | 17:33 |
| Shrews | devananda, or anyone else: Does nova make calls to the ironic nova driver in a multi-threaded way? I.e., can there be multiple threads calling ironic.nova.virt.driver.IronicDriver.spawn() ? | 17:34 |
| rameshg87 | NobodyCam: okay :-) | 17:34 |
| NobodyCam | :) | 17:34 |
| NobodyCam | bad lag today | 17:34 |
| NobodyCam | :( | 17:34 |
| NobodyCam | brb | 17:35 |
| *** rameshg87 has left #openstack-ironic | 17:35 | |
| jroll | Shrews: I think the answer is yes, if running multiple nova-computes. why do you ask? | 17:35 |
| *** blamar has quit IRC | 17:35 | |
| devananda | Shrews: yes | 17:37 |
| devananda | you dont need to run >1 n-cpu | 17:37 |
| devananda | nova-compute uses eventlet with greenthreads | 17:37 |
| devananda | there can be any number of concurrent spawn() running in a single n-cpu process | 17:37 |
| devananda | Shrews: theyre' not real threads, fwiw | 17:38 |
| Shrews | jroll: just thinking about it for a review that's up for the driver | 17:38 |
| jroll | gah, I was thinking for a single node | 17:39 |
| JayF | Shrews: jroll: IIRC ClusteredComputeManager was pulled from Nova | 17:39 |
| JayF | Shrews: jroll: So I don't think the initial permutation of the nova driver is going to be able to be used with multiple nova-computes | 17:40 |
| JayF | but IMBW? | 17:40 |
| devananda | JayF: it was rejected in my spec proposal. it's still in ironic's tree tho | 17:40 |
| jroll | JayF: it's clearly still able to be used... just not with upstreamed nova code only | 17:40 |
| devananda | JayF: and you're correct. there are some serious limitations in the default ComputeManager. | 17:40 |
| jroll | which is the point of pluggable interfaces :) | 17:40 |
| JayF | Simultaneously wrong and correct \o/ | 17:41 |
| *** rameshg87 has joined #openstack-ironic | 17:42 | |
| rameshg87 | devananda: request your inputs on the revised ilo virtual media driver spec: https://review.openstack.org/#/c/97744/ | 17:43 |
| *** davidlenwell_ is now known as davidlelnwell | 17:44 | |
| *** davidlelnwell is now known as davidlelnwell__ | 17:44 | |
| *** davidlelnwell__ is now known as davidlelnwell | 17:44 | |
| *** davidlelnwell is now known as davidlenwell | 17:44 | |
| *** geekyogi1 has quit IRC | 17:53 | |
| *** lazy_prince is now known as killer_prince | 17:55 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Make ComputeCapabilitiesFilter work with Ironic https://review.openstack.org/105802 | 18:00 |
| Shrews | devananda: lol, you stole my threads > 1 comment | 18:01 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 18:01 |
| Shrews | my brain was 10 minutes too slow | 18:02 |
| *** jcoufal has joined #openstack-ironic | 18:05 | |
| *** ellenh has quit IRC | 18:06 | |
| NobodyCam | devananda: would we want any test to ensure we are inface passing any capabilities to the scheduler??? | 18:07 |
| NobodyCam | s/inface/infact/ | 18:07 |
| devananda | NobodyCam: yes | 18:07 |
| devananda | NobodyCam: i WIP'd the first rev, forgot to tag the second | 18:07 |
| *** ellenh has joined #openstack-ironic | 18:08 | |
| NobodyCam | :) | 18:08 |
| *** amitpp has quit IRC | 18:09 | |
| *** amitpp has joined #openstack-ironic | 18:10 | |
| *** amitpp has quit IRC | 18:11 | |
| *** Nisha has quit IRC | 18:14 | |
| *** hemna_ is now known as hemna | 18:15 | |
| *** rameshg87 has left #openstack-ironic | 18:19 | |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 18:19 |
| *** eghobo has quit IRC | 18:21 | |
| *** eghobo has joined #openstack-ironic | 18:21 | |
| NobodyCam | hahhahahahaha | 18:25 |
| * NobodyCam does happy dance | 18:25 | |
| * jroll wonders why NobodyCam is dancing | 18:26 | |
| *** pelix has quit IRC | 18:27 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic-specs: Add support for retry on NodeLocked exceptions https://review.openstack.org/103996 | 18:28 |
| NobodyCam | jroll: I was able to nova boot a node on a POC that has been killing me for weeks now | 18:32 |
| jroll | lol | 18:32 |
| jroll | I know that feeling all too well ;) | 18:32 |
| JayF | I still feel that way when I call our public api and get a metal server and nothing broke | 18:32 |
| NobodyCam | :) | 18:33 |
| JayF | Although builds succeeding is the normal case there, I just have 9 months of it not existing to get over :P | 18:33 |
| *** lifeless1 is now known as lifeless | 18:33 | |
| * NobodyCam makes a bagel | 18:46 | |
| * devananda makes lunch | 18:57 | |
| jroll | oh, devananda, did you see I updated https://etherpad.openstack.org/p/ipa-todos ? | 19:00 |
| NobodyCam | brb quick walkies | 19:09 |
| *** ellenh has quit IRC | 19:22 | |
| *** blamar has joined #openstack-ironic | 19:32 | |
| *** ellenh has joined #openstack-ironic | 19:37 | |
| *** amitpp has joined #openstack-ironic | 19:39 | |
| * devananda realizes he's only going to be home for 3 of the next 8 weeks | 19:39 | |
| *** faizan has joined #openstack-ironic | 19:41 | |
| NobodyCam | :( | 19:44 |
| *** ellenh has quit IRC | 19:49 | |
| *** amitpp has quit IRC | 19:52 | |
| *** ellenh has joined #openstack-ironic | 19:56 | |
| *** jcoufal has quit IRC | 19:56 | |
| *** eguz has joined #openstack-ironic | 19:59 | |
| *** faizan has quit IRC | 19:59 | |
| *** Penick has quit IRC | 20:00 | |
| *** eguz has quit IRC | 20:02 | |
| *** eghobo has quit IRC | 20:02 | |
| *** eguz has joined #openstack-ironic | 20:02 | |
| *** Penick has joined #openstack-ironic | 20:04 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic-specs: UEFI support for Ironic deploy drivers https://review.openstack.org/99850 | 20:14 |
| *** krtaylor is now known as krtaylor_away | 20:14 | |
| *** faizan has joined #openstack-ironic | 20:15 | |
| NobodyCam | jroll: you happen to have plans to push up another rev on 100364? | 20:23 |
| jroll | NobodyCam: only if it gets a -1 :) | 20:24 |
| *** ellenh__ has joined #openstack-ironic | 20:24 | |
| *** ellenh has quit IRC | 20:25 | |
| *** ellenh__ is now known as ellenh | 20:25 | |
| NobodyCam | oh :( I was looking at a old version | 20:25 |
| NobodyCam | nm | 20:25 |
| jroll | heh | 20:25 |
| JayF | jroll: 100364 is -V | 20:25 |
| JayF | jroll: looks like py26 failed because $mirror or lol-networking | 20:25 |
| JayF | jroll: but the undercloud thing looks possibly actually broken | 20:25 |
| faizan | Hi NobodyCam: Devananda: I have addressed your review comments on uefi spec (99850) | 20:28 |
| NobodyCam | faizan: :) will have a look-see in a little bit. | 20:29 |
| faizan | request you to take a look at the new uefi spec patch | 20:29 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Document 'POST /v1/nodes/.../vendor_passthru' https://review.openstack.org/105869 | 20:30 |
| jroll | JayF: yeah I need to recheck | 20:31 |
| faizan | Devananda: NododyCam: I have listed down the nova ironic virt driver changes required for uefi boot support in this etherpad. https://etherpad.openstack.org/p/IronicUEFISupport | 20:33 |
| NobodyCam | jroll: ahhh no LOG.debug(_LD("blah %{msg}s"), {"msg": "blah"}) :( | 20:35 |
| jroll | someone want to help me find the error in this? http://logs.openstack.org/64/100364/9/check-tripleo/check-tripleo-ironic-undercloud-precise/723c3d7/ | 20:35 |
| jroll | NobodyCam: yeah, we don't translate debug messages at all | 20:35 |
| NobodyCam | I thought that was error | 20:38 |
| jroll | right, no worries | 20:39 |
| NobodyCam | my bad... :( :-p | 20:40 |
| NobodyCam | jroll: nasty looking error in neutron-server.txt | 20:42 |
| NobodyCam | in seed_logs folder | 20:43 |
| NobodyCam | not sure if thats your real error thou | 20:43 |
| NobodyCam | jroll: no Bug/BP/Spce for 100364? | 20:49 |
| NobodyCam | s/Spce/Spec/ | 20:49 |
| jroll | no... should I make one? | 20:49 |
| jroll | I'd really like to not make a spec for this | 20:49 |
| NobodyCam | kinda seems bug worthy | 20:50 |
| *** faizan has quit IRC | 20:50 | |
| jroll | yeah | 20:50 |
| NobodyCam | but overall LGTM | 20:51 |
| jroll | cool... I can add a bug | 20:51 |
| jroll | want to leave me a note on the review? | 20:51 |
| NobodyCam | have move the house, to get propaine. be back in a few minutes | 20:51 |
| jroll | sure | 20:52 |
| devananda | some errands to run, bbiafh | 20:56 |
| *** jdob has quit IRC | 21:00 | |
| NobodyCam | not moving the house | 21:02 |
| NobodyCam | :-p | 21:02 |
| NobodyCam | will do that later | 21:02 |
| *** matty_dubs is now known as matty_dubs|gone | 21:03 | |
| NobodyCam | jroll: ok your -1'd lol | 21:11 |
| jroll | ha thanks | 21:11 |
| NobodyCam | :-p | 21:12 |
| * NobodyCam thinks mr jenkins is going to get a workout when that series lands | 21:12 | |
| jroll | lol | 21:14 |
| NobodyCam | jroll: I must say mr jenkins seems you give you some really good errors : http://logs.openstack.org/34/100734/10/check/check-tempest-dsvm-ironic-postgres/85de5d9/logs/screen-ir-cond.txt.gz#_2014-07-09_16_23_11_027 | 21:17 |
| NobodyCam | s/seems you/seems to/ | 21:17 |
| jroll | that's... nice | 21:18 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 21:19 |
| Shrews | rloo: for https://review.openstack.org/102914, my ops side says "hey, this is a good idea", but my dev side says "that's a lot of code complexity just to substitute documentation that is currently missing" | 21:22 |
| Shrews | rloo: i'm torn | 21:22 |
| rloo | Shrews: as far as I'm concerned, if I never have to look at that, I'd be happy. | 21:23 |
| *** Penick has quit IRC | 21:23 | |
| rloo | Shrews: which doesn't answer your question. | 21:23 |
| Shrews | rloo: but it does make me giggle :) thx | 21:23 |
| rloo | Shrews: it isn't that much code complexity, but yeah, when I originally decided to fix the 'bug', I didn't realize it would drag on so long etc. | 21:23 |
| rloo | Shrews: as far as I am concerned, the more 'automated' docn we can do the better. | 21:24 |
| Shrews | rloo: automated is good, but when we *have* to automate b/c we are lacking in real docs, that's a big worry afaic | 21:25 |
| Shrews | it makes me a sad panda | 21:25 |
| rloo | Shrews: I am not sure what to say. You are free to -1 it and say you disagree, select one of the alternatives I suggested or add your own. | 21:27 |
| NobodyCam | https://www.youtube.com/watch?v=tRw1pkbx9V8 to your sad panda | 21:27 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Document 'POST /v1/.../vendor_passthru' https://review.openstack.org/105869 | 21:29 |
| Shrews | NobodyCam: wth? is japan building an army of attack bears? lol | 21:29 |
| NobodyCam | lol ... | 21:29 |
| NobodyCam | thought that just too strange | 21:30 |
| JayF | Shrews: I actually feel the exact opposite from an op-standpoint | 21:30 |
| Shrews | JayF: you like that we are missing documentation? :) | 21:30 |
| JayF | Shrews: I appreciate being able to call an API and find out what's /actually going on/ rather than /what the docs say/ especially given I run an environment that may run somewhat in front of master :) | 21:30 |
| JayF | Shrews: I'm saying both are bugs, and both should be fixed | 21:30 |
| JayF | Shrews: but hiding information from being easily taken programatically because it belongs in a doc... just doesn't make sense to me | 21:31 |
| Shrews | JayF: you feel exactly the same then, not opposite. :) i mentioned my ops side liking it | 21:31 |
| JayF | Shrews: I think all around, information should be available via the API that an op needs to run the cluster | 21:32 |
| *** Penick has joined #openstack-ironic | 21:32 | |
| Shrews | it's my dev side that's causing trouble | 21:32 |
| rloo | Shrews: believe me, it probably bothers my dev side more than your dev side, wrt the amount of time I've spent on it ;) | 21:33 |
| rloo | Shrews: anyway, I am not working on the code part until/if the spec gets approved :-) | 21:33 |
| Shrews | rloo: can i vote twice? | 21:36 |
| Shrews | :) | 21:36 |
| rloo | Shrews: sure. Maybe the easiest is to create two identities. Who would know? :-) | 21:36 |
| jroll | I never thought of that.. | 21:36 |
| JayF | Shrews: the ends tend to justify the means IMO :) | 21:36 |
| * jroll spams agent reviews with 1000 +1's | 21:37 | |
| rloo | gonna disappear for awhile... | 21:38 |
| *** zdin0bot has joined #openstack-ironic | 21:40 | |
| *** Penick has quit IRC | 21:40 | |
| *** radsy has joined #openstack-ironic | 21:47 | |
| *** mrda-away is now known as mrda | 21:51 | |
| NobodyCam | morning mrda | 21:51 |
| NobodyCam | :) | 21:51 |
| mrda | Morning Ironic! | 21:51 |
| mrda | lol, beat me to it! | 21:51 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 21:51 |
| NobodyCam | lol | 21:51 |
| NobodyCam | just happen to be looking at the right window | 21:52 |
| NobodyCam | jroll: just as fyi: the only thing I found looking for that dup mac address was: https://bugs.launchpad.net/ironic/+bug/1264127 | 21:54 |
| NobodyCam | :-p | 21:55 |
| NobodyCam | s/looking for/looking for a bug to go with/ | 21:55 |
| NobodyCam | jroll: your the man of the hour.. :) have any thought about adding a test (or two) for https://review.openstack.org/#/c/100735/10/ironic/common/tftp.py | 22:05 |
| *** zdin0bot has quit IRC | 22:06 | |
| jroll | NobodyCam: I guess I could do that, yeah | 22:12 |
| NobodyCam | :) I was shoucked to see that a move like that didn't break at least one test :-p | 22:13 |
| NobodyCam | shocked even | 22:14 |
| jroll | right now, it's only called from one place | 22:16 |
| jroll | I basically ripped out the middle of that function, because I want to use it too | 22:16 |
| NobodyCam | thats true, and it all worked as it should so nothing broke :) | 22:17 |
| *** Penick has joined #openstack-ironic | 22:23 | |
| *** eguz has quit IRC | 22:33 | |
| *** boris-42 has quit IRC | 22:37 | |
| *** boris-42 has joined #openstack-ironic | 22:39 | |
| *** eghobo has joined #openstack-ironic | 22:49 | |
| *** lucas-dinner has quit IRC | 22:55 | |
| *** ellenh has quit IRC | 22:55 | |
| *** ellenh has joined #openstack-ironic | 22:56 | |
| *** foexle has quit IRC | 22:57 | |
| NobodyCam | amazing how much space you can recover just by rm'ing a few 20 Gb files | 22:58 |
| lifeless | ... | 23:00 |
| NobodyCam | :-p | 23:00 |
| NobodyCam | morning lifeless | 23:00 |
| *** Penick has quit IRC | 23:00 | |
| *** ellenh has quit IRC | 23:03 | |
| *** ellenh has joined #openstack-ironic | 23:04 | |
| openstackgerrit | Michael Davies proposed a change to openstack/python-ironicclient: Expose auth_ref in ironicclient client object https://review.openstack.org/104467 | 23:05 |
| *** krtaylor_away is now known as krtaylor | 23:24 | |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Migrate CoreOS to use coreos-cloudinit https://review.openstack.org/105788 | 23:45 |
| NobodyCam | tox == Takes fOrever to eXecute | 23:46 |
| JayF | https://review.openstack.org/105788 is tested, working, and ready to land if anyone wants to review some CoreOS ramdisk internals | 23:49 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Adding support for decommissioning https://review.openstack.org/104379 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!