| *** urulama has joined #openstack-ironic | 00:00 | |
| *** jbjohnso has joined #openstack-ironic | 00:01 | |
| *** urulama_ has joined #openstack-ironic | 00:10 | |
| *** urulama has quit IRC | 00:10 | |
| rloo | Hi NobodyCam, what's up? | 00:11 |
|---|---|---|
| NobodyCam | :) | 00:11 |
| NobodyCam | was working on the nova driver | 00:12 |
| NobodyCam | and as you have put a ton of work in to power state things lately | 00:13 |
| NobodyCam | running in to a issue where a node is created via the cli | 00:13 |
| NobodyCam | its initial power state is none | 00:14 |
| NobodyCam | nova is attempting to set the instance uuid but is getting node in wrong powerstate | 00:14 |
| NobodyCam | none != on or off | 00:15 |
| rloo | hmm. | 00:15 |
| *** urulama_ has quit IRC | 00:15 | |
| rloo | So I'm not sure what the sequence of events is. it is the nova driver that creates the node via cli? or you? | 00:15 |
| NobodyCam | me | 00:16 |
| rloo | and how does one set the instance uuid? via cli too? | 00:16 |
| NobodyCam | nova is attempting to do a depoly | 00:16 |
| NobodyCam | (which it currently can not) | 00:16 |
| rloo | oh, i'm surprised it can do a deploy. the code isn't hooked up. | 00:16 |
| rloo | i mean, the code in the api to do deploy, doesn't do anything. | 00:17 |
| NobodyCam | no that is in the driver code | 00:17 |
| rloo | the nova driver? what is the error msg that you see (or in the log) when the nova driver tries to deploy? | 00:17 |
| rloo | is it expecting power to be on? | 00:17 |
| NobodyCam | I'm poking on https://review.openstack.org/#/c/51328/2 | 00:18 |
| * rloo looks. | 00:18 | |
| NobodyCam | look at line 206 | 00:18 |
| *** urulama has joined #openstack-ironic | 00:19 | |
| rloo | is that where you're getting an exception? | 00:20 |
| NobodyCam | http://paste.openstack.org/show/g0h1vtdJ35yXYDRHyz8U/ | 00:20 |
| NobodyCam | lol I actually havent run that code yet | 00:21 |
| NobodyCam | just working up to it | 00:21 |
| rloo | well, i'd say after you create your node, turn the power on. | 00:21 |
| NobodyCam | which currently would require a curl call to the api] | 00:22 |
| rloo | i already forgot the command. something like node/<uuid>/state/power_on? sec, let me look it up. hmm, might be on my other computer that is powered off now. | 00:22 |
| NobodyCam | i hve the command | 00:22 |
| NobodyCam | curl -X PUT -d '' http://127.0.0.1:6385/v1/nodes/1abdaa2b-42ff-4583-b0c1-3415c73b2979/state/power?target=power%20off | 00:23 |
| rloo | yeah, that's it. but do you want on or off? | 00:23 |
| NobodyCam | as I have just added the node to ironic I would want it off | 00:24 |
| NobodyCam | so nova can turn it on | 00:24 |
| rloo | NobodyCam: yes, just looking at the code now. It has to be in power_off. | 00:25 |
| NobodyCam | :) | 00:25 |
| rloo | Of course, it would help if the exception msg said what state it wanted it to be in :-) | 00:26 |
| NobodyCam | I'm thinking None needs to be a valid state | 00:26 |
| NobodyCam | basicly means we havent checked the actual state. | 00:27 |
| NobodyCam | but I need to thnk thru this | 00:27 |
| rloo | right, haven't checked or don't know. the nova driver *could* ask for the power state first, which would cause it to get the state. | 00:27 |
| NobodyCam | ya let me think thru it some more | 00:28 |
| NobodyCam | :) | 00:28 |
| rloo | no worries. I'm not in any hurry :D | 00:29 |
| *** urulama_ has joined #openstack-ironic | 00:29 | |
| *** urulama has quit IRC | 00:29 | |
| NobodyCam | :) lol I am.. I want it all and I want it now... (/me now has stupid song stuck in his head) | 00:30 |
| rloo | (sorry, I think) | 00:33 |
| NobodyCam | lol :) | 00:33 |
| NobodyCam | http://www.youtube.com/watch?v=1pm4fQRl72k | 00:35 |
| * NobodyCam shares | 00:35 | |
| rloo | NobodyCam, I'm embarrassed to say, I don't recognize that song. Maybe I wasn't born then? (Just kidding.) | 00:39 |
| *** urulama_ has quit IRC | 00:39 | |
| NobodyCam | lol it could be... | 00:39 |
| NobodyCam | I am old | 00:39 |
| *** urulama has joined #openstack-ironic | 00:39 | |
| NobodyCam | :-p | 00:39 |
| NobodyCam | heheheh | 00:39 |
| *** urulama has quit IRC | 00:44 | |
| *** matsuhashi has quit IRC | 00:47 | |
| *** matsuhashi has joined #openstack-ironic | 00:48 | |
| *** urulama has joined #openstack-ironic | 00:49 | |
| NobodyCam | because its that time of the year. http://www.youtube.com/watch?v=DiXjbI3kRus | 00:49 |
| rloo | NobodyCam: I'm not going to watch that. No snow yet, and not yet middle of December! | 00:51 |
| NobodyCam | lol my dad starts right after octtober | 00:52 |
| NobodyCam | its engrained in me | 00:52 |
| *** matsuhashi has quit IRC | 00:53 | |
| rloo | Wow, he doesn't even wait til Hallowe'en. And what about Thanksgiving? That's your big holiday!! | 00:53 |
| NobodyCam | we got hallowen but after that ... all xmass all the time. with octional protest | 00:56 |
| *** matsuhashi has joined #openstack-ironic | 00:58 | |
| pleia2 | hanukkah begins before thanksgiving this year, it's hurting my planning brain | 00:58 |
| NobodyCam | :( ouch | 00:58 |
| *** urulama_ has joined #openstack-ironic | 00:59 | |
| *** urulama has quit IRC | 00:59 | |
| *** nosnos has joined #openstack-ironic | 01:00 | |
| * NobodyCam hangs his head low as he makes a glass of egg nog :-) | 01:01 | |
| pleia2 | ++ | 01:01 |
| NobodyCam | :) | 01:01 |
| *** jbjohnso has quit IRC | 01:06 | |
| *** urulama_ has quit IRC | 01:08 | |
| *** urulama has joined #openstack-ironic | 01:09 | |
| *** urulama has quit IRC | 01:19 | |
| *** urulama_ has joined #openstack-ironic | 01:19 | |
| *** urulama_ has quit IRC | 01:24 | |
| *** urulama has joined #openstack-ironic | 01:28 | |
| *** urulama_ has joined #openstack-ironic | 01:38 | |
| *** urulama has quit IRC | 01:39 | |
| *** urulama_ has quit IRC | 01:48 | |
| *** urulama has joined #openstack-ironic | 01:48 | |
| *** urulama has quit IRC | 01:53 | |
| *** jbjohnso has joined #openstack-ironic | 01:57 | |
| *** urulama has joined #openstack-ironic | 01:58 | |
| *** matsuhashi has quit IRC | 02:03 | |
| *** matsuhashi has joined #openstack-ironic | 02:03 | |
| *** urulama_ has joined #openstack-ironic | 02:08 | |
| *** urulama has quit IRC | 02:09 | |
| *** urulama_ has quit IRC | 02:17 | |
| *** urulama has joined #openstack-ironic | 02:18 | |
| *** urulama has quit IRC | 02:27 | |
| *** urulama_ has joined #openstack-ironic | 02:28 | |
| *** urulama_ has quit IRC | 02:33 | |
| *** urulama has joined #openstack-ironic | 02:37 | |
| *** urulama_ has joined #openstack-ironic | 02:47 | |
| *** urulama has quit IRC | 02:48 | |
| *** urulama has joined #openstack-ironic | 02:57 | |
| *** urulama_ has quit IRC | 02:57 | |
| *** jbjohnso has quit IRC | 03:01 | |
| *** urulama has quit IRC | 03:02 | |
| *** urulama has joined #openstack-ironic | 03:06 | |
| *** dkehn has quit IRC | 03:11 | |
| *** dkehn has joined #openstack-ironic | 03:13 | |
| *** urulama has quit IRC | 03:16 | |
| *** urulama has joined #openstack-ironic | 03:16 | |
| *** urulama has quit IRC | 03:26 | |
| *** urulama has joined #openstack-ironic | 03:26 | |
| *** matsuhashi has quit IRC | 03:31 | |
| *** matsuhashi has joined #openstack-ironic | 03:31 | |
| *** urulama_ has joined #openstack-ironic | 03:36 | |
| *** urulama has quit IRC | 03:37 | |
| *** matsuhashi has quit IRC | 03:41 | |
| *** matsuhashi has joined #openstack-ironic | 03:42 | |
| *** urulama_ has quit IRC | 03:42 | |
| *** sandeepr_ltp has joined #openstack-ironic | 03:46 | |
| *** urulama has joined #openstack-ironic | 03:46 | |
| *** matsuhashi has quit IRC | 03:47 | |
| *** sandeepr_ltp has quit IRC | 03:47 | |
| *** sandeepr has quit IRC | 03:52 | |
| *** ekarlso has quit IRC | 03:52 | |
| *** ekarlso has joined #openstack-ironic | 03:54 | |
| *** urulama_ has joined #openstack-ironic | 03:56 | |
| *** urulama has quit IRC | 03:56 | |
| *** fandikurnia01 has joined #openstack-ironic | 03:58 | |
| *** urulama_ has quit IRC | 04:05 | |
| *** urulama has joined #openstack-ironic | 04:06 | |
| *** rloo has quit IRC | 04:07 | |
| *** urulama has quit IRC | 04:11 | |
| * devananda is back, starts working on sync_power_state | 04:12 | |
| *** urulama has joined #openstack-ironic | 04:16 | |
| *** jianingy is now known as jianingy_afk | 04:19 | |
| *** urulama_ has joined #openstack-ironic | 04:25 | |
| *** urulama has quit IRC | 04:26 | |
| *** urulama_ has quit IRC | 04:35 | |
| *** urulama has joined #openstack-ironic | 04:35 | |
| *** urulama has quit IRC | 04:45 | |
| *** urulama has joined #openstack-ironic | 04:45 | |
| *** urulama has quit IRC | 04:50 | |
| *** prekarat has joined #openstack-ironic | 04:51 | |
| *** prekarat has quit IRC | 04:52 | |
| *** prekarat has joined #openstack-ironic | 04:52 | |
| *** urulama has joined #openstack-ironic | 04:55 | |
| *** matsuhashi has joined #openstack-ironic | 04:56 | |
| *** urulama has quit IRC | 05:04 | |
| *** urulama has joined #openstack-ironic | 05:04 | |
| *** jamespage has quit IRC | 05:06 | |
| *** urulama has quit IRC | 05:14 | |
| *** urulama has joined #openstack-ironic | 05:14 | |
| *** urulama has quit IRC | 05:19 | |
| *** urulama has joined #openstack-ironic | 05:24 | |
| devananda | well, i got distracted | 05:33 |
| devananda | going to patch-bomb with oslo updates in a bit | 05:34 |
| *** urulama has quit IRC | 05:34 | |
| *** urulama has joined #openstack-ironic | 05:34 | |
| *** prekarat1 has joined #openstack-ironic | 05:38 | |
| *** prekarat has quit IRC | 05:41 | |
| *** urulama_ has joined #openstack-ironic | 05:44 | |
| *** urulama has quit IRC | 05:44 | |
| *** urulama_ has quit IRC | 05:54 | |
| *** urulama has joined #openstack-ironic | 05:54 | |
| *** urulama has quit IRC | 05:55 | |
| *** urulama has joined #openstack-ironic | 05:55 | |
| openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/56325 | 06:00 |
| Haomeng | devananda: ping | 06:06 |
| devananda | Haomeng: pong | 06:20 |
| Haomeng | devananda: I checked with Ceilometer guys for the auto-gen doc issue, they use https://git.openstack.org/cgit/stackforge/sphinxcontrib-docbookrestapi/ to create api, but the input is rst file | 06:21 |
| Haomeng | and I update the defect you raised - https://bugs.launchpad.net/ironic/+bug/1251011, can you take a look if you have time? | 06:21 |
| Haomeng | I have updated my findings/investigations | 06:22 |
| devananda | Haomeng: so my understanding is they are able to generate from the pecan/wsme python code somehow | 06:24 |
| devananda | Haomeng: we already have .rst file, but it is hand-written, and therefor perpetually out of sync with the python code | 06:25 |
| Haomeng | yes, I am confusing here, will double check with them, as the result yesterday, the source is RST file not our python code I think | 06:25 |
| Haomeng | yes, I understand your point | 06:26 |
| devananda | Haomeng: perhaps there is another step where we can generate teh RST file from python code | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.config.generator from oslo https://review.openstack.org/56514 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.context from oslo https://review.openstack.org/56515 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common.db.sqlalchemy changes from olso https://review.openstack.org/56516 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common utils from Oslo https://review.openstack.org/56517 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.local from oslo https://review.openstack.org/56518 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.lockutils from oslo https://review.openstack.org/56519 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.log from oslo https://review.openstack.org/56520 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.notifier.* from oslo https://review.openstack.org/56521 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.periodic_task from oslo https://review.openstack.org/56522 | 06:26 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.service from oslo https://review.openstack.org/56523 | 06:26 |
| * devananda snickers at the patch bomb | 06:26 | |
| Haomeng | but jd__ just share us the link of sphinxcontrib-docbookrestapi, and he said, Ceilometer is using sphinxcontrib-docbookrestapi | 06:26 |
| Haomeng | but the input for sphinxcontrib-docbookrestapi is rst file , it is "Sphinx extension that generates documentation for api-site from RST files." | 06:27 |
| Haomeng | let me do more investigations to see the gap between python script and rst, thank you devananda | 06:28 |
| devananda | thanks | 06:29 |
| Haomeng | and update to you the resunt on the bug you raised | 06:29 |
| Haomeng | :) | 06:29 |
| Haomeng | anytime:) | 06:29 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: update sample config https://review.openstack.org/56524 | 06:32 |
| Haomeng | devananda:looks like I found the steps to create document - https://wiki.openstack.org/wiki/Ironic#Building_Documentation_Locally, one step is "python setup.py build_sphinx", it sounds like to create doc, let me try | 06:39 |
| Haomeng | it can create a lot of rst files:) | 06:44 |
| Haomeng | for each python file I think | 06:45 |
| *** openstackgerrit has quit IRC | 06:46 | |
| *** openstackgerrit has joined #openstack-ironic | 06:46 | |
| Haomeng | but the rst files content is just 6 lines for each rst file, strange | 06:48 |
| *** jamespage has joined #openstack-ironic | 06:56 | |
| *** rwsu is now known as rwsu-pto | 07:04 | |
| devananda | Haomeng: yes, that converts the inline pthon doc strings to .rst | 07:07 |
| devananda | ... or it used to | 07:08 |
| *** r-mibu_ has quit IRC | 07:12 | |
| Haomeng | I think so, will practice to see if it works | 07:13 |
| Haomeng | devananda, I have another issue about object ID | 07:13 |
| Haomeng | we have uuid and internal id for each object | 07:13 |
| Haomeng | and I found some Ironic command can accept the internal id which is a int as input, do you think the internal id can be used for cli? | 07:14 |
| Haomeng | or api | 07:14 |
| Haomeng | should we reject such internal id as input ? | 07:15 |
| Haomeng | such as " ironic port-show 1", it works now, will return the port which internal id is 1 | 07:18 |
| Haomeng | I think we should reject such internal id as input for both CLI and API, I understand the internal id is used for get good performance by internally | 07:19 |
| GheRivero | morning Ironic | 07:38 |
| *** romcheg has joined #openstack-ironic | 07:41 | |
| Haomeng | morning GheRivero | 07:51 |
| Haomeng | I guess your local time is around 9am, right? | 07:51 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add context to TaskManager.node object https://review.openstack.org/56529 | 07:52 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement sync_power_state periodic task https://review.openstack.org/56530 | 07:52 |
| GheRivero | yeah, it's | 07:53 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.log from oslo https://review.openstack.org/56520 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.notifier.* from oslo https://review.openstack.org/56521 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.periodic_task from oslo https://review.openstack.org/56522 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.service from oslo https://review.openstack.org/56523 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: update sample config https://review.openstack.org/56524 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common.db.sqlalchemy changes from olso https://review.openstack.org/56516 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common utils from Oslo https://review.openstack.org/56517 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.local from oslo https://review.openstack.org/56518 | 07:54 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.lockutils from oslo https://review.openstack.org/56519 | 07:54 |
| romcheg | OMG | 07:54 |
| devananda | Haomeng: you are correct. int ID should never be allowed in API | 07:54 |
| romcheg | Morning everyone | 07:55 |
| devananda | Haomeng: i think lucas was working on that? | 07:55 |
| devananda | romcheg: g'morning! | 07:55 |
| devananda | and g'night :) | 07:55 |
| romcheg | g'night devananda | 07:55 |
| devananda | it's almost midnight - i should get some rest. fighting off a flu ... | 07:55 |
| Haomeng | yes, I remember theere is such patch now in review progress | 07:55 |
| devananda | but NobodyCam was waiting on the sync_power_state change, which is buried up there somewhere | 07:55 |
| Haomeng | devananda: for api doc, I have run "python setup.py build_sphinx", it can generated api doc like "http://docs.openstack.org/developer/ironic/dev/api-spec-v1.html" | 07:56 |
| Haomeng | and for the official api in api.openstack.org, I think openstack api-site owner can help to generated with sphinxcontrib-docbookrestapi extensions which is for restapi only | 07:57 |
| Haomeng | devananda: good night, you should be tired now:) | 07:58 |
| devananda | Haomeng: what you're looking for are these: http://docs.openstack.org/developer/ironic/py-modindex.html | 07:58 |
| devananda | Haomeng: but for some reason, all the ironic/api/controller/v1/* files are not there | 07:58 |
| Haomeng | yes, this is not our Ironic wiki api doc | 07:59 |
| Haomeng | I investigated already | 07:59 |
| devananda | Haomeng: we want to delete that api-spec-v1.rst file | 07:59 |
| Haomeng | we have two api document url, one is from our Ironic wiki | 07:59 |
| Haomeng | another one is in http://docs.openstack.org/developer/openstack-projects.html | 07:59 |
| devananda | Haomeng: what API doc on our wiki? | 08:00 |
| Haomeng | yes, for our Ironic wiki, it will use our Ironic project rst file to create html | 08:00 |
| devananda | no ... | 08:00 |
| devananda | wiki is just a wiki - anyone can edit it | 08:00 |
| devananda | this http://docs.openstack.org/developer/ironic/ is auto-generated from the ironic source code on every commit | 08:01 |
| Haomeng | sorry for confusing, should be same website http://docs.openstack.org/developer/ironic/ | 08:01 |
| Haomeng | yes | 08:01 |
| *** matsuhashi has quit IRC | 08:01 | |
| *** matsuhashi has joined #openstack-ironic | 08:02 | |
| *** fandikurnia01 has quit IRC | 08:02 | |
| devananda | Haomeng: our wiki contains instructions to build the docs locally, for testing purposes. that's all. | 08:02 |
| devananda | something is wrong, though. it's not generating the RST and HTML files for ironic/api/controller/v1/* | 08:03 |
| devananda | anyhow, i really need to sleep now :) | 08:03 |
| *** fandikurnia01 has joined #openstack-ironic | 08:03 | |
| devananda | o/ | 08:03 |
| Haomeng | i mean there are another api document url with different format - http://api.openstack.org/api-ref-compute.html | 08:04 |
| Haomeng | good night devanada:) | 08:04 |
| Haomeng | let me go ahead | 08:04 |
| Haomeng | have a nice dream:) | 08:04 |
| *** michchap has quit IRC | 08:12 | |
| *** michchap has joined #openstack-ironic | 08:12 | |
| *** yuriyz has quit IRC | 08:13 | |
| *** jistr has joined #openstack-ironic | 08:49 | |
| *** jistr has quit IRC | 09:01 | |
| *** jistr has joined #openstack-ironic | 09:04 | |
| *** lucasagomes has joined #openstack-ironic | 09:15 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Fix node lock in PXE driver https://review.openstack.org/55565 | 09:17 |
| *** derekh has joined #openstack-ironic | 09:18 | |
| *** matsuhashi has quit IRC | 09:25 | |
| *** matsuhashi has joined #openstack-ironic | 09:25 | |
| GheRivero | uhm... what's the differnece between ironic/db/api.py and ironic/db/sqlalchemy/api.py | 09:26 |
| *** nosnos has quit IRC | 09:26 | |
| *** nosnos has joined #openstack-ironic | 09:27 | |
| GheRivero | forget it. db/api.py are just abstract classes | 09:28 |
| Haomeng | yes, sqlalchemy/api.py is just an implement for special qlalchemy ORM | 09:29 |
| *** matsuhashi has quit IRC | 09:30 | |
| *** nosnos has quit IRC | 09:31 | |
| *** prekarat1 has left #openstack-ironic | 09:51 | |
| *** f13o has joined #openstack-ironic | 09:58 | |
| * max_lobur is away: Leaving | 10:10 | |
| *** jistr has quit IRC | 10:20 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API Ports node_id attribute to be unicode only https://review.openstack.org/56573 | 10:24 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API Ports node_id attribute to be unicode only https://review.openstack.org/56573 | 10:26 |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Don't allow change reservation at update node https://review.openstack.org/55549 | 11:04 |
| *** fandikurnia01 has quit IRC | 11:12 | |
| lucasagomes | romcheg, ping | 11:29 |
| romcheg | lucasagomes: pong | 11:29 |
| lucasagomes | romcheg, that UniqueConstraint in the sqlalchemy doesn't work with sqlite?! | 11:29 |
| romcheg | lucasagomes: AFAIR, it worked for em | 11:30 |
| romcheg | me | 11:30 |
| lucasagomes | romcheg, I see | 11:30 |
| lucasagomes | hmm weird | 11:30 |
| romcheg | lucasagomes: maybe you forgot to add the migration? | 11:31 |
| lucasagomes | I'm not changing the db, but u know the API we have a test to check if the address on ports already exists | 11:31 |
| lucasagomes | also was checking the format of the address | 11:31 |
| lucasagomes | now I'm chekcing the format differently using wsme type validations | 11:31 |
| lucasagomes | and I tried to remove that function believing that the db would then raise some execption in case the entry is duplicated | 11:32 |
| lucasagomes | but it didn't | 11:32 |
| romcheg | lucasagomes: there might be a bug in migrations | 11:32 |
| lucasagomes | on mysql it does, I tested with my devstack and it raised an IntegrityError | 11:32 |
| romcheg | So the constraint exists in the model but doesn't exist in the DB | 11:32 |
| lucasagomes | romcheg, I see | 11:33 |
| lucasagomes | romcheg, but hmm as it works with mysql, idk | 11:33 |
| romcheg | hmm | 11:33 |
| lucasagomes | weird... I will check it out | 11:33 |
| romcheg | lucasagomes: I was just about to go having lunch | 11:34 |
| romcheg | I will be here very soon | 11:34 |
| lucasagomes | romcheg, go for ur lunch, it's fine | 11:34 |
| lucasagomes | was just curiosity | 11:34 |
| lucasagomes | might be something here as well I kinda have a messy enviroment in the moment | 11:34 |
| romcheg | You can change DB address from memory to file and run a specific test on it | 11:35 |
| romcheg | Then you'll be able to verify the structure of the DB | 11:36 |
| lucasagomes | ah, cheers will take a look | 11:36 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Using the mandatory option of WSME on Ports https://review.openstack.org/56597 | 12:03 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Using the mandatory option of WSME on Ports https://review.openstack.org/56597 | 12:05 |
| *** jistr has joined #openstack-ironic | 12:07 | |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception. https://review.openstack.org/56598 | 12:13 |
| *** jbjohnso has joined #openstack-ironic | 12:15 | |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception. https://review.openstack.org/56598 | 12:18 |
| *** michchap has quit IRC | 12:23 | |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception https://review.openstack.org/56598 | 12:23 |
| *** michchap has joined #openstack-ironic | 12:24 | |
| *** jbjohnso has quit IRC | 12:29 | |
| *** michchap has quit IRC | 12:29 | |
| *** michchap has joined #openstack-ironic | 12:29 | |
| *** prekarat has joined #openstack-ironic | 12:42 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Import missing gettext _ to fix Sphinx error https://review.openstack.org/56605 | 13:21 |
| Haomeng | morning Ironic:) | 13:32 |
| * max_lobur is back (gone 03:25:38) | 13:36 | |
| *** jdob has joined #openstack-ironic | 13:36 | |
| max_lobur | morning Everyone | 13:36 |
| Haomeng | morning max_lobur | 13:45 |
| max_lobur | morning Haomeng | 13:45 |
| Haomeng | Max, I encountered new issue | 13:45 |
| Haomeng | this one - https://review.openstack.org/#/c/56605/ | 13:46 |
| max_lobur | Added myself to reviewers | 13:46 |
| max_lobur | will take a look today | 13:46 |
| Haomeng | sure | 13:46 |
| Haomeng | and I will fix other opening issue | 13:47 |
| Haomeng | thank you max | 13:47 |
| Haomeng | i just want to ask you here a short question | 13:47 |
| max_lobur | shoot | 13:47 |
| Haomeng | for such "H302 import only modules.'from ironic.openstack.common.gettextutils import _' does not import a module", should we change to "from ironic.openstack.common import gettextutils" | 13:48 |
| Haomeng | and call "_" with "gettextutils._", that is strange, but I have no idea, because we enable H302 | 13:49 |
| *** openstackgerrit has quit IRC | 13:53 | |
| *** jimjiang has quit IRC | 13:53 | |
| max_lobur | to make gettextutils stuff work there should be a call of gettextutils.install('ironic') in entry point | 13:53 |
| *** openstackgerrit has joined #openstack-ironic | 13:53 | |
| *** jimjiang has joined #openstack-ironic | 13:53 | |
| max_lobur | for REST API call the entry point is ironic/cmd/__init__.py | 13:54 |
| Haomeng | yes, let me check | 13:54 |
| max_lobur | for config sample generator for example this is done in the generator file - see /ironic/openstack/common/config/generator.py | 13:54 |
| Haomeng | our gettextutils works fine, but encoutered such issue when run "python setup.py build_sphinx" | 13:55 |
| max_lobur | so need to define what entry point is used for build_sphinx and add this call there | 13:55 |
| max_lobur | correct me someone if I'm wrong =) | 13:56 |
| Haomeng | I am not sure, just try to add gettextutils import in common.exceptins to avoid such issue, and I checked another Ceilometer project | 13:57 |
| Haomeng | found Ceilomter will " import _' " for some files | 13:57 |
| max_lobur | hm | 13:58 |
| Haomeng | so, thank you Max | 13:58 |
| Haomeng | and for other your comments about my opening patches, I will try to close this weekend, it isnot very easy to fix, sorry for delay response | 13:59 |
| Haomeng | max_lobur: I guess your local time is around 10am, right? | 14:00 |
| Haomeng | I am 10pm now:) | 14:00 |
| *** jimjiang has quit IRC | 14:01 | |
| Haomeng | tired, will have a rest:) | 14:01 |
| Haomeng | nice day:) | 14:01 |
| *** jimjiang has joined #openstack-ironic | 14:01 | |
| max_lobur | my local is 4 pm, I just haven't chance to be back to Ironic earlier today =) | 14:02 |
| max_lobur | yea, have a good rest | 14:03 |
| Haomeng | :) | 14:03 |
| *** jbjohnso has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Allow vendor passthru without Keystone authorization https://review.openstack.org/56612 | 14:06 |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Fix integer division compatibility in middleware https://review.openstack.org/56613 | 14:07 |
| *** f13o has quit IRC | 14:11 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Import missing gettext _ to fix Sphinx error https://review.openstack.org/56605 | 14:12 |
| *** russellb is now known as rustlebee | 14:13 | |
| *** anteaya is now known as anttteaya | 14:14 | |
| *** yuriyz has joined #openstack-ironic | 14:16 | |
| *** lucasagomes is now known as lucas-hungry | 14:20 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Import missing gettext _ to fix Sphinx error https://review.openstack.org/56605 | 14:21 |
| *** f13o has joined #openstack-ironic | 14:23 | |
| *** jistr has quit IRC | 14:29 | |
| *** jistr has joined #openstack-ironic | 14:32 | |
| *** Haomeng has quit IRC | 14:56 | |
| *** Haomeng has joined #openstack-ironic | 14:56 | |
| *** lucas-hungry is now known as lucasagomes | 14:57 | |
| NobodyCam | Good morning iRonic, TGIF | 15:05 |
| *** yuriyz has quit IRC | 15:08 | |
| NobodyCam | lucasagomes: you around? | 15:25 |
| lucasagomes | NobodyCam, hey yes | 15:25 |
| NobodyCam | ;) | 15:25 |
| NobodyCam | so looks like the cli will need a validate / set current power state | 15:26 |
| NobodyCam | s/cli/client | 15:26 |
| lucasagomes | NobodyCam, I see, I'm working on validations right now... not exactly that | 15:27 |
| lucasagomes | but I'm investigating wsme validations for complex types | 15:27 |
| NobodyCam | working on the nova driver, I've run into a issue with newly reated nodes having the power state of None | 15:28 |
| lucasagomes | NobodyCam, a validate for the current power state!? | 15:28 |
| NobodyCam | this prevents nova from setting the instance uuid | 15:28 |
| NobodyCam | reated = created | 15:29 |
| lucasagomes | I see | 15:29 |
| *** romcheg has quit IRC | 15:30 | |
| lucasagomes | NobodyCam, btw I wanna talk to u about another thing | 15:30 |
| lucasagomes | you know this node_id is annoying | 15:30 |
| lucasagomes | first the name node_id is actually an UUID in the API pov | 15:30 |
| lucasagomes | what you think about having an node_uuid on the API? | 15:30 |
| NobodyCam | yes .. there should now be a bug to change all refs of id to uuid | 15:31 |
| lucasagomes | + when set it will also set (internally) the node_id related to that node_uuid | 15:31 |
| NobodyCam | for user | 15:31 |
| lucasagomes | no more conversions by hand | 15:31 |
| lucasagomes | NobodyCam, yea | 15:31 |
| NobodyCam | :) | 15:31 |
| lucasagomes | I'm working on that | 15:31 |
| GheRivero | morning all | 15:31 |
| lucasagomes | will see if I can get something for today | 15:31 |
| lucasagomes | GheRivero, morning | 15:31 |
| NobodyCam | good morning GheRivero :) | 15:31 |
| *** tatyana has joined #openstack-ironic | 15:35 | |
| GheRivero | so, what's the point of having id when using uuid every place? | 15:37 |
| NobodyCam | id is use for some db ops for speed | 15:38 |
| lucasagomes | GheRivero, the id is internal use only, like for JOINs in the database | 15:38 |
| lucasagomes | so my idea is to have both in the api, node_id and node_uuid... node_id will never be exposed to the user, but it's used when converting RPC-style objects to API objects | 15:39 |
| lucasagomes | magically setting an node_id will also set the node_uuid with the relative UUID for that numeric id | 15:40 |
| lucasagomes | and vice versa | 15:40 |
| *** jdob has quit IRC | 15:40 | |
| *** jdob has joined #openstack-ironic | 15:40 | |
| lucasagomes | so we don't have to convert by hand from id->uuid when returning to the user or uuid->id when sending it via RPC | 15:41 |
| lucasagomes | I've something almost done here, it works :) | 15:41 |
| NobodyCam | lucasagomes: and just not tell the users that there is this node_id thing... | 15:41 |
| lucasagomes | NobodyCam, yes, the users won't know about the node_id at all | 15:41 |
| lucasagomes | it will exist in the api object for compatibility with the RPC objects only | 15:42 |
| NobodyCam | lucasagomes: ++ | 15:42 |
| GheRivero | ok. now I have the full picture. thanks | 15:42 |
| *** rustlebee is now known as drumkilla | 15:50 | |
| *** drumkilla is now known as rustlebee | 15:50 | |
| *** jbjohnso has quit IRC | 15:51 | |
| *** rustlebee is now known as drumkilla | 15:51 | |
| * NobodyCam gets more coffee | 15:54 | |
| *** lucasagomes is now known as lucas-afk | 15:56 | |
| *** jbjohnso has joined #openstack-ironic | 16:14 | |
| *** sandeepr has joined #openstack-ironic | 16:21 | |
| sandeepr | morning/evening ironic | 16:21 |
| NobodyCam | good morning sandeepr :) | 16:22 |
| sandeepr | hey hey NobodyCam | 16:23 |
| NobodyCam | happy Friday :) | 16:23 |
| sandeepr | thanks and u2 :) | 16:23 |
| NobodyCam | :) | 16:23 |
| sandeepr | i saw a new slideshare from devananda which has changes in the way deployment works especially the machine image part | 16:24 |
| NobodyCam | :) | 16:25 |
| *** jdob has quit IRC | 16:25 | |
| *** jdob has joined #openstack-ironic | 16:26 | |
| *** romcheg has joined #openstack-ironic | 16:27 | |
| sandeepr | what is the partition disk based on user metadata? | 16:29 |
| devananda | morning, all | 16:30 |
| NobodyCam | good morning devananda :) | 16:33 |
| *** jdob has quit IRC | 16:40 | |
| NobodyCam | bbt...brb | 16:41 |
| *** jdob has joined #openstack-ironic | 16:41 | |
| sandeepr | hey devananda, i saw your new slideshare | 16:41 |
| sandeepr | wanted to know couple of things | 16:41 |
| sandeepr | partition disks based on user metadata and fetch user image | 16:42 |
| sandeepr | what is user metadata and how is it passed? | 16:43 |
| sandeepr | what is the method used to fetch user image? | 16:43 |
| devananda | sandeepr: i haven't uploaded my slides from HK, actually | 16:43 |
| devananda | sandeepr: we had discussed several times using cinder's API as a means to specify the disk layout. but the cinder devs disagreed during the ironic design session, and we're back to no idea now | 16:44 |
| sandeepr | back to no idea on the method to fetch the user image? | 16:45 |
| sandeepr | oh disk partition | 16:46 |
| sandeepr | hmm ok | 16:46 |
| sandeepr | i guess the machine image copy is causing a bigger bottle neck | 16:46 |
| devananda | sandeepr: fetch user image -- this has been part of the pxe driver since grizzly | 16:47 |
| devananda | the bottleneck was due to file injection | 16:47 |
| devananda | the pxe driver was mounting the image and editing it. this has been "fixed" - it is no longer necessary | 16:48 |
| openstackgerrit | Max Lobur proposed a change to openstack/ironic: Redefined default value of allowed_rpc_exception_modules https://review.openstack.org/56650 | 16:49 |
| sandeepr | devananda, yes your slideshare show that. however during the disk partition the image is fetched which would still cause a bottleneck? | 16:50 |
| sandeepr | i also read the bp from dkehn - https://blueprints.launchpad.net/tripleo/+spec/bittorrent-for-image-deployments | 16:50 |
| openstackgerrit | Jenkins proposed a change to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/54430 | 16:50 |
| sandeepr | something on the lines of peer-peer- network | 16:51 |
| sandeepr | which is quite interesting | 16:51 |
| devananda | sandeepr: IMO, that is a premature optimization. interesting, yes, but very special case, and there are much more important things to solve before we | 16:51 |
| devananda | deal with the parallel-10k-node-deploy problem | 16:51 |
| sandeepr | devananda, sure there are much more important things to solve :) | 16:52 |
| NobodyCam | post bbt walkies... bbiaf | 16:52 |
| GheRivero | the problem with bt is that with a small number of images, the time to deploy is bigger than using just glance | 16:52 |
| GheRivero | besides, you need torrent trackers, servers, if the node reboots as soon as gets the images, there won't be enough seeders.. | 16:53 |
| sandeepr | my 2 cents as people out there will first do bulk provisioning and will raise this straight | 16:53 |
| dkehn | GheRivero: note: when using the bittorado tracker included | 16:57 |
| *** lucas-afk is now known as lucasagomes | 16:58 | |
| GheRivero | dkehn: cool, but bittornado last release was like... 7 years ago? | 17:00 |
| GheRivero | i would love to have a server+tracker all in one, anyway | 17:01 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.log from oslo https://review.openstack.org/56520 | 17:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.notifier.* from oslo https://review.openstack.org/56521 | 17:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.periodic_task from oslo https://review.openstack.org/56522 | 17:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.service from oslo https://review.openstack.org/56523 | 17:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: update sample config https://review.openstack.org/56524 | 17:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.lockutils from oslo https://review.openstack.org/56519 | 17:05 |
| max_lobur | devananda, will you please take a look? https://review.openstack.org/#/c/56650/ (Redefined default value of allowed_rpc_exception_modules) | 17:07 |
| devananda | max_lobur: reviewed | 17:10 |
| max_lobur | thanks =) | 17:11 |
| *** jistr has quit IRC | 17:12 | |
| lucasagomes | http://paste.openstack.org/show/53174/ any idea someone?! | 17:13 |
| lucasagomes | its the sqlalchemy-migrate | 17:14 |
| NobodyCam | the .dev? | 17:15 |
| NobodyCam | parse looking for a #? | 17:16 |
| devananda | urgh. why does jenkins hate my oslo sync patches? | 17:17 |
| lucasagomes | NobodyCam, yea, weird... I will check it later | 17:17 |
| NobodyCam | its friday, | 17:17 |
| lucasagomes | just installed another version because I want to finish my patch | 17:17 |
| openstackgerrit | Max Lobur proposed a change to openstack/ironic: Redefined default value of allowed_rpc_exception_modules https://review.openstack.org/56650 | 17:22 |
| GheRivero | from tripleo channel: https://bugs.launchpad.net/tripleo/+bug/1251700 | 17:23 |
| GheRivero | looks like there is a issue with sqlalchemy=migrate | 17:24 |
| NobodyCam | :) | 17:24 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/python-ironicclient: Comply with new hacking release https://review.openstack.org/56668 | 17:25 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement sync_power_state periodic task https://review.openstack.org/56530 | 17:36 |
| NobodyCam | devananda: do you have an option on with reguards to questions on https://review.openstack.org/#/c/56325/ I'm inclined to vote with roman | 17:37 |
| NobodyCam | and seems your patches have been bit by the bug GheRivero pasted | 17:39 |
| *** derekh has quit IRC | 17:44 | |
| devananda | NobodyCam: i have no opinion yet. i pinged clark about it to see if other projects have a policy | 17:52 |
| devananda | and yes, that bug seems to be hitting many people | 17:52 |
| NobodyCam | :) | 17:53 |
| max_lobur | my patch hit 0.7.3.dev too :( | 17:55 |
| devananda | ugh | 17:55 |
| devananda | looks like there's a fix up: https://review.openstack.org/#/c/56671/ | 17:57 |
| * devananda makes tea | 17:58 | |
| NobodyCam | lol mordred looks like rebase is needed on 56671 | 18:03 |
| devananda | heh | 18:04 |
| devananda | reminder to folks - lucasagomes, romcheg, etc - please see my email on the ML about blueprints | 18:09 |
| devananda | and generally speaking, reading https://wiki.openstack.org/wiki/Blueprints would be good :) | 18:09 |
| lucasagomes | devananda, will do, I drafted one bp but I've to do more | 18:09 |
| mordred | on it | 18:09 |
| devananda | nova team seems to have done a lot to add to that wiki recently | 18:09 |
| lucasagomes | just trying to fix one problem with the wsme validations first, can't stop until I get something working >.< | 18:10 |
| devananda | lucasagomes: np :) i know how that is | 18:10 |
| romcheg | devananda: will do them on this weekend | 18:10 |
| lucasagomes | :) | 18:10 |
| romcheg | now have to go to the spanish class | 18:11 |
| romcheg | have a good weekend folks | 18:11 |
| devananda | romcheg: thanks! you too | 18:11 |
| NobodyCam | you too romcheg | 18:11 |
| devananda | NobodyCam, lucasagomes - on the transifex patches, see -infra for info. tl;dr - it's good to do a quick scan that nothing wierd is in the patch. and we can leave it there, jenkins will re-use the patchset as necessary until we merge it | 18:13 |
| lucasagomes | gotcha | 18:14 |
| NobodyCam | devananda: ++ | 18:14 |
| lucasagomes | devananda, btw, how it works... if I want to translate Ironic to pt_BR I need an account on transifex? | 18:16 |
| *** romcheg has quit IRC | 18:17 | |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception https://review.openstack.org/56598 | 18:18 |
| NobodyCam | do we support tlh | 18:19 |
| devananda | tlh? | 18:21 |
| NobodyCam | https://www.transifex.com/languages/ | 18:21 |
| devananda | heh | 18:22 |
| NobodyCam | you know we need it | 18:22 |
| NobodyCam | lol | 18:22 |
| devananda | it doesn't seem to be in the current set of langs, but why not | 18:23 |
| NobodyCam | i was looking | 18:23 |
| lucasagomes | found here :) | 18:23 |
| lucasagomes | https://www.transifex.com/projects/p/ironic/ | 18:23 |
| lucasagomes | http://logs.openstack.org/98/56598/4/check/gate-ironic-python26/a39f857/console.html <<< same problem I had | 18:24 |
| lucasagomes | it's the sqlmigrate version | 18:24 |
| devananda | awesome | 18:24 |
| devananda | and yea, that | 18:24 |
| NobodyCam | yahttps://review.openstack.org/56671 | 18:24 |
| devananda | just waiting for the fix to propagate now | 18:25 |
| NobodyCam | i'll ping calrk to add tlh | 18:25 |
| *** hemnafk is now known as hemna | 18:28 | |
| *** jianingy_afk has quit IRC | 18:34 | |
| *** Haomeng has quit IRC | 18:36 | |
| *** yanhcdl has quit IRC | 18:37 | |
| *** jianingy_afk has joined #openstack-ironic | 18:38 | |
| *** max_lobur has quit IRC | 18:39 | |
| *** max_lobur has joined #openstack-ironic | 18:39 | |
| NobodyCam | seems I can not request a new language for ironic | 18:40 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API ports update for WSME 0.5b6 compliance https://review.openstack.org/56682 | 18:46 |
| lucasagomes | devananda, NobodyCam I marked as WIP because I will have to split it in more than one patch, but if u guys can take a look please | 18:47 |
| lucasagomes | ^ | 18:48 |
| lucasagomes | now we can use wsme to check for mandatory fields | 18:48 |
| lucasagomes | I also added a new attribute on the port api object: node_uuid, when this attribute is set it automaticaly set the node_id with the relative numeric id for that uuid | 18:49 |
| lucasagomes | same when setting node_id, it will automatically set the node_uuid | 18:49 |
| lucasagomes | so now in the API users will use node_uuid=<uuid> and internally it will be translated to node_id=<numeric id> for internal use | 18:53 |
| lucasagomes | anyway... I'm done for today | 18:53 |
| lucasagomes | have a good night NobodyCam devananda | 18:53 |
| lucasagomes | and enjoy the weekend :) | 18:53 |
| *** f13o has quit IRC | 18:53 | |
| NobodyCam | you too lucasagomes .. .and it looks pretty good | 18:54 |
| lucasagomes | thanks... I will work on a complex type to validate patches as well | 18:54 |
| NobodyCam | :) | 18:54 |
| lucasagomes | so I will try to remove all that complexity on the path method | 18:54 |
| max_lobur | have a good weekend Everyone | 18:57 |
| *** lucasagomes has quit IRC | 18:57 | |
| NobodyCam | you too max_lobur | 18:57 |
| *** urulama has quit IRC | 18:58 | |
| * max_lobur is away: AFK | 18:58 | |
| *** urulama has joined #openstack-ironic | 19:01 | |
| * NobodyCam goes and looks some food | 19:05 | |
| *** romcheg has joined #openstack-ironic | 19:09 | |
| *** urulama has quit IRC | 19:20 | |
| *** urulama has joined #openstack-ironic | 19:20 | |
| *** tatyana has quit IRC | 19:23 | |
| *** ecozzi has joined #openstack-ironic | 19:30 | |
| *** urulama has quit IRC | 19:30 | |
| *** urulama has joined #openstack-ironic | 19:31 | |
| *** jianingy_afk has quit IRC | 19:35 | |
| *** jimjiang has quit IRC | 19:35 | |
| *** blamar has quit IRC | 19:35 | |
| *** ctracey has quit IRC | 19:35 | |
| *** urulama has quit IRC | 19:35 | |
| sandeepr | date | 19:36 |
| sandeepr | ping NobodyCam | 19:36 |
| *** ctracey has joined #openstack-ironic | 19:37 | |
| *** blamar has joined #openstack-ironic | 19:37 | |
| *** urulama has joined #openstack-ironic | 19:40 | |
| *** sandeepr has quit IRC | 19:42 | |
| *** jbjohnso has quit IRC | 19:43 | |
| *** jbjohnso has joined #openstack-ironic | 19:43 | |
| *** urulama has quit IRC | 19:51 | |
| NobodyCam | https://www.transifex.com/languages/pong | 19:51 |
| NobodyCam | gah | 19:51 |
| NobodyCam | pong | 19:51 |
| *** yanhcdl has joined #openstack-ironic | 20:03 | |
| *** hemna has left #openstack-ironic | 20:06 | |
| *** hemna has joined #openstack-ironic | 20:06 | |
| *** yanhcdl has quit IRC | 20:27 | |
| *** jdob has quit IRC | 20:27 | |
| *** epim has joined #openstack-ironic | 20:29 | |
| *** jbjohnso has quit IRC | 20:38 | |
| *** yanhcdl has joined #openstack-ironic | 20:39 | |
| openstackgerrit | ramesh proposed a change to openstack/python-ironicclient: Change id->uuid in ironic cli help messages https://review.openstack.org/55941 | 20:39 |
| *** jdob has joined #openstack-ironic | 20:42 | |
| *** epim has quit IRC | 20:58 | |
| *** epim has joined #openstack-ironic | 21:02 | |
| *** tatyana has joined #openstack-ironic | 21:23 | |
| *** aratim has joined #openstack-ironic | 21:30 | |
| *** aratim has left #openstack-ironic | 21:32 | |
| *** jbjohnso has joined #openstack-ironic | 21:42 | |
| *** romcheg has quit IRC | 21:43 | |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception https://review.openstack.org/56598 | 21:48 |
| openstackgerrit | James Slagle proposed a change to openstack/ironic: Add missing ConfigNotFound exception https://review.openstack.org/56598 | 21:55 |
| *** jbjohnso has quit IRC | 21:57 | |
| * NobodyCam wanders away for a few | 21:59 | |
| *** jdob has quit IRC | 22:06 | |
| *** tatyana has quit IRC | 22:11 | |
| NobodyCam | sooo nice out side today | 22:15 |
| *** drumkilla has quit IRC | 22:19 | |
| *** russellb has joined #openstack-ironic | 22:20 | |
| NobodyCam | devananda: # FIXME(deva): determine how to get IP address and bootfile name from Ironic for the correct ironic-conductor is not easy done | 22:45 |
| NobodyCam | :-p | 22:45 |
| devananda | NobodyCam: right .... | 22:46 |
| NobodyCam | think that one going on the agenda for monday | 22:47 |
| NobodyCam | :) | 22:47 |
| devananda | NobodyCam: i think that one also requires us to associate conductor<->node | 22:48 |
| devananda | when ever an instance is on it | 22:48 |
| devananda | though that's fundamentally an implementation detail of the pxe driver | 22:49 |
| devananda | not a requirement of all provisioning tools | 22:49 |
| devananda | if we set the boot device to local, we wouldn't need to preserve that association beyond the image copy either | 22:49 |
| NobodyCam | actually that whole dhcp scrtion should be ripd out of the nova driver | 22:50 |
| devananda | NobodyCam: I wonder if we can move taht logic out of the nova driver, and into the ironic pxe driver? | 22:50 |
| devananda | ya | 22:50 |
| NobodyCam | lol | 22:50 |
| NobodyCam | :) | 22:50 |
| NobodyCam | I think that is the best thing. move it all to pxe. | 22:51 |
| NobodyCam | GheRivero: you happen to still be up ... | 22:51 |
| * NobodyCam hopes not.. but tries anyhow | 22:52 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove sqlalchemy-migrate 0.7.3 patching https://review.openstack.org/56705 | 23:00 |
| * devananda tries to unblock our gate checks | 23:00 | |
| NobodyCam | :) you +a'ing it or shal I | 23:01 |
| NobodyCam | really :P ./ironic/db/sqlalchemy/migration.py:19:1: F401 'dist_version' imported but unused | 23:05 |
| devananda | flake ... | 23:05 |
| devananda | i'll +a once it passes | 23:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove sqlalchemy-migrate 0.7.3 patching https://review.openstack.org/56705 | 23:06 |
| NobodyCam | for a friday it sure feels like a monday | 23:06 |
| devananda | heh | 23:07 |
| NobodyCam | one minute | 23:09 |
| NobodyCam | lol watching zuil | 23:09 |
| NobodyCam | zuul even | 23:09 |
| NobodyCam | +1 by mr j | 23:10 |
| NobodyCam | :) | 23:17 |
| openstackgerrit | A change was merged to openstack/ironic: Remove sqlalchemy-migrate 0.7.3 patching https://review.openstack.org/56705 | 23:20 |
| openstackgerrit | A change was merged to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/56325 | 23:27 |
| NobodyCam | devananda: looks like https://review.openstack.org/#/c/56519/ in unhappy | 23:31 |
| devananda | needs a rebase, i bet | 23:34 |
| NobodyCam | going to help john bring thing in brb | 23:35 |
| devananda | patch bomb .... | 23:35 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.log from oslo https://review.openstack.org/56520 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.notifier.* from oslo https://review.openstack.org/56521 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.periodic_task from oslo https://review.openstack.org/56522 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.service from oslo https://review.openstack.org/56523 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: update sample config https://review.openstack.org/56524 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.config.generator from oslo https://review.openstack.org/56514 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.context from oslo https://review.openstack.org/56515 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common.db.sqlalchemy changes from olso https://review.openstack.org/56516 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync common utils from Oslo https://review.openstack.org/56517 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.local from oslo https://review.openstack.org/56518 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync common.lockutils from oslo https://review.openstack.org/56519 | 23:36 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement sync_power_state periodic task https://review.openstack.org/56530 | 23:40 |
| openstackgerrit | A change was merged to openstack/ironic: Register API options under the 'api' group https://review.openstack.org/54277 | 23:45 |
| NobodyCam | :-p | 23:51 |
| NobodyCam | devananda: did you do ant looking in to lucas's question on 56514 | 23:51 |
| openstackgerrit | Michael Still proposed a change to openstack/ironic: Update openstack/common/lockutils https://review.openstack.org/56719 | 23:51 |
| NobodyCam | s/ant/any/ | 23:52 |
| devananda | NobodyCam: besides pinging dhellman? no | 23:53 |
| devananda | mikal: how are you generating the log-of-changes for ^ ? | 23:55 |
| devananda | i'm sure there's a fancy git comand for it | 23:55 |
| devananda | and i was just lazy and didn't try to figure it out | 23:56 |
| devananda | bbiab | 23:57 |
| *** michchap has quit IRC | 23:57 | |
| *** michchap has joined #openstack-ironic | 23:59 | |
| mikal | devananda: I am generating it by hand | 23:59 |
| mikal | devananda: I would love to discover a more automated way, but I haven't yet | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!