| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic-specs: Standardize logging and improve log coverage https://review.openstack.org/94924 | 00:04 |
|---|---|---|
| adam_g | does anyone know the proper way to configure nova to use all of the scheduler filters included in ironic.nova.scheduler.filters ? | 00:12 |
| adam_g | comstud, ^ | 00:13 |
| comstud | so | 00:13 |
| comstud | funny question. | 00:13 |
| comstud | but yeah | 00:14 |
| comstud | specify scheduler_available_filters many times in your conf | 00:14 |
| comstud | along with the nova default | 00:14 |
| comstud | sec | 00:14 |
| comstud | the easiest answer is that you copy all of the ironic filters into the nova/scheduler/filters directory | 00:14 |
| comstud | and then they all become available | 00:14 |
| comstud | and you can add ExactRamFilter, ExactCoreFilter etc to scheduler_default_filters conf variable | 00:15 |
| comstud | but if you want to be able to hit them out of tree, you do something like this: | 00:15 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic-specs: Support for different boot loaders and locations https://review.openstack.org/94925 | 00:15 |
| comstud | scheduler_available_filters=nova.scheduler.filters.all_filters | 00:15 |
| comstud | scheduler_available_filters=ironic.nova.scheduler.filters.exact_ram_filter.ExactRamFilter | 00:15 |
| comstud | scheduler_available_filters=ironic.nova.scheduler.filters.exact_cores_filter.ExactCoresFilter | 00:15 |
| comstud | [...] | 00:16 |
| comstud | scheduler_default_filters=RetryFilter,ComputeFilter,ExactRamFilter,ExactCoresFilter | 00:16 |
| comstud | etc.. | 00:16 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Remove duplicated test for caching image https://review.openstack.org/99230 | 00:16 |
| comstud | something like that | 00:16 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Improve cache_image testing https://review.openstack.org/99301 | 00:17 |
| adam_g | comstud, ah, okay. thanks | 00:18 |
| *** matsuhashi has joined #openstack-ironic | 00:24 | |
| *** eghobo has quit IRC | 00:26 | |
| *** blamar has quit IRC | 00:27 | |
| devananda | would love to get a few more eyes on the patch series starting with https://review.openstack.org/#/c/99026/1 | 00:45 |
| devananda | should help with scheduling filters | 00:46 |
| *** rwsu has quit IRC | 00:52 | |
| *** coolsvap is now known as coolsvap|afk | 00:56 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: ipmitool driver raises DriverLoadError https://review.openstack.org/99977 | 00:56 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Adds SingleInstanceFilter Nova scheduler filter https://review.openstack.org/100395 | 01:09 |
| NobodyCam | brain cooked... now done... will see everyone tomorrow | 01:10 |
| NobodyCam | noght y'all | 01:10 |
| NobodyCam | night even | 01:10 |
| mrda | Have a good night NobodyCam! | 01:11 |
| *** ellenh has quit IRC | 01:11 | |
| *** nosnos has joined #openstack-ironic | 01:26 | |
| *** datajerk has joined #openstack-ironic | 01:28 | |
| *** datajerk has quit IRC | 01:28 | |
| *** vinbs has joined #openstack-ironic | 01:31 | |
| *** vinbs_ has joined #openstack-ironic | 01:33 | |
| *** vinbs has quit IRC | 01:36 | |
| *** vinbs_ has quit IRC | 01:37 | |
| *** rloo_out has quit IRC | 01:39 | |
| *** rloo_out has joined #openstack-ironic | 01:39 | |
| *** foexle_ has joined #openstack-ironic | 01:48 | |
| *** foexle has quit IRC | 01:51 | |
| *** rameshg87 has joined #openstack-ironic | 01:57 | |
| *** blamar has joined #openstack-ironic | 02:17 | |
| *** bcaldwell has joined #openstack-ironic | 02:22 | |
| *** blamar has quit IRC | 02:32 | |
| *** rloo_out has quit IRC | 02:36 | |
| *** foexle_ has quit IRC | 02:41 | |
| *** rakesh_hs has joined #openstack-ironic | 02:41 | |
| *** vinbs has joined #openstack-ironic | 03:03 | |
| *** yongli has joined #openstack-ironic | 03:08 | |
| *** ramineni has joined #openstack-ironic | 03:19 | |
| *** eghobo has joined #openstack-ironic | 03:26 | |
| *** coolsvap|afk is now known as coolsvap | 03:30 | |
| *** matsuhashi has quit IRC | 03:30 | |
| *** ramineni1 has joined #openstack-ironic | 03:33 | |
| *** ramineni has quit IRC | 03:33 | |
| *** ramineni1 has left #openstack-ironic | 03:33 | |
| *** eghobo has quit IRC | 03:34 | |
| *** harlowja is now known as harlowja_away | 03:37 | |
| *** matsuhashi has joined #openstack-ironic | 03:37 | |
| *** harlowja_away is now known as harlowja | 03:39 | |
| *** nosnos has quit IRC | 03:41 | |
| *** matsuhashi has quit IRC | 03:44 | |
| *** matsuhashi has joined #openstack-ironic | 03:45 | |
| *** ramineni1 has joined #openstack-ironic | 03:48 | |
| *** matsuhashi has quit IRC | 03:49 | |
| *** coolsvap is now known as coolsvap|afk | 03:51 | |
| *** Poornima has joined #openstack-ironic | 03:56 | |
| *** matsuhashi has joined #openstack-ironic | 03:57 | |
| *** matsuhashi has quit IRC | 04:03 | |
| *** matsuhashi has joined #openstack-ironic | 04:04 | |
| *** rakesh_hs has quit IRC | 04:05 | |
| *** lazy_prince has joined #openstack-ironic | 04:05 | |
| *** rakesh_hs has joined #openstack-ironic | 04:08 | |
| *** coolsvap|afk is now known as coolsvap | 04:10 | |
| *** sabah has joined #openstack-ironic | 04:14 | |
| *** rakesh_hs has quit IRC | 04:16 | |
| *** rameshg87 has joined #openstack-ironic | 04:28 | |
| *** nosnos has joined #openstack-ironic | 04:30 | |
| *** eghobo has joined #openstack-ironic | 04:31 | |
| *** matsuhashi has quit IRC | 04:36 | |
| *** matsuhas_ has joined #openstack-ironic | 04:37 | |
| *** shausy has joined #openstack-ironic | 04:47 | |
| *** subah has joined #openstack-ironic | 04:59 | |
| *** sabah has quit IRC | 04:59 | |
| *** max_lobur has joined #openstack-ironic | 05:01 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Updates Ironic Guide with deployment information https://review.openstack.org/94604 | 05:02 |
| *** k4n0 has joined #openstack-ironic | 05:06 | |
| *** rakesh_hs has joined #openstack-ironic | 05:12 | |
| *** bcaldwell has quit IRC | 05:18 | |
| *** coolsvap is now known as coolsvap|afk | 05:33 | |
| *** harlowja is now known as harlowja_away | 05:48 | |
| *** coolsvap|afk is now known as coolsvap | 05:52 | |
| *** Haomeng|2 has quit IRC | 05:56 | |
| *** Nisha has joined #openstack-ironic | 05:57 | |
| *** Haomeng has joined #openstack-ironic | 06:00 | |
| *** bmaha has joined #openstack-ironic | 06:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/96063 | 06:01 |
| *** Nisha_ has joined #openstack-ironic | 06:07 | |
| *** Nisha_ has quit IRC | 06:07 | |
| *** Nisha has quit IRC | 06:08 | |
| *** subah has quit IRC | 06:12 | |
| *** sysexit has joined #openstack-ironic | 06:17 | |
| *** sabah has joined #openstack-ironic | 06:17 | |
| *** radsy has quit IRC | 06:21 | |
| *** rakesh_hs has quit IRC | 06:22 | |
| *** rakesh_hs has joined #openstack-ironic | 06:30 | |
| adam_g | wowsers | 06:34 |
| devananda | wtf | 06:40 |
| devananda | something broke our tests a few hours ago | 06:40 |
| devananda | adam_g: hi! are you looking at it too? | 06:41 |
| adam_g | devananda, yup | 06:41 |
| adam_g | https://bugs.launchpad.net/tempest/+bug/1330834 | 06:41 |
| *** max_lobur has quit IRC | 06:41 | |
| adam_g | really frustrating that the blatant failures in the ironic tempest jobs there were completely overlooked | 06:41 |
| *** max_lobur has joined #openstack-ironic | 06:42 | |
| adam_g | considering the change they were the relevant tests being touhced by the tempest patch | 06:42 |
| adam_g | s/the change// | 06:42 |
| devananda | wow | 06:42 |
| *** Haomeng has quit IRC | 06:46 | |
| adam_g | devananda, ill have a tempest patch up soonish | 06:46 |
| devananda | thanks | 06:46 |
| devananda | i'm debating whether to flake the ML over this | 06:46 |
| devananda | i've probably not had enough to drink yet to think that's a good idea | 06:47 |
| devananda | but srsly people | 06:47 |
| devananda | WTF | 06:47 |
| *** Haomeng has joined #openstack-ironic | 06:48 | |
| viktors|afk | devananda: still around? | 06:51 |
| devananda | yes | 06:51 |
| *** viktors|afk is now known as viktors | 06:51 | |
| devananda | sort of | 06:51 |
| devananda | depends on what youj mean by around | 06:51 |
| viktors | devananda: :) | 06:51 |
| viktors | devananda: I have a question as for patch to Nova | 06:52 |
| viktors | devananda: or it's better to wait with it? | 06:52 |
| devananda | ask away | 06:53 |
| viktors | devananda: I read your comments one more time, and I have no questions anymore :) | 06:55 |
| viktors | devananda: it's morning time... | 06:55 |
| * viktors should make some coffee | 06:56 | |
| devananda | :) | 06:57 |
| viktors | devananda: anyway i've marked patch in Nova https://review.openstack.org/#/c/99614/ (Sync "Prevent races in opportunistic db test cases") as partial bug fix. Please look at it, when you'll have a time | 06:58 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 06:59 | |
| *** foexle has joined #openstack-ironic | 07:00 | |
| devananda | viktors: +1'd as nothing else changed | 07:01 |
| devananda | thanks! | 07:01 |
| viktors | devananda: thank you! | 07:01 |
| *** yuriyz has joined #openstack-ironic | 07:06 | |
| *** vinbs has quit IRC | 07:08 | |
| *** vinbs has joined #openstack-ironic | 07:09 | |
| *** athomas has joined #openstack-ironic | 07:09 | |
| *** Manishanker has joined #openstack-ironic | 07:09 | |
| *** jcoufal has joined #openstack-ironic | 07:09 | |
| *** sabah has quit IRC | 07:10 | |
| *** eghobo has quit IRC | 07:11 | |
| *** ifarkas has joined #openstack-ironic | 07:15 | |
| *** pcrews has quit IRC | 07:17 | |
| *** Haomeng has quit IRC | 07:18 | |
| *** Haomeng has joined #openstack-ironic | 07:20 | |
| adam_g | devananda, https://review.openstack.org/#/c/100437/ | 07:22 |
| adam_g | goodnight | 07:23 |
| *** ndipanov has joined #openstack-ironic | 07:30 | |
| devananda | thanks! g'night | 07:30 |
| *** mrda is now known as mrda-away | 07:37 | |
| *** coolsvap is now known as coolsvap|afk | 07:39 | |
| openstackgerrit | jan grant proposed a change to openstack/ironic: WIP: Serialise paramiko requests https://review.openstack.org/99626 | 07:46 |
| *** matsuhas_ has quit IRC | 07:49 | |
| *** petertoft has joined #openstack-ironic | 07:49 | |
| *** matsuhashi has joined #openstack-ironic | 07:51 | |
| *** max_lobur has quit IRC | 08:01 | |
| *** shausy has quit IRC | 08:16 | |
| *** nosnos has quit IRC | 08:16 | |
| *** lazy_prince has quit IRC | 08:16 | |
| *** ramineni1 has quit IRC | 08:16 | |
| *** krtaylor has quit IRC | 08:16 | |
| *** slamont has quit IRC | 08:16 | |
| *** ifarkas has quit IRC | 08:16 | |
| *** vinbs has quit IRC | 08:16 | |
| *** rameshg87 has quit IRC | 08:16 | |
| *** rushiagr has quit IRC | 08:16 | |
| *** russell_h has quit IRC | 08:16 | |
| *** Haomeng has quit IRC | 08:16 | |
| *** harlowja_away has quit IRC | 08:16 | |
| *** mgagne has quit IRC | 08:16 | |
| *** agordeev has quit IRC | 08:16 | |
| *** yjiang5 has quit IRC | 08:16 | |
| *** adam_g has quit IRC | 08:16 | |
| *** rainya has quit IRC | 08:16 | |
| *** wendar has quit IRC | 08:16 | |
| *** stevebaker has quit IRC | 08:16 | |
| *** pquerna has quit IRC | 08:16 | |
| *** zigo has quit IRC | 08:16 | |
| *** Isotopp has quit IRC | 08:16 | |
| *** morgabra has quit IRC | 08:16 | |
| *** mrda-away has quit IRC | 08:16 | |
| *** antonym has quit IRC | 08:16 | |
| *** boris-42 has quit IRC | 08:16 | |
| *** foexle has quit IRC | 08:16 | |
| *** sysexit has quit IRC | 08:16 | |
| *** dividehex has quit IRC | 08:16 | |
| *** ndipanov has quit IRC | 08:16 | |
| *** Mikhail_D_ltp has quit IRC | 08:16 | |
| *** bmaha has quit IRC | 08:16 | |
| *** openstackgerrit has quit IRC | 08:16 | |
| *** matsuhashi has quit IRC | 08:16 | |
| *** petertoft has quit IRC | 08:16 | |
| *** yongli has quit IRC | 08:16 | |
| *** mmitchell_ has quit IRC | 08:16 | |
| *** athomas has quit IRC | 08:16 | |
| *** yuriyz has quit IRC | 08:16 | |
| *** rakesh_hs has quit IRC | 08:16 | |
| *** Poornima has quit IRC | 08:16 | |
| *** dhellmann has quit IRC | 08:16 | |
| *** jrist has quit IRC | 08:16 | |
| *** viktors has quit IRC | 08:16 | |
| *** coolsvap|afk has quit IRC | 08:16 | |
| *** JoshNang has quit IRC | 08:16 | |
| *** anteaya has quit IRC | 08:16 | |
| *** jcoufal has quit IRC | 08:16 | |
| *** k4n0 has quit IRC | 08:16 | |
| *** mkerrin has quit IRC | 08:16 | |
| *** Shrews has quit IRC | 08:16 | |
| *** devananda has quit IRC | 08:16 | |
| *** BadCub has quit IRC | 08:16 | |
| *** ekarlso has quit IRC | 08:16 | |
| *** Manishanker has quit IRC | 08:16 | |
| *** dtantsur|afk has quit IRC | 08:16 | |
| *** SpamapS has quit IRC | 08:16 | |
| *** Mikhail_D_wk has quit IRC | 08:16 | |
| *** toabctl has quit IRC | 08:16 | |
| *** Madasi has quit IRC | 08:16 | |
| *** lsmola__ has quit IRC | 08:16 | |
| *** GheRivero has quit IRC | 08:16 | |
| *** NobodyCam has quit IRC | 08:16 | |
| *** soren has quit IRC | 08:16 | |
| *** lifeless has quit IRC | 08:16 | |
| *** pradipta_away has quit IRC | 08:17 | |
| *** sseago has quit IRC | 08:17 | |
| *** tteggel has quit IRC | 08:17 | |
| *** pleia2 has quit IRC | 08:17 | |
| *** aignatov has quit IRC | 08:17 | |
| *** Ng has quit IRC | 08:17 | |
| *** sbadia has quit IRC | 08:17 | |
| *** mikal has quit IRC | 08:17 | |
| *** dshulyak has quit IRC | 08:17 | |
| *** comstud has quit IRC | 08:17 | |
| *** christopheraedo has quit IRC | 08:17 | |
| *** kevinbenton has quit IRC | 08:17 | |
| *** klindgren has quit IRC | 08:17 | |
| *** lynxman has quit IRC | 08:17 | |
| *** pelix has joined #openstack-ironic | 08:21 | |
| *** jistr has joined #openstack-ironic | 08:21 | |
| *** matsuhashi has joined #openstack-ironic | 08:21 | |
| *** petertoft has joined #openstack-ironic | 08:21 | |
| *** ndipanov has joined #openstack-ironic | 08:21 | |
| *** jcoufal has joined #openstack-ironic | 08:21 | |
| *** Manishanker has joined #openstack-ironic | 08:21 | |
| *** athomas has joined #openstack-ironic | 08:21 | |
| *** yuriyz has joined #openstack-ironic | 08:21 | |
| *** foexle has joined #openstack-ironic | 08:21 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 08:21 | |
| *** rakesh_hs has joined #openstack-ironic | 08:21 | |
| *** sysexit has joined #openstack-ironic | 08:21 | |
| *** bmaha has joined #openstack-ironic | 08:21 | |
| *** k4n0 has joined #openstack-ironic | 08:21 | |
| *** Poornima has joined #openstack-ironic | 08:21 | |
| *** yongli has joined #openstack-ironic | 08:21 | |
| *** dividehex has joined #openstack-ironic | 08:21 | |
| *** dhellmann has joined #openstack-ironic | 08:21 | |
| *** jrist has joined #openstack-ironic | 08:21 | |
| *** dtantsur has joined #openstack-ironic | 08:21 | |
| *** openstackgerrit has joined #openstack-ironic | 08:21 | |
| *** viktors has joined #openstack-ironic | 08:21 | |
| *** sseago has joined #openstack-ironic | 08:21 | |
| *** mmitchell_ has joined #openstack-ironic | 08:21 | |
| *** mkerrin has joined #openstack-ironic | 08:21 | |
| *** boris-42 has joined #openstack-ironic | 08:21 | |
| *** coolsvap|afk has joined #openstack-ironic | 08:21 | |
| *** tteggel has joined #openstack-ironic | 08:21 | |
| *** JoshNang has joined #openstack-ironic | 08:21 | |
| *** anteaya has joined #openstack-ironic | 08:21 | |
| *** Shrews has joined #openstack-ironic | 08:21 | |
| *** devananda has joined #openstack-ironic | 08:21 | |
| *** SpamapS has joined #openstack-ironic | 08:21 | |
| *** klindgren has joined #openstack-ironic | 08:21 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 08:21 | |
| *** lsmola__ has joined #openstack-ironic | 08:21 | |
| *** BadCub has joined #openstack-ironic | 08:21 | |
| *** ekarlso has joined #openstack-ironic | 08:21 | |
| *** Madasi has joined #openstack-ironic | 08:21 | |
| *** toabctl has joined #openstack-ironic | 08:21 | |
| *** mikal has joined #openstack-ironic | 08:21 | |
| *** GheRivero has joined #openstack-ironic | 08:21 | |
| *** christopheraedo has joined #openstack-ironic | 08:21 | |
| *** kevinbenton has joined #openstack-ironic | 08:21 | |
| *** NobodyCam has joined #openstack-ironic | 08:21 | |
| *** dickson.freenode.net sets mode: +o devananda | 08:21 | |
| *** lynxman has joined #openstack-ironic | 08:21 | |
| *** pleia2 has joined #openstack-ironic | 08:21 | |
| *** dshulyak has joined #openstack-ironic | 08:21 | |
| *** pradipta_away has joined #openstack-ironic | 08:21 | |
| *** soren has joined #openstack-ironic | 08:21 | |
| *** lifeless has joined #openstack-ironic | 08:21 | |
| *** Ng has joined #openstack-ironic | 08:21 | |
| *** comstud has joined #openstack-ironic | 08:21 | |
| *** aignatov has joined #openstack-ironic | 08:21 | |
| *** sbadia has joined #openstack-ironic | 08:21 | |
| *** Haomeng has joined #openstack-ironic | 08:22 | |
| *** ifarkas has joined #openstack-ironic | 08:22 | |
| *** vinbs has joined #openstack-ironic | 08:22 | |
| *** shausy has joined #openstack-ironic | 08:22 | |
| *** nosnos has joined #openstack-ironic | 08:22 | |
| *** rameshg87 has joined #openstack-ironic | 08:22 | |
| *** lazy_prince has joined #openstack-ironic | 08:22 | |
| *** ramineni1 has joined #openstack-ironic | 08:22 | |
| *** harlowja_away has joined #openstack-ironic | 08:22 | |
| *** krtaylor has joined #openstack-ironic | 08:22 | |
| *** slamont has joined #openstack-ironic | 08:22 | |
| *** yjiang5 has joined #openstack-ironic | 08:22 | |
| *** mgagne has joined #openstack-ironic | 08:22 | |
| *** agordeev has joined #openstack-ironic | 08:22 | |
| *** rushiagr has joined #openstack-ironic | 08:22 | |
| *** adam_g has joined #openstack-ironic | 08:22 | |
| *** rainya has joined #openstack-ironic | 08:22 | |
| *** russell_h has joined #openstack-ironic | 08:22 | |
| *** wendar has joined #openstack-ironic | 08:22 | |
| *** stevebaker has joined #openstack-ironic | 08:22 | |
| *** Isotopp has joined #openstack-ironic | 08:22 | |
| *** pquerna has joined #openstack-ironic | 08:22 | |
| *** zigo has joined #openstack-ironic | 08:22 | |
| *** morgabra has joined #openstack-ironic | 08:22 | |
| *** mrda-away has joined #openstack-ironic | 08:22 | |
| *** antonym has joined #openstack-ironic | 08:22 | |
| *** Haomeng has quit IRC | 08:22 | |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic-specs: DRAC power driver https://review.openstack.org/99352 | 08:23 |
| *** jcoufal has quit IRC | 08:27 | |
| *** krtaylor has quit IRC | 08:27 | |
| *** lucasagomes has joined #openstack-ironic | 08:28 | |
| *** krtaylor has joined #openstack-ironic | 08:28 | |
| *** martyntaylor has joined #openstack-ironic | 08:28 | |
| *** sabah has joined #openstack-ironic | 08:29 | |
| *** jcoufal has joined #openstack-ironic | 08:29 | |
| *** romcheg has joined #openstack-ironic | 08:30 | |
| *** lucasagomes has quit IRC | 08:33 | |
| *** shausy has quit IRC | 08:33 | |
| *** nosnos has quit IRC | 08:33 | |
| *** lazy_prince has quit IRC | 08:33 | |
| *** ramineni1 has quit IRC | 08:33 | |
| *** slamont has quit IRC | 08:33 | |
| *** ifarkas has quit IRC | 08:33 | |
| *** vinbs has quit IRC | 08:33 | |
| *** rameshg87 has quit IRC | 08:33 | |
| *** rushiagr has quit IRC | 08:33 | |
| *** russell_h has quit IRC | 08:33 | |
| *** harlowja_away has quit IRC | 08:33 | |
| *** mgagne has quit IRC | 08:33 | |
| *** agordeev has quit IRC | 08:33 | |
| *** yjiang5 has quit IRC | 08:33 | |
| *** adam_g has quit IRC | 08:33 | |
| *** rainya has quit IRC | 08:33 | |
| *** wendar has quit IRC | 08:33 | |
| *** stevebaker has quit IRC | 08:33 | |
| *** pquerna has quit IRC | 08:33 | |
| *** zigo has quit IRC | 08:33 | |
| *** Isotopp has quit IRC | 08:33 | |
| *** morgabra has quit IRC | 08:33 | |
| *** mrda-away has quit IRC | 08:33 | |
| *** antonym has quit IRC | 08:33 | |
| *** ndipanov has quit IRC | 08:34 | |
| *** ndipanov has joined #openstack-ironic | 08:43 | |
| *** lucasagomes has joined #openstack-ironic | 08:43 | |
| *** ifarkas has joined #openstack-ironic | 08:43 | |
| *** vinbs has joined #openstack-ironic | 08:43 | |
| *** shausy has joined #openstack-ironic | 08:43 | |
| *** nosnos has joined #openstack-ironic | 08:43 | |
| *** rameshg87 has joined #openstack-ironic | 08:43 | |
| *** lazy_prince has joined #openstack-ironic | 08:43 | |
| *** ramineni1 has joined #openstack-ironic | 08:43 | |
| *** harlowja_away has joined #openstack-ironic | 08:43 | |
| *** slamont has joined #openstack-ironic | 08:43 | |
| *** yjiang5 has joined #openstack-ironic | 08:43 | |
| *** mgagne has joined #openstack-ironic | 08:43 | |
| *** agordeev has joined #openstack-ironic | 08:43 | |
| *** wendar has joined #openstack-ironic | 08:43 | |
| *** stevebaker has joined #openstack-ironic | 08:43 | |
| *** Isotopp has joined #openstack-ironic | 08:43 | |
| *** pquerna has joined #openstack-ironic | 08:43 | |
| *** zigo has joined #openstack-ironic | 08:43 | |
| *** morgabra has joined #openstack-ironic | 08:43 | |
| *** mrda-away has joined #openstack-ironic | 08:43 | |
| *** antonym has joined #openstack-ironic | 08:43 | |
| *** russell_h has joined #openstack-ironic | 08:43 | |
| *** rainya has joined #openstack-ironic | 08:43 | |
| *** adam_g has joined #openstack-ironic | 08:43 | |
| *** rushiagr has joined #openstack-ironic | 08:43 | |
| *** ndipanov has quit IRC | 08:45 | |
| *** ndipanov has joined #openstack-ironic | 08:45 | |
| *** max_lobur has joined #openstack-ironic | 08:45 | |
| dtantsur | https://review.openstack.org/95789 merged \o/ | 08:48 |
| dtantsur | lucasagomes, ^^^\ | 08:48 |
| lucasagomes | yup!!! | 08:48 |
| lucasagomes | I'm rebasing the current patches right now | 08:48 |
| lucasagomes | and addresses the fixes | 08:48 |
| yuriyz | morning dtantsur, lucasagomes, Ironic | 08:49 |
| lucasagomes | yuriyz, morning | 08:50 |
| lucasagomes | dtantsur, btw u think that I should port the migration script from bash to python? | 08:50 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Nova Ironic Driver to not get deploy ramdisk and kernel from flavor https://review.openstack.org/95551 | 08:51 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add migration script to instance_info https://review.openstack.org/96136 | 08:51 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Move PXE instance level parameters to instance_info https://review.openstack.org/94855 | 08:51 |
| dtantsur | morning, yuriyz, lucasagomes :) | 08:53 |
| dtantsur | lucasagomes, I'd vote for porting, if it is complex enough | 08:54 |
| * dtantsur hates complex code in bash | 08:54 | |
| *** nosnos has quit IRC | 08:54 | |
| lucasagomes | dtantsur, heh, well it's quite small, the only complex thing is partsing json in bash :) | 08:54 |
| yuriyz | +1 for porting | 08:54 |
| dtantsur | lucasagomes, parsing json in bash? I'd like to see it | 08:54 |
| dtantsur | :) | 08:55 |
| lucasagomes | heh dtantsur https://review.openstack.org/96136 | 08:55 |
| dtantsur | lucasagomes, well, after I saw parsing JSON in pure SQL (with PostgreSQL extensions), it's hard to surprise me :D | 08:55 |
| lucasagomes | lol yeah I can figure | 08:55 |
| *** nosnos has joined #openstack-ironic | 08:55 | |
| dtantsur | lucasagomes, oh, I vote for porting Oo You're too smart, I can't understand the code :) | 08:56 |
| lucasagomes | lol ack | 08:56 |
| lucasagomes | dtantsur, lemme bug u with 2 more things | 08:56 |
| dtantsur | sure | 08:56 |
| lucasagomes | dtantsur, https://review.openstack.org/#/c/98822/4/specs/juno/ipxe-boot.rst | 08:56 |
| dtantsur | aha, I'm on it since yesterday | 08:57 |
| lucasagomes | dtantsur, so yes, I've seem that bug... but, nothing much we can do at the Ironic side I would say? I know that having to have something already configured up and running and not controlling it is not ideal | 08:57 |
| lucasagomes | but I don't see how Ironic could manage a http server | 08:57 |
| lucasagomes | it seems totally out of scope | 08:57 |
| lucasagomes | dtantsur, and, in the future we might even want to downloading things directly from glance (swift temp urls?) | 08:58 |
| dtantsur | lucasagomes, it'd _just_ rely on swift temp URLs | 08:58 |
| dtantsur | lucasagomes, we already have a spec for it | 08:58 |
| lucasagomes | dtantsur, it can be pluggable... also it needs some investigation | 08:58 |
| dtantsur | and making users configure swift claster sounds more right than making them configure HTTP server | 08:59 |
| lucasagomes | dtantsur, temp until when? can we control it? cause I want the url to go away after I tear down the node for e.g | 08:59 |
| romcheg | Good morning Ironic! | 08:59 |
| lucasagomes | dtantsur, yeah... well they might have a http server already | 08:59 |
| lucasagomes | for things like horizon etc | 08:59 |
| dtantsur | romcheg, morning | 08:59 |
| dtantsur | lucasagomes, they might, they might not... | 09:00 |
| lucasagomes | dtantsur, so this would be the first stab, once we have the swift urls we can add a flag and make it pluggable | 09:00 |
| dtantsur | lucasagomes, according to the spec, you can set lifetime of the temp URL | 09:00 |
| dtantsur | lucasagomes, for the "first stab" we'll write a lot of configuration code in e.g. devstack to actually turn it on | 09:00 |
| lucasagomes | dtantsur, yeah I have a devstack patch already configuring the iPXE enviroment and Ironic to use it | 09:01 |
| dtantsur | lucasagomes, wow, you're fast :) | 09:01 |
| lucasagomes | dtantsur, https://review.openstack.org/#/c/99677/ | 09:01 |
| lucasagomes | so u can test it already if u want | 09:01 |
| dtantsur | so, I'm -0 on having HTTP server, +1 on Swift temp URLs for it | 09:02 |
| lifeless | lucasagomes: temp swift urls for config? or images ? | 09:02 |
| lucasagomes | dtantsur, yeah what I agreed with deva yesterday is that I'm going to add a note about the swift urls | 09:02 |
| lifeless | lucasagomes: cause, non-config should be permanent, just the swift backing url, no ? | 09:02 |
| lucasagomes | lifeless, yeah I've to investigate it yet cause I dunno much about swift | 09:03 |
| dtantsur | lucasagomes, lifeless https://review.openstack.org/#/c/98904/ | 09:03 |
| lucasagomes | lifeless, but the idea would be to not put the images in a http server | 09:03 |
| *** romcheg has quit IRC | 09:03 | |
| lucasagomes | lifeless, just fetch it directly from glance when booting the node | 09:03 |
| lifeless | lucasagomes: s/glance/swift/ | 09:03 |
| dtantsur | lucasagomes, directly from swift | 09:03 |
| dtantsur | you were faster :) | 09:04 |
| lifeless | lucasagomes: if you fetch from glance with a swift backend you are forcing a copy we want to use the backing url | 09:04 |
| lucasagomes | ah right, I though we were talking about glance + swift backend | 09:04 |
| lifeless | lucasagomes: well we are, but lets be explicit | 09:04 |
| lucasagomes | right | 09:04 |
| lucasagomes | I can give it some investigation then | 09:04 |
| lucasagomes | lifeless, but you think it should be pluggable? if ppl don't want to use swift they can set a http_root and http_server and ironic will put the images there | 09:05 |
| *** bmaha has quit IRC | 09:05 | |
| dtantsur | now to something completely different, folks: any ideas, what happened to our check jobs: http://logs.openstack.org/58/96558/22/check/check-tempest-dsvm-virtual-ironic/0a7d618/logs/testr_results.html.gz | 09:05 |
| dtantsur | this is affecting everything | 09:05 |
| *** romcheg has joined #openstack-ironic | 09:06 | |
| lifeless | lucasagomes: it will have to be, because those are totally different operations | 09:06 |
| lifeless | lucasagomes: one is 'ask glance for the swift URL' | 09:06 |
| lifeless | lucasagomes: the other is 'copy the image from A to B' | 09:06 |
| lucasagomes | lifeless, I c that's good | 09:06 |
| lifeless | lucasagomes: but! | 09:06 |
| *** bmaha has joined #openstack-ironic | 09:06 | |
| lucasagomes | yeah baby steps | 09:07 |
| lucasagomes | I want to add support for HTTPS as well | 09:07 |
| lifeless | lucasagomes: I think we can start by saying 'you must have glance, and it must give us a URL we can give iPXE' | 09:07 |
| lifeless | lucasagomes: backing service is up to the user; we can then make the variation be on whether we get a direct swift url or not | 09:07 |
| lucasagomes | lifeless, only glance doesn't give u an URL without requiring authentication I think | 09:08 |
| * lucasagomes will take a look at the options u guys gave, thanks | 09:09 | |
| lifeless | lucasagomes: we're authenticated, aren't we ? | 09:09 |
| lifeless | lucasagomes: we get the url from it, pass the url to ipxe | 09:09 |
| lucasagomes | lifeless, right yeah, when generating the config we are authenticated | 09:10 |
| lucasagomes | lemme try it out, cause I know little about glance swift urls etc | 09:10 |
| dtantsur | makes sense | 09:11 |
| dtantsur | again, folks is the issue with our jobs known already? | 09:11 |
| lifeless | sorry, I haven't looked; need to crash in a minute | 09:11 |
| dtantsur | ok, I'll at least create a bug | 09:12 |
| lucasagomes | dtantsur, damn :( is tempest broken!? | 09:12 |
| lucasagomes | :( | 09:13 |
| dtantsur | https://bugs.launchpad.net/ironic/+bug/1330883 | 09:13 |
| dtantsur | seems like that | 09:13 |
| lucasagomes | damn | 09:13 |
| rameshg87 | lifeless, dtantsur, lucasagomes, we have a similar one for the proposed ilo deploy. we propose the generate swift temp urls and attach it as virtual media cdrom: https://review.openstack.org/#/c/97744/2/specs/ironic-ilo-virtualmedia-driver.rst | 09:20 |
| rameshg87 | and then boot from virtual media cdrom | 09:20 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic-specs: iPXE boot https://review.openstack.org/98822 | 09:21 |
| lucasagomes | rameshg87, nice, yeah seems we have a lot of use for those urls :) | 09:21 |
| rameshg87 | lucasagomes, :-) | 09:21 |
| *** bmaha has quit IRC | 09:22 | |
| *** bmaha has joined #openstack-ironic | 09:22 | |
| *** matsuhashi has quit IRC | 09:22 | |
| lifeless | rameshg87: can you do that with lo100 ? | 09:23 |
| rameshg87 | lucasagomes, are you not proposing swift temp urls right now ? i see that not mentioned here, https://review.openstack.org/#/c/98822/5/specs/juno/ipxe-boot.rst | 09:23 |
| rameshg87 | lucasagomes, what do we mean by lo100 ? | 09:23 |
| rameshg87 | lights out ? | 09:24 |
| *** matsuhashi has joined #openstack-ironic | 09:24 | |
| lucasagomes | rameshg87, nop I'm not. There's another spec about swift temp urls | 09:24 |
| lucasagomes | rameshg87, that was lifeless question, /me dunno what lo100 means as well :) | 09:24 |
| lucasagomes | rameshg87, https://review.openstack.org/#/c/98904/ << swift temp urls spec | 09:25 |
| rameshg87 | lucasagomes, google search gives me "HP Lights-Out 100 (LO100)" | 09:25 |
| lifeless | rameshg87: circa 2008 ilos :) | 09:25 |
| rameshg87 | lucasagomes, yeah, virtual media can work on those ilos as well | 09:25 |
| rameshg87 | lucasagomes, yes, i have seen the swift temp url spec and code review. i am just waiting for it :-) | 09:26 |
| *** Nisha has joined #openstack-ironic | 09:27 | |
| *** matsuhashi has quit IRC | 09:30 | |
| *** matsuhashi has joined #openstack-ironic | 09:30 | |
| *** matsuhashi has quit IRC | 09:34 | |
| *** matsuhashi has joined #openstack-ironic | 09:37 | |
| *** sysexit has quit IRC | 09:40 | |
| *** sabah has quit IRC | 09:42 | |
| *** sysexit has joined #openstack-ironic | 09:44 | |
| *** matsuhashi has quit IRC | 09:48 | |
| *** matsuhashi has joined #openstack-ironic | 09:50 | |
| *** r-mibu has joined #openstack-ironic | 09:57 | |
| r-mibu | Hi, Ironic CI (check-tempest-dsvm-ironic-nv) seems failed. | 09:58 |
| r-mibu | I think root cause is https://review.openstack.org/#/c/88233/ | 09:59 |
| r-mibu | Is anyone working on it? | 10:00 |
| *** martyntaylor has quit IRC | 10:00 | |
| *** martyntaylor has joined #openstack-ironic | 10:01 | |
| *** matsuhashi has quit IRC | 10:03 | |
| *** matsuhashi has joined #openstack-ironic | 10:03 | |
| *** matsuhashi has quit IRC | 10:04 | |
| *** matsuhashi has joined #openstack-ironic | 10:04 | |
| lucasagomes | r-mibu, yeah :( I think dtantsur openened a bug about it | 10:08 |
| lucasagomes | it seems to be an 1 line change to fix it | 10:08 |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/ironic: Use oslo.db library https://review.openstack.org/42159 | 10:09 |
| lucasagomes | r-mibu, will give it a go, 1 sec | 10:09 |
| r-mibu | lucasagomes, https://bugs.launchpad.net/tempest/+bug/1330834 ? | 10:14 |
| *** matsuhashi has quit IRC | 10:14 | |
| lucasagomes | r-mibu, https://bugs.launchpad.net/ironic/+bug/1330883 | 10:14 |
| lucasagomes | r-mibu, going to put a fix quickly, it's like 2 lines changes | 10:14 |
| *** Nisha has quit IRC | 10:15 | |
| r-mibu | lucasagomes, thanks for pointing, but there is patch for this bug https://review.openstack.org/#/c/100437/ | 10:19 |
| lucasagomes | r-mibu, a perfect then will assign the bug to adam_g | 10:20 |
| lucasagomes | r-mibu, I mean, will mark the bug as duplicated | 10:20 |
| lucasagomes | seems he opened one already | 10:20 |
| openstackgerrit | Phil Day proposed a change to openstack/ironic: Remove node parameter from power_off() method in virt driver https://review.openstack.org/100197 | 10:21 |
| *** matsuhashi has joined #openstack-ironic | 10:21 | |
| r-mibu | lucasagomes, thanks :) | 10:24 |
| lucasagomes | r-mibu, thank u for letting us know :) | 10:24 |
| ramineni1 | :dtansur, hi | 10:29 |
| *** bmaha2 has joined #openstack-ironic | 10:29 | |
| *** bmaha has quit IRC | 10:31 | |
| *** coolsvap|afk is now known as coolsvap | 10:33 | |
| *** matsuhashi has quit IRC | 10:35 | |
| *** matsuhashi has joined #openstack-ironic | 10:36 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic https://review.openstack.org/99318 | 10:38 |
| *** matsuhashi has quit IRC | 10:40 | |
| *** sabah has joined #openstack-ironic | 10:42 | |
| dtantsur | ramineni1, if you meant me, than hi :) | 10:51 |
| ramineni1 | dtantsur , yes..hi .. :) | 10:54 |
| ramineni1 | Reagrding the design spec review for iLO Power Driver | 10:55 |
| ramineni1 | dtansur , have addressed the comments you have mentioned ..could you please have a look | 10:56 |
| ramineni1 | dtansur , https://review.openstack.org/#/c/97455/ | 10:56 |
| dtantsur | ramineni1, sure, please wait a bit, I have a review queue already | 10:59 |
| ramineni1 | dtantsur , sure , thanks :) | 11:00 |
| *** ramineni1 has left #openstack-ironic | 11:02 | |
| *** jistr has quit IRC | 11:06 | |
| *** jistr has joined #openstack-ironic | 11:07 | |
| lucasagomes | damn why #100437 is SKIPPED in gate >.< | 11:11 |
| dtantsur | Oo | 11:12 |
| dtantsur | I haven't ever seen this status | 11:12 |
| dtantsur | maybe ask sdague? | 11:13 |
| lucasagomes | yeah, dunno if it's good or bad | 11:13 |
| lucasagomes | yeah, that patch is currently unbreaking our gate | 11:13 |
| lucasagomes | lemme ask | 11:13 |
| dtantsur | romcheg and others: need your review/suggestions on proposed bug reporting guidelines: https://etherpad.openstack.org/p/ironic-launchpad-bug-guidelines | 11:15 |
| *** romcheg has quit IRC | 11:16 | |
| *** coolsvap is now known as coolsvap|afk | 11:21 | |
| *** rameshg87 has left #openstack-ironic | 11:23 | |
| openstackgerrit | A change was merged to openstack/ironic-specs: More robust device status checking with fuser https://review.openstack.org/99605 | 11:25 |
| dtantsur | ifarkas, ^^^ | 11:25 |
| *** coolsvap|afk is now known as coolsvap | 11:28 | |
| ifarkas | \o/ | 11:30 |
| ifarkas | thanks dtantsur and lucasagomes | 11:30 |
| lucasagomes | :) | 11:30 |
| lucasagomes | u welcome | 11:30 |
| *** sabah has quit IRC | 11:34 | |
| *** Haomeng has joined #openstack-ironic | 11:35 | |
| *** coolsvap is now known as coolsvap|afk | 11:42 | |
| *** Haomeng has quit IRC | 11:43 | |
| agordeev | good monring Ironic :) | 11:44 |
| dtantsur | morning, agordeev! | 11:45 |
| *** Haomeng has joined #openstack-ironic | 11:45 | |
| agordeev | dtantsur: morning :) | 11:45 |
| *** nikunj2512 has joined #openstack-ironic | 11:46 | |
| dtantsur | Folks, I'm going to add https://etherpad.openstack.org/p/ironic-launchpad-bug-guidelines to launchpad. Any opinions& | 11:48 |
| lucasagomes | dtantsur, if possible, how to reproduce | 11:49 |
| lucasagomes | well maybe it's the same as "what you did" | 11:50 |
| lucasagomes | dtantsur, but what i mean is... like a more precise step to step on how to reproduce the error | 11:51 |
| lucasagomes | dtantsur, and maybe a more generic paragraph talking about the tags | 11:51 |
| lucasagomes | not only the "nova-driver" one | 11:51 |
| dtantsur | lucasagomes, tags are generally assigned by triager, not reporter | 11:52 |
| lucasagomes | right | 11:52 |
| lucasagomes | dtantsur, and maybe a reminder to make sure he's testing against an up-to-date ironic version | 11:52 |
| dtantsur | lucasagomes, what about bugs against released versions? | 11:53 |
| lucasagomes | dtantsur, idk how does that work tho | 11:53 |
| lucasagomes | usually we fix the bug in trunk and backport it | 11:53 |
| dtantsur | lucasagomes, see etherpad for updates | 11:54 |
| dtantsur | if people catch bug in prod, they can't just go and install development version :) | 11:54 |
| lucasagomes | right, seems good in the etherpad then | 11:54 |
| *** Haomeng has quit IRC | 11:55 | |
| lucasagomes | dtantsur, also idk if necessary, but maybe pointing the guy to the wiki about how to set the right importance for the bug (https://wiki.openstack.org/wiki/Bugs#Importance) | 11:56 |
| lucasagomes | or it's also a work for the triager? | 11:56 |
| *** Haomeng has joined #openstack-ironic | 11:57 | |
| *** lucasagomes is now known as lucas-lunch | 11:58 | |
| * lucas-lunch brb lunch is almost done | 11:58 | |
| dtantsur | lucas-lunch, also for triagers | 11:59 |
| *** nikunj2512 has quit IRC | 12:00 | |
| dtantsur | Oh, btw can we enable option "Expire "Incomplete" bug reports when they become inactive" in Launchpad? That will simplify bug team's life a bit :) | 12:00 |
| *** bmaha2 has quit IRC | 12:02 | |
| dtantsur | ok, I'll just switch it on and see, if people will complain ^_^ | 12:02 |
| *** mahalakshmi2 has joined #openstack-ironic | 12:03 | |
| *** mahalakshmi2 has quit IRC | 12:03 | |
| *** bmahalakshmi has joined #openstack-ironic | 12:04 | |
| dtantsur | both changes are in effect! | 12:05 |
| *** bmahalakshmi has quit IRC | 12:07 | |
| *** bmahalakshmi has joined #openstack-ironic | 12:08 | |
| Shrews | jenkins is being spiteful today | 12:14 |
| *** romcheg has joined #openstack-ironic | 12:18 | |
| romcheg | dtantsur: morning, around? | 12:19 |
| dtantsur | romcheg, morning, sure | 12:20 |
| romcheg | could you please gimme a link to the etherpad? | 12:20 |
| romcheg | bug specs | 12:20 |
| romcheg | I'm waiting for some official red tape with only a phone with me, so I think it might be the perfect time to check it out :) | 12:22 |
| dtantsur | romcheg, which etherpad you mean? bug reporting guidelines? | 12:22 |
| dtantsur | https://etherpad.openstack.org/p/ironic-launchpad-bug-guidelines | 12:22 |
| romcheg | yup | 12:22 |
| *** jdob has joined #openstack-ironic | 12:23 | |
| romcheg | dtantsur: thanks | 12:23 |
| *** romcheg has quit IRC | 12:26 | |
| *** romcheg has joined #openstack-ironic | 12:27 | |
| *** Poornima has quit IRC | 12:27 | |
| *** vinbs_ has joined #openstack-ironic | 12:28 | |
| *** vinbs has quit IRC | 12:28 | |
| *** vinbs_ is now known as vinbs | 12:29 | |
| *** romcheg has quit IRC | 12:30 | |
| *** bmahalakshmi has quit IRC | 12:30 | |
| *** zul has joined #openstack-ironic | 12:31 | |
| *** vinbs has quit IRC | 12:33 | |
| *** igordcard has joined #openstack-ironic | 12:35 | |
| *** Haomeng has quit IRC | 12:38 | |
| *** Haomeng has joined #openstack-ironic | 12:41 | |
| *** lucas-lunch is now known as lucasagomes | 12:44 | |
| lucasagomes | adam_g, ping u there? | 12:48 |
| lucasagomes | 100437 merge failed :( | 12:48 |
| jroll | mornin y'all | 12:49 |
| *** jistr has quit IRC | 12:50 | |
| lucasagomes | adam_g, I will rebase it, our gate is current broke cause of that. Hope u don't mind me rebasing ur patch | 12:50 |
| jroll | lucasagomes and friends: options for glance/swift urls: 1) pass glance URL along with the auth token; 2) get a direct_url (this is a config thing not available by default) from glance, contains swift credentials in the url; 3) swift temp urls, has a time limit | 12:51 |
| jroll | lucasagomes: I poked infra the other day whether I could depend on multiple patches or not so I didn't have to rebase your instance_info patches; infra told me they want to set a standard of not being afraid to touch other developers' patches :) | 12:51 |
| *** jistr has joined #openstack-ironic | 12:52 | |
| lucasagomes | jroll, ah nice, yeah indeed, I never know whether I should rebase or not someones patch | 12:52 |
| jroll | yeah | 12:53 |
| *** nosnos has quit IRC | 12:53 | |
| jroll | especially in this case, do whatever is best for the project/community :) | 12:53 |
| lucasagomes | yeah cool, will rebase fix it asap | 12:53 |
| lucasagomes | jroll, ta much | 12:54 |
| jroll | np :) | 12:54 |
| *** sysexit has quit IRC | 12:56 | |
| jroll | added our gate status to the whiteboard, btw | 12:57 |
| dtantsur | morning jroll | 13:00 |
| *** sysexit has joined #openstack-ironic | 13:00 | |
| *** sseago has quit IRC | 13:00 | |
| *** sseago_ has joined #openstack-ironic | 13:00 | |
| *** rloo has joined #openstack-ironic | 13:00 | |
| jroll | hi dtantsur | 13:01 |
| *** rakesh_hs has quit IRC | 13:02 | |
| jroll | so tempted to factor out the TFTPImageCache and related things from the pxe driver :| | 13:02 |
| dtantsur | sounds good, refactoring pxe driver never harms :) | 13:03 |
| jroll | this patch chain is so long already | 13:05 |
| *** rloo has quit IRC | 13:05 | |
| *** rloo has joined #openstack-ironic | 13:05 | |
| *** rloo has quit IRC | 13:07 | |
| jroll | (or rather will be) | 13:08 |
| *** rloo has joined #openstack-ironic | 13:08 | |
| *** zul has quit IRC | 13:08 | |
| *** zul has joined #openstack-ironic | 13:08 | |
| NobodyCam | good morning says the man make'n coffee :) | 13:10 |
| jroll | dtantsur: about my configdrive spec: I was thinking that a driver would call its own (expose|remove)_configdrive method, I'm only making those methods so there is a standard place to have that code, not that they would be called outside of a driver at all | 13:10 |
| *** Haomeng has quit IRC | 13:11 | |
| jroll | dtantsur: about the swift thing, I'm not sure I want to put a hard dependency on swift for this feature :| | 13:11 |
| dtantsur | jroll, putting image blob to JSON (and than to database) sounds too crazy, that's what I want to say | 13:12 |
| *** Haomeng has joined #openstack-ironic | 13:13 | |
| dtantsur | jroll, "Two new methods, "expose_configdrive" and "remove_configdrive", will be added to the driver API." <-- this can be read as public interface | 13:14 |
| jroll | yeah, it's absolutely insane, but so is operating your own swift cluster for a small ironic deployment :) | 13:14 |
| dtantsur | and I don't think a spec should dictate names for internal methods | 13:14 |
| dtantsur | jroll, why claster? IIRC you just need to configure Swift with some private key. You anyway have Swift, don't you? | 13:15 |
| rloo | dtantsur, jroll, I don't really know what you're talking about (haven't read the spec) but if the change is to one of the driver interfaces, then it does make sense to mention the new methods. | 13:16 |
| *** shausy has quit IRC | 13:16 | |
| dtantsur | btw morning, NobodyCam, rloo! | 13:16 |
| jroll | dtantsur: rackspace has swift, yes, but not everyone else does | 13:16 |
| rloo | afternoon dtantsur! | 13:16 |
| jroll | dtantsur: and cluster because HA :P | 13:17 |
| dtantsur | jroll, (newbie question follows) isn't Swift required by Glance? | 13:17 |
| jroll | dtantsur: no, glance has multiple storage backends | 13:17 |
| jroll | http://docs.openstack.org/developer/glance/configuring.html#configuring-glance-storage-backends | 13:18 |
| jroll | s3, heh | 13:18 |
| NobodyCam | morning dtantsur rloo and ofc jroll :) | 13:18 |
| rloo | morning NobodyCam ;) | 13:19 |
| Manishanker | morning NobodyCam :) | 13:19 |
| dtantsur | jroll, I see, thank you. Are there any other options, except for putting a blob there? Maybe just put glance uuid and let the driver fetch the image? | 13:19 |
| NobodyCam | morning Manishanker | 13:19 |
| jroll | dtantsur: hmm, maybe | 13:20 |
| jroll | 'ofc jroll' lol, morning NobodyCam | 13:20 |
| dtantsur | jroll, I guess it will be more cli-friendly to specify uuid as well | 13:20 |
| jroll | dtantsur: indeed. let me think about this a bit :) | 13:21 |
| jroll | dtantsur: I also don't want to pollute the nova driver with a ton of configdrive storage backends :P | 13:21 |
| dtantsur | jroll, heh :) I guess, Glance uuid maybe actually enough. Swift URLs maybe used, if this field will be accessed directly by IPA (if it's possible, I don't remember) | 13:23 |
| jroll | dtantsur: it would be possible to make a temp url to give to IPA, using the glance ID. hmm, not the worst idea | 13:24 |
| jroll | and we already depend heavily on swift | 13:24 |
| jroll | s/swift/glance/ | 13:24 |
| jroll | and IPA already depends on swift URLs being a thing | 13:24 |
| jroll | I like this | 13:25 |
| dtantsur | cool :) | 13:28 |
| dtantsur | lucasagomes, re https://review.openstack.org/#/c/94855/8/ironic/conductor/manager.py | 13:28 |
| lucasagomes | dtantsur, yea | 13:28 |
| dtantsur | lucasagomes, why isn't cleanup in finally block? I see you even have explicit tests on this | 13:29 |
| dtantsur | I guess https://bugs.launchpad.net/ironic/+bug/1320090 is about us not cleaning on unsuccessful deletion... | 13:29 |
| *** jgrimm_ has joined #openstack-ironic | 13:29 | |
| lucasagomes | dtantsur, right... I will take a look, yeah right now if the tear_down fails the instance_info won't be deleted | 13:30 |
| lucasagomes | dtantsur, I will take a look, mind adding a comment there? /me is rewritting the migration script | 13:30 |
| dtantsur | done! | 13:31 |
| lucasagomes | dtantsur, thanks for the review, will address it soon | 13:33 |
| *** k4n0 has quit IRC | 13:33 | |
| *** rloo has quit IRC | 13:36 | |
| NobodyCam | :) | 13:37 |
| NobodyCam | morning lucasagomes | 13:37 |
| *** rloo has joined #openstack-ironic | 13:37 | |
| *** ifarkas has quit IRC | 13:37 | |
| *** rloo_ has joined #openstack-ironic | 13:38 | |
| NobodyCam | brb | 13:38 |
| openstackgerrit | Phil Day proposed a change to openstack/ironic: Remove node parameter from power_off() method in virt driver https://review.openstack.org/100197 | 13:38 |
| openstackgerrit | Phil Day proposed a change to openstack/ironic: Add clean shutdown parameters to power_off() https://review.openstack.org/99932 | 13:38 |
| *** rloo has quit IRC | 13:40 | |
| *** romcheg has joined #openstack-ironic | 13:47 | |
| romcheg | dtantsur: Sorry, my phone died while I was waiting for beurocrats | 13:48 |
| romcheg | I left one suggestion about dumping a DB or attaching a rabbit log | 13:49 |
| *** k4n0 has joined #openstack-ironic | 13:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/96228 | 13:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 13:52 |
| *** ifarkas has joined #openstack-ironic | 13:53 | |
| *** pcrews has joined #openstack-ironic | 13:54 | |
| dtantsur | romcheg, thanks, will look | 14:01 |
| lucasagomes | NobodyCam, morning | 14:03 |
| lucasagomes | romcheg, Morning | 14:03 |
| romcheg | Morning lucasagomes, NobodyCam! | 14:03 |
| *** ifarkas_ has joined #openstack-ironic | 14:06 | |
| *** ifarkas has quit IRC | 14:09 | |
| NobodyCam | morning romcheg :) | 14:12 |
| NobodyCam | arg.. looking thru 8 pcap files trying to find out why a node would get a dhcp address but then time out on tftp | 14:14 |
| dtantsur | wow | 14:14 |
| NobodyCam | makes we want to go cross eyed | 14:15 |
| NobodyCam | :-p | 14:15 |
| jroll | dtantsur: any reason this image cache stuff wasn't in image_cache.py to begin with? | 14:18 |
| jroll | dtantsur: other than config options, is there anything pxe-specific about any of these cache classes? | 14:19 |
| dtantsur | jroll, it has PXE-specific configuration and I tried to make ImageCache generic | 14:19 |
| jroll | right, ok | 14:19 |
| dtantsur | jroll, no, only configuration (master dir and cache options) | 14:19 |
| jroll | dtantsur: so if the agent driver wanted to share the TFTPImageCache, should I refactor those config options into ironic/common/tftp.py, I guess? | 14:20 |
| jroll | although those config options are used in PXEImageCache, not TFTPImageCache | 14:20 |
| *** rwsu has joined #openstack-ironic | 14:24 | |
| *** Manishanker has quit IRC | 14:28 | |
| *** Manishanker has joined #openstack-ironic | 14:29 | |
| *** igordcard has quit IRC | 14:30 | |
| *** jdob has quit IRC | 14:31 | |
| *** jdob has joined #openstack-ironic | 14:31 | |
| *** igordcard has joined #openstack-ironic | 14:32 | |
| *** Haomeng has quit IRC | 14:34 | |
| *** mdorman has joined #openstack-ironic | 14:36 | |
| *** Haomeng has joined #openstack-ironic | 14:36 | |
| *** blamar has joined #openstack-ironic | 14:36 | |
| *** jdob has quit IRC | 14:41 | |
| *** jdob has joined #openstack-ironic | 14:41 | |
| *** jistr has quit IRC | 14:45 | |
| *** jistr has joined #openstack-ironic | 14:46 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Set fake_ipmitool in configs for dev doc https://review.openstack.org/100571 | 14:52 |
| *** jgrimm_ is now known as jgrimm | 14:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/96228 | 14:57 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 14:57 |
| *** bcaldwell has joined #openstack-ironic | 15:05 | |
| *** k4n0 has quit IRC | 15:12 | |
| *** Mikhail_D_ltp has quit IRC | 15:13 | |
| *** jistr has quit IRC | 15:21 | |
| *** sysexit has quit IRC | 15:22 | |
| dtantsur | devananda, when you're here, could you give a quick opinion on https://bugs.launchpad.net/python-ironicclient/+bug/1327517 ? | 15:31 |
| *** viktors is now known as viktors|afk | 15:45 | |
| *** blamar has quit IRC | 15:46 | |
| devananda | dtantsur: hi! lemme see ... | 15:50 |
| romcheg | Good morning devananda! | 15:50 |
| dtantsur | devananda, and good morning :) | 15:50 |
| devananda | dtantsur: so, yea, that annoys me too | 15:50 |
| * devananda triages it | 15:50 | |
| * devananda comments on it | 15:51 | |
| dtantsur | I see you also like to use IRC as twitter :D | 15:52 |
| devananda | lol | 15:52 |
| devananda | lucasagomes: so you may want to take a look at ^ as well | 15:53 |
| lucasagomes | will see | 15:53 |
| devananda | lucasagomes: tldr: i agree with lifeless that ironic's CLI should alow the node uuid to be the last parameter | 15:53 |
| devananda | so that eg "ironic node-list | grep foo | xargs ironic node-update replace foo/bar -n " can work | 15:54 |
| lucasagomes | yeah sounds better indeed | 15:54 |
| lucasagomes | we should be pipe friendly :) | 15:54 |
| *** eghobo has joined #openstack-ironic | 15:55 | |
| *** eghobo has quit IRC | 15:56 | |
| *** eghobo has joined #openstack-ironic | 15:56 | |
| NobodyCam | good morning devananda | 16:03 |
| *** blamar has joined #openstack-ironic | 16:04 | |
| *** romcheg has quit IRC | 16:08 | |
| devananda | dtantsur: i think adding an optional -n UUID param is a good way to support that without breaking compat with current users | 16:08 |
| *** jistr has joined #openstack-ironic | 16:08 | |
| dtantsur | makes sense | 16:09 |
| *** vinbs has joined #openstack-ironic | 16:13 | |
| *** foexle has quit IRC | 16:19 | |
| *** ellenh has joined #openstack-ironic | 16:19 | |
| *** hemna has joined #openstack-ironic | 16:23 | |
| dtantsur | have to go, buy! | 16:24 |
| *** dtantsur is now known as dtantsur|afk | 16:24 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Nova Ironic Driver to not get deploy ramdisk and kernel from flavor https://review.openstack.org/95551 | 16:25 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add migration script to instance_info https://review.openstack.org/96136 | 16:25 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Move PXE instance level parameters to instance_info https://review.openstack.org/94855 | 16:25 |
| lucasagomes | dtantsur|afk night | 16:25 |
| NobodyCam | night dtantsur|afk | 16:26 |
| *** athomas has quit IRC | 16:29 | |
| lucasagomes | devananda, fyi, you might want to update the topic of the channel cause we are broke at gate until https://review.openstack.org/#/c/100437/ get's merged | 16:31 |
| *** devananda changes topic to "Ironic's tempest tests are failing currently because of a bad merge last night. Fix is in progress." | 16:32 | |
| *** vinbs has quit IRC | 16:32 | |
| *** devananda changes topic to "Ironic's tempest tests are failing currently because of an incomplete patch merged to tempest last night. Fix is in progress." | 16:32 | |
| *** max_lobur has quit IRC | 16:33 | |
| *** dividehex has quit IRC | 16:35 | |
| *** ndipanov has quit IRC | 16:37 | |
| *** krtaylor has quit IRC | 16:39 | |
| ellenh | hey all, I'm getting contradictory reviews on https://review.openstack.org/#/c/99744/ about whether to format log messages with '%' or ','. can someone help clear this up for me? | 16:41 |
| *** r-mibu has quit IRC | 16:43 | |
| *** harlowja_away is now known as harlowja | 16:43 | |
| *** jistr has quit IRC | 16:44 | |
| rloo_ | ellenh: for logs, you should not use % because it is interpolated. Use ',', so that it is only done if it is going to be logged. | 16:45 |
| devananda | ellenh: hi! you should use "," in log ... yes ^ | 16:46 |
| NobodyCam | ellenh: ya , is hte way to go | 16:46 |
| devananda | ellenh: also, there is some guidance regarding when to log at certain levels here | 16:46 |
| devananda | https://wiki.openstack.org/wiki/LoggingStandards | 16:46 |
| JayF | mrda said the exact opposite in that review fwiw | 16:46 |
| devananda | and here https://review.openstack.org/#/c/94924/ | 16:46 |
| rloo_ | ellenh: it is described here: https://stackoverflow.com/questions/11615053/why-use-multiple-arguments-to-log-instead-of-interpolation | 16:47 |
| JayF | She did it with commas the first time :) | 16:47 |
| rloo_ | whoops. -1 for mrda. | 16:48 |
| devananda | so it's really a tiny optimization | 16:48 |
| devananda | and i think this is the third time we've had to discuss it | 16:48 |
| JayF | I just find the contradictory reviews a little amusing :). | 16:48 |
| devananda | either we need to stop caring, cause it's such a small amount of CPU resources | 16:48 |
| devananda | or we need to make a hacking rule and fix tehm all | 16:49 |
| lucasagomes | yeah I usually don't care | 16:49 |
| lucasagomes | +1 | 16:49 |
| rloo_ | I don't think we should stop caring. It seems 'sloppy'. If people want to be better python programmers, they should know this. | 16:49 |
| rloo_ | A hacking rule would be good, but even w/o it, I think we should use ',' instead of '%'. | 16:50 |
| lucasagomes | off-topic: how to run tempest locally? cause 100437 will need yet another review :( | 16:51 |
| lucasagomes | https://jenkins03.openstack.org/job/check-tempest-dsvm-ironic-nv/1623/console | 16:51 |
| *** jcoufal has quit IRC | 16:51 | |
| *** igordcard has quit IRC | 16:52 | |
| *** martyntaylor1 has joined #openstack-ironic | 16:52 | |
| *** Sam_S has joined #openstack-ironic | 16:52 | |
| *** Penick has joined #openstack-ironic | 16:52 | |
| *** martyntaylor has quit IRC | 16:53 | |
| JayF | devananda: had an idea last night, wanted to run it by you. What would you think about third-party CI for the agent that built + provided for download CoreOS agent images for each agent patch. (as a start) | 16:53 |
| JayF | This is assuming I'm smart enough to setup Openstack Third-Party CI :P | 16:54 |
| *** Sam_S has quit IRC | 16:54 | |
| *** Sam_S has joined #openstack-ironic | 16:56 | |
| devananda | JayF: i think that'd be very helpful | 16:57 |
| *** harlowja has quit IRC | 16:57 | |
| devananda | JayF: i've been told that ya'll have an object store or something .... | 16:57 |
| Shrews | lucasagomes: hrm, i always just use devstack which sets it up for you | 16:57 |
| rloo_ | devananda: yes, it is tiny but it could add up or not. I'd rather not worry about that. And I think it makes us better developers if we understand/use the better way. | 16:57 |
| *** Penick has quit IRC | 16:57 | |
| devananda | JayF: seems like, maybe, stashing built images in that would work? | 16:57 |
| lucasagomes | Shrews, there's a wiki somewhere? | 16:57 |
| devananda | rloo_: I agree with everything you said. | 16:58 |
| JayF | devananda: yep. I think I'm going to work on this Friday. | 16:58 |
| JayF | devananda: we have a 'hackday' at Rackspace | 16:59 |
| Shrews | lucasagomes: for using tempest? or using it with devstack? | 16:59 |
| lucasagomes | Shrews, tempest with devstack | 16:59 |
| Shrews | lucasagomes: it's enabled by default with devstack. Though with ironic, you should add 2 variables to localrc... 1 sec | 16:59 |
| lucasagomes | Shrews, no bothers found something http://docs.openstack.org/developer/tempest/overview.html | 16:59 |
| Shrews | lucasagomes: add these to localrc... | 17:00 |
| Shrews | DEFAULT_INSTANCE_TYPE=baremetal | 17:00 |
| Shrews | TEMPEST_ALLOW_TENANT_ISOLATION=False | 17:00 |
| lucasagomes | Shrews, thanks will give it a go | 17:00 |
| Shrews | lucasagomes: then follow that link you just pasted :) ask if you have trouble | 17:00 |
| lucasagomes | :) will do | 17:01 |
| *** harlowja has joined #openstack-ironic | 17:01 | |
| Shrews | lucasagomes: your tempest.conf will be setup automatically for you, so you just need to run the 'testr run <whatever_test>' command | 17:02 |
| Shrews | from /opt/stack/tempest, of course | 17:02 |
| lucasagomes | Shrews, nice... yeah that conf was what I was missing when I first tried to run it here ConfigFilesNotFoundError: Failed to read some config files: /etc/tempest/tempest.conf | 17:04 |
| lucasagomes | Shrews, ta much! | 17:04 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic-specs: Standardize logging and improve log coverage https://review.openstack.org/94924 | 17:04 |
| *** martyntaylor1 has left #openstack-ironic | 17:05 | |
| devananda | Shrews: how's your infra/config fu? | 17:05 |
| Shrews | devananda: very very stale. so much has changed since i was on infra team | 17:05 |
| *** blamar has quit IRC | 17:05 | |
| devananda | Shrews: in looking at the tempest-dsvm-virtual-ironic job results, i should have seen this earlier -- it's a superset of the tempest-dsvm-ironic job | 17:07 |
| devananda | Shrews: it's *already* doing both | 17:07 |
| Shrews | hehe. so tempest-dsvm-ironic can be removed? | 17:07 |
| openstackgerrit | Chris Behrens proposed a change to openstack/ironic: Eliminate races in Conductor _check_deploy_timeouts https://review.openstack.org/89223 | 17:08 |
| rloo_ | devananda: thx. | 17:08 |
| devananda | Shrews: i'm fairly sure the answer is yes | 17:08 |
| devananda | though.... | 17:08 |
| rloo_ | devananda: did you still want to -2 reviews linked to non-approved blueprints? | 17:08 |
| jroll | dtantsur|afk: thanks for the agent driver spec review :) | 17:11 |
| devananda | rloo_: i'm stil linclined to say yes, but le tme get coffee and food first | 17:14 |
| rloo_ | devananda: no worries. i'm not in a hurry ;) | 17:15 |
| jroll | lifeless: if I'm building a DIB element that runs some software from say, a git repository or pypi package, I would copy that software into the root filesystem in root.d, yes? | 17:16 |
| Shrews | devananda: hrm, looking at that, the api tests are allowed to run in parallel. the scenario (virtual-ironic) are not, which makes sense since they would fail in parallel. | 17:19 |
| Shrews | so if we merge, the api tests would have to go serially. not sure if that's a big deal | 17:19 |
| lucasagomes | aight folks, have a good night | 17:21 |
| Shrews | g'night lucasagomes | 17:21 |
| jroll | night lucas | 17:21 |
| *** lucasagomes is now known as lucas-dinner | 17:22 | |
| NobodyCam | night lucas-dinner | 17:22 |
| Shrews | and i honestly don't know why we even have a postgres job | 17:23 |
| Shrews | i guess 'cause that's what "the cool kids" do :) | 17:23 |
| jroll | gah, topic no longer has the whiteboard D: | 17:23 |
| * Shrews introduces jroll to bookmarks :) | 17:24 | |
| * Shrews also ducks | 17:24 | |
| jroll | pshhhhhhhhhhhh | 17:24 |
| rloo_ | whoa -- who changed the topic? | 17:25 |
| rloo_ | I thought the status was supposed to be in the whiteboard. | 17:25 |
| jroll | only deva can change the topic | 17:25 |
| jroll | I also added it to the whiteboard | 17:25 |
| jroll | with details, even! | 17:25 |
| rloo_ | jroll: so it was YOU! :) | 17:26 |
| rloo_ | jroll: I added a 'Status' above that ;) | 17:26 |
| jroll | :) | 17:26 |
| devananda | gah.... | 17:28 |
| devananda | apparently i've reached the point where i need to say no to all of you :) | 17:29 |
| *** devananda changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 17:29 | |
| rloo_ | or yes? :-) | 17:29 |
| NobodyCam | lol devananda you could try the run and hide method :-p | 17:30 |
| *** linggao has joined #openstack-ironic | 17:31 | |
| devananda | Shrews: yep. because someone frmo the db team thought we should test against both back ends | 17:32 |
| devananda | not just mysql | 17:32 |
| Shrews | hogwash, i say! | 17:32 |
| * Shrews lets his mysql background override logic and reason | 17:33 | |
| jroll | anyone know what to do with deprecated_group when moving an option from no group to a group? | 17:36 |
| jroll | deprecated_group='DEFAULT' maybe? | 17:36 |
| rloo_ | jroll: sounds good. Let us know if that works ;) | 17:38 |
| jroll | yeah, I just dug around and it looks reasonable | 17:39 |
| comstud | so many comments on just logging changes | 17:42 |
| openstackgerrit | Chris Behrens proposed a change to openstack/ironic: Virt driver logging improvements https://review.openstack.org/97047 | 17:42 |
| openstackgerrit | Chris Behrens proposed a change to openstack/ironic: Allow overriding the log level for ironicclient https://review.openstack.org/97048 | 17:42 |
| *** dwalleck has joined #openstack-ironic | 17:44 | |
| *** theDavidAiken has joined #openstack-ironic | 17:44 | |
| *** foexle has joined #openstack-ironic | 17:44 | |
| *** igordcard has joined #openstack-ironic | 17:50 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-specs: Swift Temporary URLs Spec https://review.openstack.org/98904 | 17:51 |
| devananda | comstud: srsly | 17:51 |
| comstud | hehe | 17:53 |
| jroll | is it reasonable that the pxe and agent driver could have separate configs for image cache size/ttl, or should that be a single setting? | 17:54 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/conductor https://review.openstack.org/99744 | 17:55 |
| jroll | and by image cache I mean TFTP image cache, specifically | 17:56 |
| *** Penick has joined #openstack-ironic | 17:58 | |
| *** petertoft has quit IRC | 17:58 | |
| Shrews | comstud: re: you comments on 97047, i think you might be working from an outdated code base. The polling interval *used* to be 10 seconds (hardcoded value), but now uses api_retry_interval which is 2 seconds | 18:03 |
| Shrews | s/working from/remembering/ maybe | 18:05 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/common https://review.openstack.org/99495 | 18:06 |
| Shrews | comstud: https://review.openstack.org/91366 | 18:06 |
| *** max_lobur has joined #openstack-ironic | 18:10 | |
| *** krtaylor has joined #openstack-ironic | 18:12 | |
| comstud | Shrews: I'm not working from outdated code base | 18:17 |
| comstud | I'm working from outdated memories | 18:17 |
| *** ellenh has quit IRC | 18:17 | |
| Shrews | lol | 18:17 |
| comstud | however, we do happen to use 10 as a .conf value, also | 18:17 |
| comstud | so, gee, I can put in a different interval for the logging or something | 18:18 |
| comstud | but that seems to be getting quite ridiculous | 18:18 |
| comstud | These lines are not very long and are easy to grep out if you need to | 18:19 |
| comstud | There's larger issues... like logging 100K of data in 1 log message from ironicclient, etc | 18:19 |
| *** krtaylor has quit IRC | 18:21 | |
| Shrews | comstud: i don't think that's necessary. i posted comments on the review | 18:21 |
| comstud | hehe | 18:22 |
| comstud | thanks | 18:22 |
| comstud | I wouldn't necessarily mind removing the log in get_available_nodes() | 18:23 |
| comstud | That one I've not really found useful.. at least, the uuids | 18:23 |
| Shrews | i think that one only runs 1/min | 18:23 |
| Shrews | so not a big deal | 18:23 |
| comstud | nod | 18:23 |
| Shrews | the others add 60 log entries per minute, but... eh... :) | 18:24 |
| jroll | Shrews: when you have hundreds/thousands of nodes, the one in get_available_nodes is pretty obnoxious :P | 18:25 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 18:25 | |
| *** zul has quit IRC | 18:25 | |
| comstud | Shrews: 30 per minute per VM doing something, yeah.. with the defaults | 18:26 |
| comstud | Shrews: I suspect I'm going to find I want to tone it back | 18:27 |
| comstud | VM/node | 18:27 |
| Shrews | jroll: well, that sounds like a 1st-world-testing problem :) | 18:27 |
| comstud | I'll think about it.. I dunno, maybe should only log on a different interval | 18:27 |
| Shrews | i can't imagine what the logs would look like under debug with hundreds/thousands of nodes | 18:30 |
| * Shrews mind explodes | 18:31 | |
| *** harlowja has quit IRC | 18:33 | |
| jroll | Shrews: well, this is the first world :) | 18:34 |
| jroll | logs get large but not horrible | 18:34 |
| * jroll lunches | 18:34 | |
| *** krtaylor has joined #openstack-ironic | 18:36 | |
| *** foexle has quit IRC | 18:37 | |
| lifeless | jroll: no, typically install.d | 18:39 |
| *** harlowja has joined #openstack-ironic | 18:40 | |
| NobodyCam | brb | 18:50 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Test fake console driver https://review.openstack.org/97629 | 18:52 |
| *** martyntaylor has joined #openstack-ironic | 18:55 | |
| *** lazy_prince has quit IRC | 19:02 | |
| *** ellenh has joined #openstack-ironic | 19:05 | |
| jroll | lifeless: ok, thanks | 19:05 |
| *** foexle has joined #openstack-ironic | 19:06 | |
| *** foexle has quit IRC | 19:13 | |
| comstud | so, i've not been paying attention | 19:16 |
| comstud | What's busted with the gate this time? | 19:16 |
| rloo_ | comstud: take a look at the topic for this channel. | 19:17 |
| comstud | ty | 19:18 |
| *** sysexit has joined #openstack-ironic | 19:19 | |
| adam_g | https://review.openstack.org/#/c/100437/ should hopefully merging soon (its in gate tests atm) and unblock ironic gate | 19:21 |
| NobodyCam | lifeless: off the top of your head do you recall if the CE guys where using tagged vlans for their tests? | 19:26 |
| lifeless | not the ones I was involved in | 19:26 |
| *** romcheg has joined #openstack-ironic | 19:27 | |
| NobodyCam | ack TY | 19:27 |
| *** Penick has quit IRC | 19:32 | |
| NobodyCam | gah... gots to run out for a second, well maybe a few minutes.. bbiafm... | 19:36 |
| JayF | dtantsur|afk: https://review.openstack.org/#/c/99666/ I put a comment on this. I'm not sure if you mean 'stdout will already be logged' or 'stdout is not useful' with your comment there | 19:38 |
| *** pelix has quit IRC | 19:43 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Interactive console support for ipminative driver https://review.openstack.org/97331 | 19:45 |
| *** Penick has joined #openstack-ironic | 19:45 | |
| *** dwalleck has left #openstack-ironic | 19:45 | |
| *** dwalleck has joined #openstack-ironic | 19:46 | |
| *** dwalleck has quit IRC | 19:49 | |
| * NobodyCam is back | 19:49 | |
| adam_g | https://review.openstack.org/#/c/100437/ is merged. please recheck stuff against bug 1330834 | 19:57 |
| rloo_ | yay adam_g! | 19:59 |
| *** Penick has quit IRC | 20:00 | |
| *** Penick has joined #openstack-ironic | 20:02 | |
| *** Penick has quit IRC | 20:06 | |
| *** Penick has joined #openstack-ironic | 20:14 | |
| *** martyntaylor has quit IRC | 20:19 | |
| *** igordcard has quit IRC | 20:19 | |
| *** linggao has quit IRC | 20:19 | |
| *** bcaldwell has quit IRC | 20:19 | |
| *** rushiagr has quit IRC | 20:19 | |
| *** russell_h has quit IRC | 20:19 | |
| *** harlowja has quit IRC | 20:19 | |
| *** mgagne has quit IRC | 20:19 | |
| *** agordeev has quit IRC | 20:19 | |
| *** yjiang5 has quit IRC | 20:19 | |
| *** adam_g has quit IRC | 20:19 | |
| *** rainya has quit IRC | 20:19 | |
| *** wendar has quit IRC | 20:19 | |
| *** stevebaker has quit IRC | 20:19 | |
| *** pquerna has quit IRC | 20:19 | |
| *** zigo has quit IRC | 20:19 | |
| *** Isotopp has quit IRC | 20:19 | |
| *** morgabra has quit IRC | 20:19 | |
| *** mrda-away has quit IRC | 20:19 | |
| *** antonym has quit IRC | 20:19 | |
| *** martyntaylor has joined #openstack-ironic | 20:20 | |
| *** harlowja has joined #openstack-ironic | 20:20 | |
| *** igordcard has joined #openstack-ironic | 20:20 | |
| *** linggao has joined #openstack-ironic | 20:20 | |
| *** bcaldwell has joined #openstack-ironic | 20:20 | |
| *** yjiang5 has joined #openstack-ironic | 20:20 | |
| *** mgagne has joined #openstack-ironic | 20:20 | |
| *** agordeev has joined #openstack-ironic | 20:20 | |
| *** rushiagr has joined #openstack-ironic | 20:20 | |
| *** adam_g has joined #openstack-ironic | 20:20 | |
| *** rainya has joined #openstack-ironic | 20:20 | |
| *** russell_h has joined #openstack-ironic | 20:20 | |
| *** antonym has joined #openstack-ironic | 20:20 | |
| *** mrda-away has joined #openstack-ironic | 20:20 | |
| *** morgabra has joined #openstack-ironic | 20:20 | |
| *** zigo has joined #openstack-ironic | 20:20 | |
| *** pquerna has joined #openstack-ironic | 20:20 | |
| *** Isotopp has joined #openstack-ironic | 20:20 | |
| *** stevebaker has joined #openstack-ironic | 20:20 | |
| *** wendar has joined #openstack-ironic | 20:20 | |
| devananda | adam_g: \o/ | 20:24 |
| *** eghobo has quit IRC | 20:24 | |
| *** lucas-dinner has quit IRC | 20:24 | |
| *** slamont has quit IRC | 20:24 | |
| adam_g | devananda, FYI im doing a full audit of everything we currently skip in tempest, to address some of the concerns earlier in -qa: https://etherpad.openstack.org/p/IronicTempestFeatures | 20:24 |
| adam_g | devananda, among other improvements, im hoping we can get rid of that regex soonish and the ironic jobs can be good citizens finally instead of a wart | 20:25 |
| devananda | adam_g: fantastic, thank you!! | 20:25 |
| devananda | adam_g: also, I think it's time we drop one of our tempest jobs | 20:25 |
| devananda | tempest-dsvm-ironic is a subset of tempest-dsvm-virtual-ironic now | 20:26 |
| adam_g | devananda, the non-virtual one? | 20:26 |
| devananda | yah | 20:26 |
| devananda | except | 20:26 |
| devananda | we should actually drop the virtual one | 20:26 |
| devananda | and make the other two do that | 20:26 |
| devananda | so we are left with tempest-dsvm-ironic and tempest-dsvm-ironic-postgres, both of which run tempest/api/baremetal and tempest/schenario/baremetal stuffs | 20:27 |
| adam_g | devananda, i dont think we'll be able to do that. AFAIK the non-virtual one is a cookie cutter tempest jenkins job, the virutal-ironic requires some special stuff in its job (regex, UCA, etc) | 20:27 |
| devananda | adam_g: right. i mean, we need all the stuff that the virtual job has in it. but we can rename it | 20:27 |
| devananda | keep that job but just call it "tempest-dsvm-ironic" | 20:28 |
| devananda | or we just have one job named "tempest-dsvm-virtual-ironic" | 20:28 |
| devananda | i'm really fine with either one. just seems like there's no point in saying "this is the virtual job" when it will be the _only_ job | 20:29 |
| adam_g | devananda, ah, okay. hmm i thought the *tempest-dsvm-* jobs were derived from a single template but use localrc as input. i'd have to look closer | 20:29 |
| *** jgrimm has quit IRC | 20:29 | |
| devananda | adam_g: i could be wrong. there might be an actual reason for the separate name that I'm not aware of | 20:29 |
| adam_g | devananda, but yeah, i agree they should all be consolidated. after i get done going thru these tests, ill start a wiki for ironic CI Juno where we can track our goals | 20:29 |
| devananda | thanks! | 20:29 |
| adam_g | im still worried the feature flag stuff in tempest is not sufficient, but ill talk to -qa about that | 20:30 |
| devananda | i haven't looked in detail at it. would love to know what you figure out | 20:31 |
| ryanpetrello | adam_g: “so pls poke adam_g in #openstack-ironic to approve it” re: https://review.openstack.org/#/c/97475/ | 20:32 |
| ryanpetrello | says devananda | 20:32 |
| ryanpetrello | so *poke* | 20:32 |
| adam_g | ryanpetrello, poked | 20:33 |
| *** eghobo has joined #openstack-ironic | 20:33 | |
| * ryanpetrello knew he could do it | 20:33 | |
| adam_g | ryanpetrello, i need to try to reproduce that locally from stable/icehouse. i can't seem to find anything useful in the logs | 20:34 |
| *** lucas-dinner has joined #openstack-ironic | 20:36 | |
| *** slamont has joined #openstack-ironic | 20:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/96228 | 21:00 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 21:00 |
| *** dwalleck has joined #openstack-ironic | 21:04 | |
| *** jdob has quit IRC | 21:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/96263 | 21:05 |
| *** dwalleck has quit IRC | 21:05 | |
| *** dwalleck has joined #openstack-ironic | 21:06 | |
| *** romcheg has quit IRC | 21:11 | |
| openstackgerrit | A change was merged to openstack/ironic: Use .png instead of .gif images https://review.openstack.org/99759 | 21:12 |
| openstackgerrit | A change was merged to openstack/ironic: Enforce a minimum time between all IPMI commands https://review.openstack.org/96558 | 21:13 |
| openstackgerrit | A change was merged to openstack/ironic: Replace mknod() with chmod() https://review.openstack.org/99484 | 21:13 |
| *** dwalleck has quit IRC | 21:15 | |
| linggao | NobodyCam, ping | 21:16 |
| NobodyCam | png | 21:16 |
| *** Penick has quit IRC | 21:16 | |
| NobodyCam | pong even | 21:16 |
| linggao | Hey NobodyCam. it seems a lot of patches failed Jekins test. | 21:17 |
| linggao | I mean before 3:30pm EDT | 21:18 |
| linggao | Is it system problem or problems with the patches. | 21:19 |
| NobodyCam | the gate broke | 21:19 |
| jroll | linggao: a tempest commit broke our gate | 21:19 |
| linggao | I see. | 21:19 |
| NobodyCam | jroll: is htere a recheck bug #? | 21:19 |
| NobodyCam | there even | 21:19 |
| jroll | 1330834 | 21:19 |
| jroll | I just put it in the whiteboard | 21:19 |
| linggao | how do I restart the Jekins for my patch? | 21:19 |
| *** max_lobur has quit IRC | 21:19 | |
| jroll | linggao: "recheck bug 1330834" | 21:20 |
| *** max_lobur has joined #openstack-ironic | 21:20 | |
| NobodyCam | linggao: ^^^ as a comment | 21:20 |
| linggao | jroll, thanks!!! | 21:20 |
| jroll | np :) | 21:20 |
| *** max_lobur has quit IRC | 21:20 | |
| adam_g | ryanpetrello, i think its a devstack change that needs to be backported to stable | 21:21 |
| linggao | great, I have restarted the Jekins varification. feeling good :). | 21:22 |
| linggao | Talk to you guys tomorrow. | 21:23 |
| ryanpetrello | adam_g: blargh | 21:23 |
| linggao | NobodyCam, thank you!!! too. | 21:23 |
| *** foexle has joined #openstack-ironic | 21:23 | |
| *** linggao has quit IRC | 21:24 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding swift temp url support https://review.openstack.org/81391 | 21:25 |
| *** jgrimm has joined #openstack-ironic | 21:28 | |
| *** Penick has joined #openstack-ironic | 21:36 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding swift temp url support https://review.openstack.org/81391 | 21:42 |
| adam_g | ryanpetrello, this should fix it https://review.openstack.org/#/c/100698/ | 21:43 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding swift temp url support https://review.openstack.org/81391 | 21:43 |
| ryanpetrello | wow, *thanks* for hunting this down O_O | 21:44 |
| *** sysexit has quit IRC | 21:47 | |
| *** Mikhail_D_ltp has quit IRC | 21:50 | |
| *** mrda-away is now known as mrda | 21:54 | |
| mrda | morning Ironic! | 21:55 |
| devananda | g'morning, mrda ! | 21:55 |
| mrda | \o | 21:55 |
| jroll | heya mrda | 21:58 |
| mrda | hey jroll | 21:59 |
| * devananda finishes meetings and emails, takes a break | 22:16 | |
| NobodyCam | mornign mrda | 22:16 |
| mrda | hey NobodyCam | 22:17 |
| mrda | rloo_: thanks for the email! | 22:18 |
| *** radsy has joined #openstack-ironic | 22:32 | |
| NobodyCam | quick walkies brb | 22:36 |
| rloo_ | mrda: yw :-) | 22:40 |
| *** jgrimm has quit IRC | 22:52 | |
| *** foexle has quit IRC | 23:03 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Drivers determine acceptable power states https://review.openstack.org/86744 | 23:14 |
| *** lucas-dinner has quit IRC | 23:16 | |
| *** mdorman_ has joined #openstack-ironic | 23:25 | |
| *** mdorman_ has quit IRC | 23:26 | |
| *** mdorman has quit IRC | 23:28 | |
| *** martyntaylor has quit IRC | 23:31 | |
| *** igordcard has quit IRC | 23:35 | |
| *** hemna has quit IRC | 23:51 | |
| devananda | rloo_: hi! | 23:55 |
| devananda | rloo_: i need to run in a few minutes, but want to chat about BPs with you for a minute | 23:55 |
| devananda | i think we're all trying to figure out how to work with the new spec process | 23:56 |
| devananda | the standardize-logging spec that i tossed up is an interesting example | 23:56 |
| lifeless | devananda: as is async API :) | 23:57 |
| devananda | it's not really a feature ... or a bug ... the more we discuss it the less I think it's appropriate for this process | 23:57 |
| * JayF finds the agent spec to be a good example ;) | 23:57 | |
| devananda | lifeless: the async API is a great example of where the process IS needed | 23:57 |
| devananda | JayF: as is the agent :) | 23:57 |
| lifeless | devananda: Oh :( | 23:57 |
| lifeless | erm | 23:57 |
| lifeless | :) | 23:57 |
| lifeless | cold fingers | 23:57 |
| devananda | so, yes, i think we should be -2'ing patches (particularly large patches) which are clearly implementing a blueprint/spec which has not been approved yet | 23:57 |
| devananda | i think that will help us encourange ourselves to go review specs (which we're starting to do this week, which is great) | 23:58 |
| devananda | we shouldn't uphold process merely for the sake of process | 23:58 |
| devananda | which I think sometimes we can all do | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!