*** mmethot has joined #openstack-ironic | 00:00 | |
*** mmethot has quit IRC | 00:08 | |
*** mmethot has joined #openstack-ironic | 00:09 | |
*** mmethot has quit IRC | 00:14 | |
*** mmethot has joined #openstack-ironic | 00:14 | |
*** mmethot has quit IRC | 00:14 | |
*** mmethot has joined #openstack-ironic | 00:15 | |
*** mmethot has quit IRC | 00:16 | |
*** mmethot has joined #openstack-ironic | 00:17 | |
*** mmethot has quit IRC | 00:19 | |
*** TxGirlGeek has quit IRC | 00:29 | |
*** mjturek has quit IRC | 00:36 | |
*** TxGirlGeek has joined #openstack-ironic | 00:40 | |
*** dsneddon has quit IRC | 01:00 | |
*** _fragatina has quit IRC | 01:10 | |
*** _fragatina has joined #openstack-ironic | 01:11 | |
*** _fragatina has quit IRC | 01:15 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Fix listing nodes with conductor could raise https://review.openstack.org/632605 | 01:24 |
---|---|---|
*** dsneddon has joined #openstack-ironic | 01:25 | |
*** dsneddon has quit IRC | 01:32 | |
*** TxGirlGeek has quit IRC | 01:39 | |
*** dsneddon has joined #openstack-ironic | 02:00 | |
*** dsneddon has quit IRC | 02:04 | |
*** rloo has quit IRC | 02:20 | |
*** rh-jelabarre has quit IRC | 02:28 | |
*** rh-jelabarre has joined #openstack-ironic | 02:32 | |
*** dsneddon has joined #openstack-ironic | 02:35 | |
*** mmethot has joined #openstack-ironic | 02:36 | |
*** dsneddon has quit IRC | 02:40 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Add description field to node: api-ref https://review.openstack.org/632877 | 02:47 |
*** dsneddon has joined #openstack-ironic | 03:13 | |
*** devananda has quit IRC | 03:17 | |
*** dsneddon has quit IRC | 03:18 | |
*** mmethot has quit IRC | 03:29 | |
*** mmethot has joined #openstack-ironic | 03:29 | |
*** gabys has quit IRC | 03:37 | |
openstackgerrit | zhouxinyong proposed openstack/tenks master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632913 | 03:38 |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Remove duplicated fault code https://review.openstack.org/632920 | 03:51 |
*** dsneddon has joined #openstack-ironic | 03:54 | |
*** dsneddon has quit IRC | 03:58 | |
*** _fragatina has joined #openstack-ironic | 03:59 | |
*** _fragatina has quit IRC | 03:59 | |
*** _fragatina has joined #openstack-ironic | 03:59 | |
*** rh-jelabarre has quit IRC | 04:06 | |
*** dsneddon has joined #openstack-ironic | 04:32 | |
*** irclogbot_1 has quit IRC | 04:35 | |
*** dnuka has joined #openstack-ironic | 04:36 | |
dnuka | good morning folks o/ | 04:36 |
*** ijw has quit IRC | 04:37 | |
*** dsneddon has quit IRC | 04:37 | |
*** gyee has quit IRC | 05:01 | |
*** e0ne has joined #openstack-ironic | 05:03 | |
*** e0ne has quit IRC | 05:03 | |
*** dsneddon has joined #openstack-ironic | 05:12 | |
*** dsneddon has quit IRC | 05:17 | |
*** _fragatina has quit IRC | 05:33 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `dateutil` https://review.openstack.org/625889 | 05:39 |
*** dsneddon has joined #openstack-ironic | 05:47 | |
*** dsneddon has quit IRC | 05:52 | |
*** hwoarang has quit IRC | 06:08 | |
*** hwoarang has joined #openstack-ironic | 06:09 | |
*** gabys has joined #openstack-ironic | 06:09 | |
*** dsneddon has joined #openstack-ironic | 06:21 | |
openstackgerrit | Kaifeng Wang proposed openstack/python-ironicclient master: Support node description https://review.openstack.org/632935 | 06:31 |
*** dsneddon has quit IRC | 06:36 | |
openstackgerrit | Merged openstack/ironic master: Make ipmi_force_boot_device more user friendly https://review.openstack.org/620453 | 06:45 |
*** moshele has joined #openstack-ironic | 07:03 | |
*** dsneddon has joined #openstack-ironic | 07:06 | |
*** dsneddon has quit IRC | 07:19 | |
*** hamdyk has joined #openstack-ironic | 07:23 | |
arne_wiebalck | good morning, ironic! | 07:28 |
dnuka | good morning arne_wiebalck :) | 07:29 |
arne_wiebalck | dnuka: o/ | 07:29 |
*** e0ne has joined #openstack-ironic | 07:35 | |
*** dsneddon has joined #openstack-ironic | 07:47 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 07:49 |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 07:51 |
*** dsneddon has quit IRC | 07:51 | |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: Support non-persistent boot device https://review.openstack.org/632774 | 07:56 |
arne_wiebalck | etingof: I’m still replying to your comments while you already comment on my comments :-D | 07:59 |
* arne_wiebalck is too slow for etingof | 07:59 | |
etingof | hey arne_wiebalck o/ | 08:00 |
dnuka | good morning etingof o/ | 08:00 |
etingof | that explains why gerrit has not taken over the world of instant messaging yet | 08:00 |
etingof | good morning dnuka o/ | 08:00 |
*** rpittau has joined #openstack-ironic | 08:06 | |
dnuka | good morning rpittau :) | 08:06 |
rpittau | good morning ironic! o/ | 08:06 |
rpittau | hi dnuka :) | 08:07 |
arne_wiebalck | etingof: do you have a sec to discuss this? | 08:09 |
etingof | arne_wiebalck, sure | 08:09 |
arne_wiebalck | etingof: I basically agree with all your (and Kaifeng’s) comments | 08:10 |
arne_wiebalck | etingof: esp. as there is another persistency option | 08:10 |
arne_wiebalck | etingof: it’d be confusing | 08:10 |
arne_wiebalck | etingof: andthe name does | 08:11 |
arne_wiebalck | etingof: not refelct the intent in a good way | 08:11 |
arne_wiebalck | etingof: so, would a “preserve_bios_boot” order which is used in all drivers be better (or is the whole idea not worth pursuing)? | 08:12 |
arne_wiebalck | etingof: story is here: https://storyboard.openstack.org/#!/story/2004846 | 08:12 |
etingof | arne_wiebalck, to me, the new name sounds self-documenting | 08:13 |
arne_wiebalck | etingof: agreed | 08:13 |
etingof | arne_wiebalck, I personally can see why this use-case makes sense | 08:13 |
etingof | arne_wiebalck, interestingly, we still write the boot loader on the local drive even if we do not intend to boot from it | 08:14 |
etingof | perhaps that still makes sense... | 08:14 |
arne_wiebalck | etingof: it’s special, I agree, but we are one use case at least :) | 08:14 |
arne_wiebalck | etingof: we do boot from the local drive | 08:14 |
arne_wiebalck | etingof: I need to explain this | 08:14 |
etingof | yeah, that feels being against the whole desire to preserve boot mode ;) | 08:15 |
etingof | *boot device | 08:15 |
arne_wiebalck | etingof: our boxes try to boot from the network, fail (unless there is sth set up), and then fall back to the local drive | 08:15 |
* arne_wiebalck needs to leave for daily stand-up, back in a couple of mins | 08:16 | |
etingof | okay, I thought the must always boot from the trusted network | 08:16 |
* etingof wishes arne_wiebalck to enjoy his kanban | 08:17 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: [WIP] Build tinyipa with python3 https://review.openstack.org/495385 | 08:18 |
*** dnuka is now known as dnuka|brb | 08:19 | |
*** dsneddon has joined #openstack-ironic | 08:27 | |
*** rcernin has quit IRC | 08:29 | |
*** amoralej|off is now known as amoralej | 08:31 | |
*** dsneddon has quit IRC | 08:32 | |
*** robbbe has quit IRC | 08:40 | |
iurygregory | morning everyone o/ | 08:41 |
rpioso | Good morning, iurygregory | 08:46 |
iurygregory | morning rpioso o/ | 08:50 |
*** ianychoi has quit IRC | 08:55 | |
*** dsneddon has joined #openstack-ironic | 09:00 | |
*** dnuka|brb is now known as dnuka | 09:01 | |
dnuka | good morning iurygregory, rpioso o/ | 09:01 |
iurygregory | morning dnuka | 09:01 |
rpioso | dnuka: Hey \o | 09:02 |
dnuka | hey iurygregory, any idea, why is the openstack-tox-docs CI fails on https://review.openstack.org/#/c/625889 | 09:03 |
patchbot | patch 625889 - sushy - Introduce `convert_to_date` - 21 patch sets | 09:03 |
iurygregory | dnuka, http://logs.openstack.org/89/625889/21/check/openstack-tox-docs/e8ac045/job-output.txt.gz#_2019-01-24_07_56_26_842092 | 09:03 |
iurygregory | =) | 09:03 |
dnuka | thanks iurygregory \o/ | 09:04 |
iurygregory | you can run tox -e docs to see i think | 09:04 |
dnuka | iurygregory, got it :) | 09:04 |
iurygregory | if doesnt work let me know i think i have a workaround to test XD | 09:05 |
dnuka | ack | 09:05 |
*** dsneddon has quit IRC | 09:07 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 09:12 |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 09:15 |
*** dnuka is now known as dnuka|brb | 09:18 | |
rpittau | hey iurygregory :) | 09:19 |
iurygregory | rpittau, o/ | 09:19 |
arne_wiebalck | good morning iurygregory o/ | 09:19 |
iurygregory | good morning arne_wiebalck o/ | 09:19 |
iurygregory | beautiful log: http://logs.openstack.org/53/632453/3/check/virtualbmc-tempest-ironic-ipmi-iscsi/3e1bbe4/controller/logs/screen-ir-cond.txt.gz?level=ERROR | 09:22 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: Preserve BIOS boot order upon deployment https://review.openstack.org/632774 | 09:25 |
rpittau | iurygregory, wow, all the red :D | 09:25 |
iurygregory | rpittau, yeah =( | 09:26 |
iurygregory | same problem for both patches | 09:26 |
iurygregory | so maybe i missed a configuration | 09:27 |
mgoddard | morning | 09:27 |
rpittau | probably | 09:27 |
iurygregory | but, on devstack it works fine | 09:27 |
arne_wiebalck | etingof: two more points concerning the “preserve bios boot order” patch | 09:27 |
rpittau | hi mgoddard :) | 09:27 |
iurygregory | "Well it work on devstack" hehe | 09:27 |
iurygregory | morning mgoddard o/ | 09:27 |
mgoddard | hi rpittau iurygregory | 09:27 |
arne_wiebalck | etingof: should that move out of the agent conf (to deploy)? | 09:27 |
rpittau | iurygregory, "it works on devstack" is the new "it works on my pc" xD | 09:27 |
iurygregory | rpittau, but "it works on devstack" have cool t-shirts XD | 09:28 |
arne_wiebalck | etingof: how do I indentify places where the bios boot order might be changed (byeond ipmi) | 09:28 |
rpittau | iurygregory, lol, right! | 09:28 |
arne_wiebalck | good morning mgoddard o/ | 09:28 |
mgoddard | hi arne_wiebalck | 09:29 |
etingof | arne_wiebalck, is not that in the management interface of each hardware type? | 09:29 |
*** derekh has joined #openstack-ironic | 09:29 | |
openstackgerrit | Merged openstack/tenks master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632913 | 09:32 |
*** dnuka|brb is now known as dnuka | 09:32 | |
dnuka | good morning mgoddard o/ | 09:32 |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Introspection data storage backend follow up https://review.openstack.org/632969 | 09:34 |
*** vills_ has joined #openstack-ironic | 09:35 | |
*** vills_ has left #openstack-ironic | 09:35 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Introspection data storage backend follow up https://review.openstack.org/632969 | 09:36 |
*** e0ne has quit IRC | 09:41 | |
*** dsneddon has joined #openstack-ironic | 09:42 | |
*** gabys has quit IRC | 09:44 | |
*** gabys has joined #openstack-ironic | 09:44 | |
openstackgerrit | Hamdy Khader proposed openstack/ironic master: Add support for Smart NICs https://review.openstack.org/583573 | 09:46 |
*** gabys has quit IRC | 09:48 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-specs master: Better explain how do we know that the Smart NIC is ready to be programmed https://review.openstack.org/632971 | 09:50 |
*** tssurya has joined #openstack-ironic | 09:51 | |
*** dsneddon has quit IRC | 09:56 | |
*** e0ne has joined #openstack-ironic | 09:56 | |
*** livelace has joined #openstack-ironic | 10:02 | |
*** tssurya has quit IRC | 10:05 | |
*** tssurya has joined #openstack-ironic | 10:06 | |
*** trungnv has quit IRC | 10:06 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Move to zuulv3 https://review.openstack.org/632692 | 10:06 |
*** trungnv has joined #openstack-ironic | 10:10 | |
arne_wiebalck | etingof: Thx! Yes, the function is part of the mgmt interface, but then called from the supported boot interfaces. | 10:15 |
etingof | arne_wiebalck, so will the proposed change work across all hw types right away? | 10:16 |
arne_wiebalck | etingof: Now, do you think it’s wortwhile to touch all boot interfaces? Another option would be to limit this to the ipmi hardware type. | 10:16 |
arne_wiebalck | etingof: :-D | 10:16 |
etingof | arne_wiebalck, if it is not just your specific use-case, is this feature anyhow specific to ipmi? | 10:17 |
arne_wiebalck | etingof: I’d say no, it’s not specific to ipmi. | 10:18 |
etingof | arne_wiebalck, so may be it should be general then? | 10:18 |
arne_wiebalck | etingof: Could be done on all h/w types. | 10:18 |
arne_wiebalck | etingof: Q is if that is useful to anyone. | 10:18 |
etingof | arne_wiebalck, may be bring this Q to the upstream meeting on Monday? | 10:19 |
arne_wiebalck | etingof: Good idea! | 10:19 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/virtualbmc master: Move to zuulv3 https://review.openstack.org/632453 | 10:24 |
*** moshele has quit IRC | 10:27 | |
*** csmart has joined #openstack-ironic | 10:28 | |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: Preserve BIOS boot order upon deployment https://review.openstack.org/632774 | 10:29 |
*** dsneddon has joined #openstack-ironic | 10:32 | |
*** dsneddon has quit IRC | 10:45 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Move to zuulv3 https://review.openstack.org/632692 | 10:55 |
openstackgerrit | Nikolay Fedotov proposed openstack/ironic-inspector master: Remove the host resolution in get_ipmi_address https://review.openstack.org/626552 | 11:10 |
*** dsneddon has joined #openstack-ironic | 11:17 | |
*** hjensas has quit IRC | 11:18 | |
*** hjensas has joined #openstack-ironic | 11:21 | |
*** dsneddon has quit IRC | 11:28 | |
*** dnuka has quit IRC | 11:35 | |
openstackgerrit | Harald Jensås proposed openstack/python-ironicclient master: Add Events support https://review.openstack.org/345934 | 11:37 |
*** moshele has joined #openstack-ironic | 11:43 | |
*** robbbe has joined #openstack-ironic | 11:50 | |
*** Bhujay has joined #openstack-ironic | 11:55 | |
*** rpittau has quit IRC | 12:00 | |
*** dsneddon has joined #openstack-ironic | 12:02 | |
*** amoralej is now known as amoralej|lunch | 12:05 | |
*** rpittau has joined #openstack-ironic | 12:06 | |
*** dsneddon has quit IRC | 12:15 | |
*** rpittau has quit IRC | 12:15 | |
*** rpittau has joined #openstack-ironic | 12:16 | |
*** rpittau has quit IRC | 12:30 | |
*** rpittau has joined #openstack-ironic | 12:31 | |
*** dsneddon has joined #openstack-ironic | 12:32 | |
*** dnuka has joined #openstack-ironic | 12:36 | |
*** bfournie has quit IRC | 12:38 | |
*** pcaruana has quit IRC | 12:40 | |
*** dsneddon has quit IRC | 12:45 | |
*** rpittau has quit IRC | 12:45 | |
*** rpittau has joined #openstack-ironic | 12:46 | |
*** e0ne has quit IRC | 12:59 | |
*** livelace has quit IRC | 12:59 | |
dnuka | folks, any idea how to resolve this :) | 13:08 |
dnuka | https://review.openstack.org/#/c/625889/23/sushy/utils.py@298 | 13:08 |
patchbot | patch 625889 - sushy - Introduce `convert_to_date` - 23 patch sets | 13:08 |
dnuka | The reviewer makes a good point. I could add a default value(like the current time), but I'm not sure, this is the correct move here(since we are dealing with dates). I mean the caller of convert_to_date() should pass it a correct date_string and if not we could return None | 13:09 |
dnuka | :) | 13:10 |
*** bfournie has joined #openstack-ironic | 13:12 | |
*** trown|outtypewww is now known as trown | 13:13 | |
*** rpittau has quit IRC | 13:15 | |
*** rpittau has joined #openstack-ironic | 13:16 | |
*** hwoarang has quit IRC | 13:18 | |
*** hwoarang has joined #openstack-ironic | 13:19 | |
*** pcaruana has joined #openstack-ironic | 13:20 | |
*** e0ne has joined #openstack-ironic | 13:20 | |
*** dsneddon has joined #openstack-ironic | 13:20 | |
*** dnuka is now known as dnuka|brb | 13:22 | |
*** derekh has quit IRC | 13:26 | |
iurygregory | dnuka|brb, since you are using dateutil libary you can use the function for ISO 8601 format and if the string doesnt follow the format it will raise ValueError (this way you dont need the regex you are using and the if | 13:28 |
*** rpittau has quit IRC | 13:31 | |
*** rpittau has joined #openstack-ironic | 13:31 | |
*** dnuka|brb is now known as dnuka | 13:32 | |
dnuka | thanks iurygregory. Ah! the parser.isoparse(), right? :) | 13:35 |
iurygregory | yup | 13:36 |
dnuka | iurygregory, got it | 13:36 |
iurygregory | do some tests in your env and see if it works fine | 13:36 |
iurygregory | =) | 13:36 |
dnuka | ack :) | 13:36 |
iurygregory | i did a quick search only | 13:36 |
dnuka | ha :D | 13:36 |
arne_wiebalck | dnuka: iurygregory: I just updated the review. To support iurygregory’s suggestion I think there is also a mistake in the regex … :) | 13:36 |
dnuka | thanks arne_wiebalck :) | 13:37 |
rpittau | that may not work unfortunately, isoparse doesn't support standard sub second digits, at least not up to 2.7.5 :/ | 13:39 |
*** dsneddon has quit IRC | 13:39 | |
dnuka | oh! | 13:39 |
rpittau | https://github.com/dateutil/dateutil/blob/2.7.x/dateutil/parser/isoparser.py#L91 | 13:42 |
dnuka | ack | 13:42 |
iurygregory | damm =( | 13:43 |
iurygregory | but sushy was to be only python 3 + no? | 13:43 |
rpittau | oh well, give it a try anyway, but I'm not optimistic :/ | 13:43 |
* iurygregory remember some discussion about this in a meeting or something | 13:43 | |
rpittau | iurygregory, yes, in theory | 13:43 |
dnuka | rpittau, any suggestions :) | 13:43 |
rpittau | I don't think it's there yet | 13:44 |
iurygregory | still can use the simple parser i think | 13:44 |
iurygregory | with the pattern | 13:44 |
arne_wiebalck | dnuka: what’s wrong with only parser.parse? | 13:44 |
dnuka | arne_wiebalck, parser.parse works. but - https://review.openstack.org/#/c/625889/18/sushy/tests/unit/resources/test_settings.py@26 | 13:45 |
patchbot | patch 625889 - sushy - Introduce `convert_to_date` - 23 patch sets | 13:45 |
*** amoralej|lunch is now known as amoralej | 13:45 | |
iurygregory | dnuka, didnt get '-' | 13:46 |
*** rpittau has quit IRC | 13:46 | |
iurygregory | i dont see problem with parser.parse | 13:47 |
* arne_wiebalck doesn’t see the issue … | 13:47 | |
dnuka | :) | 13:47 |
*** rpittau has joined #openstack-ironic | 13:48 | |
dnuka | arne_wiebalck: so, parser.parse works with anything. for ex- 2019-01-24, regex part need to check for the ISO 8601 format(2016-03-07T14:44.30-05:05), I think :) | 13:52 |
arne_wiebalck | dnuka: ah, you’d like to enforce 8601 | 13:53 |
dnuka | arne_wiebalck, yes :) | 13:53 |
arne_wiebalck | dnuka: ah, ok, sorry I missed that | 13:53 |
arne_wiebalck | dnuka: so my comment about the regex being not correct is valid ;-P | 13:54 |
iurygregory | ++ | 13:54 |
dnuka | arne_wiebalck: no worries :) | 13:54 |
iurygregory | also is valid see if sushy will be running only after 3.x python | 13:54 |
iurygregory | so you could use the isoparser \o/ | 13:54 |
dnuka | iurygregory: I'll try :) | 13:55 |
iurygregory | other wise regex + parser.parse will do the job | 13:55 |
rpittau | oh wait, before I meant dateutil version, not python version | 13:55 |
dnuka | yes | 13:55 |
iurygregory | rpittau, oh | 13:55 |
iurygregory | so we would need to add to req only lol | 13:55 |
*** derekh has joined #openstack-ironic | 13:58 | |
*** jcoufal has joined #openstack-ironic | 13:59 | |
*** rloo has joined #openstack-ironic | 13:59 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/virtualbmc master: Move to zuulv3 https://review.openstack.org/632453 | 14:02 |
*** yolanda has joined #openstack-ironic | 14:02 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 14:06 |
dnuka | :) | 14:06 |
arne_wiebalck | dnuka: you chnaged to allow both ‘.’ and ‘:’ ? | 14:11 |
dnuka | arne_wiebalck, https://github.com/openstack/sushy/blob/master/sushy/tests/unit/json_samples/settings.json#L7 :) | 14:12 |
*** e0ne has quit IRC | 14:14 | |
*** hjensas has quit IRC | 14:14 | |
*** dsneddon has joined #openstack-ironic | 14:14 | |
arne_wiebalck | dnuka: ‘.’ is definitely allowed, however: it’s allowed for all three time elements. And FWIU ‘.30’ does not mean ’30 seconds’, but is the decimal fraction of the minute, so ’20 seconds’ ... | 14:15 |
arne_wiebalck | dnuka: 18 secs, actually | 14:16 |
arne_wiebalck | dnuka: also, ‘,’ is also allowed (and preferred over ‘.’) | 14:16 |
arne_wiebalck | dnuka: is it possible that the ‘.’ in the example is just a mistake | 14:16 |
arne_wiebalck | dnuka: ? | 14:16 |
arne_wiebalck | dnuka: s/example/test/ | 14:17 |
dnuka | arne_wiebalck :) still processing the info. I'm not sure about the example json values. may need to talk to etingof :) | 14:18 |
dnuka | etingof: are you around :) | 14:20 |
rpittau | arne_wiebalck, dnuka, I believe the `.` is correct in this case http://redfish.dmtf.org/schemas/DSP0266_1.5.0.html#datetime-values | 14:21 |
dnuka | rpittau, thank for clearing that up | 14:21 |
arne_wiebalck | rpittau: the ‘.’ in your link is fraction of a second, no? | 14:22 |
rpittau | arne_wiebalck, correct | 14:22 |
arne_wiebalck | rpittau: in the regex it is fraction of minutes | 14:22 |
rpittau | oh in that case it should be just `:` | 14:23 |
arne_wiebalck | rpittau: well :) | 14:23 |
dnuka | :) | 14:23 |
arne_wiebalck | rpittau: ‘.’ is allowed for all time elements | 14:23 |
etingof | dnuka, sure | 14:23 |
rpittau | arne_wiebalck, I agree, but I'm considering only this particular schema | 14:24 |
arne_wiebalck | rpittau: makes sense, but even if limited to secs, the regex is wrong | 14:24 |
dnuka | I'm scratching my head :) | 14:25 |
rpittau | arne_wiebalck, you're totally right, maybe the json schema used for that is wrong then, since it's missing seconds, or hours :/ | 14:26 |
dnuka | etingof, can you please take a look at https://review.openstack.org/#/c/625889/ | 14:26 |
patchbot | patch 625889 - sushy - Introduce `convert_to_date` - 24 patch sets | 14:26 |
*** mjturek has joined #openstack-ironic | 14:27 | |
mjturek | heyo ironic | 14:27 |
dnuka | hi mjturek | 14:27 |
*** dsneddon has quit IRC | 14:27 | |
arne_wiebalck | dnuka: rpittau: the more complete the regex is supposed to cover iso8061, the more complex it will get: I found a “complete” one on stackoverflow and it has 167 characters … I think we should draw the line somewhere :) | 14:32 |
rpittau | lol | 14:32 |
*** mmethot has quit IRC | 14:32 | |
dnuka | arne_wiebalck, I agree with you 100% :) | 14:32 |
rpittau | to keep dmtf standards I think `:` is enough as separator | 14:33 |
iurygregory | would make easy if we require a higher version of dateutil XD | 14:33 |
rpittau | so the example should be changed | 14:33 |
arne_wiebalck | dnuka: how about looking at the code that the future version of dateutil is using? | 14:33 |
arne_wiebalck | dnuka: and “get inspiration” from it ;) | 14:33 |
dnuka | arne_wiebalck, will do that :) | 14:34 |
*** mmethot has joined #openstack-ironic | 14:34 | |
*** e0ne has joined #openstack-ironic | 14:34 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: [WIP] Build tinyipa with python3 https://review.openstack.org/495385 | 14:34 |
arne_wiebalck | rpittau: with example you mean the test in https://github.com/openstack/sushy/blob/master/sushy/tests/unit/json_samples/settings.json#L7 ? | 14:35 |
rpittau | arne_wiebalck, yes, the `.` should be changed to `:` | 14:36 |
rpittau | as the microseconds are optional | 14:36 |
rpittau | well, sub-seconds | 14:36 |
dnuka | rpittau, is it ok to update the json examples in this same patch | 14:38 |
rpittau | dnuka, I think so, it looks like a mistake | 14:38 |
dnuka | rpittau, got it | 14:38 |
rpittau | again, talking about dmtf standards | 14:38 |
dnuka | ack :) | 14:39 |
*** arne_wiebalck_ has joined #openstack-ironic | 14:39 | |
*** arne_wiebalck has quit IRC | 14:41 | |
*** arne_wiebalck_ is now known as arne_wiebalck | 14:41 | |
*** mjturek has quit IRC | 14:42 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 14:43 |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `convert_to_date` https://review.openstack.org/625889 | 14:45 |
dnuka | thanks everyone :) It's 8:15 PM. see you tomorrow o/ | 14:46 |
arne_wiebalck | dnuka: o/ | 14:47 |
*** rh-jelabarre has joined #openstack-ironic | 14:47 | |
*** dnuka has quit IRC | 14:47 | |
*** hjensas has joined #openstack-ironic | 14:56 | |
*** dsneddon has joined #openstack-ironic | 14:57 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Move to zuulv3 https://review.openstack.org/633010 | 15:01 |
*** hamdyk has quit IRC | 15:01 | |
*** mjturek has joined #openstack-ironic | 15:02 | |
*** mjturek has quit IRC | 15:04 | |
*** hwoarang has quit IRC | 15:07 | |
*** mjturek has joined #openstack-ironic | 15:08 | |
*** hwoarang has joined #openstack-ironic | 15:08 | |
*** dsneddon has quit IRC | 15:10 | |
openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce `to_datetime` https://review.openstack.org/625889 | 15:13 |
*** baha has joined #openstack-ironic | 15:21 | |
*** moshele has quit IRC | 15:24 | |
*** mbeierl has quit IRC | 15:34 | |
*** pcaruana has quit IRC | 15:35 | |
*** mbeierl has joined #openstack-ironic | 15:37 | |
*** dsneddon has joined #openstack-ironic | 15:38 | |
*** mjturek has quit IRC | 15:41 | |
*** dsneddon has quit IRC | 15:43 | |
*** mbeierl has quit IRC | 15:44 | |
*** ianychoi has joined #openstack-ironic | 15:44 | |
*** bfournie has quit IRC | 15:46 | |
*** mbeierl has joined #openstack-ironic | 15:46 | |
*** mjturek has joined #openstack-ironic | 15:46 | |
*** mjturek has quit IRC | 15:51 | |
*** pcaruana has joined #openstack-ironic | 15:55 | |
*** TxGirlGeek has joined #openstack-ironic | 15:58 | |
*** dsneddon has joined #openstack-ironic | 16:00 | |
*** mjturek has joined #openstack-ironic | 16:02 | |
*** gyee has joined #openstack-ironic | 16:06 | |
*** tssurya has quit IRC | 16:08 | |
*** e0ne has quit IRC | 16:09 | |
*** dustinc has joined #openstack-ironic | 16:13 | |
*** gyee has quit IRC | 16:14 | |
*** _fragatina has joined #openstack-ironic | 16:20 | |
*** _fragatina has quit IRC | 16:20 | |
*** _fragatina has joined #openstack-ironic | 16:21 | |
*** gyee has joined #openstack-ironic | 16:25 | |
*** hjensas has quit IRC | 16:25 | |
*** dustinc has quit IRC | 16:33 | |
*** Bhujay has quit IRC | 16:44 | |
*** pcaruana has quit IRC | 16:48 | |
*** hjensas has joined #openstack-ironic | 16:57 | |
*** livelace has joined #openstack-ironic | 17:09 | |
*** hjensas has quit IRC | 17:15 | |
NobodyCam | good morning Ironic'ers | 17:15 |
iurygregory | morning NobodyCam | 17:17 |
*** mmethot has quit IRC | 17:17 | |
iurygregory | and bye everyone time to go home XD | 17:17 |
NobodyCam | Good morning iurygregory | 17:18 |
NobodyCam | have a great night :) | 17:19 |
iurygregory | ty =D | 17:19 |
*** mmethot has joined #openstack-ironic | 17:19 | |
rpittau | hi NobodyCam :) | 17:25 |
NobodyCam | Good Morning rpittau | 17:26 |
NobodyCam | \o/ | 17:26 |
*** robbbe has quit IRC | 17:28 | |
*** hjensas has joined #openstack-ironic | 17:32 | |
rpittau | bye all! good evening! o/ | 17:34 |
*** rpittau has quit IRC | 17:34 | |
*** _fragatina has quit IRC | 17:43 | |
*** dims has quit IRC | 17:43 | |
*** dims has joined #openstack-ironic | 17:48 | |
*** bfournie has joined #openstack-ironic | 17:49 | |
*** dims has quit IRC | 17:54 | |
*** dims has joined #openstack-ironic | 17:55 | |
*** trown is now known as trown|lunch | 17:59 | |
*** e0ne has joined #openstack-ironic | 17:59 | |
*** derekh has quit IRC | 18:09 | |
*** e0ne has quit IRC | 18:13 | |
*** e0ne has joined #openstack-ironic | 18:27 | |
*** _fragatina has joined #openstack-ironic | 18:30 | |
*** robbbe has joined #openstack-ironic | 18:38 | |
*** amoralej is now known as amoralej|off | 18:44 | |
*** luzC has joined #openstack-ironic | 18:48 | |
*** manjeets_ has joined #openstack-ironic | 18:59 | |
*** TxGirlGe_ has joined #openstack-ironic | 18:59 | |
*** TxGirlGeek has quit IRC | 19:00 | |
*** manjeets has quit IRC | 19:00 | |
*** e0ne has quit IRC | 19:02 | |
*** livelace has quit IRC | 19:03 | |
*** ijw has joined #openstack-ironic | 19:07 | |
*** baha has quit IRC | 19:07 | |
*** baha has joined #openstack-ironic | 19:15 | |
*** manjeets_ is now known as manjeets | 19:17 | |
*** robbbe has quit IRC | 19:23 | |
*** e0ne has joined #openstack-ironic | 19:26 | |
openstackgerrit | Nikolay Fedotov proposed openstack/ironic-inspector master: Remove the host resolution in get_ipmi_address https://review.openstack.org/626552 | 19:27 |
mjturek | what's the best way to get out of clean-failed? | 19:27 |
*** mjturek has quit IRC | 19:30 | |
*** mjturek has joined #openstack-ironic | 19:30 | |
*** TxGirlGe_ has quit IRC | 19:32 | |
mjturek | nvm! I got it (manage then provide) | 19:34 |
*** gabys has joined #openstack-ironic | 19:38 | |
*** gabys has quit IRC | 19:42 | |
*** baha has quit IRC | 19:47 | |
*** baha has joined #openstack-ironic | 19:49 | |
openstackgerrit | Harald Jensås proposed openstack/ironic master: API - Implement /events endpoint https://review.openstack.org/631946 | 19:54 |
*** e0ne has quit IRC | 19:54 | |
openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Enable ironic notifications https://review.openstack.org/576873 | 20:08 |
*** robbbe has joined #openstack-ironic | 20:15 | |
jidar | howdy people! I'm trying to figure out how to do a nic mapping to node name in the param defaults section as outlined in the ch12 of the adv docs, in the docs there is the following code snippet: https://gist.github.com/74cd81ae1988ad16ffeb9e46f71b9c31 but it has "node1" and "node2" how are those nodes named? Can I pick any name for the nodes as long as it matches the nics on the node? | 20:18 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allocation API: REST API implementation https://review.openstack.org/629260 | 20:26 |
*** TxGirlGeek has joined #openstack-ironic | 20:26 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [DO NOT MERGE] [PROTOTYPE] Use JSON-RPC instead of oslo.messaging https://review.openstack.org/633052 | 20:27 |
*** moshele has joined #openstack-ironic | 21:00 | |
*** rcernin has joined #openstack-ironic | 21:02 | |
*** ElCoyote_ has joined #openstack-ironic | 21:08 | |
*** moshele has quit IRC | 21:35 | |
*** trown|lunch is now known as trown|outtypewww | 21:48 | |
*** mjturek has quit IRC | 22:03 | |
*** rcernin has quit IRC | 22:11 | |
*** rcernin has joined #openstack-ironic | 22:12 | |
*** rh-jelabarre has quit IRC | 22:21 | |
*** baha has quit IRC | 22:35 | |
*** jcoufal has quit IRC | 22:36 | |
*** ElCoyote_ has quit IRC | 22:43 | |
*** MattMan has quit IRC | 22:50 | |
*** MattMan has joined #openstack-ironic | 22:50 | |
*** mmethot has quit IRC | 23:01 | |
*** mmethot has joined #openstack-ironic | 23:03 | |
*** bfournie has quit IRC | 23:11 | |
*** TxGirlGeek has quit IRC | 23:15 | |
*** robbbe has quit IRC | 23:15 | |
*** robbbe has joined #openstack-ironic | 23:18 | |
*** robbbe has quit IRC | 23:18 | |
*** robbbe has joined #openstack-ironic | 23:21 | |
*** robbbe has quit IRC | 23:21 | |
*** robbbe has joined #openstack-ironic | 23:22 | |
*** robbbe has quit IRC | 23:23 | |
*** robbbe has joined #openstack-ironic | 23:24 | |
*** robbbe has quit IRC | 23:24 | |
*** robbbe has joined #openstack-ironic | 23:26 | |
*** robbbe has joined #openstack-ironic | 23:29 | |
*** robbbe has joined #openstack-ironic | 23:37 | |
*** robbbe has joined #openstack-ironic | 23:37 | |
*** relrod has left #openstack-ironic | 23:38 | |
*** robbbe has joined #openstack-ironic | 23:38 | |
*** robbbe has joined #openstack-ironic | 23:39 | |
*** robbbe has quit IRC | 23:39 | |
*** robbbe has joined #openstack-ironic | 23:40 | |
*** robbbe has joined #openstack-ironic | 23:47 | |
*** robbbe has joined #openstack-ironic | 23:47 | |
*** robbbe has quit IRC | 23:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!