| *** masber has joined #openstack-ironic | 00:05 | |
| *** csmart has joined #openstack-ironic | 00:14 | |
| *** mkrai has joined #openstack-ironic | 00:14 | |
| *** rama_y has joined #openstack-ironic | 00:44 | |
| *** hshiina has joined #openstack-ironic | 00:50 | |
| *** rama_y has quit IRC | 00:52 | |
| *** deep-book-gk has joined #openstack-ironic | 00:58 | |
| *** deep-book-gk has left #openstack-ironic | 01:00 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:01 | |
| *** phuongnh has joined #openstack-ironic | 01:06 | |
| *** jkilpatr has quit IRC | 01:06 | |
| *** rama_y has joined #openstack-ironic | 01:07 | |
| *** litao__ has joined #openstack-ironic | 01:11 | |
| *** sdague has joined #openstack-ironic | 01:16 | |
| *** yufei has joined #openstack-ironic | 01:25 | |
| *** namnh has joined #openstack-ironic | 01:27 | |
| *** baoli has joined #openstack-ironic | 01:27 | |
| *** yangyapeng has joined #openstack-ironic | 01:41 | |
| *** OctopusZhang has joined #openstack-ironic | 01:44 | |
| *** bfournie has joined #openstack-ironic | 01:45 | |
| *** bfournie has quit IRC | 01:46 | |
| *** yufei has quit IRC | 01:46 | |
| *** OctopusZhang_ has joined #openstack-ironic | 01:47 | |
| *** OctopusZhang_ is now known as yufei | 01:47 | |
| *** slagle has joined #openstack-ironic | 01:49 | |
| *** OctopusZhang has quit IRC | 01:50 | |
| *** OctopusZhang_ has joined #openstack-ironic | 01:50 | |
| *** yufei has quit IRC | 01:52 | |
| *** OctopusZhang_ is now known as yufei | 01:52 | |
| *** yangyape_ has joined #openstack-ironic | 01:52 | |
| *** yangyapeng has quit IRC | 01:55 | |
| *** bfournie has joined #openstack-ironic | 01:57 | |
| *** bfournie has quit IRC | 01:58 | |
| *** bfournie has joined #openstack-ironic | 01:59 | |
| *** yufei has quit IRC | 02:04 | |
| *** yufei has joined #openstack-ironic | 02:04 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver https://review.openstack.org/480477 | 02:07 |
|---|---|---|
| *** Finix has joined #openstack-ironic | 02:28 | |
| *** tuanluong has joined #openstack-ironic | 02:38 | |
| *** baoli has quit IRC | 02:58 | |
| jlvillal | TheJulia, As a note on 60 seconds being too short. Before it was 5 seconds... | 03:21 |
| *** yangyape_ has quit IRC | 03:35 | |
| *** yangyapeng has joined #openstack-ironic | 03:35 | |
| *** baoli has joined #openstack-ironic | 03:43 | |
| *** udesale has joined #openstack-ironic | 03:47 | |
| *** baoli has quit IRC | 03:48 | |
| *** jaganathan has joined #openstack-ironic | 03:49 | |
| *** yangyape_ has joined #openstack-ironic | 03:51 | |
| *** yangyapeng has quit IRC | 03:51 | |
| *** yangyape_ has quit IRC | 03:55 | |
| *** yangyapeng has joined #openstack-ironic | 03:55 | |
| *** hshiina has quit IRC | 03:56 | |
| *** hshiina has joined #openstack-ironic | 03:58 | |
| *** aparnav has joined #openstack-ironic | 04:04 | |
| *** junbo has joined #openstack-ironic | 04:07 | |
| *** baoli has joined #openstack-ironic | 04:08 | |
| *** baoli has quit IRC | 04:13 | |
| *** amarao has joined #openstack-ironic | 04:18 | |
| *** udesale__ has joined #openstack-ironic | 04:31 | |
| *** udesale has quit IRC | 04:33 | |
| *** udesale__ has quit IRC | 04:38 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 04:43 | |
| *** rama_y has quit IRC | 04:55 | |
| *** udesale__ has joined #openstack-ironic | 05:07 | |
| *** karthiks has joined #openstack-ironic | 05:08 | |
| *** jaganathan has quit IRC | 05:15 | |
| *** GK1wmSU has joined #openstack-ironic | 05:21 | |
| *** GK1wmSU has left #openstack-ironic | 05:23 | |
| *** dsneddon has quit IRC | 05:30 | |
| *** _GK1wmSU has joined #openstack-ironic | 05:35 | |
| *** _GK1wmSU has left #openstack-ironic | 05:36 | |
| *** Finix has quit IRC | 05:58 | |
| *** Finix has joined #openstack-ironic | 05:58 | |
| *** ChubYann has quit IRC | 06:05 | |
| *** rcernin has joined #openstack-ironic | 06:11 | |
| *** pcaruana has joined #openstack-ironic | 06:19 | |
| *** tuanluong has quit IRC | 06:19 | |
| openstackgerrit | fupingxie proposed openstack/ironic master: Replace http with https(Part-1) https://review.openstack.org/481516 | 06:31 |
| *** amotoki__ is now known as amotoki | 06:33 | |
| *** Finix has quit IRC | 06:53 | |
| *** Finix has joined #openstack-ironic | 06:53 | |
| *** jaganathan has joined #openstack-ironic | 06:56 | |
| *** e0ne has joined #openstack-ironic | 07:01 | |
| *** jaosorior has joined #openstack-ironic | 07:04 | |
| *** amarao1 has joined #openstack-ironic | 07:07 | |
| *** ccamacho has quit IRC | 07:11 | |
| *** amarao1 has quit IRC | 07:13 | |
| *** rbartal has joined #openstack-ironic | 07:16 | |
| *** ccamacho has joined #openstack-ironic | 07:18 | |
| *** tuanluong has joined #openstack-ironic | 07:18 | |
| *** amarao1 has joined #openstack-ironic | 07:19 | |
| *** aarefiev_afk is now known as aarefiev | 07:32 | |
| *** phuongnh has quit IRC | 07:38 | |
| *** Finix has quit IRC | 07:39 | |
| *** dtantsur|afk is now known as dtantsur | 07:41 | |
| dtantsur | morning ironic | 07:41 |
| dtantsur | I see our gate still does not feel well.. | 07:41 |
| *** udesale has joined #openstack-ironic | 07:58 | |
| *** milan has joined #openstack-ironic | 07:58 | |
| *** udesale__ has quit IRC | 08:00 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 08:00 |
| yufei | morning ironicers, I want to ask a questions about nosetest in ironic, after I install requirement and test_requirements in ironic, I try to run nosetest ironic/tests/unit, but got a lot of error like 'AttributeError: 'NoneType' object has no attribute 'load'... | 08:04 |
| yufei | Is there any steps I missed before I run nosetest? | 08:05 |
| *** etingof has joined #openstack-ironic | 08:06 | |
| dtantsur | yufei: hi! I'm not sure we support nosetest. Why not use "tox" as a test runner? | 08:08 |
| etingof | morning Ironic! ;) | 08:08 |
| dtantsur | morning etingof, happy Monday :) | 08:08 |
| milan | morning Ironic! :) | 08:10 |
| milan | morning dtantsur etingof! :) | 08:10 |
| dtantsur | yo milan! | 08:10 |
| milan | :D | 08:10 |
| *** derekh has joined #openstack-ironic | 08:11 | |
| milan | morning derekh! :) | 08:11 |
| yufei | dtantsur, I will try to run unittest with tox, thanks | 08:11 |
| *** ralonsoh has joined #openstack-ironic | 08:12 | |
| *** rcarrill1 is now known as rcarrillocruz | 08:17 | |
| * dtantsur updates to Fedora 26, wish me luck :) | 08:18 | |
| pas-ha | morning ironic :) | 08:23 |
| dtantsur | hi pas-ha! | 08:24 |
| pas-ha | hi dtantsur :) | 08:24 |
| dtantsur | pas-ha: time to revive the ansible-deploy spec? ;) (no hurry, just reminding) | 08:24 |
| pas-ha | yep, on my to-do list | 08:24 |
| dtantsur | awesome! | 08:24 |
| vdrok | morning ironic! | 08:28 |
| vdrok | dtantsur: yeah. seems like sometimes rabbit is stuck for more than two minutes | 08:29 |
| dtantsur | meh.. we should figure out why it happens. do neutron folks have any ideas? | 08:29 |
| vdrok | I have not seen any new comments in a bug | 08:30 |
| * dtantsur moves to their channel | 08:31 | |
| *** udesale__ has joined #openstack-ironic | 08:31 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Update reno for stable/pike https://review.openstack.org/488774 | 08:34 |
| *** udesale has quit IRC | 08:34 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 08:34 |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic-specs master: Add spec for specific faults support https://review.openstack.org/334113 | 08:43 |
| milan | morning pas-ha, vdrok! :) | 08:43 |
| *** racedo has joined #openstack-ironic | 08:44 | |
| aarefiev | morning all! | 08:53 |
| vdrok | morning milan aarefiev :) | 08:58 |
| aarefiev | o/ | 08:58 |
| milan | morning aarefiev! :) | 08:58 |
| aarefiev | milan: morning :) | 08:59 |
| aarefiev | milan: do you have a minute ? re: https://review.openstack.org/#/c/463768/ | 09:00 |
| patchbot | patch 463768 - ironic-inspector - Use declarative reader/writer separation | 09:00 |
| * milan checks | 09:01 | |
| aarefiev | milan: I'd like to introduce db api for accessing db both from api and worker | 09:02 |
| aarefiev | and keep cache on worker side only | 09:03 |
| milan | aarefiev, what is the motivation for that? | 09:04 |
| *** yangyapeng has quit IRC | 09:04 | |
| aarefiev | first of all there is no needs to talk cache from api | 09:05 |
| aarefiev | it would be worker entity only | 09:05 |
| aarefiev | also if we introduce versional objects | 09:05 |
| vdrok | dtantsur: hrm, actually the 485773 seems to be failing because of us http://logs.openstack.org/73/485773/4/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/39a2ad0/console.html#_2017-07-31_07_34_13_187140 | 09:06 |
| aarefiev | it would be much easy to ьщму щт | 09:06 |
| aarefiev | * move on | 09:06 |
| aarefiev | heh | 09:06 |
| aarefiev | slow lang switching | 09:07 |
| vdrok | dtantsur: this one is the thing otoh http://logs.openstack.org/73/485773/4/check/gate-grenade-dsvm-ironic-ubuntu-xenial/a146b14/console.html | 09:07 |
| aarefiev | milan: it's one more abstraction layer for avoid code duplication and it's easy to use new session model | 09:09 |
| milan | hmmm | 09:10 |
| * milan needs to digest that with another cup of coffee | 09:10 | |
| milan | :) | 09:10 |
| *** sambetts|afk is now known as sambetts | 09:11 | |
| aarefiev | milan: also I'm open to any ideas :) | 09:12 |
| vdrok | ugh, I don't see http_timeout in the tempest.conf | 09:12 |
| *** etingof has quit IRC | 09:14 | |
| aarefiev | milan: anyway we need to structure db code somehow, otherwise it will become mess | 09:14 |
| milan | aarefiev, yeah, | 09:17 |
| aarefiev | dtantsur has some ideas for sure :) | 09:18 |
| milan | I'm wondering: we can actually serialise a node object (it has the "etag") so we can use a node anywhere without too many worries | 09:19 |
| milan | we could maybe use the node cache as the shared "library"/layer then | 09:19 |
| *** jtomasek has joined #openstack-ironic | 09:20 | |
| *** jtomasek has quit IRC | 09:20 | |
| *** jtomasek has joined #openstack-ironic | 09:21 | |
| aarefiev | milan: 99% of cache api shouldn't be available for api, I'd prefer to split it anyway | 09:21 |
| milan | aarefiev, rest api only for state fetch/introspection start&abort? | 09:22 |
| milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L159 | 09:22 |
| milan | I'm not so sure we can totally split things apart as, say the abort, has to happen in rest api and be conducted later by the worker, but together it's a single work item | 09:26 |
| *** jtomasek has quit IRC | 09:27 | |
| *** etingof has joined #openstack-ironic | 09:27 | |
| aarefiev | milan: we don't tie node to worker, so sync local version with db would be a challenge | 09:28 |
| milan | ... aarefiev, so the shared thing from the node_cache would be just a node_info object, the remote call would use it as a locked&serialized arg | 09:28 |
| milan | ... where the "lock" is the version_uuid | 09:29 |
| milan | aarefiev, we don't have to tie a node to a worker or am I missing something? | 09:30 |
| aarefiev | yeah, we don't | 09:30 |
| aarefiev | so one worker could start introspection and other one continue | 09:30 |
| milan | aarefiev, yup, modulo the processing call | 09:31 |
| milan | the rest is ensured thru the state machine | 09:31 |
| aarefiev | for passing objects via rpc it would be better use version objects | 09:31 |
| milan | the rest of the "consistency" | 09:31 |
| milan | aarefiev, agreed, we could store the version in db? | 09:32 |
| aarefiev | and don't reinvent the wheel | 09:32 |
| aarefiev | sure | 09:32 |
| milan | aarefiev, another thing is | 09:32 |
| milan | the node_info itself doesn't expose the functionality we're trying to version | 09:33 |
| milan | as that is provided by the modules processing.py and introspect.py so maybe I've just proved myself wrong O:-) | 09:33 |
| milan | aarefiev, is it the version of the rpc api we're talking about? | 09:34 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR https://review.openstack.org/489153 | 09:34 |
| aarefiev | milan: it will be both db and rpc api version | 09:34 |
| milan | so e.g wsgi v1 can talk to worker v2 after an upgrade? | 09:34 |
| milan | aarefiev, right | 09:34 |
| aarefiev | yep, worker v1 an v2 at the same time | 09:35 |
| aarefiev | also "race condition if version_id changed outside of this node_info" is not true now | 09:35 |
| milan | aarefiev, why? | 09:36 |
| aarefiev | milan: prev example: worker 1 starts introspection and update the node with version 10 | 09:37 |
| aarefiev | worker 2 has version 9 | 09:37 |
| dtantsur | vdrok: node locked, hmmm | 09:37 |
| dtantsur | aarefiev: what do I have ideas about? :) | 09:38 |
| milan | aarefiev, but the api version is a different column | 09:38 |
| aarefiev | dtantsur: we are talking about https://review.openstack.org/#/c/463768/ | 09:39 |
| patchbot | patch 463768 - ironic-inspector - Use declarative reader/writer separation | 09:39 |
| dtantsur | ah, ok. I haven't checked it yet, I think | 09:39 |
| milan | s/is/going to be | 09:39 |
| aarefiev | milan: what about https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L80 | 09:39 |
| aarefiev | if worker 2 gets `continue` request it will fail | 09:40 |
| milan | aarefiev, the naming might not be ideal, but it's this thing: http://docs.sqlalchemy.org/en/latest/orm/versioning.html | 09:40 |
| milan | it's an uuid actually that changes at each update | 09:40 |
| aarefiev | yep | 09:41 |
| milan | so the version wouldn't be part of the key or? | 09:41 |
| milan | api_version | 09:41 |
| aarefiev | agree | 09:41 |
| milan | OK so one thing is the DB record version i.e new column added | 09:42 |
| milan | or renamed/deleted | 09:42 |
| milan | other thing is the RPC API version | 09:42 |
| milan | method added/removed/renamed | 09:42 |
| aarefiev | yeah, and I mean node version now | 09:43 |
| milan | OK so what is the concern with the former? | 09:43 |
| milan | aarefiev, node being the record in the db, correct? | 09:43 |
| aarefiev | yep | 09:43 |
| aarefiev | so, it's says that if node is changed outside the existing worker it's wrong | 09:45 |
| milan | aarefiev, that means e.g https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L907 has to ensure backwards compatibility | 09:46 |
| milan | with e.g https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L931 | 09:46 |
| aarefiev | milan: you are talking about upgrades | 09:47 |
| milan | aarefiev, yeah but that's where the different versions of workers will appear, no? | 09:48 |
| aarefiev | yep, but imo it's too early for upgrades :) | 09:48 |
| milan | I'm trying to figure out what needs doing in this case w/r db? | 09:48 |
| aarefiev | we don't have workers | 09:48 |
| *** stendulker has joined #openstack-ironic | 09:49 | |
| milan | so currently the only motivation for the change you're introducing is the reader/writer concern separation ;) | 09:49 |
| aarefiev | milan: also I don't want to use cache in api | 09:50 |
| milan | aarefiev, so you'll use another db layer in both. How's that better? | 09:50 |
| milan | btw that layer going to be "below" the cache one, isn't it? | 09:50 |
| milan | * beneath | 09:51 |
| aarefiev | milan: yep, it will be more easy code in api and I don't need to copy-paste db queries in api | 09:52 |
| milan | aarefiev, would be nicer to avoid that indeed, but I think the OO way is through the node_info object that encapsulates all the details | 09:53 |
| aarefiev | milan: which details | 09:53 |
| milan | aarefiev, about how to e.g talk to the db to translate node-id to node-uuid | 09:54 |
| milan | so that e.g post @ v1/abort/node-1 works | 09:54 |
| *** yufei has quit IRC | 09:55 | |
| *** jtomasek has joined #openstack-ironic | 09:55 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Do not merge https://review.openstack.org/489161 | 10:00 |
| aarefiev | milan: imo we are mixing a lot staff in cache: node details store, operations with nodes, versioning, fsm transitions, locking | 10:04 |
| aarefiev | dunno how to deal with this | 10:05 |
| openstackgerrit | fupingxie proposed openstack/ironic master: Replace http with https(Part-1) https://review.openstack.org/481516 | 10:07 |
| milan | aarefiev, yeah that is a valid concern but the node_info object is sort of a nexus of the higher-level inspector modules (introspect.py & process.py) | 10:10 |
| openstackgerrit | Merged openstack/python-ironicclient master: Update reno for stable/pike https://review.openstack.org/488777 | 10:11 |
| aarefiev | milan: yeah, and all functional part of introspect.py & process.py will be in worker | 10:11 |
| milan | aarefiev, not all but definitely those async* ones | 10:12 |
| milan | s/async _background | 10:12 |
| aarefiev | api contains only verification and pass all the work to worker | 10:13 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Remove usage of some of the deprecated methods https://review.openstack.org/487025 | 10:14 |
| milan | aarefiev, yeah, if only there was a magical @RPC('v2.0') decorator here https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/introspect.py#L74 | 10:14 |
| aarefiev | milan: yeah I remember that decorator :) | 10:16 |
| milan | O:-) | 10:16 |
| *** namnh has quit IRC | 10:17 | |
| *** ccamacho has quit IRC | 10:22 | |
| *** dsneddon has joined #openstack-ironic | 10:23 | |
| *** bcornec has quit IRC | 10:29 | |
| *** ralonsoh has quit IRC | 10:32 | |
| *** ralonsoh has joined #openstack-ironic | 10:33 | |
| *** ccamacho has joined #openstack-ironic | 10:37 | |
| *** jaosorior has quit IRC | 10:39 | |
| *** yuriyz has joined #openstack-ironic | 10:48 | |
| *** jaosorior has joined #openstack-ironic | 10:50 | |
| *** jkilpatr has joined #openstack-ironic | 10:59 | |
| openstackgerrit | fupingxie proposed openstack/ironic master: Replace http with https(Part-1) https://review.openstack.org/481516 | 11:00 |
| *** stendulker has quit IRC | 11:06 | |
| *** stendulker_ has joined #openstack-ironic | 11:06 | |
| TheJulia | Good morning everyone | 11:06 |
| *** tuanluong has quit IRC | 11:08 | |
| *** stendulker_ has quit IRC | 11:09 | |
| dtantsur | morning TheJulia | 11:09 |
| *** jistr is now known as jistr|mtg | 11:12 | |
| *** xavierr has joined #openstack-ironic | 11:17 | |
| *** jroll has quit IRC | 11:24 | |
| *** jroll has joined #openstack-ironic | 11:24 | |
| *** ccamacho is now known as ccamacho|lunch | 11:25 | |
| TheJulia | I need to run to a doctor's appointment. I should be back before the meeting. | 11:31 |
| dtantsur | TheJulia: good luck! | 11:34 |
| *** fxpester has quit IRC | 11:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Optimize node locking on heartbeat https://review.openstack.org/456528 | 11:36 |
| *** jistr|mtg is now known as jistr | 11:39 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 11:44 |
| *** aparnav has quit IRC | 11:49 | |
| *** litao__ has quit IRC | 11:50 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 11:50 |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Change ramdisk log filename template https://review.openstack.org/481001 | 11:52 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config https://review.openstack.org/480161 | 11:54 |
| openstackgerrit | Merged openstack/ironic master: Changes log level of a message https://review.openstack.org/476847 | 11:54 |
| *** csmart has quit IRC | 12:00 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config https://review.openstack.org/480161 | 12:00 |
| *** mkrai has quit IRC | 12:00 | |
| *** retr0h has quit IRC | 12:01 | |
| *** dprince has joined #openstack-ironic | 12:05 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Remove SSH-based driver interfaces and drivers https://review.openstack.org/481688 | 12:09 |
| *** bfournie has quit IRC | 12:10 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic-specs master: Support baremetal inspection abort operation https://review.openstack.org/482867 | 12:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Properly set http_timeout as workaround https://review.openstack.org/489161 | 12:11 |
| vdrok | morning TheJulia | 12:11 |
| vdrok | dtantsur: ^^ | 12:11 |
| * vdrok was dumb and has not checked the config option was set | 12:11 | |
| dtantsur | vdrok: thanks, let's see! | 12:12 |
| vdrok | dtantsur: you can see the logs from the first PS run, the value was properly set in config | 12:12 |
| vdrok | second PS is just a commit message update | 12:12 |
| dtantsur | ah | 12:12 |
| dtantsur | seems so | 12:13 |
| sambetts | shame the CI failure is so racey that we thought it was fixed before :( | 12:13 |
| dtantsur | yeah | 12:13 |
| dtantsur | and shame we cannot fix it properly | 12:13 |
| dtantsur | vdrok: please update the whiteboard | 12:13 |
| sambetts | yeah... | 12:13 |
| vdrok | doing now | 12:14 |
| sambetts | I don't really understand why the neutron grenade job doesn't see the same issues :/ | 12:15 |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic-specs master: Support baremetal inspection abort operation https://review.openstack.org/482867 | 12:17 |
| *** bcornec has joined #openstack-ironic | 12:17 | |
| vdrok | sambetts: maybe because our jobs are more resource consuming? | 12:17 |
| sambetts | you think rabbit itself is crashing because of a lack of resources? :/ it could be I guess thats something we should look at | 12:19 |
| *** udesale__ has quit IRC | 12:19 | |
| sambetts | I don't see any issues in the rabbit logs though | 12:20 |
| *** jcoufal has joined #openstack-ironic | 12:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/489199 | 12:23 |
| *** baoli has joined #openstack-ironic | 12:24 | |
| *** nicodemos has quit IRC | 12:26 | |
| *** baoli has quit IRC | 12:27 | |
| *** slagle has quit IRC | 12:27 | |
| *** nicodemos has joined #openstack-ironic | 12:27 | |
| *** slagle has joined #openstack-ironic | 12:29 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Owner-aware baremetal nodes https://review.openstack.org/415512 | 12:31 |
| *** ralonsoh has quit IRC | 12:32 | |
| *** ccamacho|lunch is now known as ccamacho | 12:34 | |
| *** fellypefca has joined #openstack-ironic | 12:35 | |
| *** bfournie has joined #openstack-ironic | 12:41 | |
| *** SerenaFe_ has joined #openstack-ironic | 12:41 | |
| * milan errands -> afk a bit | 12:42 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Owner-aware baremetal nodes https://review.openstack.org/415512 | 12:43 |
| fellypefca | Good morning, Ironic :) | 12:52 |
| *** hw_wutianwei has quit IRC | 12:52 | |
| fellypefca | dtantsur Hi! We need backport this patch (https://review.openstack.org/#/c/482209/) to Newton, because this will fix our agent job on our CI for Newton patchs. But I need to make a backport to Ocata too or just for Newton? | 12:52 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers (MERGED) | 12:52 |
| nicodemos | morning fellypefca, vdrok, TheJulia, ironic | 12:53 |
| dtantsur | fellypefca: there should be no missing versions, so yes, you start with ocata | 12:53 |
| nicodemos | and dtantsur :) | 12:53 |
| fellypefca | nicodemos o/ | 12:54 |
| dtantsur | morning fellypefca, nicodemos | 12:54 |
| fellypefca | dtantsur ok, thanks! | 12:54 |
| *** baoli has joined #openstack-ironic | 12:56 | |
| vdrok | morning fellypefca nicodemos | 12:56 |
| fellypefca | vdrok hey o/ | 12:57 |
| *** baoli has quit IRC | 12:59 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config https://review.openstack.org/480161 | 12:59 |
| *** baoli has joined #openstack-ironic | 13:00 | |
| *** baoli has quit IRC | 13:00 | |
| *** sasha2 has joined #openstack-ironic | 13:02 | |
| *** cdearborn has joined #openstack-ironic | 13:03 | |
| *** mtanino has joined #openstack-ironic | 13:03 | |
| *** mtanino has quit IRC | 13:03 | |
| *** baoli has joined #openstack-ironic | 13:04 | |
| *** mtanino has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Enhanced form-field radio functionality https://review.openstack.org/489215 | 13:05 |
| *** bcornec has quit IRC | 13:06 | |
| *** rloo has joined #openstack-ironic | 13:10 | |
| *** ralonsoh has joined #openstack-ironic | 13:18 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 13:19 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Functional tests: clean up test config file https://review.openstack.org/476877 | 13:22 |
| *** mjturek has joined #openstack-ironic | 13:26 | |
| *** fxpester has joined #openstack-ironic | 13:27 | |
| *** makowals has quit IRC | 13:28 | |
| *** makowals has joined #openstack-ironic | 13:31 | |
| *** dsneddon has quit IRC | 13:35 | |
| *** edleafe- is now known as edleafe | 13:40 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add test for set/unset node target_raid_config https://review.openstack.org/480623 | 13:44 |
| *** gcb has joined #openstack-ironic | 13:46 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 13:49 |
| *** bcornec has joined #openstack-ironic | 13:52 | |
| *** rama_y has joined #openstack-ironic | 13:53 | |
| *** crushil has joined #openstack-ironic | 13:57 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 13:59 |
| *** hw_wutianwei has joined #openstack-ironic | 13:59 | |
| *** jaypipes has joined #openstack-ironic | 14:03 | |
| *** Travis_ has joined #openstack-ironic | 14:04 | |
| Travis_ | Hey there | 14:04 |
| Travis_ | I am trying to setup ironic and i believe i have ran into this bug: 1704497 | 14:05 |
| openstack | bug 1704497 in Ironic "radosgw tempurl: Error: path must be full path to an object e.g. /v1/a/c/o" [Undecided,New] https://launchpad.net/bugs/1704497 | 14:05 |
| Travis_ | Can anyone provide any thoughts on this bug since it has not been classified? | 14:06 |
| dtantsur | hi Travis_! I haven't worked with RadosGW, but the description seems reasonable.. | 14:07 |
| dtantsur | we'll need someone with a reproducer to fix it, I am afraid.. | 14:07 |
| *** bnemec-pto is now known as bnemec | 14:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Address review feedback for ipxe boot file fix https://review.openstack.org/486853 | 14:16 |
| *** adisky__ has quit IRC | 14:16 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Pass os_identity_api_version into functional tests https://review.openstack.org/414957 | 14:18 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config https://review.openstack.org/480161 | 14:21 |
| *** hshiina has quit IRC | 14:27 | |
| *** rbartal has quit IRC | 14:29 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 14:29 |
| Travis_ | @dtantsur thank you for your reply, do you have any suggestions on how i can continue my deployment without this bugfix (possibly a work around). | 14:32 |
| *** openstackgerrit has quit IRC | 14:33 | |
| *** ralonsoh_ has joined #openstack-ironic | 14:34 | |
| *** fxpester has quit IRC | 14:35 | |
| dtantsur | Travis_: not sure, I haven't worked with radosgw.. maybe worth asking on the ML? | 14:35 |
| xavierr | good morning ironic | 14:35 |
| dtantsur | morning xavierr | 14:37 |
| Travis_ | Alright- ill give the ML a try Thanks! | 14:38 |
| *** ralonsoh has quit IRC | 14:38 | |
| TheJulia | Good morning folks | 14:40 |
| TheJulia | again | 14:40 |
| mjturek | gm TheJulia | 14:40 |
| *** mkrai has joined #openstack-ironic | 14:41 | |
| mjturek | and dtantsur and xavierr :) | 14:41 |
| *** ElCoyote_ has left #openstack-ironic | 14:43 | |
| dtantsur | morning mjturek and another good morning to TheJulia :) | 14:45 |
| * dtantsur kindly remind folks to update their statuses and go over the agenda (it's there today!) | 14:45 | |
| *** skazi__ has joined #openstack-ironic | 14:47 | |
| rloo | hi and good morning dtantsur, mjturek, TheJulia, xavierr, Travis_ and everyone else in ironic-land | 14:54 |
| *** krtaylor has joined #openstack-ironic | 14:54 | |
| dtantsur | morning rloo, happy Monday! | 14:54 |
| dtantsur | what can be better than a Monday with gate passing ~ 60% of times? | 14:54 |
| rloo | dtantsur: a gate that passes 75% of the time? :) or :( | 14:56 |
| NobodyCam | Good morning ironic folks. | 14:56 |
| dtantsur | yeah.. well, it's better than Friday for some reason | 14:56 |
| dtantsur | we did manage to push some patches down its throat :) | 14:56 |
| dtantsur | morning NobodyCam :) | 14:57 |
| TheJulia | good morning NobodyCam | 14:57 |
| Travis_ | Good morning | 14:57 |
| NobodyCam | Morning dtantsur and TheJulia ;) | 14:57 |
| *** jaganathan has quit IRC | 14:57 | |
| rloo | dtantsur: just looked at whiteboard. so the nova issues were fixed, we're back to neutron related gate issues? | 14:57 |
| dtantsur | rloo: yep. the timeout bump was incorrect (hence "for some reason" above) | 14:58 |
| dtantsur | we're trying a correct patch now, but, well.. it's just a work around | 14:58 |
| dtantsur | I pinged Neutron folks today again without too much effect | 14:58 |
| rloo | dtantsur: ok thx. wondering why we're the only ones noticing it, but then, we're ironic and thus special... | 14:58 |
| dtantsur | this as well | 14:58 |
| rloo | dtantsur: :-) or :-( ha ha. | 14:59 |
| dtantsur | vdrok (?) suggested it's because our jobs consume more resources | 14:59 |
| *** openstackgerrit has joined #openstack-ironic | 14:59 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 14:59 |
| dtantsur | but dunno. really dunno... | 14:59 |
| *** makowals has quit IRC | 15:00 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/489199 | 15:00 |
| dtantsur | on the bright side: I was going through the priorities some time ago. You all have done really great job this week! :) | 15:01 |
| *** makowals has joined #openstack-ironic | 15:02 | |
| *** rcernin has quit IRC | 15:03 | |
| rloo | dtantsur: :) always a bright side! | 15:03 |
| *** pcaruana has quit IRC | 15:04 | |
| *** Travis_ has quit IRC | 15:04 | |
| *** fellypefca has quit IRC | 15:05 | |
| *** dprince has quit IRC | 15:06 | |
| *** rama_y has quit IRC | 15:07 | |
| *** retr0h has joined #openstack-ironic | 15:08 | |
| *** csmart has joined #openstack-ironic | 15:08 | |
| vdrok | morning rloo and NobodyCam :) | 15:13 |
| rloo | morning vdrok and NobodyCam (how'd i miss you?) | 15:13 |
| vdrok | :) | 15:14 |
| *** eglute has quit IRC | 15:14 | |
| *** eglute has joined #openstack-ironic | 15:16 | |
| *** MattMan has quit IRC | 15:17 | |
| *** MattMan has joined #openstack-ironic | 15:17 | |
| *** penick has joined #openstack-ironic | 15:19 | |
| *** gcb has quit IRC | 15:23 | |
| *** crushil has quit IRC | 15:23 | |
| *** baoli has quit IRC | 15:26 | |
| *** rbudden has joined #openstack-ironic | 15:28 | |
| *** baoli has joined #openstack-ironic | 15:29 | |
| NobodyCam | Hey hey good morning vdrok and rloo. | 15:31 |
| *** rama_y has joined #openstack-ironic | 15:36 | |
| *** namnh has joined #openstack-ironic | 15:42 | |
| *** fellypefca has joined #openstack-ironic | 15:44 | |
| *** crushil has joined #openstack-ironic | 15:45 | |
| namnh | rloo: hi Ruby, i see path https://review.openstack.org/#/c/455811/ got -1 from gate-grenade. so do I wait or continue testing this patch? | 15:46 |
| patchbot | patch 455811 - ironic - [POC|DNM|TEST] Add node.configdrive | 15:46 |
| rloo | namnh: hi, the ironic gate is having issues. random failures. so i haven't bothered with 'recheck' | 15:46 |
| rloo | namnh: although i may be updating the dependent patch to that, most likely today. so maybe wait until i update it. i can ping you. thx! | 15:47 |
| rloo | hey vdrok, if you have a few minutes. i didn't understand your comment at L87: https://review.openstack.org/#/c/408556/65/ironic/cmd/dbsync.py | 15:48 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 15:48 |
| *** skazi__ has quit IRC | 15:48 | |
| vdrok | rloo: which part? :) | 15:48 |
| rloo | vdrok: i think the problem here is ... | 15:49 |
| vdrok | upgrade command is run on new code I think. which means the new online migrations are required. If you intend to use it for all the online migrations we'll have. If it is going to be only checking the version column, then yes, uncommenting in next release should be fine | 15:50 |
| vdrok | s/are required/are present | 15:50 |
| rloo | vdrok: yes, upgrade command is run on new code | 15:50 |
| vdrok | ugh, I wrote something really uncomprehensible | 15:50 |
| rloo | vdrok: new online migrations is needed. | 15:51 |
| *** rpioso has joined #openstack-ironic | 15:51 | |
| *** e0ne has quit IRC | 15:51 | |
| vdrok | upgrade command is run on new code I think. which means the new online migrations are present. If you intend to use it for all the online migrations we'll have, then we'll have the same problem you point out in the comment each time. If it is going to be only checking the version column, then yes, uncommenting in next release should be fine | 15:51 |
| rloo | vdrok: order is, Pike, dbsync-upgrade, do rolling upgrade dance, db-sync-online-migrations. Queens. enalbe that check, upgrade, etc... | 15:51 |
| vdrok | rloo: yup. comment above is what I mean there | 15:52 |
| rloo | vdrok: i don't understand the part about 'if you intend to use it for all the online migrations we'll have, then we'll ahve the same problem'. | 15:53 |
| rloo | vdrok: it is only going to be checking the version column | 15:53 |
| *** vishwanathj has quit IRC | 15:53 | |
| rloo | vdrok: oh, did you mean *if* we change the check to check other things in addition to version column? | 15:53 |
| vdrok | rloo: yeah | 15:53 |
| *** vishwanathj has joined #openstack-ironic | 15:53 | |
| rloo | vdrok: ah. yes. then what you say is true. but it is only checking the version column. | 15:53 |
| vdrok | it seemed like version is added in online migration, so we might add more checks like that | 15:53 |
| vdrok | yup, ok, then it's fine. | 15:54 |
| rloo | vdrok: online migration is only done this one time. after that, when an object is saved to the DB, that .version is updated. | 15:54 |
| rloo | vdrok: the .version is special, since that is what is being used to know/check that the objects are in sync. | 15:55 |
| rloo | vdrok: i have wondered whether it was worth having the .version; none of the other projects have it i think, but that was in the spec and it is supposed to make upgrades easier. | 15:55 |
| vdrok | rloo: yes, I remember some bits of those discussions. maybe we'll be able to get rid of the config pinning at some point and use only version column | 15:57 |
| *** amarao1 has quit IRC | 15:57 | |
| vdrok | (some point being after all the objects we have have version populated?) | 15:58 |
| *** amarao has quit IRC | 15:58 | |
| rloo | vdrok: some point after I do not have to deal with upgrade code? :) | 15:58 |
| vdrok | anyway, that seems like a separate discussion | 15:58 |
| vdrok | :) | 15:58 |
| rloo | vdrok: fwiw, nova doesn't insist on a config pin. so probably possible, but i haven't looked into it. | 15:59 |
| *** hw_wutianwei has quit IRC | 15:59 | |
| vdrok | rloo: regarding the lists in the release mapping, is it only done for convenience? | 16:00 |
| rloo | vdrok: hmm, 'convenience'... it is done cuz it is needed. do you mean 'why are they lists'? | 16:01 |
| vdrok | yes | 16:01 |
| rloo | vdrok: cuz i hate dbs | 16:01 |
| vdrok | :) | 16:01 |
| rloo | vdrok: ok, cuz i wanted to do something like request in db, if object.version < 1.8 | 16:01 |
| rloo | vdrok: but that doesn't work. so i ended up doing if object.version != value | 16:02 |
| rloo | vdrok: and then i realized there could be more than one value | 16:02 |
| rloo | vdrok: hence the list | 16:02 |
| rloo | vdrok: a pain but i suspect we will rarely change the version of an object more than once in a release. but one never knows. | 16:02 |
| rloo | vdrok: i can add a comment about that somewhere | 16:03 |
| vdrok | rloo: hrm, I see. adding a comment there about that would be good ++ | 16:03 |
| vdrok | yup, I see it now | 16:03 |
| rloo | vdrok: L1128 if i recall: https://review.openstack.org/#/c/408556/52/ironic/db/sqlalchemy/api.py | 16:05 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 16:05 |
| namnh | rloo: sure, can you leave your comment on your patch to notify that it is really to test. because now is 11 pm in my country. | 16:06 |
| vdrok | rloo: otoh, we could just leave it as is and generate the versions on the fly? anyway, that can be done as further improvement | 16:07 |
| rloo | namnh: will do! and please go away and get some sleep :) | 16:07 |
| namnh | rloo: :) thanks, looking forward to your patch ;) | 16:08 |
| rloo | vdrok: we always have to leave room for improvement :) | 16:08 |
| NobodyCam | Just a flash back from 2015: | 16:09 |
| NobodyCam | https://usercontent.irccloud-cdn.com/file/KSc02Zd7/howIronicReallyWorks.png | 16:09 |
| dtantsur | ;) | 16:09 |
| NobodyCam | hehehehe | 16:09 |
| * rloo remembers. is that good or bad? :) | 16:09 | |
| NobodyCam | :) | 16:09 |
| *** SerenaFe_ has quit IRC | 16:10 | |
| NobodyCam | I think it's *GOOD* | 16:11 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/newton: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480903 | 16:11 |
| rloo | NobodyCam: :D | 16:11 |
| dtantsur | good old times :) | 16:11 |
| NobodyCam | :) | 16:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480904 | 16:14 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480905 | 16:17 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480905 | 16:18 |
| *** dsneddon has joined #openstack-ironic | 16:18 | |
| * sambetts has to run early today | 16:20 | |
| NobodyCam | have a good night sambetts :) | 16:21 |
| sambetts | if there is anything I need to follow up on in the meeting, ping me and my bouncer should catch it and I'll follow up | 16:21 |
| sambetts | night NobodyCam | 16:21 |
| *** sambetts is now known as sambetts|afk | 16:21 | |
| jlvillal | Hi vdrok | 16:22 |
| jlvillal | vdrok: I'm out of the loop. Just thought I would check in on the gate status. | 16:22 |
| *** ricardoas has joined #openstack-ironic | 16:22 | |
| jlvillal | vdrok: I saw the Whiteboard says mostly broken :( | 16:22 |
| *** namnh has quit IRC | 16:24 | |
| NobodyCam | jlvillal: I think we're passing about 60% ~ 75% of the time | 16:24 |
| jlvillal | NobodyCam: Thanks. Makes it tough to merge at that rate :( | 16:25 |
| dtantsur | we should promote it to "Half-Broken" :) | 16:25 |
| NobodyCam | lol!!! +++ | 16:25 |
| *** crushil has quit IRC | 16:25 | |
| *** baoli has quit IRC | 16:26 | |
| *** baoli has joined #openstack-ironic | 16:27 | |
| *** krtaylor has quit IRC | 16:34 | |
| *** crushil has joined #openstack-ironic | 16:36 | |
| *** dprince has joined #openstack-ironic | 16:40 | |
| vdrok | jlvillal: morning, yeah half broken is more like it :) | 16:41 |
| vdrok | I still think the workaround referenced there should help with that | 16:42 |
| *** efried_zzz is now known as efried | 16:44 | |
| *** stendulker has joined #openstack-ironic | 16:45 | |
| *** jcoufal has quit IRC | 16:48 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480905 | 16:50 |
| *** bcornec has quit IRC | 16:52 | |
| *** derekh has quit IRC | 16:53 | |
| *** kaifeng has joined #openstack-ironic | 16:53 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Add a flag to always peform persistent boot on PXE interface https://review.openstack.org/483430 | 16:53 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:54 | |
| dtantsur | meeting reminder! | 16:59 |
| *** mrtenio|afk is now known as mrtenio | 17:01 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 17:02 |
| *** yolanda has quit IRC | 17:03 | |
| *** jcoufal has joined #openstack-ironic | 17:03 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480904 | 17:04 |
| *** Goneri has joined #openstack-ironic | 17:10 | |
| openstackgerrit | Merged openstack/ironic master: Remove usage of some of the deprecated methods https://review.openstack.org/487025 | 17:21 |
| openstackgerrit | Merged openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 17:23 |
| openstackgerrit | Merged openstack/ironic master: Refactor get_physnets_by_portgroup_id https://review.openstack.org/485772 | 17:23 |
| *** ralonsoh_ has quit IRC | 17:29 | |
| *** ijw has quit IRC | 17:37 | |
| vdrok | uh-oh, ocata tests: http://logs.openstack.org/04/480904/8/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/ccee502/console.html#_2017-07-31_17_34_23_958098 | 17:38 |
| vdrok | we'll need to fix the tempest tests | 17:39 |
| dtantsur | oh, this is bad | 17:39 |
| dtantsur | speaking of branchless tempest ;) | 17:39 |
| *** yolanda has joined #openstack-ironic | 17:55 | |
| *** aarefiev is now known as aarefiev_afk | 17:58 | |
| pas-ha | who's running the HP CI? you need to add couple of lines to your local.conf it seems, new swift-related stuff... | 17:59 |
| pas-ha | SWIFT_START_ALL_SERVICES=False | 17:59 |
| *** yolanda has quit IRC | 18:00 | |
| *** kaifeng has quit IRC | 18:00 | |
| dtantsur | stendulker: ^^^? | 18:01 |
| dtantsur | Nisha_Agarwal: ^^^ | 18:01 |
| stendulker | dtantsur: Hi | 18:01 |
| Nisha_Agarwal | dtantsur, hi | 18:01 |
| dtantsur | o/ please see pas-ha's suggestion above | 18:02 |
| Nisha_Agarwal | pas-ha, thanks | 18:02 |
| Nisha_Agarwal | pas-ha, will see and fix | 18:02 |
| stendulker | dtantsur, pas-ha: sure, will update | 18:03 |
| pas-ha | Nisha_Agarwal: np, was bitten by this the first day after vacation when reinstalling DevStack :) | 18:03 |
| dtantsur | cool! | 18:03 |
| stendulker | its failing since yday evening. were traiging the same. | 18:03 |
| dtantsur | pas-ha: should we add this to the docs? or does it affect only bare metal? | 18:03 |
| Nisha_Agarwal | pas-ha, do you know the exact changes for swift | 18:03 |
| stendulker | pas-ha: thank you for this suggestion | 18:03 |
| *** ChubYann has joined #openstack-ironic | 18:03 | |
| pas-ha | Nisha_Agarwal: I added only that and SWIFT_REPLICAS=1 just in case | 18:03 |
| vdrok | good night everyone! | 18:04 |
| pas-ha | bye vdrok | 18:04 |
| *** ijw has joined #openstack-ironic | 18:04 | |
| Nisha_Agarwal | pas-ha, thanks | 18:04 |
| * dtantsur goes as well | 18:04 | |
| dtantsur | good night! | 18:04 |
| *** raddaoui has joined #openstack-ironic | 18:04 | |
| pas-ha | copied that from a successfull agent_ipmitool gate run | 18:04 |
| *** dtantsur is now known as dtantsur|afk | 18:05 | |
| Nisha_Agarwal | good night dtantsur vdrok | 18:05 |
| pas-ha | Nisha_Agarwal: stendulker hopefully that helps :) | 18:05 |
| Nisha_Agarwal | pas-ha, yes | 18:05 |
| Nisha_Agarwal | pas-ha, thanks for pointing it. | 18:05 |
| stendulker | good night ! | 18:06 |
| * pas-ha signing off, back to rebasing chain of 7 patches :/ | 18:06 | |
| pas-ha | g'night all | 18:06 |
| *** ijw has quit IRC | 18:06 | |
| *** ijw has joined #openstack-ironic | 18:07 | |
| Nisha_Agarwal | pas-ha, g'night | 18:07 |
| openstackgerrit | Merged openstack/ironic master: Rolling upgrades support for create_port RPCAPI https://review.openstack.org/485773 | 18:07 |
| rloo | ^^ wow! it made it through the obstacle course! | 18:07 |
| *** stendulker has quit IRC | 18:10 | |
| *** harlowja has joined #openstack-ironic | 18:11 | |
| jlvillal | rloo: Congrats Ruby!!! | 18:11 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Set target raid configuration on a node https://review.openstack.org/483205 | 18:13 |
| rloo | jlvillal: for what? | 18:15 |
| jlvillal | rloo: The rolling upgrade obstacle course I thought. | 18:15 |
| rloo | jlvillal: heh, i meant the gate. that's for handling changes for physical network awareness feature | 18:16 |
| jlvillal | rloo: I misunderstood. I was thinking rolling upgrade was done. I think you meant the patch made it through the gate :) | 18:16 |
| rloo | jlvillal: :) | 18:17 |
| jlvillal | But that reminds me to remove that patch id from my recheck bot. Now I only have one on the list | 18:17 |
| rloo | jlvillal: or leave it there, it won't be rechecked :) | 18:18 |
| vdrok | lol, the one patch meant to fix the gate did not merge, while a bunch of others passed fine | 18:19 |
| rloo | vdrok: sigh | 18:19 |
| vdrok | reading through the interoperability guideline I see "interoperability should be considered when cloud A and cloud B have the same sets of drivers available." which I think dtantsur|afk asked to add :) | 18:23 |
| NobodyCam | "the one patch meant to fix the gate did not merge, while a bunch of others passed fine" sounds very Ironic to /me | 18:23 |
| vdrok | :) /me now gone | 18:26 |
| rloo | have a good evening vdrok! | 18:28 |
| xavierr | Nisha_Agarwal: please, if that change on the conf works, update 3rd party gate debugging etherpad https://etherpad.openstack.org/p/IronicThirdPartyGateDebugging | 18:28 |
| NobodyCam | night vdrok | 18:28 |
| xavierr | we from oneview drivers are having issues with devstack | 18:28 |
| Nisha_Agarwal | xavierr, sure. I would be able to see it tomorrow morning. already signing off for the day | 18:28 |
| xavierr | Nisha_Agarwal: thank you. In this case we will try that change here now and tomorrow we can talk to see if that worked for both of us, ok? :) | 18:30 |
| Nisha_Agarwal | xavierr, :) sure | 18:30 |
| xavierr | o/ | 18:30 |
| *** yolanda has joined #openstack-ironic | 18:34 | |
| *** yolanda has quit IRC | 18:34 | |
| *** yolanda has joined #openstack-ironic | 18:34 | |
| *** yolanda has quit IRC | 18:38 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 18:42 |
| *** karthiks has quit IRC | 18:43 | |
| *** jcoufal has quit IRC | 18:47 | |
| *** jcoufal has joined #openstack-ironic | 18:50 | |
| *** penick has quit IRC | 18:59 | |
| *** penick has joined #openstack-ironic | 19:16 | |
| *** racedo has quit IRC | 19:17 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Limit CPU usage for iscsiadm to 30% https://review.openstack.org/489364 | 19:27 |
| *** e0ne has joined #openstack-ironic | 19:29 | |
| *** racedo has joined #openstack-ironic | 19:31 | |
| *** e0ne has quit IRC | 19:31 | |
| *** milan has quit IRC | 19:34 | |
| *** rpioso has quit IRC | 19:34 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 19:40 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Limit CPU usage for iscsiadm to 30% https://review.openstack.org/489364 | 19:40 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP: playing limit cpu usage https://review.openstack.org/489364 | 19:41 |
| *** jcoufal has quit IRC | 19:47 | |
| *** krtaylor has joined #openstack-ironic | 19:47 | |
| *** mtanino_ has joined #openstack-ironic | 19:52 | |
| *** crushil has quit IRC | 19:53 | |
| *** e0ne has joined #openstack-ironic | 20:04 | |
| *** fellypefca has quit IRC | 20:07 | |
| rloo | rama_y: hi, i just reviewed your osprofiler patch: https://review.openstack.org/#/c/358142/ | 20:10 |
| patchbot | patch 358142 - ironic - Enable OSProfiler support in Ironic | 20:10 |
| rloo | rama_y: it looks good except for one problem. it doesn't seem to work well with trace_sqlalchemy=false. | 20:11 |
| rloo | rama_y: do you know if it is osprofiler issue? | 20:11 |
| rama_y | Hi rloo: thanks! let me read your comments | 20:12 |
| *** persia has quit IRC | 20:13 | |
| rama_y | hi rloo: I am not aware of this issue with trace_alchemy=false | 20:13 |
| rama_y | I will check on that. | 20:14 |
| *** Sukhdev has joined #openstack-ironic | 20:14 | |
| rloo | rama_y: please check; we can't allow that in w/o knowing what's going on. | 20:14 |
| rama_y | rloo: yes; agree. | 20:15 |
| *** persia has joined #openstack-ironic | 20:16 | |
| openstackgerrit | Merged openstack/ironic master: Address review feedback for ipxe boot file fix https://review.openstack.org/486853 | 20:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP: playing limit cpu usage https://review.openstack.org/489364 | 20:26 |
| *** persia has quit IRC | 20:27 | |
| *** jkilpatr has quit IRC | 20:28 | |
| *** persia has joined #openstack-ironic | 20:28 | |
| *** xavierr has quit IRC | 20:38 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 20:39 |
| *** Nisha_Agarwal has quit IRC | 20:45 | |
| *** etingof has quit IRC | 20:45 | |
| *** ijw has quit IRC | 20:51 | |
| *** ijw has joined #openstack-ironic | 20:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP: playing limit cpu usage https://review.openstack.org/489364 | 20:52 |
| openstackgerrit | Eric Fried proposed openstack/ironic master: DNM: Test nova.utils.get_endpoint_data https://review.openstack.org/488177 | 20:52 |
| *** dprince has quit IRC | 20:58 | |
| *** jtomasek has quit IRC | 21:00 | |
| *** etingof has joined #openstack-ironic | 21:11 | |
| *** ijw has quit IRC | 21:14 | |
| *** ijw has joined #openstack-ironic | 21:14 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters https://review.openstack.org/478825 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient https://review.openstack.org/476171 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient https://review.openstack.org/476170 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance https://review.openstack.org/467728 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation https://review.openstack.org/476173 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient https://review.openstack.org/476172 | 21:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients https://review.openstack.org/469627 | 21:21 |
| *** Goneri has quit IRC | 21:27 | |
| *** jkilpatr has joined #openstack-ironic | 21:34 | |
| *** yolanda has joined #openstack-ironic | 21:34 | |
| *** e0ne has quit IRC | 21:37 | |
| *** yolanda has quit IRC | 21:41 | |
| *** baoli has quit IRC | 21:44 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 22:01 |
| *** bfournie has quit IRC | 22:06 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 22:06 |
| *** cdearborn has quit IRC | 22:09 | |
| *** baoli has joined #openstack-ironic | 22:10 | |
| *** baoli has quit IRC | 22:15 | |
| *** lihi has quit IRC | 22:20 | |
| *** lihi has joined #openstack-ironic | 22:20 | |
| *** baoli has joined #openstack-ironic | 22:22 | |
| *** baoli_ has joined #openstack-ironic | 22:29 | |
| *** baoli has quit IRC | 22:30 | |
| *** penick has quit IRC | 22:35 | |
| *** rama_y has quit IRC | 23:00 | |
| *** https_GK1wmSU has joined #openstack-ironic | 23:20 | |
| *** https_GK1wmSU has left #openstack-ironic | 23:22 | |
| *** Sukhdev has quit IRC | 23:43 | |
| NobodyCam | slow day in channel today :) | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
