| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Replace use of functools.wraps() with six.wraps() https://review.openstack.org/540187 | 00:26 |
|---|---|---|
| *** rcernin has joined #openstack-ironic | 00:27 | |
| *** yangyapeng has quit IRC | 00:31 | |
| *** yangyapeng has joined #openstack-ironic | 00:32 | |
| *** hshiina|afk is now known as hshiina | 00:32 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector master: Replace use of functools.wraps() with six.wraps() https://review.openstack.org/540191 | 00:33 |
| jlvillal | hshiina, Good catch on figuring out it was the METRIC.timer() decorator breaking things | 00:33 |
| jlvillal | hshiina, I guess we have been broken all this time for the 'timeout' method when we have used get_signature. Sadly :( | 00:34 |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient master: Replace use of functools.wraps() with six.wraps() https://review.openstack.org/540193 | 00:35 |
| *** yangyapeng has quit IRC | 00:36 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Remove unused code in unittest https://review.openstack.org/540194 | 00:38 |
| openstackgerrit | Merged openstack/ironic master: node_tag_exists(): raise exception if bad node https://review.openstack.org/533700 | 00:38 |
| openstackgerrit | Merged openstack/ironic master: [api-ref] clarify what /v1/lookup returns https://review.openstack.org/539676 | 00:38 |
| hshiina | jlvillal: power methods with timeout seems to be saved by task_manager.require_exclusive_lock if it is used. | 00:42 |
| hshiina | this uses six.wraps | 00:42 |
| openstackgerrit | James E. Blair proposed openstack/networking-generic-switch master: Zuul: Remove project name https://review.openstack.org/540195 | 00:42 |
| jlvillal | hshiina, Hmm. I'm not sure. I would think each decorator would have to use it... | 00:45 |
| jlvillal | hshiina, But I haven't actually done any testing or investigating. | 00:46 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Deprecate Classic Drivers https://review.openstack.org/540153 | 00:55 |
| openstackgerrit | wangdequn proposed openstack/ironic master: correct referenced url in comments https://review.openstack.org/534710 | 01:08 |
| *** priteau has joined #openstack-ironic | 01:08 | |
| *** priteau has quit IRC | 01:12 | |
| rpioso | Good nite | 01:12 |
| *** rpioso is now known as rpioso|afk | 01:12 | |
| *** liuzz has joined #openstack-ironic | 01:14 | |
| *** liuzz_ has quit IRC | 01:14 | |
| *** jlabarre has quit IRC | 01:22 | |
| *** tiendc has joined #openstack-ironic | 01:23 | |
| *** yangyapeng has joined #openstack-ironic | 01:26 | |
| *** mjturek has quit IRC | 01:27 | |
| *** zhangfei has joined #openstack-ironic | 01:39 | |
| *** crushil has joined #openstack-ironic | 01:49 | |
| *** yolanda_ has joined #openstack-ironic | 02:06 | |
| *** yolanda has quit IRC | 02:07 | |
| *** namnh has joined #openstack-ironic | 02:46 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: DNM: See if 'timeout' paremeter is visible with reflection https://review.openstack.org/540219 | 02:55 |
| *** racedo has quit IRC | 03:00 | |
| *** racedo has joined #openstack-ironic | 03:01 | |
| *** liuzz has quit IRC | 03:06 | |
| *** priteau has joined #openstack-ironic | 03:09 | |
| *** priteau has quit IRC | 03:13 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 03:15 |
| rloo | hshiina: I just saw your patch ^^. Did you see jroll's patch that merged today: https://review.openstack.org/539961 | 03:16 |
| patchbot | patch 539961 - ironic - Remove mode argument from boot.(prepare|clean_up)_... (MERGED) | 03:16 |
| rloo | hshiina: also, jlvillal has a patch to fix the metrics decorator in ironic-lib: https://review.openstack.org/#/c/540031/ | 03:17 |
| patchbot | patch 540031 - ironic-lib - Use six.wraps() for Metrics so decorated methods c... | 03:17 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Fix handling of 'timeout' parameter to power methods https://review.openstack.org/540150 | 03:18 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update a CI job for rescue mode https://review.openstack.org/528704 | 03:18 |
| rloo | hshiina: and as far as timeout goes, it was broken too :-( there's the fix ^^ | 03:18 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode https://review.openstack.org/537821 | 03:19 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests https://review.openstack.org/538119 | 03:20 |
| *** lennyb has quit IRC | 03:22 | |
| *** sapd__ has joined #openstack-ironic | 03:23 | |
| *** sapd_ has quit IRC | 03:23 | |
| *** rloo has quit IRC | 03:26 | |
| openstackgerrit | inspurericzhang proposed openstack/ironic-inspector master: fixed some "ironic" misspelling https://review.openstack.org/540233 | 03:27 |
| hshiina | rloo, jroll: thank you for the fix. rescue mode worked in my local evnironment with the latest master. | 04:04 |
| *** harlowja_ has quit IRC | 04:09 | |
| *** devananda has quit IRC | 04:38 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] Firmware based boot from volume for iLO drivers https://review.openstack.org/539158 | 04:48 |
| *** harlowja has joined #openstack-ironic | 04:56 | |
| *** vsaienk0 has joined #openstack-ironic | 04:59 | |
| *** vsaienk0 has quit IRC | 05:09 | |
| *** links has joined #openstack-ironic | 05:15 | |
| *** dpawar has joined #openstack-ironic | 05:16 | |
| *** jaganathan has quit IRC | 05:18 | |
| *** yangyapeng has quit IRC | 05:25 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: TESTING: Is gate broken? https://review.openstack.org/540259 | 05:54 |
| *** dpawar has quit IRC | 05:58 | |
| *** zhangfei has quit IRC | 05:59 | |
| jlvillal | dtantsur|afk, TheJulia sambett|afk vdrok: The ironic-tempest-plugin gate "may" be broken. I updated the whiteboard. There is a test job running now: https://review.openstack.org/540259 | 06:00 |
| patchbot | patch 540259 - ironic-tempest-plugin - TESTING: Is gate broken? | 06:00 |
| jlvillal | I'm going to go to sleep :) | 06:00 |
| *** yangyapeng has joined #openstack-ironic | 06:00 | |
| jlvillal | hshiina, ^^^ as an FYI | 06:01 |
| hshiina | jlvillal: good night | 06:02 |
| openstackgerrit | James E. Blair proposed openstack/ironic-inspector master: Zuul: Remove project name https://review.openstack.org/540262 | 06:05 |
| *** marios has joined #openstack-ironic | 06:06 | |
| *** rcernin has quit IRC | 06:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.openstack.org/540264 | 06:21 |
| *** tiendc has quit IRC | 06:23 | |
| *** vsaienk0 has joined #openstack-ironic | 06:39 | |
| *** crushil has quit IRC | 06:43 | |
| *** mjura has joined #openstack-ironic | 06:51 | |
| *** olivierbourdon38 has quit IRC | 06:56 | |
| *** charcol-v2 has quit IRC | 06:57 | |
| *** jaganathan has joined #openstack-ironic | 06:59 | |
| *** makowals has quit IRC | 07:01 | |
| *** makowals has joined #openstack-ironic | 07:02 | |
| *** threestrands has quit IRC | 07:05 | |
| *** jaganathan has quit IRC | 07:09 | |
| *** priteau has joined #openstack-ironic | 07:10 | |
| *** priteau has quit IRC | 07:15 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:18 | |
| openstackgerrit | octopuszhang proposed openstack/networking-generic-switch master: Remove save configuration implement https://review.openstack.org/540295 | 07:18 |
| *** dpawar has joined #openstack-ironic | 07:21 | |
| *** vsaienk0 has quit IRC | 07:23 | |
| *** yolanda_ has quit IRC | 07:23 | |
| *** jaganathan has joined #openstack-ironic | 07:25 | |
| openstackgerrit | wangdequn proposed openstack/ironic master: correct referenced url in comments https://review.openstack.org/534710 | 07:26 |
| *** vsaienk0 has joined #openstack-ironic | 07:27 | |
| *** olivierbourdon38 has joined #openstack-ironic | 07:30 | |
| *** olivierbourdon38 has quit IRC | 07:32 | |
| pas-ha | morning ironic, TGIF :) | 07:38 |
| etingof | happy Friday, pas-ha ;) | 07:41 |
| *** AlexeyAbashkin has quit IRC | 07:47 | |
| *** tuanla____ has joined #openstack-ironic | 07:48 | |
| *** jtomasek has joined #openstack-ironic | 07:50 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:55 | |
| *** zhenguo has joined #openstack-ironic | 07:59 | |
| *** vabada2 has joined #openstack-ironic | 08:06 | |
| *** vabada2 has quit IRC | 08:06 | |
| *** vabada has quit IRC | 08:06 | |
| *** vabada has joined #openstack-ironic | 08:07 | |
| *** pcaruana has joined #openstack-ironic | 08:09 | |
| *** charcol-v2 has joined #openstack-ironic | 08:15 | |
| *** e0ne has joined #openstack-ironic | 08:23 | |
| *** tesseract has joined #openstack-ironic | 08:26 | |
| *** amarao has joined #openstack-ironic | 08:31 | |
| *** amarao1 has joined #openstack-ironic | 08:32 | |
| *** rbartal has joined #openstack-ironic | 08:42 | |
| *** yangyapeng has quit IRC | 08:55 | |
| *** yangyapeng has joined #openstack-ironic | 08:55 | |
| *** makowals has quit IRC | 08:57 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 08:59 |
| *** makowals has joined #openstack-ironic | 08:59 | |
| *** mgoddard_ has joined #openstack-ironic | 09:03 | |
| *** athomas has joined #openstack-ironic | 09:03 | |
| openstackgerrit | octopuszhang proposed openstack/networking-generic-switch master: Remove save configuration implement https://review.openstack.org/540295 | 09:03 |
| *** rbartal has quit IRC | 09:10 | |
| *** hshiina is now known as hshiina|afk | 09:13 | |
| *** exodusftw has quit IRC | 09:16 | |
| *** exodusftw has joined #openstack-ironic | 09:24 | |
| openstackgerrit | Merged openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 09:24 |
| openstackgerrit | Merged openstack/ironic master: Fix configure-networking docs https://review.openstack.org/540085 | 09:24 |
| *** marios has quit IRC | 09:26 | |
| *** jaganathan has quit IRC | 09:27 | |
| *** jaganathan has joined #openstack-ironic | 09:27 | |
| *** marios has joined #openstack-ironic | 09:28 | |
| *** marios has joined #openstack-ironic | 09:29 | |
| *** racedo has quit IRC | 09:31 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add creation of openrc file when keystone enabled https://review.openstack.org/540096 | 09:37 |
| *** jaganathan has quit IRC | 09:44 | |
| *** derekh has joined #openstack-ironic | 09:44 | |
| openstackgerrit | octopuszhang proposed openstack/networking-generic-switch master: Fix bind_port error https://review.openstack.org/540295 | 09:52 |
| openstackgerrit | octopuszhang proposed openstack/networking-generic-switch master: Fix bind_port error https://review.openstack.org/540295 | 09:54 |
| *** yangyapeng has quit IRC | 09:55 | |
| *** hshiina|afk has quit IRC | 09:59 | |
| *** jaganathan has joined #openstack-ironic | 09:59 | |
| *** makowals has quit IRC | 10:01 | |
| *** tuanla____ has quit IRC | 10:03 | |
| *** makowals has joined #openstack-ironic | 10:05 | |
| *** dsneddon has quit IRC | 10:09 | |
| *** dsneddon has joined #openstack-ironic | 10:12 | |
| *** ijw has joined #openstack-ironic | 10:13 | |
| *** hieulq_ has quit IRC | 10:13 | |
| *** AlexeyAbashkin has quit IRC | 10:14 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 10:14 | |
| *** ijw has quit IRC | 10:18 | |
| *** ijw has joined #openstack-ironic | 10:19 | |
| *** racedo has joined #openstack-ironic | 10:19 | |
| *** AlexeyAbashkin has quit IRC | 10:23 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 10:23 | |
| *** priteau has joined #openstack-ironic | 10:29 | |
| *** priteau has quit IRC | 10:30 | |
| *** priteau has joined #openstack-ironic | 10:30 | |
| *** ijw has quit IRC | 10:36 | |
| *** ijw has joined #openstack-ironic | 10:36 | |
| *** olivierbourdon38 has joined #openstack-ironic | 10:39 | |
| *** ijw_ has joined #openstack-ironic | 10:42 | |
| *** ijw has quit IRC | 10:45 | |
| *** d0ugal has quit IRC | 10:48 | |
| *** smoriya has joined #openstack-ironic | 10:50 | |
| *** smoriya has quit IRC | 10:50 | |
| *** d0ugal has joined #openstack-ironic | 10:56 | |
| *** lucas-afk is now known as lucasagomes | 11:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove sample policy and config files https://review.openstack.org/536349 | 11:21 |
| *** exodusftw has quit IRC | 11:24 | |
| *** exodusftw has joined #openstack-ironic | 11:29 | |
| *** dsneddon has quit IRC | 11:33 | |
| *** ijw_ has quit IRC | 11:34 | |
| jroll | morning everyone | 11:37 |
| * jroll says you're welcome to hshiina even though they are not here | 11:38 | |
| sambett|afk | Morning jroll | 11:38 |
| jroll | oh no, jlvillal was right about the tempest plugin gate :( | 11:38 |
| sambett|afk | :( | 11:39 |
| jroll | some of this: Details: (BaremetalAgentIpmitoolWholedisk:setUpClass) Node 85d5de24-94fd-4b6e-9c86-c14666b8e9f1 failed to reach power_state=['power on'] within the required time (600 s). | 11:40 |
| jroll | some of this (ruh roh): Details: {u'error_message': u'{"debuginfo":null,"faultcode":"Client","faultstring":"Driver ipmi does not support inspect (disabled or not implemented)."}'} | 11:40 |
| sambett|afk | :/ | 11:41 |
| *** sambett|afk is now known as sambetts | 11:41 | |
| jroll | not sure if the first one is sporadic or not, but the second one probably comes from https://review.openstack.org/#/c/537415 | 11:42 |
| patchbot | patch 537415 - ironic-inspector - Switch the CI to hardware types (MERGED) | 11:42 |
| jroll | dtantsur|afk: ^^ fyi | 11:42 |
| jroll | aha, that's the ocata job :x | 11:43 |
| *** ijw has joined #openstack-ironic | 11:45 | |
| * jroll will fix in a moment | 11:46 | |
| jroll | though I don't see why it wouldn't be supported | 11:47 |
| sambetts | jroll: it is supported, as long as inspector is enabled in the config file | 11:48 |
| jroll | right | 11:48 |
| jroll | [inspect]/enabled = True | 11:48 |
| sambetts | huh :/ | 11:48 |
| jroll | oh, enabled_inspect_interfaces isn't set despite the https://review.openstack.org/#/c/537415/2/devstack/plugin.sh | 11:49 |
| patchbot | patch 537415 - ironic-inspector - Switch the CI to hardware types (MERGED) | 11:49 |
| sambetts | jroll: I feel like that might be a devstack plugin order thing | 11:51 |
| jroll | ah, the devstack bits need to be backported, I think | 11:51 |
| jroll | because the ironic-tempest-dsvm-ironic-inspector-ocata will use the master zuul config but ocata devstack plugins | 11:51 |
| sambetts | oh right? the bits that process the *_INTERFACES options | 11:51 |
| sambetts | ? | 11:52 |
| jroll | yeah | 11:52 |
| jroll | or, or, we make this change not happen on ocata: https://review.openstack.org/#/c/537415/2/playbooks/legacy/ironic-inspector-tempest-dsvm-discovery/run.yaml | 11:52 |
| patchbot | patch 537415 - ironic-inspector - Switch the CI to hardware types (MERGED) | 11:52 |
| sambetts | Not sure why we are making it happen on ocata personally, I missed that conversation | 11:52 |
| * jroll backs up | 11:53 | |
| jroll | here's the patch, it's to our tempest plugin on master: https://review.openstack.org/#/c/540259/ | 11:53 |
| patchbot | patch 540259 - ironic-tempest-plugin - TESTING: Is gate broken? | 11:53 |
| jroll | which has jobs that run ocata/pike code, like ironic-tempest-dsvm-ironic-inspector-ocata | 11:53 |
| jroll | those jobs still use the zuul config from master, because that's where the patch is | 11:54 |
| sambetts | yup got that, because its branchless etc etc | 11:54 |
| jroll | so when this patch landed, it changed how that ocata job is configured: https://review.openstack.org/#/c/537415/ | 11:54 |
| patchbot | patch 537415 - ironic-inspector - Switch the CI to hardware types (MERGED) | 11:54 |
| jroll | but we don't have the devstack code in ocata to support it | 11:54 |
| jroll | it should really only change the job config for queens | 11:54 |
| sambetts | jroll: right thats why I'm trying to work out why it broke ocata | 11:55 |
| sambetts | jroll: is it because its using the same job definition | 11:55 |
| sambetts | and therefore the same local.conf | 11:55 |
| sambetts | for all releases | 11:55 |
| sambetts | ? | 11:55 |
| jroll | kind of | 11:56 |
| jroll | it's using the same job definition, and therefore the same local.conf, for any patches to ironic-tempest-plugin master branch | 11:56 |
| jroll | including the ocata/pike jobs that run against those changes | 11:56 |
| sambetts | jroll: because its pulling the job definition from ironic-insepctors master branch nots its ocata/pike branches | 11:57 |
| *** namnh has quit IRC | 11:57 | |
| sambetts | ? | 11:57 |
| jroll | sambetts: yes, AIUI | 11:58 |
| *** dtantsur|afk is now known as dtantsur | 11:58 | |
| dtantsur | morning/afternoon | 11:58 |
| sambetts | o/ dtantsur | 11:58 |
| jroll | morning dtantsur, just in time :P | 11:58 |
| dtantsur | oh fun, we're using normal playbooks on ocata... | 11:59 |
| dtantsur | jroll: I hope I did not step on your shoes with https://review.openstack.org/#/c/536349/ | 12:00 |
| patchbot | patch 536349 - ironic - Remove sample policy and config files | 12:00 |
| jroll | dtantsur: how dare you fix my release note :) | 12:01 |
| * dtantsur EVIL | 12:01 | |
| dtantsur | jroll: did not expect you to be up at this time | 12:01 |
| * jroll notices he did not publish his comments on PS4 | 12:01 | |
| jroll | dtantsur: it's 7am, not too early, honestly I got online to send someone a reddit thread and wound up in a CI rabbithole :P | 12:02 |
| dtantsur | that's how it works! | 12:02 |
| * dtantsur needs to practice a bit of bass before people make him fix the CI | 12:02 | |
| jroll | heh | 12:03 |
| jroll | I'll put up a patch shortly | 12:03 |
| dtantsur | jroll: just note that the inspector change is likely unrelated, the inspector job is defined in ironic | 12:03 |
| *** pcaruana has quit IRC | 12:04 | |
| dtantsur | jroll: https://review.openstack.org/#/c/536875/5/playbooks/legacy/tempest-dsvm-ironic-inspector/run.yaml | 12:04 |
| patchbot | patch 536875 - ironic - Switch the CI to hardware types (MERGED) | 12:04 |
| dtantsur | jroll: and I wonder if we can just backport the change introducing support for inspect interfaces.. | 12:04 |
| dtantsur | anyway, thanks for looking into it! | 12:04 |
| jroll | dtantsur: ah, I see, thanks | 12:05 |
| sambetts | jroll, dtantsur: Just putting up a patch quick to test something, zuul recently changed branch_override to override_checkout, they did support both for a while, but I wonder if they've just removed the the old one | 12:06 |
| dtantsur | sambetts: I don't think it's related. ironic-tempest-plugin master pulls playbooks from ironic master | 12:07 |
| * dtantsur brb | 12:07 | |
| sambetts | thats why I'm wondering if its the override thats not working right, e.g. its not reading the override and is looking for the job on master | 12:09 |
| jroll | sambetts: no, job configs are intentionally all combined from the same branch, that's how it's always worked | 12:10 |
| openstackgerrit | Sam Betts proposed openstack/ironic-tempest-plugin master: Switch to override_checkout for stable branch jobs https://review.openstack.org/540355 | 12:10 |
| sambetts | hmmm so we really need to make our CI jobs branchless too ... like infra's ones | 12:11 |
| jroll | not sure what you mean | 12:12 |
| jroll | jobs like this one are meant to make sure we don't break ocata CI with tempest-plugin changes | 12:12 |
| jroll | not sure how to do that any other way | 12:12 |
| sambetts | project-config / openstack-zuul-jobs etc are all branchless, so they don't have this issue everyone pulls the same version of the jobs for all branches | 12:12 |
| sambetts | and you just handle it in the job itself | 12:13 |
| sambetts | I guess :/ | 12:13 |
| sambetts | like we did before with the flags in the devstack local.conf, except this time we can do it in a real templating language | 12:13 |
| sambetts | j2 | 12:13 |
| jroll | I'm not sure that solves the problem, rather just pushes it to all jobs on all branches so we're forced to think about it | 12:14 |
| sambetts | just a possble solution (obviously not ideal) | 12:14 |
| jroll | gotcha | 12:15 |
| sambetts | I think it was in denver, when monty was talking about zuulv3, the impression I got was that zuul would only load jobs off master for each project for all branches, I think at some point that changed | 12:16 |
| jroll | yeah, I believe it did, not sure when, wasn't around for the transition | 12:19 |
| dtantsur | yep, it did change (thankfully) | 12:19 |
| sambetts | "This value is also used to help select which variants of a job to run. If override-checkout is set, then Zuul will use this value instead of the branch of the item being tested when collecting jobs to run." | 12:19 |
| jroll | orly | 12:19 |
| dtantsur | sambetts: oh, this sounds like something we need! | 12:20 |
| jroll | nice, let's see how it goes then | 12:20 |
| * jroll puts up the other, just in case it's needed | 12:20 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-tempest-plugin master: Switch to override-checkout for stable branch jobs https://review.openstack.org/540355 | 12:22 |
| sambetts | just revised it a little because I think we were using it slightly wrong too | 12:23 |
| *** pcaruana has joined #openstack-ironic | 12:23 | |
| sambetts | I've used this option heavily for networking-cisco because we support mitaka to pike backwards | 12:23 |
| sambetts | on our master branch | 12:23 |
| jroll | surprising that branch_override didn't do the same thing | 12:24 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Don't use hardware types on Ocata CI https://review.openstack.org/540359 | 12:24 |
| *** dpawar has quit IRC | 12:24 | |
| sambetts | I think that mgiht have been because of how it was being used in the "vars:" section | 12:24 |
| jroll | ah | 12:24 |
| sambetts | its should have been "branch-override" in the top of the job | 12:25 |
| jroll | yes, this makes sense | 12:25 |
| jroll | thanks :) | 12:25 |
| sambetts | it was our feature request that changed it from branch-override to override-checkout because we needed the ability to checkout tags e.g. "mitaka-eol" | 12:25 |
| sambetts | and they used to specifically check for a branch | 12:26 |
| sambetts | instead of just using it as a commitish reference | 12:26 |
| jroll | heh, nice | 12:26 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-tempest-plugin master: TESTING: Is gate broken? https://review.openstack.org/540259 | 12:29 |
| jroll | added depends-on for mine, but I think yours will do the right thing | 12:29 |
| sambetts | if the docs are to be believed :) | 12:30 |
| dtantsur | docs cannot lie | 12:30 |
| sambetts | ;) | 12:31 |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-create-vm-nodes: Set NIC model to 'virtio' https://review.openstack.org/539258 | 12:39 |
| *** yangyapeng has joined #openstack-ironic | 12:42 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Update docs and generate config file example https://review.openstack.org/539010 | 12:51 |
| openstackgerrit | Merged openstack/sushy-tools master: Zuul: Remove project name https://review.openstack.org/539906 | 12:54 |
| *** jlabarre has joined #openstack-ironic | 12:55 | |
| *** gcb has quit IRC | 12:57 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/virtualbmc master: Remove devices/*/boot when setting bootdev https://review.openstack.org/540368 | 12:58 |
| *** ijw has quit IRC | 13:00 | |
| *** ijw has joined #openstack-ironic | 13:01 | |
| *** ricardoas-away is now known as ricardoas | 13:02 | |
| *** ijw_ has joined #openstack-ironic | 13:05 | |
| *** ijw has quit IRC | 13:07 | |
| *** ijw_ has quit IRC | 13:09 | |
| *** hamzy has quit IRC | 13:13 | |
| *** vsaienk0 has quit IRC | 13:14 | |
| *** d0ugal has quit IRC | 13:16 | |
| *** d0ugal has joined #openstack-ironic | 13:16 | |
| *** d0ugal has quit IRC | 13:16 | |
| *** d0ugal has joined #openstack-ironic | 13:16 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/virtualbmc master: Remove devices/*/boot when setting bootdev https://review.openstack.org/540368 | 13:18 |
| *** hamzy has joined #openstack-ironic | 13:18 | |
| *** vsaienk0 has joined #openstack-ironic | 13:29 | |
| dtantsur | sambetts: if you have a minute: https://review.openstack.org/#/c/536349/ | 13:33 |
| patchbot | patch 536349 - ironic - Remove sample policy and config files | 13:33 |
| dtantsur | jroll: do you plan on a similar ^^^ change for ironic-inspector? | 13:33 |
| *** saop has joined #openstack-ironic | 13:34 | |
| *** saop has quit IRC | 13:35 | |
| *** trown|outtypewww is now known as trown | 13:36 | |
| *** ricardoas has quit IRC | 13:39 | |
| *** pcaruana has quit IRC | 13:46 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent master: Add documentation for rescue mode https://review.openstack.org/431622 | 13:48 |
| *** ijw has joined #openstack-ironic | 14:00 | |
| *** ijw has quit IRC | 14:05 | |
| *** pcaruana has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Follow-up patch for api-ref documentation for rescue https://review.openstack.org/540392 | 14:07 |
| jroll | dtantsur: I can do that, if you like | 14:08 |
| *** jcoufal has joined #openstack-ironic | 14:08 | |
| dtantsur | jroll: if you have a minute :) I'd like the two projects to be consistent | 14:08 |
| jroll | of course | 14:09 |
| jroll | dtantsur: would you like me to keep devstack/example.local.conf, I assume? I don't see something similar in docs | 14:11 |
| *** edleafe is now known as figleaf | 14:11 | |
| dtantsur | jroll: yep, it is for docs actually | 14:12 |
| jroll | cool, thank you | 14:12 |
| *** ijw has joined #openstack-ironic | 14:18 | |
| *** ijw has quit IRC | 14:19 | |
| *** ijw has joined #openstack-ironic | 14:19 | |
| *** Goneri has joined #openstack-ironic | 14:22 | |
| *** ijw_ has joined #openstack-ironic | 14:23 | |
| *** ijw has quit IRC | 14:23 | |
| *** jaganathan has quit IRC | 14:24 | |
| *** ijw_ has quit IRC | 14:25 | |
| *** ijw has joined #openstack-ironic | 14:25 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add creation of openrc file when keystone enabled https://review.openstack.org/540096 | 14:26 |
| *** lucasagomes is now known as lucas-hungry | 14:26 | |
| *** links has quit IRC | 14:30 | |
| *** cdearborn has joined #openstack-ironic | 14:32 | |
| *** rbudden has joined #openstack-ironic | 14:32 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-inspector master: Remove sample policy and config files https://review.openstack.org/540402 | 14:36 |
| jroll | dtantsur: that should do it | 14:36 |
| dtantsur | thanks! | 14:36 |
| *** mjturek has joined #openstack-ironic | 14:47 | |
| openstackgerrit | Merged openstack/ironic master: Update api-ref for port group create https://review.openstack.org/537803 | 14:48 |
| *** ricardoas has joined #openstack-ironic | 14:50 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 14:51 |
| * TheJulia tries to wake up | 14:52 | |
| dtantsur | morning TheJulia | 14:52 |
| *** r-daneel has joined #openstack-ironic | 14:52 | |
| TheJulia | is it really morning? | 14:52 |
| dtantsur | dunno, there was some sun a few hours ago | 14:57 |
| *** saop has joined #openstack-ironic | 14:59 | |
| saop | TheJulia, Hello I updated the comments please review this https://review.openstack.org/#/c/539158/ | 14:59 |
| patchbot | patch 539158 - ironic - [docs] Firmware based boot from volume for iLO dri... | 14:59 |
| TheJulia | I overslept a little and now feel like a train hit me | 15:00 |
| saop | TheJulia, :) take your time | 15:00 |
| *** r-daneel has quit IRC | 15:01 | |
| *** Goneri has quit IRC | 15:01 | |
| dtantsur | TheJulia: this can happen, yeah :( sometimes too much sleep is just as bad as not enough sleep | 15:04 |
| dtantsur | saop: lemme check it | 15:04 |
| TheJulia | dtantsur: thanks | 15:05 |
| * TheJulia is sipping coffee to try and feel awake enough to start her day | 15:05 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Fix nits in networking-baremetal docs https://review.openstack.org/540413 | 15:06 |
| dtantsur | saop: reviewed. it's close, just a few minor things | 15:10 |
| *** pcaruana has quit IRC | 15:13 | |
| *** pcaruana has joined #openstack-ironic | 15:14 | |
| *** Goneri has joined #openstack-ironic | 15:17 | |
| *** rpioso|afk is now known as rpioso | 15:22 | |
| rpioso | Morning | 15:23 |
| *** livelace-link has joined #openstack-ironic | 15:23 | |
| TheJulia | good morning rpioso | 15:24 |
| *** vsaienk0 has quit IRC | 15:24 | |
| rpioso | TheJulia: Hey :) | 15:24 |
| cdearborn | i9-]'= | 15:25 |
| cdearborn | oops! wiping off keyboard :-) | 15:25 |
| *** livelace-link has quit IRC | 15:26 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface https://review.openstack.org/526773 | 15:28 |
| *** livelace-link has joined #openstack-ironic | 15:28 | |
| dtantsur | morning rpioso, cdearborn :) | 15:28 |
| *** lucas-hungry is now known as lucasagomes | 15:29 | |
| rpioso | dtantsur: Happy Friday! | 15:29 |
| rpioso | dtantsur: Your weekend is almost here. | 15:30 |
| dtantsur | \o/ | 15:30 |
| cdearborn | good morning dtantsur, TheJulia | 15:32 |
| dtantsur | can someone please tell me if we still have any of FFEs not finished? | 15:39 |
| dtantsur | hjensas: is your FFE done? | 15:39 |
| openstackgerrit | Sam Betts proposed openstack/ironic-tempest-plugin master: Switch to override-checkout for stable branch jobs https://review.openstack.org/540355 | 15:39 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type https://review.openstack.org/453012 | 15:40 |
| *** makowals has quit IRC | 15:44 | |
| dtantsur | this is the last bit for rescue ^^^ right? | 15:45 |
| *** dsneddon has joined #openstack-ironic | 15:47 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: added redfish boot mode management https://review.openstack.org/528314 | 15:47 |
| *** vsaienk0 has joined #openstack-ironic | 15:48 | |
| hjensas | dtantsur: 524709 and 539600 did not land yet, I don't have time for adding tests to 524709. But I am fine with leaving these for Rocky. | 15:48 |
| hjensas | dtantsur: so, yes. | 15:48 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Node state configuration - add log_agent_heartbeat https://review.openstack.org/539600 | 15:49 |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] Firmware based boot from volume for iLO drivers https://review.openstack.org/539158 | 15:50 |
| hjensas | dtantsur: to be honest, I only added 539600 because the auto generated docs include the option. :) | 15:50 |
| dtantsur | okay, rescue needs fixing, see my comments on https://review.openstack.org/#/c/453012/17/ironic/drivers/modules/agent.py@171 | 15:50 |
| patchbot | patch 453012 - ironic - Implements rescue interface for 'ilo' hardware type | 15:50 |
| dtantsur | how did we end up merging it initially, I wonder? | 15:50 |
| TheJulia | dtantsur:just routed networks hash ringt | 15:51 |
| dtantsur | ack | 15:52 |
| dtantsur | TheJulia, hjensas, https://review.openstack.org/#/c/539600/ can be merged, but if it goes in Queens, it does not need a release note | 15:52 |
| patchbot | patch 539600 - networking-baremetal - Node state configuration - add log_agent_heartbeat | 15:52 |
| dtantsur | because it's a small addition to a feature in the same release | 15:53 |
| dtantsur | okay, so only rescue is of a concern | 15:53 |
| TheJulia | I thought that merged for some reason | 15:53 |
| *** dsneddon has quit IRC | 15:54 | |
| TheJulia | hmm, must have gotten lost in the mix and is not on the whiteboard | 15:54 |
| jroll | TheJulia: is the hash ring thing a must have for q? | 15:54 |
| hjensas | TheJulia: yeah, I forgot to put that on the whiteboard. | 15:54 |
| dtantsur | TheJulia: having explicit checks on an interface implementation breaks the whole idea of the driver composition :( | 15:55 |
| *** saop has quit IRC | 15:55 | |
| * hjensas will take out the reno then. | 15:55 | |
| TheJulia | jroll: I do not believe so, I think it is a nice to have for hjensas, although having it makes it usable out of the gate | 15:55 |
| dtantsur | (and potentially out-of-tree drivers) | 15:55 |
| TheJulia | dtantsur: *sigh* | 15:55 |
| *** dsneddon has joined #openstack-ironic | 15:55 | |
| dtantsur | okay, we may treat it as a bug next week | 15:55 |
| jroll | TheJulia: got it, it has some work to do, unless I'm missing something. so, don't think it will make it | 15:55 |
| jroll | (it being the hash ring bits) | 15:56 |
| TheJulia | dtantsur: so if the class structure can't be validated, and we shouldn't use the interface name in the db how exactly do we know what path the node should take for logic that is interface specific in other areas? | 15:56 |
| dtantsur | TheJulia: it means that such logic is misplaced | 15:58 |
| dtantsur | i.e. it should go to the interface in question | 15:58 |
| TheJulia | so we need to add a method to call the other interface, that will have to be abstracted off base because base would need to be something like pass | 15:59 |
| TheJulia | or... we just statically tag and see if the driver loaded has it | 16:00 |
| * TheJulia has a headache | 16:00 | |
| dtantsur | TheJulia: yep, something like task.driver.network.validate_rescue OR task.driver.network.supports_rescue_network | 16:00 |
| jroll | dtantsur: ++ | 16:00 |
| dtantsur | but what we do makes it impossible to create an out-of-tree network interface that supports a rescue network | 16:00 |
| *** dsneddon has quit IRC | 16:01 | |
| dtantsur | without inheriting it from NeutronNetwork, even if it does not use Neutron :) | 16:01 |
| jroll | dtantsur: and the base RescueInterface.validate should be calling task.driver.network.validate_rescue | 16:01 |
| dtantsur | yes | 16:01 |
| * jroll could cook that patch | 16:01 | |
| TheJulia | yeah, I agree with that approach, I think I'm just brain fried at the moment | 16:01 |
| *** r-daneel has joined #openstack-ironic | 16:02 | |
| jroll | dtantsur: that said, this doesn't block out-of-tree network interfaces that support a rescue network, it just skips validation of them :) | 16:02 |
| dtantsur | well, right, okay | 16:02 |
| dtantsur | unless we have something like that in other places | 16:02 |
| * jroll notes it is pedantic friday for jroll | 16:02 | |
| dtantsur | jroll: please do, but leave a note on that patch so that Shiv does not duplicate the effort | 16:02 |
| jroll | dtantsur: I'll leave a note if I start working on it, not sure I'll have time today | 16:03 |
| dtantsur | ack | 16:03 |
| *** dsneddon has joined #openstack-ironic | 16:03 | |
| rpioso | I have a couple of silly questions. What is the purpose of our MissingParameterValue exception? When should it be used? | 16:05 |
| openstackgerrit | Sam Betts proposed openstack/ironic-tempest-plugin master: Switch to override-checkout for stable branch jobs https://review.openstack.org/540355 | 16:05 |
| rpioso | If a function/method does not have an explicit kwargs arg, can a missing parameter be detected by the function/method? | 16:07 |
| *** dsneddon has quit IRC | 16:08 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type https://review.openstack.org/453012 | 16:08 |
| * rpioso needs to hang a caffeine pump on his IV pole. | 16:09 | |
| dtantsur | rpioso: it's e.g. when your driver needs a driver_info field, but it's not provided | 16:09 |
| *** olivierbourdon38 has quit IRC | 16:10 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Use zuul.override_checkout instead of custom branch_override var https://review.openstack.org/540432 | 16:10 |
| *** pcaruana has quit IRC | 16:11 | |
| *** dmellado has quit IRC | 16:11 | |
| rpioso | dtantsur: So it typically doesn't directly apply to a function's formal parameters, correct? | 16:11 |
| dtantsur | rpioso: correct | 16:11 |
| rpioso | dtantsur: ty | 16:11 |
| dtantsur | it's a user-facing error | 16:11 |
| dtantsur | np | 16:11 |
| TheJulia | rpioso: If you find such for an IV pole, please let me know, I'm still asleep | 16:12 |
| *** dsneddon has joined #openstack-ironic | 16:12 | |
| * jroll throws a handful of these in the channel https://img.medscapestatic.com/pi/features/drugdirectory/octupdate/AMR00200.jpg | 16:13 | |
| TheJulia | ooooooo | 16:13 |
| rpioso | jroll: That's what I'm talking about! | 16:13 |
| jroll | I had no idea this was a thing, but webmd says it is: https://www.webmd.com/drugs/2/drug-144281/caffeine-citrate-intravenous/details | 16:14 |
| *** dmellado has joined #openstack-ironic | 16:14 | |
| *** AlexeyAbashkin has quit IRC | 16:14 | |
| *** ijw has quit IRC | 16:14 | |
| *** ijw has joined #openstack-ironic | 16:14 | |
| *** livelace-link has quit IRC | 16:15 | |
| *** ijw has quit IRC | 16:15 | |
| *** ijw has joined #openstack-ironic | 16:15 | |
| rpioso | jroll: The overdose tab must be a mistake ;-) | 16:15 |
| jroll | heh | 16:15 |
| *** yangyapeng has quit IRC | 16:15 | |
| jroll | overdose symptoms: euphoria | 16:15 |
| *** livelace-link has joined #openstack-ironic | 16:16 | |
| *** yangyapeng has joined #openstack-ironic | 16:16 | |
| rpioso | or alive | 16:16 |
| *** rpioso is now known as rpioso|caffeine | 16:17 | |
| jroll | :P | 16:17 |
| *** livelace-link has quit IRC | 16:17 | |
| *** dsneddon has quit IRC | 16:18 | |
| *** links has joined #openstack-ironic | 16:18 | |
| *** ijw has quit IRC | 16:20 | |
| *** yangyapeng has quit IRC | 16:20 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Node state configuration - add log_agent_heartbeat https://review.openstack.org/539600 | 16:23 |
| *** dsneddon has joined #openstack-ironic | 16:24 | |
| TheJulia | overdose of caffeine is bad | 16:29 |
| *** saop has joined #openstack-ironic | 16:29 | |
| *** yangyapeng has joined #openstack-ironic | 16:30 | |
| saop | TheJulia, dtantsur Updated comments please review, https://review.openstack.org/#/c/539158/ | 16:33 |
| patchbot | patch 539158 - ironic - [docs] Firmware based boot from volume for iLO dri... | 16:33 |
| *** dsneddon has quit IRC | 16:34 | |
| *** saop has quit IRC | 16:34 | |
| *** rpioso|caffeine is now known as rpioso | 16:35 | |
| TheJulia | mgoddard: replied to https://review.openstack.org/#/c/534441/6 | 16:38 |
| patchbot | patch 534441 - ironic - Don't try to lock for vif actions | 16:38 |
| TheJulia | also sambetts, dtantsur, jroll may want to take a look | 16:39 |
| dtantsur | k | 16:39 |
| * jroll sets aside for later today | 16:40 | |
| *** dsneddon has joined #openstack-ironic | 16:44 | |
| *** stendulker has joined #openstack-ironic | 16:44 | |
| mgoddard | TheJulia: thanks, I'll take a look | 16:45 |
| *** ijw has joined #openstack-ironic | 16:47 | |
| *** devananda has joined #openstack-ironic | 16:50 | |
| *** trown is now known as trown|lunch | 16:51 | |
| *** ijw has quit IRC | 16:52 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add validate_rescue() method to network interface https://review.openstack.org/540445 | 16:55 |
| jroll | stendulker: nice! thank you for that | 16:55 |
| stendulker | jroll: Thank you for the suggestion. Please have a look. | 16:55 |
| *** AlexeyAbashkin has joined #openstack-ironic | 16:56 | |
| dtantsur | thanks stendulker! | 16:57 |
| *** e0ne has quit IRC | 16:57 | |
| jroll | stendulker: +1'd with a question about adding a release note, looks good though, thank you! :) | 16:58 |
| stendulker | ok. Thanks | 17:00 |
| *** mjura has quit IRC | 17:01 | |
| *** vsaienk0 has quit IRC | 17:07 | |
| TheJulia | now, where did I put that brain de-scrambler | 17:09 |
| *** marios has quit IRC | 17:09 | |
| *** r-daneel has quit IRC | 17:10 | |
| spotz | Hey guys is there a way to troubleshoot why a node is enroll, manage doesn't change the state on it | 17:10 |
| openstackgerrit | James E. Blair proposed openstack/virtualbmc master: Zuul: Remove project name https://review.openstack.org/540452 | 17:11 |
| TheJulia | spotz: anything in last_error? | 17:11 |
| dtantsur | spotz: hi, check the last_error field. chances are very high that ironic cannot verify the power credentials | 17:11 |
| spotz | dtantsur TheJulia - It definitely could be the power creds its virtual nodes for testing vs baremetal | 17:12 |
| TheJulia | spotz: that is generally what will cause a failure | 17:13 |
| *** amarao has quit IRC | 17:13 | |
| openstackgerrit | Merged openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/534935 | 17:13 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/537761 | 17:13 |
| spotz | dtantsur TheJulia - I'll troubleshoot from that angle, thanks! | 17:14 |
| *** athomas has quit IRC | 17:16 | |
| *** amarao1 has quit IRC | 17:16 | |
| mgoddard_ | Hi TheJulia: just finished catching up on those IRC logs re VIFs | 17:17 |
| TheJulia | mgoddard: I like your comments w/r/t adding more locking possibly. PTG discussion for sure | 17:18 |
| mgoddard_ | TheJulia: there was a lot of back and forth, so it's hard to work out exactly where everyone ended up, but AFAICT, the thing that dtantsur agreed to was jroll's comment: "TheJulia: I guess I'm back to thinking we probably don't need to lock for vif-detach, but I don't have evidence to support that case in either direction" | 17:18 |
| mgoddard_ | TheJulia: so detach, but not attach | 17:19 |
| *** ijw has joined #openstack-ironic | 17:19 | |
| mgoddard_ | TheJulia: I may have misinterpreted | 17:19 |
| TheJulia | yeah, and shifting points of view based on the discussion | 17:19 |
| TheJulia | I took the endpoint as vif actions in general | 17:19 |
| TheJulia | endpoint of the discussion | 17:19 |
| mgoddard_ | TheJulia: perhaps it would be easiest to just ask (again)? | 17:21 |
| jlvillal | Good morning Ironic :) | 17:21 |
| mgoddard_ | I'm quite strongly -1 to removing the lock for *attach*, although I wouldn't block a consensus | 17:21 |
| mgoddard_ | I'm ok with removing it for detach | 17:22 |
| TheJulia | I'd rather jroll sambetts and dtantsur to just review and see if we reach consensus at this point. My brain is not quite up for lots of irc discussion today. I'll just hang a sign up saying "purple is the color for the bikeshed" | 17:22 |
| dtantsur | morning jlvillal | 17:22 |
| dtantsur | mgoddard_, TheJulia, my brain is also not up to speed :( | 17:22 |
| TheJulia | whiskey o'clock? | 17:23 |
| *** racedo has quit IRC | 17:23 | |
| *** ijw has quit IRC | 17:23 | |
| mgoddard_ | TheJulia: sure, I'll reply on the review, and discussion can continue there | 17:23 |
| devananda | g'morning, folks :) | 17:24 |
| TheJulia | good morning devananda | 17:24 |
| openstackgerrit | John L. Villalovos proposed openstack/networking-generic-switch master: Mark multinode job as non-voting https://review.openstack.org/540459 | 17:25 |
| TheJulia | jlvillal: let me guess, 50% failure rate or something? | 17:26 |
| jlvillal | TheJulia, 100% failure rate I think. | 17:27 |
| jlvillal | TheJulia, It is basically your patch for Ironic ported to networking-generic-switch | 17:27 |
| TheJulia | woot! | 17:27 |
| TheJulia | sigh | 17:27 |
| jlvillal | TheJulia, They used the same job as Ironic that got disabled. | 17:28 |
| TheJulia | fun fun :( | 17:28 |
| *** r-daneel has joined #openstack-ironic | 17:28 | |
| *** derekh has quit IRC | 17:29 | |
| *** tesseract has quit IRC | 17:30 | |
| jroll | morning devananda, mgoddard_ | 17:30 |
| jroll | I also am not in an irc discussion mood, heh | 17:31 |
| dtantsur | morning devananda | 17:31 |
| * jroll gets lunch | 17:31 | |
| TheJulia | perhaps it is time for more coffee? | 17:33 |
| mgoddard_ | morning all, looks like it will be a quiet day today then :) | 17:33 |
| *** dmellado has quit IRC | 17:34 | |
| *** harlowja has quit IRC | 17:37 | |
| sambetts | dtantsur, jlvillal: latest https://review.openstack.org/#/c/540355/ patch should fix the tempest plugin CI | 17:38 |
| patchbot | patch 540355 - ironic-tempest-plugin - Switch to override-checkout for stable branch jobs | 17:38 |
| sambetts | and I've added a note and a patch about how to get rid of branch_override and use the native zuul options instead of custom ones | 17:38 |
| sambetts | https://review.openstack.org/#/c/540418/ | 17:39 |
| jlvillal | sambetts, Cool! and thanks | 17:39 |
| patchbot | patch 540418 - openstack-infra/zuul - Add ZUUL_OVERRIDE_BRANCH support for legacy jobs (ABANDONED) | 17:39 |
| sambetts | thats the wrong patch... hold on | 17:39 |
| *** AlexeyAbashkin has quit IRC | 17:39 | |
| sambetts | https://review.openstack.org/#/c/540432/ | 17:39 |
| patchbot | patch 540432 - ironic - Use zuul.override_checkout instead of custom bran... | 17:39 |
| * sambetts --> home | 17:40 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add validate_rescue() method to network interface https://review.openstack.org/540445 | 17:40 |
| sambetts | Night all o/ | 17:40 |
| *** sambetts is now known as sambetts|afk | 17:40 | |
| jlvillal | sambetts, Good night | 17:40 |
| sambetts|afk | cya'll Monday | 17:40 |
| NobodyCam | Good Morning Ironic'ers : night sambetts|afk | 17:40 |
| TheJulia | mgoddard_: quiet is good. Quiet is very good. | 17:43 |
| *** SerenaFeng has joined #openstack-ironic | 17:47 | |
| *** baha has joined #openstack-ironic | 17:48 | |
| *** racedo has joined #openstack-ironic | 17:50 | |
| *** dsneddon has quit IRC | 17:51 | |
| mgoddard_ | bon weekend all o/ | 17:55 |
| NobodyCam | have a good one mgoddard_ | 17:57 |
| *** dsneddon has joined #openstack-ironic | 17:59 | |
| *** mgoddard_ has quit IRC | 17:59 | |
| *** jbags has joined #openstack-ironic | 18:01 | |
| jbags | Hello and good morning! I have recently switched from pike to queens , and an experiencing a problem. When I execute the step: "openstack overcloud node configure --all-manageable --root-device=sda" I encounter an error that I havent before, and a google search is turning up nothing. | 18:02 |
| jbags | The error is "got an unexpected keyword argument 'auth_token'"], u'result': None}" | 18:02 |
| jbags | the full stacktrace is here : https://pastebin.com/i7TnEXy8 | 18:03 |
| jbags | can anyone assist? | 18:03 |
| jroll | jbags: that looks like a tripleo issue (iroinc doesn't use mistral), you might check with #tripleo | 18:08 |
| jlvillal | jbags, That looks like triplo-o stuff | 18:08 |
| * jroll requests his soda | 18:08 | |
| jbags | ok thank you | 18:08 |
| *** harlowja has joined #openstack-ironic | 18:21 | |
| *** dtantsur is now known as dtantsur|afk | 18:23 | |
| dtantsur|afk | have a nice weekend | 18:23 |
| NobodyCam | have a good one dtantsur|afk :) | 18:23 |
| *** trown|lunch is now known as trown | 18:26 | |
| *** racedo has quit IRC | 18:29 | |
| *** racedo has joined #openstack-ironic | 18:29 | |
| *** racedo has quit IRC | 18:29 | |
| *** racedo has joined #openstack-ironic | 18:30 | |
| *** racedo has quit IRC | 18:30 | |
| *** SerenaFeng has quit IRC | 18:34 | |
| *** dmellado has joined #openstack-ironic | 18:35 | |
| *** kjw3 has joined #openstack-ironic | 18:41 | |
| *** links has quit IRC | 18:41 | |
| *** yangyape_ has joined #openstack-ironic | 18:41 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type https://review.openstack.org/453012 | 18:44 |
| *** yangyapeng has quit IRC | 18:45 | |
| *** Goneri has quit IRC | 18:47 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type https://review.openstack.org/453012 | 18:48 |
| *** ricardoas is now known as ricardoas-away | 18:57 | |
| *** lucasagomes is now known as lucas-afk | 19:05 | |
| *** kjw3 has quit IRC | 19:05 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:11 | |
| *** stendulker has quit IRC | 19:20 | |
| *** Goneri has joined #openstack-ironic | 19:41 | |
| *** amarao has joined #openstack-ironic | 19:46 | |
| *** Goneri has quit IRC | 19:46 | |
| openstackgerrit | James E. Blair proposed openstack/python-ironic-inspector-client master: Zuul: Remove project name https://review.openstack.org/540500 | 19:52 |
| *** AlexeyAbashkin has quit IRC | 19:58 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:04 | |
| *** Goneri has joined #openstack-ironic | 20:06 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Deprecate Classic Drivers https://review.openstack.org/540153 | 20:08 |
| *** AlexeyAbashkin has quit IRC | 20:19 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25-27 support https://review.openstack.org/471750 | 20:22 |
| *** amarao has quit IRC | 20:25 | |
| *** rbudden_ has joined #openstack-ironic | 20:28 | |
| *** rbudden has quit IRC | 20:29 | |
| *** rbudden_ is now known as rbudden | 20:29 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/538482 | 20:33 |
| *** charcol-v2 has quit IRC | 20:46 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Make Fedora26 CI job voting https://review.openstack.org/540524 | 20:52 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Make Fedora26 CI job voting https://review.openstack.org/540524 | 21:12 |
| *** e0ne has joined #openstack-ironic | 21:17 | |
| *** priteau has quit IRC | 21:33 | |
| *** baha has quit IRC | 21:46 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fix upper-constraints use https://review.openstack.org/540538 | 21:57 |
| *** e0ne has quit IRC | 21:58 | |
| *** trown is now known as trown|outtypewww | 22:06 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fix upper-constraints use https://review.openstack.org/540538 | 22:11 |
| *** Goneri has quit IRC | 22:11 | |
| *** jcoufal has quit IRC | 22:21 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fix upper-constraints use https://review.openstack.org/540538 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fix non-venv keystone handling https://review.openstack.org/498910 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Experimental Neutron Installation https://review.openstack.org/452514 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: VM networking for Neutron testing https://review.openstack.org/498271 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: WIP: Initial neutron task changes https://review.openstack.org/452515 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: DNM: Execute tests for neutron integration https://review.openstack.org/498268 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Add LLDP discovery for IPA/Inspector https://review.openstack.org/498972 | 22:32 |
| *** jlabarre has quit IRC | 22:34 | |
| *** cdearborn has quit IRC | 22:56 | |
| *** figleaf is now known as edleafe | 22:58 | |
| *** saop has joined #openstack-ironic | 23:15 | |
| saop | TheJulia, Hello | 23:15 |
| saop | TheJulia, Need one workflow https://review.openstack.org/#/c/539158/, would you please give me. | 23:17 |
| patchbot | patch 539158 - ironic - [docs] Firmware based boot from volume for iLO dri... | 23:17 |
| *** saop has quit IRC | 23:23 | |
| *** priteau has joined #openstack-ironic | 23:34 | |
| *** MattMan has quit IRC | 23:36 | |
| *** MattMan has joined #openstack-ironic | 23:37 | |
| *** priteau has quit IRC | 23:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
