| *** bfournie has joined #openstack-ironic | 00:00 | |
| *** bfournie has quit IRC | 00:01 | |
| *** bfournie has joined #openstack-ironic | 00:01 | |
| *** rama_y has quit IRC | 00:03 | |
| *** ijw has quit IRC | 00:07 | |
| *** ijw has joined #openstack-ironic | 00:07 | |
| *** ijw has quit IRC | 00:11 | |
| *** rcernin has quit IRC | 00:12 | |
| *** masber has joined #openstack-ironic | 00:37 | |
| *** UForgotten has quit IRC | 00:43 | |
| *** UForgotten has joined #openstack-ironic | 00:43 | |
| *** yuanying_ has joined #openstack-ironic | 00:58 | |
| *** yuanying has quit IRC | 01:02 | |
| *** yuanying has joined #openstack-ironic | 01:02 | |
| *** yuanying_ has quit IRC | 01:02 | |
| *** phuongnh has joined #openstack-ironic | 01:08 | |
| *** ijw has joined #openstack-ironic | 01:14 | |
| *** ijw has quit IRC | 01:19 | |
| *** masber has quit IRC | 01:20 | |
| *** zshi_laptop has joined #openstack-ironic | 01:30 | |
| *** gcb has quit IRC | 01:33 | |
| *** gcb has joined #openstack-ironic | 01:34 | |
| *** yuanying has quit IRC | 01:50 | |
| *** masber has joined #openstack-ironic | 01:51 | |
| *** yuanying has joined #openstack-ironic | 02:01 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic stable/ocata: Fix broken links to ocata installation guide https://review.openstack.org/491966 | 02:02 |
|---|---|---|
| *** yangyape_ has joined #openstack-ironic | 02:07 | |
| *** yangyapeng has quit IRC | 02:10 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 02:24 | |
| *** libin_ has joined #openstack-ironic | 02:28 | |
| openstackgerrit | D G Lee proposed openstack/ironic master: Adds more exception handling for ironic-conductor heartbeat https://review.openstack.org/475210 | 02:33 |
| *** litao__ has joined #openstack-ironic | 02:33 | |
| *** baoli has joined #openstack-ironic | 02:34 | |
| *** baoli has quit IRC | 02:38 | |
| *** zhangfei has joined #openstack-ironic | 02:46 | |
| *** zhangfei has quit IRC | 02:47 | |
| *** zhangfei has joined #openstack-ironic | 02:48 | |
| *** Nisha_Agarwal has quit IRC | 02:56 | |
| *** hw_wutianwei has joined #openstack-ironic | 02:57 | |
| *** rama_y has joined #openstack-ironic | 03:11 | |
| *** rama_y has quit IRC | 03:15 | |
| *** zshi_laptop has quit IRC | 03:17 | |
| *** zshi_laptop has joined #openstack-ironic | 03:17 | |
| *** links has joined #openstack-ironic | 03:18 | |
| *** aparnav has joined #openstack-ironic | 03:24 | |
| *** vmud213 has joined #openstack-ironic | 03:43 | |
| *** rama_y has joined #openstack-ironic | 04:24 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:36 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Unit tests for set target raid configuration https://review.openstack.org/496857 | 04:36 |
| *** MattMan_1 has quit IRC | 04:46 | |
| *** MattMan_1 has joined #openstack-ironic | 04:46 | |
| *** sasha2 has quit IRC | 04:52 | |
| *** udesale has joined #openstack-ironic | 04:57 | |
| *** rbudden has quit IRC | 05:05 | |
| *** rcernin has joined #openstack-ironic | 05:11 | |
| *** rcernin has quit IRC | 05:27 | |
| *** amarao has joined #openstack-ironic | 05:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements https://review.openstack.org/497018 | 05:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/497032 | 05:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/497033 | 05:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/497034 | 05:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/494817 | 05:50 |
| *** rcernin has joined #openstack-ironic | 06:12 | |
| *** ChubYann has quit IRC | 06:16 | |
| *** pcaruana has joined #openstack-ironic | 06:21 | |
| *** rama_y has quit IRC | 06:24 | |
| *** makowals has joined #openstack-ironic | 06:27 | |
| *** rbartal has joined #openstack-ironic | 06:32 | |
| *** jtomasek has joined #openstack-ironic | 06:33 | |
| *** mjura has joined #openstack-ironic | 06:34 | |
| *** amarao1 has joined #openstack-ironic | 06:34 | |
| *** amarao1 has quit IRC | 06:53 | |
| *** parlos has joined #openstack-ironic | 07:15 | |
| *** tesseract has joined #openstack-ironic | 07:20 | |
| *** Tengu has left #openstack-ironic | 07:40 | |
| *** racedo has joined #openstack-ironic | 07:44 | |
| *** dtantsur|afk is now known as dtantsur | 07:46 | |
| dtantsur | morning ironic | 07:47 |
| dtantsur | we have released, finally \o/ | 07:47 |
| *** tesseract-RH has joined #openstack-ironic | 07:49 | |
| *** tesseract-RH has quit IRC | 07:50 | |
| *** tesseract-RH has joined #openstack-ironic | 07:50 | |
| *** tesseract has quit IRC | 07:52 | |
| *** mgoddard has joined #openstack-ironic | 07:54 | |
| *** karimb has joined #openstack-ironic | 07:56 | |
| Nisha_Agarwal | dtantsur, yay | 08:00 |
| Nisha_Agarwal | dtantsur, morning | 08:00 |
| Nisha_Agarwal | :) | 08:00 |
| dtantsur | morning (afternoon?)! | 08:01 |
| *** tesseract-RH has quit IRC | 08:01 | |
| Nisha_Agarwal | morning for u :) afternon for me | 08:01 |
| Nisha_Agarwal | :) | 08:01 |
| Nisha_Agarwal | afternoon* | 08:01 |
| openstackgerrit | Merged openstack/ironic-ui stable/pike: Imported Translations from Zanata https://review.openstack.org/496200 | 08:01 |
| Nisha_Agarwal | hey dtantsur ...i was just reading yesterday's IRC converstaion | 08:02 |
| Nisha_Agarwal | i read we are still ok tomerge docs patches | 08:02 |
| *** tesseract-RH has joined #openstack-ironic | 08:03 | |
| Nisha_Agarwal | dtantsur, is my understanding correct? | 08:03 |
| dtantsur | Nisha_Agarwal: docs patches are ok to merge and to backport in the most cases | 08:03 |
| *** vmud213 has quit IRC | 08:03 | |
| *** zshi_laptop has quit IRC | 08:03 | |
| Nisha_Agarwal | oh you mean we can backport them to pike? | 08:03 |
| dtantsur | Nisha_Agarwal: yes, if they are valid for pike | 08:04 |
| dtantsur | but they will get in the next point release | 08:04 |
| Nisha_Agarwal | dtantsur, so just to know whats your opinion on this one if stendulker updates it by today https://review.openstack.org/496480 | 08:04 |
| patchbot | patch 496480 - ironic - Update documentation for ``ilo`` hardware type | 08:04 |
| dtantsur | Nisha_Agarwal: this is good to go, when updated, yes | 08:05 |
| Nisha_Agarwal | dtantsur, ok thanks. | 08:05 |
| Nisha_Agarwal | :) | 08:05 |
| dtantsur | np | 08:07 |
| Nisha_Agarwal | dtantsur, on resource class thing, i think we need to have a way from nova/ironic to schedule based on quantity things. I am not sure if that makes sense now also after all the latest changes in nova and ironic for resource class | 08:07 |
| dtantsur | Nisha_Agarwal: could you please collect your requirements and post it to the ML with [ironic] [nova] tags? This has been raised several times, but we need to be clear on why we need them, and let's make it official by posting it to the ML. | 08:08 |
| Nisha_Agarwal | dtantsur, and i am not sure if ironic thinks that way. but we need to have scheduling based on resource classes (quantity too i guess) as we used to have under capabilities | 08:08 |
| Nisha_Agarwal | dtantsur, ok. | 08:09 |
| dtantsur | a resource class is more like a mandatory capability. it's not quantitative. | 08:09 |
| Nisha_Agarwal | dtantsur, yes but resource class structure in itself has quantitative capability | 08:09 |
| Nisha_Agarwal | like DISK_GB | 08:09 |
| dtantsur | ah, you mean standard resource classes? yes, they are quantitative. but nova plans on removing support for them.. | 08:10 |
| Nisha_Agarwal | dtantsur, is it? | 08:11 |
| *** libin_ has quit IRC | 08:11 | |
| Nisha_Agarwal | i actually means standard + custom resource classes both. I just gave an example of DISK_GB. | 08:12 |
| dtantsur | Nisha_Agarwal: nova will only have the one custom resource class for an ironic node in Queens. tracking of DISK_GB is planned to go away. | 08:12 |
| Nisha_Agarwal | dtantsur, any link for this where i can read it ....but nova plans on removing support for them | 08:13 |
| dtantsur | lemme get a link | 08:13 |
| Nisha_Agarwal | oh u mean removing support of DISK_GB for ironic node | 08:13 |
| Nisha_Agarwal | Nova will still has it, am i correct? | 08:14 |
| dtantsur | I thought it's in https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/custom-resource-classes-in-flavors.html but it's not explicit there | 08:14 |
| dtantsur | Nisha_Agarwal: aha, here https://docs.openstack.org/releasenotes/nova/pike.html#deprecation-notes | 08:14 |
| Nisha_Agarwal | dtantsur, this one i am aware. | 08:17 |
| Nisha_Agarwal | what i meant is that do you(ironic) think we need to have scheduling based on quantitative capabilities/properties still? like if we want to schedule an ironic node based on any quantitative capability like say persistent_memory size. Do we want to do that through custom resource classes structure similar to what nova has for resoure classes | 08:20 |
| *** zshi_laptop has joined #openstack-ironic | 08:21 | |
| dtantsur | Nisha_Agarwal: note that ironic does not have scheduling currently, it's fully within Nova | 08:21 |
| Nisha_Agarwal | because i remember jroll even mentioning that capabilities also will eventually go off and nova will just support resource classes and traits | 08:21 |
| dtantsur | as the nova is changing, you'll have to use either resource classes or capabilities for your case | 08:21 |
| dtantsur | Nisha_Agarwal: I still have to get up-to-date about the capabilities future. I don't know the latest plans, sorry | 08:21 |
| Nisha_Agarwal | dtantsur, :) | 08:22 |
| *** karimb has quit IRC | 08:29 | |
| *** Nisha_Agarwal has quit IRC | 08:32 | |
| *** lucas-afk is now known as lucasagomes | 08:41 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs https://review.openstack.org/494549 | 08:47 |
| *** karimb has joined #openstack-ironic | 08:53 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic-specs master: Support node abandon https://review.openstack.org/497156 | 09:02 |
| *** etingof has quit IRC | 09:03 | |
| *** sambetts|afk is now known as sambetts | 09:11 | |
| sambetts | Morning all | 09:12 |
| dtantsur | morning sambetts | 09:13 |
| openstackgerrit | Aparna proposed openstack/ironic master: Support SUM based firmware update as clean step for iLO drivers https://review.openstack.org/422572 | 09:13 |
| sambetts | hey dtantsur | 09:13 |
| *** etingof has joined #openstack-ironic | 09:17 | |
| *** tesseract-RH has quit IRC | 09:18 | |
| *** tesseract has joined #openstack-ironic | 09:23 | |
| *** etingof has quit IRC | 09:23 | |
| *** milan has joined #openstack-ironic | 09:28 | |
| milan | #pixiesay -mlikeaboss pixie's back! | 09:29 |
| PixieBoots | ʕ▀̿ᴥ▀̿ʔ: pixie's back! | 09:29 |
| milan | morning ironic! :) | 09:29 |
| *** etingof has joined #openstack-ironic | 09:29 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs https://review.openstack.org/494549 | 09:32 |
| *** zshi_laptop has quit IRC | 09:38 | |
| dtantsur | hi milan :) | 09:42 |
| *** athomas has joined #openstack-ironic | 09:42 | |
| milan | hey hey dtantsur! :) | 09:43 |
| *** zshi_laptop has joined #openstack-ironic | 09:51 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [admin docs] ironic -> openstack baremetal CLI https://review.openstack.org/494335 | 09:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: remove REST API examples from RAID doc https://review.openstack.org/496840 | 09:53 |
| openstackgerrit | Merged openstack/ironic master: [doc] change absolute to relative URL https://review.openstack.org/496804 | 09:54 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs https://review.openstack.org/494549 | 09:54 |
| *** makowals has quit IRC | 10:03 | |
| *** zhangfei has quit IRC | 10:07 | |
| *** wong_ has quit IRC | 10:08 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs https://review.openstack.org/494549 | 10:14 |
| *** dtantsur is now known as dtantsur|brb | 10:14 | |
| milan | dtantsur|brb: did we make the release? | 10:21 |
| * milan too lazy to open mailbox O:-) | 10:22 | |
| *** karimb has quit IRC | 10:23 | |
| *** aparnav has quit IRC | 10:29 | |
| *** karimb has joined #openstack-ironic | 10:32 | |
| *** jjung has joined #openstack-ironic | 10:35 | |
| milan | * milan 's mailbox is too big to open :D | 10:37 |
| *** jkilpatr has quit IRC | 10:39 | |
| *** zhangfei has joined #openstack-ironic | 10:50 | |
| *** zhangfei has quit IRC | 10:53 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: Change perms for the VM logs https://review.openstack.org/494549 | 10:58 |
| *** phuongnh has quit IRC | 10:59 | |
| *** namnh has quit IRC | 11:01 | |
| *** udesale has quit IRC | 11:04 | |
| pester | hm | 11:08 |
| pester | ironic node-set-provision-state --config-drive /tmp/configdrive d7f0d663-aaf4-47d5-90b3-7e80f0417c53 active | 11:08 |
| pester | must be encoded string without NULL bytes, not str | 11:08 |
| *** jkilpatr has joined #openstack-ironic | 11:13 | |
| *** zshi_laptop has quit IRC | 11:33 | |
| *** amarao1 has joined #openstack-ironic | 11:35 | |
| *** mat128 has joined #openstack-ironic | 11:39 | |
| sambetts | pester: is /tmp/configdrive a file or a directory?? | 11:40 |
| pester | file | 11:40 |
| pester | now I mounted it and pass directory path to ironic - it worked | 11:41 |
| pester | now I hit this - https://bugs.launchpad.net/ironic-lib/+bug/1670239 | 11:41 |
| openstack | Launchpad bug 1670239 in ironic-lib "create_config_drive_partition failed because ironic-lib call cmd "partprobe /dev/sda" failed" [Medium,In progress] - Assigned to Hao Li (lihaosz) | 11:41 |
| *** srobert has joined #openstack-ironic | 11:42 | |
| *** makowals has joined #openstack-ironic | 11:43 | |
| *** srobert has quit IRC | 11:44 | |
| pester | easy way to fix it ? | 11:54 |
| *** rbudden has joined #openstack-ironic | 11:54 | |
| *** litao__ has quit IRC | 11:55 | |
| *** bfournie has quit IRC | 11:56 | |
| *** bfournie has joined #openstack-ironic | 11:56 | |
| *** bfournie has quit IRC | 12:00 | |
| *** trown|outtypewww is now known as trown | 12:01 | |
| sambetts | The patch to fix that bug is here, https://review.openstack.org/#/c/443604/ you could patch and build a custom IPA image to see if that fixes the issue you are seeing | 12:04 |
| patchbot | patch 443604 - ironic-lib - Retry when execute partprobe cmd failed | 12:04 |
| sambetts | if you do test out the fix please review the patch | 12:04 |
| pester | I`m trying images from this - http://tarballs.openstack.org/ironic-python-agent/coreos/files/coreos_production_pxe.vmlinuz | 12:07 |
| pester | I guess it is the 'freshest' and should contain patch | 12:08 |
| pester | I`m ironic newton in standalone mode | 12:08 |
| pester | *using | 12:08 |
| *** fellypefca has joined #openstack-ironic | 12:11 | |
| jroll | pester: first thing I'd try is using the newton build - the files with *stable-newton* in the name here: http://tarballs.openstack.org/ironic-python-agent/coreos/files/ | 12:11 |
| jroll | (oh, I see what sam said, that might not help) | 12:14 |
| * jroll chugs coffee | 12:14 | |
| *** dyasny has quit IRC | 12:14 | |
| *** dyasny has joined #openstack-ironic | 12:15 | |
| *** dprince has joined #openstack-ironic | 12:16 | |
| *** cdearborn has joined #openstack-ironic | 12:18 | |
| sambetts | o/ jroll | 12:19 |
| jroll | \o | 12:19 |
| sambetts | pester: the proposed fix for that particular bug hasn't been merged yet, so you;ll have to build your own ramdisk if you want test it out and confirm that fix works | 12:20 |
| *** ricardoas has quit IRC | 12:23 | |
| pester | is there instructions how to mount coreos_production_pxe_image-oem.cpio.gz ? want to apply patch to it | 12:25 |
| sambetts | pester: you don't want to mount and apply the patch to the image like that, you'll need to clone git.openstack.org/ironic-python-agent and then checkout the patch from gerrit, and then use the tools provided in the imagebuild directory to build a new image | 12:27 |
| *** bfournie has joined #openstack-ironic | 12:29 | |
| *** jlabarre has joined #openstack-ironic | 12:32 | |
| *** karimb has quit IRC | 12:33 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update documentation for ``ilo`` hardware type https://review.openstack.org/496480 | 12:42 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update documentation for ``ilo`` hardware type https://review.openstack.org/496480 | 12:45 |
| *** jcoufal has joined #openstack-ironic | 12:46 | |
| fellypefca | Good morning, Ironic :) | 12:52 |
| milan | morning fellypefca! :) | 12:53 |
| fellypefca | hey milan o/ | 12:53 |
| *** ricardoas has joined #openstack-ironic | 12:54 | |
| *** mat128 has quit IRC | 12:55 | |
| *** udesale has joined #openstack-ironic | 12:55 | |
| *** rloo has joined #openstack-ironic | 12:56 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Updating the minimum version of python-oneviewclient https://review.openstack.org/493983 | 12:59 |
| *** Goneri has joined #openstack-ironic | 13:00 | |
| *** sasha2 has joined #openstack-ironic | 13:00 | |
| *** dtantsur|brb is now known as dtantsur | 13:00 | |
| dtantsur | milan: yep, we did | 13:01 |
| milan | #pixiesay -mrnr Pike is Ironic! | 13:02 |
| PixieBoots | \m/ʕ>ᴥ<ʔ\m/: Pike is Ironic! | 13:02 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add documentation describing each Ironic state https://review.openstack.org/496719 | 13:02 |
| *** mat128 has joined #openstack-ironic | 13:04 | |
| *** mat128 has quit IRC | 13:04 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update documentation for ``ilo`` hardware type https://review.openstack.org/496480 | 13:11 |
| *** zshi_laptop has joined #openstack-ironic | 13:11 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Switch API ref to use versionadded syntax throughout https://review.openstack.org/490008 | 13:13 |
| *** jjung has quit IRC | 13:17 | |
| *** mjturek has joined #openstack-ironic | 13:21 | |
| fellypefca | TheJulia dtantsur Can you take a look at this patch? https://review.openstack.org/#/c/493983/ I also need to backport this patch for stable/pike since this one was also backported https://review.openstack.org/#/c/476968/ | 13:22 |
| patchbot | patch 493983 - ironic - Updating the minimum version of python-oneviewclient | 13:22 |
| patchbot | patch 476968 - ironic - [devstack] use resource classes by default (MERGED) | 13:22 |
| fellypefca | TheJulia dtantsur We fix our CI but we need to update the driver-requirements with the new version of our client. | 13:22 |
| dtantsur | fellypefca: this cannot be done on stable branch, as it's a feature release (and note that we don't bump requirements for bug fixes, unless they're end-of-the-world kind of an issue) | 13:23 |
| fellypefca | dtantsur Oh, ok. So, no backport for this one :) | 13:25 |
| dtantsur | fellypefca: yep. we still expect operators to follow your client's release notes, and upgrade if needed | 13:25 |
| *** udesale has quit IRC | 13:26 | |
| *** udesale has joined #openstack-ironic | 13:26 | |
| *** udesale has quit IRC | 13:26 | |
| *** xavierr has joined #openstack-ironic | 13:27 | |
| fellypefca | dtantsur Nice. Thank you :) But for master it's everything ok? | 13:27 |
| dtantsur | fellypefca: I'd prefer we don't bump versions for bug fixes only, but it's not a blocking issue | 13:28 |
| fellypefca | dtantsur hmm ok | 13:28 |
| dtantsur | fellypefca: anyway, aren't we moving away from oneviewclient is the very near future? | 13:29 |
| fellypefca | dtantsur yes | 13:30 |
| rloo | milan: if you are around have have a few (or many, depends on ...) do you want to discuss https://review.openstack.org/#/c/490008/ | 13:31 |
| patchbot | patch 490008 - ironic - Switch API ref to use versionadded syntax throughout | 13:31 |
| * dtantsur is ready to watch the promised fight | 13:32 | |
| rloo | hi and good morning fellypefca, dtantsur, sambetts, milan (although i'm not sure about how i feel about milan this morning :)) | 13:32 |
| dtantsur | Game of Docs, Episode ... | 13:32 |
| dtantsur | morning rloo :) | 13:32 |
| fellypefca | Morning rloo | 13:32 |
| * rloo wonders what fight dtantsur is referring to... :) | 13:32 | |
| fellypefca | brance yourself milan hahah | 13:32 |
| dtantsur | rloo: you promised a fight with milan on that patch, no? :D | 13:33 |
| milan | morning rloo! :D | 13:33 |
| sambetts | o/ milan rloo | 13:33 |
| rloo | dtantsur: i don't recall 'fight' being used? or did i? | 13:33 |
| rloo | milan: is this what you like? http://docs-draft.openstack.org/08/490008/7/check/gate-ironic-api-ref/4bd711c//api-ref/build/html/#drivers-drivers | 13:33 |
| dtantsur | aha, it was "duel". duel is nice too! /me keeps popcorn ready | 13:33 |
| rloo | by the way, others can chime in. we're trying to figure out something about layout of info in the docs; so it has to make sense to our readers | 13:34 |
| milan | dtantsur: it supposed to be a duel which I ran from ;) | 13:34 |
| rloo | dtantsur: ah, duel, yeah :) | 13:34 |
| * milan looks | 13:34 | |
| sambetts | rloo: my personal preference is for it not to be indented and for it to just be at the top of that section of logical disk properties | 13:34 |
| rloo | (and again, if only there were guidelines...) | 13:35 |
| sambetts | I think the grey bars are ugly | 13:35 |
| rloo | sambetts: i agree about indentation; it is ugly and doesn't make sense | 13:35 |
| rloo | sambetts: i would like it to be in both places (ie, for it to be wherever it is applicable) | 13:35 |
| sambetts | rloo: the sphinx docs for the versionadded syntax "When this applies to an entire module, it should be placed at the top of the module section before any prose." so thats the guide I was going with for that section | 13:36 |
| rloo | sambetts: i like having the version info here, at a glance you can see which version applies for that request. | 13:36 |
| sambetts | oviously this isn't a module its an API endpoint but you get the idea | 13:36 |
| sambetts | rloo: same | 13:36 |
| rloo | sambetts: yeah, so it applies to that request, so you put it at the top of that request. | 13:36 |
| rloo | sambetts: adding it to the top of the 'drivers' is fine with me too, but it is really applicable here, at the request level. | 13:36 |
| * rloo wonders why milan is so quiet, did he really run away? | 13:37 | |
| sambetts | +1 and thats consistent with the rest of the API docs too | 13:37 |
| *** yufei has joined #openstack-ironic | 13:37 | |
| sambetts | heh we've scared him off | 13:37 |
| *** yufei has left #openstack-ironic | 13:37 | |
| sauloaislan | Morning! | 13:37 |
| rloo | sambetts: so do we agree, no indent, put the version info below the request. and also at top of the 'module' (driver)? | 13:38 |
| fellypefca | Good morning sauloaislan | 13:38 |
| * milan contemplating :D | 13:38 | |
| rloo | 'sok milan, take your time... | 13:38 |
| milan | rloo, sambetts, cool with me, at least it's not green anymore | 13:38 |
| sambetts | rloo: personally I don't think we need it in the top of the drivers bit if its already in the request part itself | 13:38 |
| sambetts | milan: main contention is the logical volumes part I think | 13:38 |
| rloo | sambetts: ok, if we don't want to put both, then we shouldn't put the other version at the very top. maybe back to the bottom, before the requests. | 13:39 |
| sambetts | rloo: huh? | 13:40 |
| milan | sambetts yeah rloo doesn't like the indentation | 13:40 |
| rloo | sambetts: https://review.openstack.org/#/c/490008/7/api-ref/source/baremetal-api-v1-drivers.inc | 13:40 |
| patchbot | patch 490008 - ironic - Switch API ref to use versionadded syntax throughout | 13:40 |
| rloo | sambetts: might be easier to refer to that | 13:40 |
| rloo | sambetts: i mean, L7 | 13:40 |
| rloo | sambetts: if we don't put both versions there, i think it would be better to move L7 to where it was before | 13:41 |
| * milan also wonders whether .. versionadded:: is expressed also in some TOC | 13:41 | |
| rloo | sambetts: before L18 | 13:41 |
| rloo | sambetts: oh. except for your description of where they said to put it. sigh. | 13:41 |
| milan | btw rloo why is the indentation inappropriate @204? | 13:42 |
| rloo | sambetts: ok fine. i | 13:42 |
| rloo | milan: cuz 1. it looks ugly with the gray vertical bar. | 13:42 |
| rloo | milan: 2. it makes no sense to be indented. nothing else is. | 13:43 |
| rloo | milan: i think on a line of its own, if you take away the colour/symbol, it renders like a single-line paragraph. | 13:43 |
| rloo | milan: which means it is applicable to that entire section, just like any other paragraph there. | 13:44 |
| rloo | milan: why do you like the indentation? | 13:44 |
| rloo | milan: let me ask you this other question, why indent it at L204, and not indent it at L7? what's the diff? | 13:45 |
| milan | rloo I found it more bound to the paragraph header if it was under it so referring visually to the header: Show driver logical disk properties: New in version 1.12. | 13:45 |
| milan | rloo: I actually wanted to indent on @L7 too | 13:45 |
| *** links has quit IRC | 13:46 | |
| rloo | milan: ha ha. at least you are consistent then! | 13:46 |
| milan | make it a section the same way | 13:46 |
| rloo | milan: if that 'thing' didn't have that ugly + symbol and wasn't in some greenish colour but looked like normal text, would you still want it indented? | 13:46 |
| rloo | personally, i would have rendered it w/o that symbol, in black italicized. | 13:47 |
| milan | rloo: I would otherwise it looks like a broken header | 13:47 |
| rloo | but clearly, i seem to be the only one that does NOT want additional colour besides black... | 13:47 |
| milan | rloo: don't get me wrong, I also prefer less | 13:48 |
| rloo | milan: but the other text below it doesn't look like a broken header. | 13:48 |
| rloo | milan: if it was more verbose, 'This is a new something that is available starting with version 1.32.' would you still want it indented? | 13:48 |
| milan | rloo nope, but the version tag looks like an attribute of the rightful header, at least to me | 13:48 |
| rloo | milan: yeah, so it is cuz the tag is ugly. | 13:49 |
| milan | rloo: that way it would duplicate the header, no? | 13:49 |
| rloo | milan: you lost me there | 13:49 |
| rloo | milan: what would duplicate the header. oh, 'header' == the 'New in version..'? | 13:49 |
| milan | so the header is the digest of the paragraph, so is the version tag text | 13:50 |
| milan | rloo ++ exactly | 13:50 |
| rloo | milan: not sure what 'digest' means | 13:50 |
| milan | rloo, wrong word, essence? | 13:50 |
| rloo | milan: my example above wrt being more verbose, was referring to the version tag text. | 13:50 |
| rloo | milan: i'm lost. what is the 'header' again? | 13:51 |
| milan | rloo I get it, so both the tag and the header have the same function of communicating the essence of the paragraph | 13:51 |
| rloo | milan: do you mean, that when you see 'GET /v1drivers/...' followed by 'Show driver logical disk properties' followed by blank line followed by 'New in version..', that you consider all that to be the 'header'? | 13:52 |
| rloo | milan: i disagree with you there, that all the above have the same function... essence, but i think i can understand why you see it that way. | 13:53 |
| milan | rloo, rather: Show driver logical disk properties New in version 1.12: <Show driver logical disk properties> Show driver logical disk properties is our new cool feature | 13:53 |
| rloo | milan: ok. i think for you, it might be better if the 'New in version 1.12' was below that 'Show the required...' | 13:53 |
| milan | rloo yeah, maybe | 13:54 |
| milan | I'm not -1 strongly on that, just IMO those the tag when used without indentation breaks the flow, if used with the optional <brief> parameter it duplicates the header | 13:55 |
| rloo | milan: sambetts is following the guidelines. much as i can see your rationale, i don't see how we can follow the guidelines and satisfy your needs in a reasonable way. | 13:55 |
| rloo | milan: i am also fine NOT following the guidelines if we can agree/be consistent. | 13:55 |
| rloo | milan: having said that, i feel like to be consistent with other openstack projects, we might want to just follow the guidelines. unless we really disagree with those guidelines. | 13:56 |
| rloo | milan: so i think i'll leave it for you to let us know what you prefer. | 13:56 |
| rloo | milan: i totally disagree with the indentation though. that gray veritcal line is really ugly. i suppose if you can figure out a way to render w/o it i might be able to stomach it. | 13:57 |
| milan | rloo I've just one question: does it make sense to you what I'm saying or do you find it inconsistent/false? | 13:57 |
| rloo | milan: oh, it makes sense. i suspect it is the visual thing that really 'enhances' your feeling/thinking of that tag/text and how/where it applies. | 13:58 |
| rloo | for the life of me, i have no idea why i need to see blue for GET, orange for PATCH, red for DELETE and green for POST. | 13:59 |
| milan | rloo, cool, yeah, I thought of the tag more as an "attribute" of the header of the paragraph | 13:59 |
| rloo | milan: yeah, cuz it isn't in 'normal' text like the rest. | 13:59 |
| milan | rloo, I'm OK with following the guidelines and being consistent with the rest of the projects | 14:00 |
| rloo | milan: openstack seems to like their tags to be in interesting colours/looks... | 14:00 |
| milan | rloo, ++, it's not really functional IMO | 14:00 |
| rloo | milan: thx for agreeing :) | 14:01 |
| rloo | milan: if you would please summarize on the patch... | 14:01 |
| milan | rloo, for instance, if the Show driver logical disk properties was a header of a subsection — just like in the source code of the text — and the version added was indented, it would be more functional and clean IMO | 14:01 |
| milan | rloo, sure | 14:02 |
| * milan comments | 14:02 | |
| rloo | sambetts: does this help? ^^ (and let's hope no one else has a diff opinion...) | 14:03 |
| *** mat128 has joined #openstack-ironic | 14:03 | |
| *** derekh has joined #openstack-ironic | 14:04 | |
| *** mat128 has quit IRC | 14:05 | |
| *** shoaib has joined #openstack-ironic | 14:13 | |
| sauloaislan | dtantsur: Hey, did you had some time to take a look at this backport patch https://review.openstack.org/#/c/489598/ | 14:13 |
| patchbot | patch 489598 - ironic (stable/ocata) - Removes agent mixin from oneview drivers | 14:13 |
| dtantsur | hi, not yet. I'm getting closer to it, just finishing the last releases | 14:14 |
| sauloaislan | dtantsur: Ok thank you :D | 14:15 |
| *** zhangfei has joined #openstack-ironic | 14:16 | |
| milan | sambetts, rloo I'm still writing, testing one more thing with that… sry it takes some time... | 14:18 |
| rloo | milan: ok | 14:18 |
| *** makowals has quit IRC | 14:20 | |
| *** mat128 has joined #openstack-ironic | 14:21 | |
| *** jcoufal has quit IRC | 14:25 | |
| *** jcoufal has joined #openstack-ironic | 14:26 | |
| *** karimb has joined #openstack-ironic | 14:29 | |
| *** jcoufal has quit IRC | 14:33 | |
| *** jcoufal has joined #openstack-ironic | 14:36 | |
| *** mat128 has quit IRC | 14:36 | |
| *** rpioso has joined #openstack-ironic | 14:44 | |
| shoaib | Hi When setting a flavor for Baremetal, Does a resource class need to be set ? And if so, what should it be set to ? Thanks | 14:44 |
| dtantsur | shoaib: in pike - yes. please check https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html#scheduling-based-on-resource-classes | 14:45 |
| pester | hm, I failed to use configdrive cause cloudinit trying to mount partition with -o ro,sync - and fails, correct is mount -o loop,ro,sync | 14:46 |
| *** ralonsoh has joined #openstack-ironic | 14:46 | |
| pester | on freshly builded xenial | 14:46 |
| *** ralonsoh has quit IRC | 14:47 | |
| pester | sambetts: btw my problem is dissapeared as I switched to master coreos images | 14:47 |
| *** patriciadomin has joined #openstack-ironic | 14:49 | |
| sambetts | pester: thats good :) could you leave a comment on the bug report with you findings? | 14:57 |
| pester | I`m not sure where) cloudinit or ironic :D cloudinit guys kinda also asked to file a bug | 14:59 |
| sambetts | pester: now I'm confused are you deploying coreos user images?? | 15:00 |
| sambetts | pester: or do you mean the coreos Ironic ramdisk image | 15:00 |
| pester | I deploy xenial by coreos images | 15:00 |
| sambetts | So for the bug with the ironic python agent regarding the /dev/sda you should update the bug report that you linked to me earlier | 15:01 |
| sambetts | but definatly go and add a bug to cloud-init regarding the bad mounting for xenial if the version of cloud-init is broken there | 15:02 |
| *** rbartal has quit IRC | 15:02 | |
| pester | https://bugs.launchpad.net/cloud-init/+bug/1712851 | 15:07 |
| openstack | Launchpad bug 1712851 in ironic-python-agent "cloudinit can`t mount configdrive partition" [Undecided,New] | 15:07 |
| *** zhangfei has quit IRC | 15:09 | |
| *** jistr|trng is now known as jistr | 15:09 | |
| *** jlabarre has quit IRC | 15:10 | |
| *** zshi_laptop has quit IRC | 15:10 | |
| *** jlabarre has joined #openstack-ironic | 15:12 | |
| *** mjura has quit IRC | 15:13 | |
| anupn | Hi sambetts: wrt https://review.openstack.org/#/c/495385/, shall i download get-pip.py from 3.2 here https://github.com/openstack/ironic-python-agent/blob/da2ee32a8b2c15a232397e4ed5bc01b695cba00c/imagebuild/tinyipa/finalise-tinyipa.sh#L62 | 15:26 |
| patchbot | patch 495385 - ironic-python-agent - Build tinyipa with python3 | 15:26 |
| sambetts | why? its an older version, the version we download is the most up to date and offical version | 15:28 |
| *** karimb has quit IRC | 15:28 | |
| *** rama_y has joined #openstack-ironic | 15:29 | |
| *** yufei has joined #openstack-ironic | 15:34 | |
| *** mgoddard has quit IRC | 15:36 | |
| *** parlos has quit IRC | 15:36 | |
| anupn | sambetts, hmm yes i think so it's the latest version 9.0.1 | 15:37 |
| *** ijw has joined #openstack-ironic | 15:37 | |
| NobodyCam | Good Morning Ironic'er | 15:38 |
| anupn | sambetts, but not using that complained me on this http://paste.openstack.org/show/619335/ | 15:38 |
| anupn | NobodyCam, Good morning :) | 15:38 |
| NobodyCam | Morning anupn :) | 15:39 |
| *** mgoddard has joined #openstack-ironic | 15:39 | |
| anupn | sambetts, i have changed here to python3 instead of python2.7 https://github.com/openstack/ironic-python-agent/blob/da2ee32a8b2c15a232397e4ed5bc01b695cba00c/imagebuild/tinyipa/finalise-tinyipa.sh#L173 | 15:40 |
| sambetts | anupn: looking at https://bootstrap.pypa.io/ the one that we download is from Nov 2016 and the 3.2 one is from Jan 2016 so its older | 15:41 |
| * milan wonders whether .. versionchanged:: tags should be stacked | 15:41 | |
| sambetts | milan: huh? | 15:42 |
| anupn | sambetts, ah ok, so no need to change to 3.2 | 15:42 |
| milan | sambetts, as the api evolves, should the tags be kept or just the last one? | 15:42 |
| sambetts | milan: they all need to be there unless we remove a microversion | 15:43 |
| sambetts | milan: so that people can see what is in the microversion they are requesting and what was added afterwards | 15:43 |
| milan | sambetts: cool, so stacking I guess | 15:43 |
| *** mat128 has joined #openstack-ironic | 15:43 | |
| sambetts | milan: have you looked at the nodes' page? there are like 10 versionadded tags in there :-P | 15:44 |
| sambetts | on the node create call | 15:44 |
| milan | wow :D now add the versionchanged to that stack for e.g some modified attributes semantics | 15:44 |
| *** mat128 has quit IRC | 15:45 | |
| sambetts | there is one in there | 15:45 |
| anupn | sambetts, so will we be compiling here python3, with python3 executable https://github.com/openstack/ironic-python-agent/blob/da2ee32a8b2c15a232397e4ed5bc01b695cba00c/imagebuild/tinyipa/finalise-tinyipa.sh#L171 | 15:45 |
| sambetts | anupn: yes we should be | 15:45 |
| anupn | seems that is complaining like this when i have changed this http://paste.openstack.org/show/619335/ | 15:45 |
| milan | sambetts: omg | 15:45 |
| sambetts | milan: it actuall makes a lot of sense once rendered out | 15:46 |
| sambetts | milan: http://docs-draft.openstack.org/08/490008/7/check/gate-ironic-api-ref/4bd711c//api-ref/build/html/#create-node | 15:46 |
| sambetts | its a really useful history of that API | 15:46 |
| * sambetts --> got to run | 15:47 | |
| milan | sambetts: yeah it's basically an ugly changelog ;) | 15:47 |
| *** sambetts is now known as sambetts|afk | 15:47 | |
| milan | see you sambetts|afk | 15:47 |
| sambetts|afk | milan: its not that ugly ;) | 15:47 |
| anupn | goodbye sambetts|afk | 15:47 |
| sambetts|afk | milan: at least its all in one place | 15:47 |
| sambetts|afk | :-P | 15:47 |
| milan | true :D | 15:47 |
| *** amotoki is now known as amotoki__away | 15:47 | |
| *** mat128 has joined #openstack-ironic | 15:48 | |
| *** srobert has joined #openstack-ironic | 15:49 | |
| milan | rloo in the light of http://docs-draft.openstack.org/08/490008/7/check/gate-ironic-api-ref/4bd711c//api-ref/build/html/#create-node I admit my way sucks more | 15:49 |
| rloo | milan: but i love that! :) | 15:49 |
| milan | it would just add to the ugly mess | 15:50 |
| milan | seems it's all a change log | 15:50 |
| *** karimb has joined #openstack-ironic | 15:50 | |
| rloo | milan: i am actually fine with that; it makes it clear it only refers to that text. | 15:50 |
| milan | I didn't understand | 15:50 |
| *** yufei has quit IRC | 15:50 | |
| rloo | milan: you mean the 'Changed in version 1.6: A node may be referenced ...' | 15:50 |
| rloo | milan: i hate this url stuff it doesn't work. sorry, i think you mean for me to look at the 'detail' for create node | 15:51 |
| *** jjung has joined #openstack-ironic | 15:51 | |
| milan | rloo, rather the whole stack of the additions and updates | 15:51 |
| milan | yeah linking to it is crazy too | 15:52 |
| milan | how does one refer to a particular change? | 15:52 |
| rloo | milan: heh. it is kind of cool. we made a lot of changes to the node :) | 15:52 |
| milan | yeah and they're uglyfied in the api docs :D | 15:52 |
| milan | but it is what it is… never mind me | 15:53 |
| milan | colours are nice after all | 15:53 |
| rloo | milan: yup. i'm going for 'correct info in a way/format that most people will understand' | 15:53 |
| rloo | milan: i don't like diff colours. visually too distracting. but i think i am the exception, no one else seems to care. | 15:54 |
| milan | rloo, if feel the same | 15:54 |
| *** rcernin has quit IRC | 15:54 | |
| rloo | milan: two exceptions! :) | 15:54 |
| milan | it's formatting in formatting in formatting | 15:54 |
| milan | interleaved with, more formatting :D | 15:55 |
| rloo | milan: heh. i figure i have to stop thinking about the docs; get back to 'real' work... | 15:55 |
| * milan shut's up his ranting as he has a nice Circular reference stack trace he has to figure out ;) | 15:56 | |
| *** cdearborn has quit IRC | 15:57 | |
| *** pcaruana has quit IRC | 15:57 | |
| *** rpioso has quit IRC | 15:57 | |
| *** jeblair has joined #openstack-ironic | 15:59 | |
| *** ianychoi has joined #openstack-ironic | 16:00 | |
| jeblair | hi! if someone can take a look at this devstack-gate change to the ironic_key_file and tell me if it is safe to make (or what job needs to run with it to verify that) i would appreciate it: https://review.openstack.org/497470 | 16:00 |
| patchbot | patch 497470 - openstack-infra/devstack-gate - Remove IRONIC_KEY_FILE | 16:00 |
| *** dyasny has quit IRC | 16:00 | |
| *** dyasny has joined #openstack-ironic | 16:01 | |
| rloo | vsaienk0, vdrok, jlvillal, any of you know about jeblair's question/patch? ^^ | 16:02 |
| rloo | vsaienk0: there is a comment in that file from you ^^ | 16:02 |
| *** tesseract has quit IRC | 16:03 | |
| dtantsur | jeblair: the option is used by stable versions of ironic devstack plugin, though I'm not sure it's 100% required there.. | 16:03 |
| *** openstackgerrit has quit IRC | 16:04 | |
| jeblair | dtantsur: that's in the ironic repo? | 16:04 |
| dtantsur | jeblair: yep | 16:04 |
| jeblair | dtantsur: ah i see it in newton and ocata | 16:06 |
| *** links has joined #openstack-ironic | 16:06 | |
| *** karimb has quit IRC | 16:07 | |
| jlvillal | I remember vsaienk0 putting that in the file. I think it was for grenade, but I am not sure | 16:08 |
| *** openstackgerrit has joined #openstack-ironic | 16:08 | |
| openstackgerrit | James E. Blair proposed openstack/ironic stable/newton: DNM: testing devstack-gate change https://review.openstack.org/497474 | 16:08 |
| openstackgerrit | James E. Blair proposed openstack/ironic stable/ocata: DNM: testing devstack-gate change https://review.openstack.org/497475 | 16:09 |
| jeblair | dtantsur, jlvillal: ^ so hopefully one of those patches will catch it | 16:09 |
| dtantsur | cool, thanks! | 16:10 |
| dtantsur | does this var get in the way somehow? | 16:10 |
| jeblair | now that i know where to look, it does look like it's used in newton/ocata, so i expect it to fail. but i think we can keep the variable and point it at the normal ssh key. i'll let this test finish, then assuming they fail, update and recheck. | 16:11 |
| *** yufei has joined #openstack-ironic | 16:11 | |
| jeblair | dtantsur: i'm working on the zuulv3 devstack job, and we're doing all the ssh setup in a different place, so i'm trying to find out if this extra key is necessary -- the key file is more troublesome than the variables, at the moment. | 16:12 |
| dtantsur | ack, I see | 16:12 |
| dtantsur | we can try working around it somehow, if it's really required | 16:12 |
| *** etingof has quit IRC | 16:13 | |
| jeblair | thanks, i'll check back in after these two attempts; we should have more info then | 16:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Update Nova configuration documentation https://review.openstack.org/497478 | 16:13 |
| dtantsur | dansmith: ^^^ | 16:14 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:18 | |
| *** yufei has quit IRC | 16:26 | |
| *** dyasny_ has joined #openstack-ironic | 16:32 | |
| *** racedo has quit IRC | 16:32 | |
| *** dyasny has quit IRC | 16:34 | |
| *** links has quit IRC | 16:38 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Unit tests for set target raid configuration https://review.openstack.org/496857 | 16:42 |
| *** yufei has joined #openstack-ironic | 16:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Minor improvements to the resource classes documentation https://review.openstack.org/496309 | 16:45 |
| *** Nisha_Agarwal has quit IRC | 16:46 | |
| *** dtantsur is now known as dtantsur|afk | 16:47 | |
| dtantsur|afk | o/ | 16:47 |
| *** lucasagomes is now known as lucas-afk | 16:51 | |
| *** amarao1 has quit IRC | 16:53 | |
| *** derekh has quit IRC | 16:55 | |
| *** amarao has quit IRC | 16:57 | |
| *** mgoddard has quit IRC | 17:01 | |
| jlvillal | $ host dig download.cirros-cloud.net | 17:04 |
| jlvillal | ;; connection timed out; no servers could be reached | 17:04 |
| jlvillal | :( | 17:04 |
| jlvillal | $ host download.cirros-cloud.net | 17:05 |
| jlvillal | Host download.cirros-cloud.net not found: 2(SERVFAIL) | 17:05 |
| *** yufei has left #openstack-ironic | 17:05 | |
| jlvillal | I mean that. I'm still :( | 17:05 |
| jlvillal | So I guess Dreamhost DNS is down at the moment. Hopefully it will be back up soon. Likely that will break our gate jobs. | 17:10 |
| rloo | jlvillal: oh. i wonder if it is intermittent. seems like i did a recheck earlier today cuz something wasn't accessible. maybe it was cirros-cloud, i don't recall now. | 17:12 |
| rloo | jlvillal: nope, it was deb.debian.org. | 17:13 |
| *** trown is now known as trown|lunch | 17:15 | |
| jlvillal | https://twitter.com/dhstatus They say a DDoS attack on their DNS | 17:16 |
| *** etingof has joined #openstack-ironic | 17:17 | |
| rloo | jlvillal: ouch | 17:21 |
| *** rcernin has joined #openstack-ironic | 17:27 | |
| *** jtomasek has quit IRC | 17:27 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Update Pike release title to include version range https://review.openstack.org/497528 | 17:33 |
| *** etingof has quit IRC | 17:34 | |
| *** etingof has joined #openstack-ironic | 17:34 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [docs] Update Releasing Ironic Projects https://review.openstack.org/497536 | 18:01 |
| rloo | dtantsur|afk, TheJulia: ^^ | 18:02 |
| milan | o/ | 18:06 |
| *** milan has left #openstack-ironic | 18:07 | |
| *** trown|lunch is now known as trown | 18:10 | |
| *** fellypefca has quit IRC | 18:19 | |
| *** dprince has quit IRC | 18:38 | |
| *** ijw has quit IRC | 18:40 | |
| *** mjturek has quit IRC | 18:44 | |
| *** rpioso has joined #openstack-ironic | 18:46 | |
| TheJulia | Good afternoon folks | 18:59 |
| * TheJulia feels bad, but got up early and began driving first thing | 18:59 | |
| *** masber has quit IRC | 19:00 | |
| rloo | hi TheJulia. | 19:03 |
| rloo | TheJulia: sorry you feel bad :-( | 19:03 |
| *** ChubYann has joined #openstack-ironic | 19:06 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [docs] Update Releasing Ironic Projects https://review.openstack.org/497536 | 19:09 |
| TheJulia | rloo: heh, I never noticed the "release automation" bit either :( | 19:13 |
| rloo | TheJulia: yeah. I was wondering whether it was cuz we were focussed on other stuff at the time, not sure. I saw it today when I went to update it :) | 19:13 |
| TheJulia | The bullet points make perfect sense as todos without the automation, so I suspect those lines were from before the automation | 19:14 |
| TheJulia | without looking at git blame output | 19:14 |
| rloo | TheJulia: so you are the release liaison now. i think it should be written in a way that you'll understand it :) | 19:16 |
| rloo | TheJulia: by the way, there's stuff there about rolling upgrades. i'm working on those. | 19:17 |
| TheJulia | heh, I do actually like the break of the paragraph | 19:17 |
| TheJulia | It stands out much better now :) | 19:17 |
| rloo | TheJulia: found a, err, issue with upgrades code. which could be a bug. or is a bug... | 19:18 |
| TheJulia | Bug filed yet? | 19:19 |
| rloo | TheJulia: nope. cuz i'm gonna fix it so it isn't a bug. i think. it means delaying deletion of some code til Rocky. | 19:19 |
| rloo | TheJulia: alternative is to fix/push another pike version. i don't need that stress :) | 19:20 |
| TheJulia | no need to stress :) | 19:31 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: conductor saves version in db https://review.openstack.org/497593 | 19:36 |
| * TheJulia ponders | 19:40 | |
| TheJulia | rloo: I guess that actually makes sense to land now, kind of. | 19:45 |
| rloo | TheJulia: yup. well, actually, anytime in Queens. cuz... i'm going to change the ironic-dbsync online_data_migrations to make sure that the conductor versions are all set (yet again). | 19:46 |
| rloo | TheJulia: so if you can keep track. ocata - no version in db, pike - version in db & operator invokes ironic-dbsync online_data_migrations so all db versions are populated. BUT after this call, if new conductors - they won't have version set. in Queens, conductor version is set & ironic-dbsyn online_data_migration will set any missing conductor.version | 19:47 |
| rloo | TheJulia: alternative is to backport that patch ^^ to stable/pike and release that. | 19:48 |
| rloo | TheJulia: and assume that whoever upgrades to pike, uses that new released version. | 19:49 |
| TheJulia | I'm not immediately aware of a reason we would need it backported since any future logic can know what the past looked like w/r/t the field | 19:49 |
| TheJulia | yeah, lets avoid that :) | 19:49 |
| rloo | TheJulia: we don't need it backported as long as i put in code to do the online-data-migration in queens. | 19:50 |
| TheJulia | wfm | 19:50 |
| TheJulia | is gerrit super slow, or is it my internet connection? | 19:50 |
| rloo | TheJulia: i think super slow. doc changes are taking more time than usual. | 19:51 |
| * TheJulia should have started git pull with time | 19:52 | |
| TheJulia | it looks like my connection :\ | 19:58 |
| TheJulia | yeah, next git command that talks to a remote, I'm timing | 20:00 |
| rloo | TheJulia: *how* slow it is? it is slow for me, but not terribly slow. i mean more seconds, not minutes (I don't think) | 20:02 |
| TheJulia | well, over 4 minutes so far for `git fetch --all`, but | 20:03 |
| TheJulia | web pages are also loading super slowly | 20:03 |
| * TheJulia shrugs | 20:03 | |
| *** ijw has joined #openstack-ironic | 20:05 | |
| *** mat128 has quit IRC | 20:11 | |
| *** srobert has quit IRC | 20:18 | |
| *** [1]rpioso has joined #openstack-ironic | 20:26 | |
| *** ijw has quit IRC | 20:28 | |
| jeblair | dtantsur|afk, jlvillal, vsaienk0: it looks like https://review.openstack.org/497474 and https://review.openstack.org/497475 passed tests (one job failed because it hit the timeout by a hair's breadth, but it had finished running tests and they succeeded). but i don't see any grenade jobs which is where folks were expecting it to be used. is there any other testing i should do to validate https://review.openstack.org/497470 ? | 20:28 |
| patchbot | patch 497474 - ironic (stable/newton) - DNM: testing devstack-gate change | 20:28 |
| patchbot | patch 497475 - ironic (stable/ocata) - DNM: testing devstack-gate change | 20:28 |
| patchbot | patch 497470 - openstack-infra/devstack-gate - Remove IRONIC_KEY_FILE | 20:28 |
| *** rpioso has quit IRC | 20:29 | |
| *** arnewiebalck_ has joined #openstack-ironic | 20:30 | |
| rloo | jeblair: is that file used by tests in master branch? | 20:30 |
| *** ijw has joined #openstack-ironic | 20:31 | |
| jeblair | rloo: that variable only appears in the newton and ocata branches | 20:31 |
| *** jkilpatr has quit IRC | 20:31 | |
| *** arnewiebalck has quit IRC | 20:31 | |
| *** arnewiebalck_ is now known as arnewiebalck | 20:31 | |
| TheJulia | rloo: I it disappeared with the ssh driver usage in gate testing | 20:32 |
| TheJulia | err, i guess | 20:32 |
| *** [1]rpioso has quit IRC | 20:32 | |
| *** [1]rpioso has joined #openstack-ironic | 20:34 | |
| rloo | jeblair: without looking into it and the min info i have here, it seems fine to remove. as long as it doesn't break master, I'm not too concerned. we could always revert... | 20:34 |
| *** derekh has joined #openstack-ironic | 20:35 | |
| TheJulia | looks like there is a running ocata grenade job, fwiw | 20:35 |
| TheJulia | I think we can definitely remove it, since it was for *_ssh drivers | 20:35 |
| jeblair | TheJulia: oh you're right, i missed the grenade in there | 20:37 |
| jeblair | so that should have tested the newton->ocata path. would you like me to push up a DNM change to master to verify the ocata->master path? | 20:38 |
| TheJulia | jeblair: stable/pike at this point :) | 20:38 |
| *** amarao has joined #openstack-ironic | 20:38 | |
| jeblair | how time flies :) | 20:39 |
| TheJulia | A complete, and total blur | 20:39 |
| openstackgerrit | James E. Blair proposed openstack/ironic stable/pike: DNM: testing devstack-gate change https://review.openstack.org/497608 | 20:40 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: conductor saves version in db https://review.openstack.org/497593 | 20:40 |
| openstackgerrit | James E. Blair proposed openstack/ironic master: DNM: testing devstack-gate change https://review.openstack.org/497609 | 20:40 |
| jeblair | that should cover it :) | 20:41 |
| rloo | jeblair: heh | 20:41 |
| *** hamzy has quit IRC | 20:42 | |
| *** srobert has joined #openstack-ironic | 20:43 | |
| *** dsneddon_pto has quit IRC | 20:44 | |
| *** dsneddon_pto has joined #openstack-ironic | 20:44 | |
| *** dsneddon_pto is now known as dsneddon | 20:44 | |
| *** derekh has quit IRC | 20:46 | |
| *** jjung has quit IRC | 20:47 | |
| *** srobert has quit IRC | 20:48 | |
| *** trown is now known as trown|outtypewww | 20:49 | |
| *** shweta has joined #openstack-ironic | 20:50 | |
| shweta | Hello everyone.. Does ironic not support deploying windows or esx user images on the baremetal nodes? | 20:51 |
| shweta | https://docs.openstack.org/developer/diskimage-builder/user_guide/supported_distros.html - this doc only lists Linux as the supported OS for target and build host | 20:51 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add Sem-Ver flag to increment master branch version https://review.openstack.org/497615 | 20:54 |
| TheJulia | rloo: ^^^ $ python setup.py --version | 20:54 |
| TheJulia | 9.1.0.dev68 | 20:54 |
| rloo | TheJulia: thx! | 20:55 |
| TheJulia | shweta: In theory, your an deploy anything, but you generally need some way to get credentials in. If I remember correctly, There is a utility for windows to read config drives. ESX on the other hand, is designed to only support installation/deployment over networks using kickstart | 20:56 |
| TheJulia | s/your an/you can/ | 20:56 |
| TheJulia | shweta: Specifically, this is with whole disk images | 20:56 |
| shweta | So since ironic needs vmlinuz and initrd images passed separately, we can provision windows image with cloudinit but not esx? Which credentials are these exactly? | 20:58 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Set target raid configuration on a node https://review.openstack.org/483205 | 20:58 |
| TheJulia | shweta: So... That is three completely separate things | 20:59 |
| *** jkilpatr has joined #openstack-ironic | 21:00 | |
| TheJulia | 1) ironic uses a linux based deployment ramdisk in order to write the disk image, it uses a kernel and ramdisk to network or like an ISO to virtual media boot the node from. | 21:00 |
| TheJulia | 2) config-drive format, not cloud-init, unless cloud-init has implemented windows support. There was a presentation or two about it a few summits ago. | 21:01 |
| TheJulia | 3) Credential wise, it varies. Typically it is the key or password to login to the deployed instance. | 21:02 |
| shweta | TheJulia: thanks for clarification! | 21:03 |
| *** amarao has quit IRC | 21:04 | |
| shweta | To clarify: Does "disk-image-create ironic-agent fedora -o ironic-deploy" this command mean the deployment ramdisk is fedora based, but the actual user image could be ubuntu or something else? | 21:06 |
| *** jkilpatr has quit IRC | 21:07 | |
| TheJulia | shweta: exactly | 21:10 |
| *** jtomasek has joined #openstack-ironic | 21:10 | |
| *** jtomasek has quit IRC | 21:11 | |
| shweta | TheJulia: okay, thanks a lot! | 21:14 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add Sem-Ver flag to increment master branch version https://review.openstack.org/497615 | 21:17 |
| *** jkilpatr has joined #openstack-ironic | 21:19 | |
| *** rcernin has quit IRC | 21:19 | |
| *** Goneri has quit IRC | 21:22 | |
| shoaib | Hi, after my Nova-boot goes through, my BareMetal node fails to launch, because Neutron doesn't send a DHCP response back to the BareMetal node that was brought up, And i dont see the Baremetal Host (with its MAC addr) added to the /var/lib/neutron/dhcp/<namespace>/host file | 21:23 |
| shoaib | I'm wondering if setting the "network_interface" option for an ironic node, has something to do with it | 21:24 |
| shoaib | Thanks | 21:24 |
| *** ijw has quit IRC | 21:28 | |
| TheJulia | shoaib: What is network_interface set to presently? | 21:29 |
| shoaib | TheJulia: Its not set presently | 21:30 |
| TheJulia | shoaib: Additionally, did you check the namespace is defined for your provisioning network? | 21:30 |
| TheJulia | shoaib: What driver are you using? | 21:30 |
| TheJulia | do you see anything on the node for an error? | 21:30 |
| shoaib | TheJulia: Yes there is a namespace set for the provisioning network, but in that namespace, i dont see the new Baremetal host that I enrolled | 21:31 |
| shoaib | TheJulia: after nova-boot, the node was in spawning state for a long time, but eventually failed, because it didnt get any DHCP response | 21:31 |
| TheJulia | It won't be actually acted upon you attempt to deploy | 21:31 |
| TheJulia | if you do `openstack baremetal node show <uuid>` do you see anything in the last_error field? | 21:32 |
| shoaib | TheJulia: I dont see anything in last error, its back into available state | 21:34 |
| openstackgerrit | Merged openstack/bifrost master: Updated from global requirements https://review.openstack.org/497018 | 21:36 |
| *** rpioso has joined #openstack-ironic | 21:36 | |
| *** [1]rpioso has quit IRC | 21:37 | |
| shoaib | TheJulia: after looking up online, Neutron is supposed to add the Baremetal host to the provisioning network in this file eg: /var/lib/neutron/dhcp/70295064-0d99-4013-8406-eba65cc1577b/host , and only this would allow neutron dhcp-server to send a response back to the baremetal node deployed | 21:40 |
| shoaib | TheJulia: And thats not happening for me sadly | 21:41 |
| openstackgerrit | Merged openstack/bifrost master: Change perms for the VM logs https://review.openstack.org/494549 | 21:42 |
| TheJulia | shoaib: do you see anything in your ironic-conductor.log file? | 21:43 |
| shoaib | TheJulia: Started the process again, and monitoring the ironic conductor logs carefully, ill let u know what i see | 21:47 |
| *** ijw has joined #openstack-ironic | 21:49 | |
| shoaib | TheJulia: Also btw, when a node is booted, is the /instance/nova_host_id parameter for a baremetal node supposed to be the new Baremetal node that is booted , or the controller node where you are booting from ? | 21:50 |
| TheJulia | shoaib: no idea, I would look at the ironic api node instance_id field | 21:51 |
| shoaib | TheJulia: Ok, so the ironic-conductor doesn't show any Errors or anything significant, the node just goes into "wait call-back" state from "deploying" , | 21:52 |
| *** bfournie has quit IRC | 21:53 | |
| *** ijw has quit IRC | 21:57 | |
| *** ijw has joined #openstack-ironic | 22:02 | |
| TheJulia | shoaib: it really sounds like your baremetal node is not just physically attached to the flat network your using. Does the dnsmasq log indicate that it is seeing requests but doesn't have a record for it? do you see the dhcp requests if you sniff in the namespace? | 22:04 |
| *** jlabarre has quit IRC | 22:06 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add online data migrations for conductor version https://review.openstack.org/497666 | 22:07 |
| *** dsneddon is now known as dsneddon_away | 22:11 | |
| *** jcoufal has quit IRC | 22:18 | |
| jlvillal | $ export OS_BAREMETAL_API_VERSION=latest | 22:23 |
| jlvillal | stack@ubuntu-xenial:~/devstack$ openstack baremetal node list | 22:23 |
| jlvillal | baremetal version latest is not in supported versions: 1, 1.1, 1.2, 1.3, 1.4, 1.5, 1.6, 1.7, 1.8, 1.9, 1.10, 1.11, 1.12, 1.13, 1.14, 1.15, 1.16, 1.17, 1.18, 1.19, 1.20, 1.21, 1.22, 1.23, 1.24, 1.25, 1.26, 1.27, 1.28, 1.29, 1.30, 1.31, 1.32, 1.33, 1.34 | 22:23 |
| jlvillal | openstack: 'baremetal node list' is not an openstack command. See 'openstack --help'. | 22:23 |
| jlvillal | :( | 22:23 |
| rloo | jlvillal: i thought we fixed it so it understood latest, but maybe we didn't. | 22:25 |
| jlvillal | rloo: the --os-baremetal-api-version latest does work | 22:25 |
| rloo | jlvillal: that's what i thought we had fixed. but i guess not. you have the latest client package? | 22:25 |
| jlvillal | rloo: This is in a devstack run | 22:26 |
| jlvillal | python-ironicclient (1.16.0) | 22:26 |
| rloo | jlvillal: i guess we never fixed it then :-( | 22:27 |
| jlvillal | rloo: I'm taking a look at the code now | 22:27 |
| rloo | jlvillal: it should be simple enough to change 'latest' to the max version; i think it is hard-coded -- the max version. i think it would be harder for osc to actually send 'latest' to the server. i think it checks the value and ... | 22:28 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Disallow rolling upgrade from Ocata https://review.openstack.org/497685 | 22:29 |
| jlvillal | rloo: yeah the command line parser converts it to LAST_KNOWN_API_VERSION | 22:29 |
| rloo | jlvillal: if we haven't opened a bug, might be good to, so we can track. if it is considered a bug. i think it is. | 22:29 |
| rloo | jlvillal: gotta run. ciao. | 22:29 |
| jlvillal | rloo: Okay, will do | 22:29 |
| jlvillal | rloo: Have fun! | 22:29 |
| *** ijw has quit IRC | 22:33 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing RPC https://review.openstack.org/497689 | 22:43 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Add the ability to leverage a separate rabbitmq server https://review.openstack.org/440843 | 22:45 |
| jlvillal | rloo: https://bugs.launchpad.net/python-ironicclient/+bug/1712935 | 22:47 |
| openstack | Launchpad bug 1712935 in python-ironicclient "Using OS_BAREMETAL_API_VERSION=latest fails" [High,New] | 22:47 |
| *** bfournie has joined #openstack-ironic | 22:51 | |
| *** rbudden has quit IRC | 22:52 | |
| *** bfournie has quit IRC | 22:53 | |
| *** bfournie has joined #openstack-ironic | 23:11 | |
| *** jcoufal has joined #openstack-ironic | 23:17 | |
| *** jcoufal has quit IRC | 23:21 | |
| *** jkilpatr has quit IRC | 23:40 | |
| *** jkilpatr has joined #openstack-ironic | 23:40 | |
| *** rbudden has joined #openstack-ironic | 23:43 | |
| *** rpioso has quit IRC | 23:53 | |
| *** yuanying_ has joined #openstack-ironic | 23:59 | |
| *** zhangfei has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!