jroll | oh, cool | 00:01 |
---|---|---|
*** achanda has quit IRC | 00:05 | |
*** hemna has quit IRC | 00:10 | |
*** oomichi has joined #openstack-ironic | 00:11 | |
*** romcheg1 has quit IRC | 00:12 | |
*** dprince has joined #openstack-ironic | 00:26 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Hide new fields via single method https://review.openstack.org/166056 | 00:26 |
*** rloo has quit IRC | 00:27 | |
*** Marga_ has quit IRC | 00:29 | |
*** rloo has joined #openstack-ironic | 00:29 | |
*** Marga_ has joined #openstack-ironic | 00:29 | |
*** Haomeng has joined #openstack-ironic | 00:37 | |
*** Marga_ has quit IRC | 00:37 | |
*** Haomeng|2 has quit IRC | 00:39 | |
openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 00:40 |
*** rloo has quit IRC | 00:44 | |
*** rloo has joined #openstack-ironic | 00:45 | |
*** rloo has quit IRC | 01:19 | |
*** chenglch has joined #openstack-ironic | 01:19 | |
openstackgerrit | Merged openstack/ironic: Changes for secure boot support for iLO drivers https://review.openstack.org/165907 | 01:37 |
*** harlowja_ is now known as harlowja_away | 01:46 | |
*** rameshg87 has joined #openstack-ironic | 01:48 | |
*** wanyen has quit IRC | 01:48 | |
*** Haomeng has quit IRC | 01:53 | |
rameshg87 | good morning ironic | 01:54 |
openstackgerrit | Merged openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 02:19 |
jroll | heya rameshg87 :) | 02:25 |
rameshg87 | hello jroll | 02:27 |
rameshg87 | staying late night again :) | 02:27 |
rameshg87 | jroll: which timezone are you in ? | 02:27 |
jroll | rameshg87: no, just happen to be on irc, not working :) | 02:28 |
jroll | I'm west coast US, PST/PDT (idk which is which) | 02:28 |
mrda | hey rameshg87 | 02:29 |
*** stendulker has joined #openstack-ironic | 02:29 | |
rameshg87 | hello mrda | 02:31 |
rameshg87 | jroll: okay :) | 02:31 |
* rameshg87 will be back in a few houts | 02:32 | |
*** jamielennox is now known as jamielennox|lunc | 02:32 | |
rameshg87 | hours | 02:32 |
*** rameshg87 has quit IRC | 02:32 | |
*** jamielennox|lunc is now known as jamielennox|food | 02:32 | |
openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 02:32 |
*** Haomeng has joined #openstack-ironic | 02:36 | |
*** stendulker_ has joined #openstack-ironic | 02:41 | |
*** stendulker has quit IRC | 02:41 | |
*** achanda has joined #openstack-ironic | 02:45 | |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: Fail deploy if root uuid or disk id isn't available https://review.openstack.org/143919 | 02:50 |
*** stendulker has joined #openstack-ironic | 02:51 | |
*** stendulker_ has quit IRC | 02:53 | |
*** greghaynes has quit IRC | 02:58 | |
*** jamielennox|food is now known as jamielennox | 03:01 | |
*** ramineni has joined #openstack-ironic | 03:05 | |
*** greghaynes has joined #openstack-ironic | 03:05 | |
*** subscope_ has joined #openstack-ironic | 03:05 | |
*** coolsvap has joined #openstack-ironic | 03:16 | |
*** stendulker has quit IRC | 03:17 | |
*** stendulker has joined #openstack-ironic | 03:17 | |
*** coolsvap|afk has joined #openstack-ironic | 03:23 | |
*** coolsvap has quit IRC | 03:25 | |
*** coolsvap|afk is now known as coolsvap | 03:26 | |
*** coolsvap is now known as coolsvap|afk | 03:26 | |
*** coolsvap|afk is now known as coolsvap | 03:27 | |
*** Marga_ has joined #openstack-ironic | 03:43 | |
*** Marga_ has quit IRC | 03:46 | |
*** achanda has quit IRC | 03:49 | |
*** yog_ has joined #openstack-ironic | 03:55 | |
*** Marga_ has joined #openstack-ironic | 03:56 | |
*** achanda has joined #openstack-ironic | 03:59 | |
*** achanda has quit IRC | 04:01 | |
*** Marga_ has quit IRC | 04:02 | |
*** stendulker_ has joined #openstack-ironic | 04:03 | |
*** stendulker has quit IRC | 04:04 | |
*** Marga_ has joined #openstack-ironic | 04:11 | |
*** achanda has joined #openstack-ironic | 04:12 | |
*** achanda has quit IRC | 04:15 | |
*** Marga_ has quit IRC | 04:17 | |
*** rameshg87 has joined #openstack-ironic | 04:21 | |
*** Marga_ has joined #openstack-ironic | 04:25 | |
*** wuhg has joined #openstack-ironic | 04:26 | |
*** Marga_ has quit IRC | 04:31 | |
*** Marga_ has joined #openstack-ironic | 04:40 | |
*** Marga_ has quit IRC | 04:53 | |
*** Nisha has joined #openstack-ironic | 04:56 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: ILO: Add unit tests for ris.RISOperations._rest_op https://review.openstack.org/165718 | 04:59 |
rameshg87 | ramineni: Nisha: stendulker_: ^^^^ | 04:59 |
Nisha | :) | 04:59 |
* ramineni looks | 05:00 | |
*** chlong has quit IRC | 05:02 | |
*** pradipta has joined #openstack-ironic | 05:04 | |
rameshg87 | mrda: some pep8 issues - https://review.openstack.org/#/c/163730/ . don't know if you noticed :) | 05:04 |
Haomeng | rameshg87: morning:) thanks for your patch, and I commited new patch to fix the merge conflict issue for this one - https://review.openstack.org/#/c/143919/ | 05:05 |
*** mdbooth has quit IRC | 05:07 | |
*** achanda has joined #openstack-ironic | 05:08 | |
mrda | thanks rameshg87 | 05:08 |
Haomeng | rameshg87: :) | 05:09 |
*** mdbooth has joined #openstack-ironic | 05:11 | |
*** Marga_ has joined #openstack-ironic | 05:16 | |
rameshg87 | Haomeng: thanks for that :) | 05:16 |
*** Marga_ has quit IRC | 05:16 | |
*** Marga_ has joined #openstack-ironic | 05:16 | |
Haomeng | rameshg87: should say 'thanks for you', ramesh | 05:16 |
Haomeng | rameshg87: :) | 05:16 |
*** chlong has joined #openstack-ironic | 05:19 | |
openstackgerrit | Aparna proposed stackforge/proliantutils: ILO: Support for configuring httpboot through RIS https://review.openstack.org/163325 | 05:24 |
rameshg87 | ramineni: ^^ | 05:27 |
*** coolsvap is now known as coolsvap|afk | 05:28 | |
*** absubram has quit IRC | 05:30 | |
Haomeng | rameshg87: :) | 05:31 |
*** alexpilotti has quit IRC | 05:35 | |
*** coolsvap|afk is now known as coolsvap | 05:36 | |
*** Marga_ has quit IRC | 05:37 | |
*** Marga_ has joined #openstack-ironic | 05:38 | |
*** mdbooth has quit IRC | 05:45 | |
openstackgerrit | Merged openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 05:45 |
*** mdbooth has joined #openstack-ironic | 05:50 | |
openstackgerrit | Michael Davies proposed openstack/ironic: Fix is_hostname_safe for RFC compliance https://review.openstack.org/165666 | 05:55 |
*** pradipta has quit IRC | 06:00 | |
*** chlong has quit IRC | 06:01 | |
*** saripurigopi has joined #openstack-ironic | 06:02 | |
*** pradipta has joined #openstack-ironic | 06:04 | |
openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 06:07 |
*** chlong has joined #openstack-ironic | 06:13 | |
*** pradipta has quit IRC | 06:15 | |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: Fail deploy if root uuid or disk id isn't available https://review.openstack.org/143919 | 06:16 |
rameshg87 | mrda: one question | 06:17 |
rameshg87 | mrda: https://review.openstack.org/#/c/165666/4/ironic/common/utils.py | 06:17 |
*** mdbooth has quit IRC | 06:17 | |
mrda | yes rameshg87 ? | 06:18 |
rameshg87 | mrda: oh sorry, i missed a part of regex | 06:18 |
rameshg87 | nothing :) | 06:18 |
openstackgerrit | Aparna proposed stackforge/proliantutils: ILO: Support for configuring httpboot through RIS https://review.openstack.org/163325 | 06:18 |
mrda | whew! :) | 06:18 |
mrda | rameshg87: do you know anything about the migrations tests in ironic/tests/db/sqlalchemy/test_migrations.py ? | 06:19 |
rameshg87 | mrda: have an idea | 06:19 |
mrda | I have no idea how _check_* functions run | 06:20 |
mrda | that is question 1) | 06:20 |
mrda | oh, I think I see it now | 06:21 |
rameshg87 | mrda: i have seen them only once and that too recently | 06:21 |
mrda | I'm also struggling how to test a column length extension, no-one's done anything like that before | 06:22 |
* rameshg87 might not be the right person to ask :) | 06:22 | |
mrda | that's fine rameshg87, I'll keep plugging away | 06:23 |
rameshg87 | mrda: okay. even i don't have a straight answer | 06:23 |
*** mdbooth has joined #openstack-ironic | 06:23 | |
*** marios has joined #openstack-ironic | 06:24 | |
*** zhenguo has joined #openstack-ironic | 06:26 | |
mrda | I just wrote a test that would deliberately fail, and it didn't :-S | 06:26 |
rameshg87 | :D | 06:27 |
rameshg87 | mrda: can hostnames end with a period ? | 06:27 |
* rameshg87 have never seen such hostnames | 06:27 | |
mrda | try ssh'ing to your favourite destination with a '.' on the end | 06:27 |
* rameshg87 tries | 06:28 | |
mrda | It does work, because all fqdns implicitly have a period on the end | 06:28 |
mrda | i.e. slashdot.org is really slashdot.org. | 06:28 |
mrda | because slashdot.org is really slashdot.org.<null> | 06:28 |
mrda | where <null> is a root nameserver | 06:28 |
rameshg87 | great .. it works :) | 06:29 |
rameshg87 | thanks | 06:29 |
rameshg87 | tried sshing to review.openstack.org. | 06:29 |
rameshg87 | :) | 06:29 |
* mrda is glad that's your favourite destination :) | 06:29 | |
* rameshg87 doesn't only has ip addresses to ssh apart from review.openstack.org | 06:30 | |
* rameshg87 has only ip addresses to ssh apart from review.openstack.org | 06:30 | |
* mrda needs to go afk for a while, as his brain is full | 06:30 | |
mrda | thanks rameshg87 for the reviews | 06:31 |
rameshg87 | mrda: wc..relax for a while to free up brain :) | 06:31 |
*** mrda is now known as mrda-afk | 06:31 | |
*** Marga_ has quit IRC | 06:34 | |
*** jamielennox is now known as jamielennox|away | 06:35 | |
*** dlpartain has joined #openstack-ironic | 06:41 | |
*** dlpartain has left #openstack-ironic | 06:41 | |
*** subscope_ has quit IRC | 06:42 | |
openstackgerrit | Merged stackforge/proliantutils: ILO: Support for configuring httpboot through RIS https://review.openstack.org/163325 | 06:44 |
*** yuanying has quit IRC | 06:51 | |
*** pradipta has joined #openstack-ironic | 06:53 | |
*** killer_prince is now known as lazy_prince | 06:53 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: ILO: Add unit tests for ris.RISOperations._rest_op https://review.openstack.org/165718 | 06:54 |
*** yuanying has joined #openstack-ironic | 06:55 | |
*** coolsvap is now known as coolsvap_ | 07:09 | |
*** achanda has quit IRC | 07:17 | |
openstackgerrit | Merged stackforge/proliantutils: ILO: Add unit tests for ris.RISOperations._rest_op https://review.openstack.org/165718 | 07:20 |
*** Nisha has quit IRC | 07:26 | |
*** achanda has joined #openstack-ironic | 07:29 | |
*** saripurigopi has quit IRC | 07:34 | |
*** stendulker_ has quit IRC | 07:43 | |
GheRivero | morning | 07:43 |
*** chlong has quit IRC | 07:45 | |
*** saripurigopi has joined #openstack-ironic | 07:57 | |
*** jistr has joined #openstack-ironic | 07:58 | |
*** takadayuiko has joined #openstack-ironic | 07:59 | |
openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 07:59 |
ramineni | rameshg87: ^^ | 08:00 |
*** jcoufal has joined #openstack-ironic | 08:01 | |
*** dtantsur|afk is now known as dtantsur | 08:02 | |
dtantsur | Morning, TGIF! | 08:03 |
takadayuiko | Good evening, Ironic :) | 08:04 |
dtantsur | takadayuiko, o/ | 08:09 |
*** oomichi has quit IRC | 08:12 | |
takadayuiko | dtantsur, Hi! I'm fixing my patch, wait please... | 08:12 |
dtantsur | which one? :) | 08:13 |
*** viktors|afk is now known as viktors | 08:13 | |
takadayuiko | dtantsur, supporting generating conf file. https://review.openstack.org/#/c/163281/ | 08:13 |
dtantsur | takadayuiko, cool, thanks. did you come up with upgrade path from previous keystone configuration? | 08:14 |
dtantsur | and btw, are you working on https://bugs.launchpad.net/bugs/1401579 you're assigned to? | 08:19 |
openstack | Launchpad bug 1401579 in ironic-discoverd "Missing unit tests for main, client and node_cache" [Low,Triaged] - Assigned to Yuiko Takada (takada-yuiko) | 08:19 |
*** ndipanov has joined #openstack-ironic | 08:20 | |
*** athomas has joined #openstack-ironic | 08:20 | |
*** openstackgerrit has quit IRC | 08:22 | |
*** openstackgerrit has joined #openstack-ironic | 08:22 | |
*** achanda has quit IRC | 08:23 | |
*** achanda has joined #openstack-ironic | 08:27 | |
*** lsmola has quit IRC | 08:28 | |
rameshg87 | ramineni: looking | 08:31 |
rameshg87 | dtantsur: takadayuiko: o/ | 08:31 |
takadayuiko | dtantsur, Yes, I will delete some existing values like os_auth_url and change to ketstone middleware's value | 08:34 |
takadayuiko | about https://bugs.launchpad.net/bugs/1401579 I have not done, but I'll do from the next week! | 08:34 |
openstack | Launchpad bug 1401579 in ironic-discoverd "Missing unit tests for main, client and node_cache" [Low,Triaged] - Assigned to Yuiko Takada (takada-yuiko) | 08:34 |
*** ifarkas has joined #openstack-ironic | 08:34 | |
takadayuiko | rameshg87, o/ | 08:35 |
dtantsur | takadayuiko, what do you mean by "delete existing values"? installations using old values in the configuration file should continue to work, there is some tooling in oslo.config to deprecate options btw | 08:35 |
dtantsur | takadayuiko, re timing: I'm going to come up with FF and release dates today. FF will probably be next week. by that time oslo.config migration should be done to be included in the current release :) | 08:36 |
takadayuiko | dtantsur, Ah! I should not delete values! mmm | 08:36 |
dtantsur | takadayuiko, see deprecated_group and deprecated_opts in http://docs.openstack.org/developer/oslo.config/opts.html#oslo_config.cfg.Opt | 08:37 |
dtantsur | not sure if it's documented properly anywhere, but it should allow you to use both new and old options (though not sure how it's going to plan with external libs) | 08:38 |
dtantsur | deprecated_for_removal argument might help too... | 08:38 |
dtantsur | takadayuiko, I think we still might land it in this release, but it needs some upgrade story :) thanks | 08:39 |
* dtantsur now has to run, sorry. see you later. | 08:41 | |
*** dtantsur is now known as dtantsur|bbl | 08:41 | |
*** ramineni has quit IRC | 08:42 | |
takadayuiko | dtantsur OK, deprecated_group and deprecated_opts. I will study from now :) | 08:43 |
*** lsmola has joined #openstack-ironic | 08:44 | |
*** saripurigopi has quit IRC | 08:45 | |
*** achanda has quit IRC | 08:50 | |
*** pas-ha has joined #openstack-ironic | 08:54 | |
*** rsalevsky has joined #openstack-ironic | 08:54 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add whole disk image support for iscsi_ilo using agent ramdisk https://review.openstack.org/165847 | 08:55 |
rameshg87 | yuriyz: need your +2 again after rebase if this looks okay - https://review.openstack.org/#/c/143919/22. please have a look | 08:57 |
yuriyz | hello rameshg87 ok | 08:58 |
*** andreykurilin_ has joined #openstack-ironic | 08:58 | |
rameshg87 | yuriyz: thanks :) | 08:59 |
*** bauzas is now known as bauwser | 09:00 | |
*** lazy_prince is now known as killer_prince | 09:00 | |
*** andreykurilin_ has quit IRC | 09:03 | |
*** andreykurilin_ has joined #openstack-ironic | 09:04 | |
*** chenglch has quit IRC | 09:08 | |
*** andreykurilin__ has joined #openstack-ironic | 09:08 | |
*** andreykurilin_ has quit IRC | 09:09 | |
*** lucasagomes has joined #openstack-ironic | 09:13 | |
*** jamielennox|away is now known as jamielennox | 09:14 | |
rameshg87 | lucasagomes: hello, when you get time, need your +2 again on https://review.openstack.org/#/c/143919/ after rebase .. | 09:20 |
*** andreykurilin__ has quit IRC | 09:27 | |
*** andreykurilin_ has joined #openstack-ironic | 09:27 | |
*** derekh has joined #openstack-ironic | 09:28 | |
lucasagomes | rameshg87, right.... you know if we are approving code? It seems to be a bug fix | 09:28 |
lucasagomes | I think it's fine, but wanted to ask first | 09:28 |
*** tiagogomes_ has joined #openstack-ironic | 09:28 | |
*** pradipta has quit IRC | 09:30 | |
rameshg87 | lucasagomes: yeah, even i am not sure :) | 09:30 |
lucasagomes | ok | 09:30 |
rameshg87 | lucasagomes: kilo-3 has been released | 09:31 |
lucasagomes | right yeah I think we are good | 09:31 |
rameshg87 | lucasagomes: so i assume rc-1 has started which is a point where bugs can be merged ? | 09:31 |
lucasagomes | rameshg87, that seems correct | 09:31 |
lucasagomes | for safety I will just +2 and approve when I'm sure of it | 09:31 |
rameshg87 | yeah sure .. | 09:32 |
rameshg87 | thanks | 09:32 |
rameshg87 | lucasagomes: at some point of time we have to move the approved specs from kilo directory to liberty, right ? | 09:39 |
rameshg87 | lucasagomes: i hope all of them will be automatically approved for liberty :) | 09:39 |
lucasagomes | rameshg87, I believe they will have to be resubmitted as liberty (yes that's sorta of "moving") | 09:40 |
lucasagomes | but that's the author that should do | 09:40 |
rameshg87 | lucasagomes: oh okay .. | 09:41 |
* rameshg87 leaves for home | 09:43 | |
*** rameshg87 has quit IRC | 09:43 | |
*** andreykurilin_ has quit IRC | 09:43 | |
* lucasagomes is trying to use google inbox, it's a bit confusing | 10:00 | |
tiagogomes_ | Hi, I am trying to use ironic as a standalone service, but when I run `node-validate` I get a "Remote error: HTTPNotFound HTTPNotFound (HTTP 404)" error. This how the node is configured: http://paste.baserock.org/owopupimac | 10:03 |
tiagogomes_ | This happens with the stable/juno branch | 10:05 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: add retry capability to wsman client operations https://review.openstack.org/165860 | 10:11 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: config options for retry values https://review.openstack.org/166156 | 10:11 |
lucasagomes | tiagogomes_, hmm I'm not sure whether Juno supports non-glance image | 10:13 |
lucasagomes | for the pxe_deploy_{ramdisk,kernel} | 10:13 |
lucasagomes | was it backported? I don't think so | 10:13 |
*** MattMan has quit IRC | 10:14 | |
tiagogomes_ | lucasagomes, then the documentation is wrong: http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 10:15 |
tiagogomes_ | http://docs.openstack.org/developer/ironic/deploy/install-guide.html#using-ironic-as-a-standalone-service | 10:15 |
lucasagomes | tiagogomes_, right... confusing... usually the docs reflect to the latest version. Perhaps a note saying it's not supported on versions < kilo | 10:17 |
lucasagomes | lemme check if it was backported | 10:17 |
lucasagomes | yeah it wasn't backported | 10:19 |
lucasagomes | tiagogomes_, I'm afraid you'll need glance for juno :( | 10:20 |
*** MattMan has joined #openstack-ironic | 10:20 | |
tiagogomes_ | lucasagomes, np, the end goal was always doing it with glance. I was trying to use it standalone first to isolate possible integration issues. I'll try with glance. Thanks | 10:21 |
lucasagomes | tiagogomes_, cool np | 10:21 |
lucasagomes | tiagogomes_, I will update the documentation to make it clear as well that it won't work on previous versions | 10:21 |
lucasagomes | sorry for that | 10:21 |
tiagogomes_ | lucasagomes no worries | 10:22 |
lucasagomes | tiagogomes_, br? | 10:23 |
tiagogomes_ | lucasagomes pt :) | 10:23 |
lucasagomes | gotcha :) | 10:23 |
lucasagomes | salve ^_^ | 10:24 |
*** saripurigopi has joined #openstack-ironic | 10:25 | |
*** mgoddard has joined #openstack-ironic | 10:37 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP server in PXE driver https://review.openstack.org/159819 | 10:39 |
*** saripurigopi_ has joined #openstack-ironic | 10:44 | |
*** saripurigopi has quit IRC | 10:46 | |
*** saripurigopi_ is now known as saripurigopi | 10:46 | |
*** yog_ has quit IRC | 10:47 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update "Ironic as a standalone service" documentation https://review.openstack.org/166163 | 10:52 |
*** killer_prince is now known as lazy_prince | 10:54 | |
*** enikanorov has quit IRC | 10:54 | |
*** enikanorov has joined #openstack-ironic | 10:55 | |
*** yuanying has quit IRC | 11:02 | |
*** enikanorov has quit IRC | 11:04 | |
*** enikanorov has joined #openstack-ironic | 11:05 | |
*** enikanorov has quit IRC | 11:06 | |
*** enikanorov has joined #openstack-ironic | 11:07 | |
*** pradipta has joined #openstack-ironic | 11:14 | |
*** jcoufal has quit IRC | 11:16 | |
*** athomas has quit IRC | 11:21 | |
*** enikanorov has quit IRC | 11:26 | |
*** athomas has joined #openstack-ironic | 11:26 | |
*** enikanorov has joined #openstack-ironic | 11:27 | |
*** enikanorov has quit IRC | 11:30 | |
*** enikanorov has joined #openstack-ironic | 11:31 | |
*** enikanorov has quit IRC | 11:39 | |
*** enikanorov has joined #openstack-ironic | 11:40 | |
*** dtantsur|bbl is now known as dtantsur | 11:41 | |
*** dprince has quit IRC | 11:43 | |
dtantsur | yuriyz, hi! I have a gut feeling that feature freeze applies to https://review.openstack.org/#/c/159819/ it's kind of bug fix, but looks pretty dangerous. wdyt? | 11:44 |
*** rameshg87 has joined #openstack-ironic | 11:44 | |
yuriyz | hello dtantsur yes I marked this as workinprogress, do not merge | 11:46 |
dtantsur | ack | 11:46 |
rameshg87 | lucasagomes and other folks - i made some changes in devstack for supporting ilo drivers - https://review.openstack.org/#/c/156887/3 | 11:49 |
rameshg87 | let me know your thoughts on this | 11:50 |
rameshg87 | wanted to start a ci for ilo driver(s) | 11:50 |
*** Marga_ has joined #openstack-ironic | 11:54 | |
*** pelix has joined #openstack-ironic | 11:59 | |
lucasagomes | will take a look later, but sounds good :) | 11:59 |
rameshg87 | sure..thanks | 12:00 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add whole disk image support for iscsi_ilo using agent ramdisk https://review.openstack.org/165847 | 12:00 |
rameshg87 | vdrok_afk: ^^^ | 12:01 |
tiagogomes_ | hi, I bumped in another error now when running node-validate: ''NoneType' object has no attribute 'GlanceImageService' (HTTP 500)'. Conductor's log: http://paste.baserock.org/ibiqoxaqex | 12:09 |
TheJulia | good morning | 12:10 |
*** dprince has joined #openstack-ironic | 12:11 | |
tiagogomes_ | `glance image-list` is working fine | 12:11 |
rameshg87 | tiagogomes_: which ironic are you using ? | 12:12 |
rameshg87 | tiagogomes_: that part of the code was recently modified in ironic-k3 | 12:12 |
tiagogomes_ | rameshg87 stable/june | 12:12 |
rameshg87 | tiagogomes_: oh okay | 12:12 |
* rameshg87 checks stable/juno | 12:13 | |
*** stendulker has joined #openstack-ironic | 12:15 | |
*** pradipta has quit IRC | 12:15 | |
dtantsur | TheJulia, morning | 12:19 |
rameshg87 | tiagogomes_: from the code i don't see a reason why it would fail with that error | 12:19 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Intrduction to grub2 bootloader for uefi boot mode https://review.openstack.org/166192 | 12:20 |
rameshg87 | tiagogomes_: if you have conductor logs (conductor logs above this in paste would also help) and steps to reproduce, i think you can file a bug | 12:20 |
rameshg87 | tiagogomes_: and i hope settings in glance_host, glance_port, glance_protocol, glance_api_servers, glance_api_insecure, glance_num_retries, auth_strategy anything if specified are correct | 12:21 |
rameshg87 | i mean in /etc/ironic/ironic.conf | 12:22 |
tiagogomes_ | rameshg87 ok, thanks for looking, I'll double check the configuration | 12:22 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: grub2 bootloader support for uefi boot mode https://review.openstack.org/166192 | 12:22 |
*** rameshg87 is now known as rameshg87-away | 12:25 | |
*** rameshg87-away is now known as rameshg87 | 12:36 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add supporting generate config files https://review.openstack.org/163281 | 12:40 |
*** lucasagomes is now known as lucas-hungry | 12:42 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 12:43 |
*** takadayuiko has quit IRC | 12:46 | |
devananda | g'morning, all | 12:47 |
dtantsur | devananda, morning | 12:47 |
lucas-hungry | devananda, morning | 12:48 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 12:48 |
* lucas-hungry now goes for lunch | 12:48 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 12:49 |
dtantsur | lucas-hungry can't go lunching without proper config driver docs :D | 12:50 |
*** pelix has quit IRC | 12:53 | |
*** ChuckC_ has joined #openstack-ironic | 13:00 | |
*** ChuckC has quit IRC | 13:01 | |
*** bradjones has joined #openstack-ironic | 13:01 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 13:04 |
*** ChuckC_ has quit IRC | 13:05 | |
tiagogomes_ | rameshg87 sorted now, I didn't have pysendfile installed | 13:06 |
rameshg87 | tiagogomes_: oh | 13:07 |
rameshg87 | tiagogomes_: that's the wierdest error message you can get for that :( | 13:08 |
*** enikanorov has quit IRC | 13:09 | |
*** enikanorov has joined #openstack-ironic | 13:11 | |
rameshg87 | anyway wants to see something strange - http://logs.openstack.org/56/166056/1/gate/gate-ironic-python27/574031a/console.html | 13:11 |
rameshg87 | a test that was merged yesterday is failing now on the gate pipeline (not in the check) | 13:11 |
*** trown|outttypeww is now known as trown | 13:13 | |
rameshg87 | anyone see some problem with this test case : https://github.com/openstack/ironic/blob/master/ironic/tests/drivers/ilo/test_deploy.py#L664-L675 ? | 13:15 |
rameshg87 | it was merged yesterday, but is failing now in gate pipeline on http://logs.openstack.org/56/166056/1/gate/gate-ironic-python27/574031a/console.html | 13:15 |
rameshg87 | it passes locally for me | 13:15 |
*** stendulker has quit IRC | 13:15 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP server in PXE driver https://review.openstack.org/159819 | 13:16 |
* rameshg87 is going to wait until someone has look at it | 13:18 | |
*** ijw has joined #openstack-ironic | 13:18 | |
*** ChuckC_ has joined #openstack-ironic | 13:19 | |
*** ijw has quit IRC | 13:20 | |
dtantsur | rameshg87, it passes locally for me too. I'll check with the patch now | 13:20 |
*** ijw has joined #openstack-ironic | 13:20 | |
*** absubram has joined #openstack-ironic | 13:21 | |
*** ijw has joined #openstack-ironic | 13:21 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Hide new fields via single method https://review.openstack.org/166056 | 13:21 |
dtantsur | rameshg87, attempt to rebase ^^^ | 13:22 |
rameshg87 | dtantsur: yeah okay | 13:22 |
rameshg87 | dtantsur: i see no reason it should fail like that there :) | 13:22 |
*** kkoski has joined #openstack-ironic | 13:24 | |
*** JoshNang has quit IRC | 13:24 | |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: eDeploy: Store all of the facts collected by edeploy in Ironic DB https://review.openstack.org/165538 | 13:26 |
*** JoshNang has joined #openstack-ironic | 13:26 | |
devananda | dtantsur: that's a strange failure indeed | 13:26 |
*** Marga_ has quit IRC | 13:26 | |
dtantsur | devananda, yeah... could we break something in the middle of git log? | 13:27 |
* dtantsur is checking | 13:27 | |
devananda | it failed in a py27 unit test. it's possible to have race conditions there, yes | 13:27 |
devananda | eg, where different test threads interfere with each other | 13:28 |
dtantsur | hmm yeah, maybe | 13:28 |
rameshg87 | devananda: values can go wrong | 13:28 |
rameshg87 | devananda: but it was an assertion whether a mocked method was called | 13:28 |
rameshg87 | devananda: can it go wrong ? if it has, it has to do something with the mock library itself i think | 13:29 |
rameshg87 | devananda: and i had one more question | 13:32 |
rameshg87 | devananda: i have 2 patches with 2*+2s but not approved - one is a nit fix (https://review.openstack.org/#/c/165297/) and other is a bug (https://review.openstack.org/#/c/143919/) | 13:32 |
*** mgoddard has quit IRC | 13:32 | |
rameshg87 | devananda: lucas-hungry wasn't sure if it's open to submit those. are they open now since k3 is released ? | 13:33 |
devananda | rameshg87: it could be an issue in the ironic/common/dhcp_factory module, such that the provider (NeutronDHCPApi) was not initialized, and so the mock wasn't called | 13:33 |
* devananda looks | 13:33 | |
rameshg87 | oh | 13:34 |
devananda | that said, dhcp_factory is using a semaphore during __init__ to protect against two callers accessing it prior to initialization | 13:34 |
devananda | so it SHOULD be safe ... | 13:35 |
devananda | I tried to use logstash to search for more instances of this error cropping up -- if we see it again, we need to investigate :-/ | 13:35 |
devananda | but logstash didn't seem to match it at all | 13:35 |
*** jamielennox is now known as jamielennox|away | 13:36 | |
devananda | rameshg87: 165297 is just doc string fixes and a variable rename, right? | 13:37 |
rameshg87 | devananda: yes | 13:37 |
* rameshg87 confirms | 13:37 | |
rameshg87 | devananda: yes it is | 13:37 |
devananda | that's exactly the sort of thing we should do during RC :) | 13:38 |
rameshg87 | devananda: :) | 13:38 |
rameshg87 | devananda: but this test is from code that merged yesterday | 13:38 |
devananda | ok... | 13:39 |
rameshg87 | devananda: i am not sure we started initializing dhcp_factory too many times in our code before | 13:39 |
rameshg87 | devananda: iirc it was only at one place before | 13:39 |
*** wuhg has quit IRC | 13:39 | |
devananda | right. so we could be hitting a race we never exposed before :( | 13:39 |
rameshg87 | devananda: now it's in 3 places (or more) | 13:39 |
rameshg87 | devananda: yeah | 13:39 |
devananda | rameshg87: so 143919 is changing translated strings, and I dont think it needs to | 13:41 |
rameshg87 | devananda: yeah so i can throw in a patch without the change for string | 13:42 |
rameshg87 | devananda: and would it be fine ? i will just add a todo above so that someone notices it later and fixes it in L1 | 13:42 |
*** saripurigopi has quit IRC | 13:43 | |
devananda | yup | 13:43 |
*** kkoski has quit IRC | 13:43 | |
rameshg87 | okay | 13:43 |
devananda | you could even propose that change in a second patch and -W it yourself until then :) | 13:43 |
devananda | either way, the patch looks fine to me aside from that | 13:44 |
*** lucas-hungry is now known as lucasagomes | 13:45 | |
lucasagomes | dtantsur, lol yeah | 13:45 |
rameshg87 | devananda: that's better i propose one and will -W it. thanks | 13:46 |
*** mtanino has joined #openstack-ironic | 13:50 | |
*** kkoski has joined #openstack-ironic | 13:50 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Fail deploy if root uuid or disk id isn't available https://review.openstack.org/143919 | 13:52 |
rameshg87 | devananda: lucasagomes: without translation string change ^^^ | 13:52 |
*** rameshg87 is now known as rameshg87-brb | 13:53 | |
lucasagomes | rameshg87-brb, thanks | 13:53 |
lucasagomes | devananda, that bug is very annoying btw, I think we should get it in the RC | 13:53 |
devananda | rameshg87-brb: urgh. please do not rebase when fixing something small in a patch :( | 13:54 |
devananda | rameshg87-brb: the diff between rev22 and rev23 is now quite large, which makes the re-review harder | 13:54 |
*** raginbajin has joined #openstack-ironic | 13:55 | |
lucasagomes | yeah i saw the rebase on the tests | 13:55 |
lucasagomes | but seems good | 13:55 |
devananda | either i'm reading wrong (which is possible) or something went sideways with the rebase | 13:56 |
*** alexpilotti has joined #openstack-ironic | 13:56 | |
* lucasagomes re-looks | 13:57 | |
*** rameshg87-brb is now known as rameshg87 | 13:57 | |
rameshg87 | devananda: okay. but normally before raising any patch for review, i make sure i apply it on the HEAD of origin/master and then review | 13:57 |
devananda | rameshg87: that's not necessarily the most helpful thing | 13:58 |
rameshg87 | devananda: isn't it necessary to make sure that some other change (that didn't merge conflict with me) didn't break my change ? | 13:58 |
rameshg87 | devananda: if it did, i can get to know when my unit test fail locally | 13:58 |
devananda | rameshg87: you should be sure it can be rebased on HEAD. however, reviewers that have reviewed a previous version of the patch will often compare between revisions | 13:58 |
rameshg87 | devananda: okay got it | 13:58 |
devananda | just to see what you changed between eg, rev22 -> 23, | 13:59 |
rameshg87 | devananda: test it locally on HEAD, my don't rebase on HEAD | 13:59 |
rameshg87 | right ? | 13:59 |
lucasagomes | 22 and 23 does the same, if you compare with base | 13:59 |
*** MattMan has left #openstack-ironic | 13:59 | |
lucasagomes | but the diff on 22 and 23 includes the rebase differences | 13:59 |
* lucasagomes is ok with that | 14:00 | |
NobodyCam | good morning and ofc..... TGIF!!!!! | 14:00 |
lucasagomes | NobodyCam, morning! TGIF indeed | 14:00 |
rameshg87 | lucasagomes: yeah on the files that i didn't change :( | 14:00 |
lucasagomes | rameshg87, it's grand | 14:00 |
NobodyCam | moorning lucasagomes rameshg87 and devananda :) | 14:00 |
rameshg87 | NobodyCam: morning | 14:01 |
NobodyCam | :) | 14:01 |
rameshg87 | devananda: one question | 14:01 |
rameshg87 | devananda: so if i didn't wouldn't gerrit automatically try to apply on HEAD of origin/master ? | 14:02 |
rameshg87 | devananda: i mean if i didn't rebase on HEAD, wouldn't gerrit automatically try to apply on HEAD of origin/master ? | 14:02 |
devananda | rameshg87: when you git review, you can use "-R" flag to tell gerrit do not rebase | 14:02 |
* rameshg87 thought git review just sends the patch to the gerrit server | 14:02 | |
devananda | rameshg87: when jenkins tests the change, it does the rebase in the test env | 14:02 |
rameshg87 | oh okay | 14:03 |
devananda | rameshg87: git review can also pull changes from gerrit | 14:03 |
devananda | anyway, i diff'd the patches for rev 22 and 23 and yea, everyhting's fine | 14:03 |
rameshg87 | devananda: yeah i see option now | 14:04 |
rameshg87 | devananda: man page says "This may confuse many reviewers that would expect to see a much simpler difference." | 14:04 |
rameshg87 | :) | 14:04 |
devananda | hehehe | 14:04 |
lucasagomes | I'm wondering if I should create a script that generates a config drive (gzipped + base64'd) from a directory on the filesystem | 14:04 |
devananda | TheJulia: ^ | 14:04 |
lucasagomes | so operators can use that to generate a config drive when using Ironic stand alone | 14:04 |
rameshg87 | lucasagomes: +1 | 14:04 |
devananda | lucasagomes: i believe we already did ... | 14:04 |
lucasagomes | sounds like a good idea instead of descibring the whole process | 14:04 |
rameshg87 | lucasagomes: i do that manually today | 14:04 |
lucasagomes | devananda, oh | 14:04 |
rameshg87 | lucasagomes: with nocloud configuration file | 14:05 |
devananda | hum... we should probably announce bifrost to the ML | 14:05 |
rameshg87 | lucasagomes: i mean nocloud image file | 14:05 |
lucasagomes | yeah if you guys have one, that would be aewsome to put under /tools | 14:05 |
lucasagomes | rameshg87, I see | 14:05 |
dtantsur | NobodyCam, such morning much TGIF wow :) | 14:05 |
lucasagomes | I something here, that I used when I was starting doing the work, but I need to polish it | 14:05 |
* rameshg87 doesn't have one | 14:06 | |
rameshg87 | lucasagomes: i created long back and kept only the gzipped-base64-encoded file with me in a shell script | 14:06 |
NobodyCam | :) morning dtantsur :) | 14:06 |
lucasagomes | rameshg87, gotcha, yeah I have some base code for that | 14:07 |
openstackgerrit | Merged stackforge/ironic-discoverd: eDeploy: Store all of the facts collected by edeploy in Ironic DB https://review.openstack.org/165538 | 14:08 |
lucasagomes | rameshg87, devananda http://paste.openstack.org/show/193935/ | 14:08 |
lucasagomes | I will polish it | 14:08 |
lucasagomes | make a proper script | 14:08 |
devananda | lucasagomes: https://github.com/juliakreger/bifrost/blob/master/roles/bifrost-configdrives/tasks/main.yml | 14:09 |
lucasagomes | devananda, oh cool | 14:10 |
devananda | lucasagomes: https://github.com/juliakreger/bifrost < the readme here is useful, the per-task readme's need work | 14:11 |
rameshg87 | lucasagomes: in http://paste.openstack.org/show/193935/, it should be vfat, right ? | 14:12 |
devananda | lucasagomes: short version - stand alone ironic with ansible. we're still working on it, obviously, but plan to propose it to the project once it's got CI and such | 14:13 |
rameshg87 | lucasagomes: if we write an iso image into a partition in bios partition table, will it work ? | 14:13 |
lucasagomes | devananda, I wonder if having a script in ironic under tools/ might help with that too, so in the ansible playbook you can just ./make-configdrive --output httpdir/config.iso /dir/with-configdrive-files | 14:13 |
devananda | lucasagomes: it certainly might | 14:13 |
lucasagomes | rameshg87, it does, because we create a hybrid ISO | 14:14 |
NobodyCam | lucasagomes: I think that is a very good idea | 14:14 |
rameshg87 | lucasagomes: okay | 14:14 |
lucasagomes | rameshg87, so you can dd it onto a device | 14:14 |
lucasagomes | like the normal distros ISOs today were you can DD it to ur USB | 14:14 |
rameshg87 | okay | 14:14 |
lucasagomes | and boot from it | 14:14 |
rameshg87 | got it | 14:14 |
lucasagomes | devananda, NobodyCam rameshg87 ok, will work on a quick script for it then | 14:15 |
* rameshg87 will take leave earlier today | 14:15 | |
rameshg87 | good night all | 14:15 |
*** rameshg87 has quit IRC | 14:15 | |
NobodyCam | night rameshg87 | 14:16 |
lucasagomes | the ansible thing looks good :) | 14:17 |
lucasagomes | go bifrost! | 14:17 |
NobodyCam | ya! | 14:17 |
* lucasagomes was reading the README | 14:17 | |
NobodyCam | lucasagomes: ya its really quite kewl! | 14:18 |
devananda | lucasagomes: we've been talking about stand-alone ironic for a while ... :) | 14:19 |
*** BadCub_Hiding is now known as BadCub | 14:19 | |
BadCub | morning | 14:19 |
NobodyCam | and making it happen:) | 14:19 |
NobodyCam | morning BadCub :) | 14:20 |
lucasagomes | devananda, indeed | 14:20 |
lucasagomes | BadCub, morning | 14:20 |
BadCub | morning lucasagomes , devananda, NobodyCam | 14:20 |
devananda | lucasagomes: one of the things i want to talk with everyone about at the summit is moving ironic to have stand-alone functional testing | 14:24 |
devananda | lucasagomes: and i think bifrost has the potential to do that for us quite soon | 14:24 |
lucasagomes | devananda, I'm 100% with it. I like the idea of Ironic being able to be used as a tool standalone and I've seems enough people wanting to use it that way (reminds the mid-cycle at intel) | 14:26 |
lucasagomes | we are outgrowing openstack, and I think it's good | 14:26 |
lucasagomes | and bifrost yeah, it seems it's ahead of any other tool we have to do that | 14:27 |
lucasagomes | so +1 | 14:27 |
devananda | i'd say ironic is useful both in and outside of openstack. the tent is big :) | 14:28 |
devananda | *of an openstack cloud | 14:28 |
lucasagomes | oh yeah | 14:28 |
* lucasagomes never remembers the syntax to use argparse in python >.< | 14:29 | |
devananda | oh, monday i wont be able to make the meeting | 14:30 |
NobodyCam | :) yep! we gots ya coveder! | 14:31 |
devananda | k k | 14:31 |
*** mgoddard has joined #openstack-ironic | 14:31 | |
NobodyCam | :) | 14:31 |
lucasagomes | yeah I will be present too | 14:31 |
NobodyCam | (/me assigns all things to devananda) | 14:31 |
NobodyCam | hehehehe | 14:31 |
NobodyCam | :-p | 14:31 |
devananda | I'm adding links to the status pads on the agenda | 14:32 |
NobodyCam | Thank you :) | 14:32 |
devananda | so they get shared and discussed -- and i imagine we'll keep tracking on them for several weeks | 14:32 |
* BadCub needs larger coffee intake before engaging in talking | 14:32 | |
dtantsur | anyone has idea what this is and how it works: https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L500-L535 ? | 14:33 |
Shrews | dtantsur: sort of | 14:34 |
dtantsur | I'm trying to setup discoverd under devstack and for now it looks like there's no network connection between vms and the host (which is doubtfully true) | 14:34 |
dtantsur | Shrews, my final goal is to have my own dnsmasq listening for requests from vms | 14:34 |
Shrews | dtantsur: the VMs are connected to a bridge and that bridge needs to be connected to the network that is created by neutron | 14:34 |
Shrews | dtantsur: oh, i just did that in bifrost, but we don't use neutron there | 14:35 |
dtantsur | Shrews, I suspect that brbm is where I want dnsmasq to listen, but it fails as brbm has now address... | 14:35 |
jlvillal | lucasagomes: A little late in responding. But glad you think vim-fugitive looks interesting. My favorite feature is 'Gblame' and then using '~' or '^' to walk back in-time | 14:36 |
Shrews | dtantsur: yeah, i think the bridge must be connected to a real interface (e.g., eth0) for that to work. you'll need to attach dnsmasq to the neutron created one | 14:36 |
Shrews | the taps will connect the VM bridge to the neutron one | 14:37 |
dtantsur | Shrews, which is it? br-ex? it does not receive dnsmasq queries... | 14:37 |
Shrews | dtantsur: i think it's the one identified by this bit of code: https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L502-L507 | 14:38 |
Shrews | dtantsur: oh, wait... maybe just br-int | 14:38 |
dtantsur | br-int? | 14:38 |
* dtantsur is trying | 14:38 | |
dtantsur | Shrews, this device is not even up on my devstack... | 14:39 |
Shrews | dtantsur: you have killed all of the libvirt managed dnsmasq processes, yes? | 14:39 |
*** gabrielbezerra has quit IRC | 14:40 | |
dtantsur | Shrews, honest answer is "I don't know" :) should I or should I not? | 14:40 |
dtantsur | I can rerun devstack from scratch, if it helps | 14:40 |
* Shrews thinking | 14:41 | |
tiagogomes_ | ~ # ironic node-set-provision-state $NODE_UUID active | 14:41 |
tiagogomes_ | RPC do_node_deploy called for 137ec78d-52aa-4c25-957e-1339d8d0abe9, but provision state is already error. (HTTP 500) | 14:41 |
tiagogomes_ | how do I clear out "error" in the provision_state | 14:41 |
Shrews | dtantsur: let me check my devstack for a minute... | 14:42 |
dtantsur | Shrews, yes please | 14:42 |
Shrews | dtantsur: honestly, i've never tried a devstack configuration where you used neutron but not for dhcp... intriguing setup :) | 14:43 |
devananda | adam_g: is there a way, on a python-ironicclient patch, to trigger the juno->kilo grenade job? | 14:43 |
devananda | dtantsur: dont you still need neutron for the instance dhcp?? | 14:44 |
BadCub | morning Shrews, dtantsur | 14:46 |
*** enikanorov has quit IRC | 14:47 | |
dtantsur | BadCub, morning | 14:48 |
*** enikanorov has joined #openstack-ironic | 14:48 | |
dtantsur | devananda, neutron does not support catch-all rules, so no, for inspection DHCP Neutron is not used.. | 14:49 |
devananda | dtantsur: right. i mean that you still need neutron to handle DHCP when instances are created | 14:49 |
dtantsur | devananda, yeah, so I can't jsut kill it with fire :D | 14:49 |
devananda | dtantsur: or are you developing / planning this to completely not use neutron for dhcp at all? | 14:49 |
dtantsur | no, and that's the problem | 14:50 |
devananda | k | 14:50 |
devananda | dtantsur: fwiw, it is possible to use ironic without neutron or dhcp (in a single, trusted tenant environment) | 14:50 |
dtantsur | I want to leave neutron and somehow coexist with it's crazy devstack configuration | 14:50 |
devananda | dtantsur: and i would like to be able to use discoverd with bifrost | 14:50 |
dtantsur | devananda, with our tripleo-based thingy we're pretty much coexist with neutron on both vms and bare metal | 14:51 |
dtantsur | devananda, oh nice to hear! | 14:51 |
devananda | dtantsur: eg, have a single-node ironic only (no nova or neutron) that can do discovery | 14:51 |
devananda | and provisioning | 14:51 |
dtantsur | interesting. should not be hard IMO | 14:51 |
*** gabrielbezerra has joined #openstack-ironic | 14:52 | |
devananda | this would only be for environments that have single tenant, do not need nova scheduling, flavors, etc... | 14:52 |
dtantsur | pretty reasonable use case actually | 14:53 |
devananda | cool | 14:53 |
dtantsur | devananda, I still wonder what will need from discoverd to support long-running ramdisks. WHat is on top of my head is: 1. option not to power off after inspection; 2. return node UUID to the ramdisk | 14:55 |
jroll | morning all :) | 14:55 |
dtantsur | jroll, morning | 14:55 |
NobodyCam | morning jroll :) and ofc TGIF! | 14:56 |
BadCub | mornng jroll | 14:56 |
jroll | tgif indeed :) | 14:56 |
jroll | devananda: thinking about hacking on bifrost today -- anything I should know that isn't in the readme? | 14:56 |
BadCub | Friday means only 3 days till Monday *ducks and hides* LOL | 14:57 |
devananda | jroll: oooh. awesome. um... good quetion for TheJulia, too | 14:57 |
* devananda throws a wet cat at BadCub | 14:57 | |
jroll | devananda: also... can I make it use dhcp for the instance network? | 14:57 |
* BadCub throws a soggy hippo at devananda lol | 14:57 | |
* dtantsur is wondering about this new bifrost thingy | 14:57 | |
* jroll going to need to find some computers and an image | 14:57 | |
jroll | dtantsur: https://github.com/juliakreger/bifrost | 14:58 |
NobodyCam | lol omg ... I needs more coffee | 14:58 |
jroll | devananda: DIB :( | 14:58 |
Shrews | dtantsur: so, looks like neutron uses dnsmasq within a network namespace (the qdhcp-* thing). It seems to connect to a tap device interface "tapSOMETHING". Not sure which interface you'd use for another dnsmasq process :( | 15:00 |
Shrews | or if it's even possible | 15:00 |
* Shrews is no help | 15:00 | |
Shrews | jroll: Cool. Intend to try bifrost with real h/w or VMs? | 15:01 |
jroll | hardware | 15:01 |
jroll | I want to set it up in our lab where I don't need the full stack | 15:01 |
jroll | thinking about maybe using it for CI or something eventually | 15:02 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 15:03 |
TheJulia | jroll: The only thing that has bitten us is really firewalling on various test machines and the networking drivers for hardware we've deployed to, which is actually why we switched over to DIB based images since it just seemed to "work" :) | 15:03 |
NobodyCam | morning TheJulia :) | 15:03 |
jroll | TheJulia: huh, ok. maybe I'll make a tag to not use DIB :) | 15:03 |
BadCub | mornin TheJulia | 15:03 |
jroll | since I know I have an agent image that works on these machines :P | 15:04 |
TheJulia | jroll: there are config flags for that already :) | 15:04 |
jroll | oh neat | 15:04 |
jroll | I assume you're taking pull requests btw? | 15:04 |
TheJulia | jroll: also to add a boot loader | 15:04 |
TheJulia | jroll: absoloutely | 15:04 |
jroll | cool | 15:04 |
jroll | already noticed some stuff to clean up... e.g. using the git/pip modules rather than shell commands | 15:05 |
TheJulia | lucasagomes: I need to split it out, but we did put in logic to build configdrives in bifrost | 15:06 |
lucasagomes | TheJulia, cool, no worries I got one script almost working now | 15:06 |
lucasagomes | will put it in Ironic | 15:06 |
TheJulia | I step away for a little while and you guys start talking about bifrost like crazy. What great timing :) | 15:06 |
lucasagomes | so you guys can use it in bifrost too | 15:06 |
openstackgerrit | Merged openstack/ironic: Address nits in uefi agent iscsi deploy commit https://review.openstack.org/165297 | 15:06 |
jroll | TheJulia: today is a hackday for me :P | 15:06 |
*** lazy_prince is now known as killer_prince | 15:08 | |
TheJulia | jroll: we went the shell command route partially because for a while we were pulling in patches that had not yet landed, so all of that can be refactored now :) | 15:08 |
jroll | TheJulia: right on | 15:08 |
TheJulia | I suspect my next thing might be agent_ilo | 15:10 |
*** mjturek1 has joined #openstack-ironic | 15:11 | |
*** openstackgerrit has quit IRC | 15:21 | |
*** openstackgerrit has joined #openstack-ironic | 15:22 | |
NobodyCam | btw, If you have not heard me say it. AWESOME effort this week EVERYONE!!! :) | 15:24 |
dtantsur | \o/ | 15:25 |
NobodyCam | :) | 15:25 |
NobodyCam | wow what a job title: "Happiness Ambassador" | 15:27 |
dtantsur | awesome | 15:27 |
NobodyCam | that a real title for someone on linkedIn | 15:28 |
devananda | jroll: you probably could. right now the IP addresses are statically assigned and injected via configdrive | 15:28 |
jroll | devananda: yeah, I might try, or I might just use static | 15:28 |
jroll | the dhcp configs in this environment are static anyway | 15:29 |
devananda | jroll: one of my goals with bifrost is to remain as independent as possible from any infrastructure requirements as possible | 15:29 |
devananda | jroll: eg, I'd really like it not to use DHCP at all | 15:29 |
devananda | so that we can use ironic to image machines in an environment where we don't control DHCP | 15:30 |
jroll | devananda: sure. I think supporting DHCP would be ok, though, having a DHCP server around isn't too insane | 15:30 |
jroll | I agree it should support not dhcp | 15:30 |
devananda | :) | 15:30 |
jroll | but when you need to pxe boot things anyway, it's not insane to assume there's a dhcp server :P | 15:30 |
TheJulia | Supporting dhcp would really be as simple as just turning off the config drive.... or making a boolean that toggled IP address configuration in the configdrive. | 15:31 |
TheJulia | at least, from the standpoint of a deployed node | 15:31 |
devananda | TheJulia: unless we add a metadata service, we'll still need the config drive for anything beyond IP address, aiui | 15:31 |
jroll | right | 15:32 |
jroll | I think you could inject "dhcp this interface" via configdrive | 15:32 |
devananda | sure | 15:32 |
jroll | maybe do that if there's no IP in the csv | 15:32 |
devananda | in my mind, that path leads to needing to route 169.254.169.254 for metadata, which quickly becomes impossible in many pre-existing environments | 15:33 |
devananda | *when trying to use this in .. | 15:33 |
TheJulia | I'm really not a fan of the CSV format, but it kind of just worked since thats what we had to start with. | 15:33 |
jroll | I don't love the CSV either | 15:34 |
devananda | not a fan either | 15:34 |
* NobodyCam thinks of how he could embed message in an Ethernet frame with out ip on top of it :/ | 15:35 | |
TheJulia | NobodyCam: raw io controls | 15:35 |
NobodyCam | :) | 15:35 |
*** humble_ has joined #openstack-ironic | 15:35 | |
* TheJulia backs away from NobodyCam slowly | 15:35 | |
NobodyCam | hehehe | 15:35 |
* BadCub pats TheJulia on the back. It's okay.. NobodyCam can be scary at times LOL | 15:36 | |
NobodyCam | :-p | 15:36 |
devananda | NobodyCam: token ring | 15:37 |
TheJulia | Uhh... not afraid, more... don't want to do anything that has to deal with raw frames over ethernet | 15:37 |
NobodyCam | :) hehehe | 15:37 |
* TheJulia sees it now... nobodycam invents a new protocol to tunnel.... ethernet over ethernet over ethernet! | 15:37 | |
NobodyCam | lol | 15:38 |
*** lsmola has quit IRC | 15:38 | |
*** ndipanov has quit IRC | 15:38 | |
TheJulia | Wait, that would have to be ethernet over ethernet over ethernet over ethernet. | 15:38 |
BadCub | oh, don't give him that challenge. It never ends well for my sanity! LOL | 15:38 |
* TheJulia grins | 15:38 | |
NobodyCam | hehehe | 15:38 |
*** ndipanov has joined #openstack-ironic | 15:39 | |
* BadCub remembers an old PC disemboweled and all the components hung on the wall, fully functional and a 286 that did things it was never designed to do | 15:39 | |
*** ChuckC_ is now known as ChuckC | 15:39 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Documentation update https://review.openstack.org/165852 | 15:40 |
NobodyCam | lol, some one bet me I couldn't get a 286 to run NT back in the day | 15:40 |
NobodyCam | brb | 15:42 |
jroll | NobodyCam: http://www.secdev.org/projects/scapy/ | 15:43 |
*** lsmola has joined #openstack-ironic | 15:48 | |
* NobodyCam clicks | 15:51 | |
NobodyCam | hummmm | 15:51 |
*** Marga_ has joined #openstack-ironic | 15:52 | |
jroll | it's super cool | 15:56 |
NobodyCam | jroll: it looks like it. a little out of date but very cool | 16:00 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 16:01 |
lucasagomes | TheJulia, devananda ^ when you guys get some time | 16:02 |
lucasagomes | the script is there too | 16:02 |
* lucasagomes perhaps should update the commit message | 16:03 | |
NobodyCam | :-p | 16:03 |
devananda | lucasagomes: cheers | 16:03 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 16:05 |
* lucasagomes updated the commit message | 16:05 | |
*** lsmola has quit IRC | 16:05 | |
lucasagomes | for those looking for a tiny patch to approve on the documentation see https://review.openstack.org/#/c/166163/ | 16:12 |
lucasagomes | this just clarifies that standalone ironic without glance started in Kilo | 16:12 |
lucasagomes | so won't work on stable branches | 16:12 |
*** kbyrne has quit IRC | 16:18 | |
*** rsalevsky has quit IRC | 16:20 | |
* NobodyCam looks | 16:20 | |
NobodyCam | bam | 16:22 |
lucasagomes | the Kilo or Kilo realease? | 16:23 |
devananda | lucasagomes dtantsur NobodyCam - as a reminder, we're NOT running any functional testing of the API above 1.1 right now | 16:23 |
BadCub | "The Kilo" :-) | 16:23 |
devananda | so things like 166056 are completely untested, aside form unit tests | 16:23 |
NobodyCam | I'm ok with Kilo release | 16:23 |
lucasagomes | :( | 16:23 |
* lucasagomes thought only Kilo made sense too | 16:23 | |
BadCub | but I can go either way LOL | 16:23 |
lucasagomes | english is too hard folks, damn | 16:23 |
dtantsur | devananda, at least we're testing that we don't break what used to work... | 16:23 |
jroll | lol | 16:24 |
devananda | dtantsur: indeed | 16:25 |
NobodyCam | devananda: This is why I would like us (ironic) to talk about our third part testing requirments at the summit. I am starting to feel like we will need to start the process of enfourcing the as a requirment. as I don't have the local hardware to test many of the new cool features we landed this cycle | 16:28 |
NobodyCam | s/part/party/ | 16:30 |
devananda | features or drivers? | 16:30 |
devananda | things like discovery, cleaning, non-glance-image-ref, these can be tested in the gate upstream | 16:31 |
NobodyCam | well "driver features" like uefi | 16:31 |
devananda | they aren't - and they should be - and we should require that | 16:31 |
devananda | but driver things that require hardware. third-party | 16:31 |
NobodyCam | ya | 16:31 |
NobodyCam | and so far the only 3rd party test is IBM | 16:32 |
jlvillal | lucasagomes: You are not the only one who complains about English | 16:32 |
* BadCub supports 3rd party testing because he doesn't want racks of servers in the house | 16:32 | |
jlvillal | lucasagomes: My wife does too. She is Russian. She asks me why is it a certain way. I don't have good answer. I say it sounds better. She doesn't like my answer :) | 16:32 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Removed hardcoded IDs from "chassis" test resources https://review.openstack.org/133205 | 16:33 |
lucasagomes | jlvillal, lol. Yeah there's just too many ways to say the same thing | 16:36 |
lucasagomes | + the way you would say something in ur native language | 16:37 |
jlvillal | lucasagomes: English is like Perl ;) | 16:37 |
TheJulia | I could use a rack of servers in my house.... except they likely wouldn't be on generator unless I buy a larger generator. | 16:37 |
lucasagomes | like "I'm hungry" in english, in portuguese I would say somehting like "I have hungry" | 16:37 |
lucasagomes | jlvillal, lol yeah | 16:37 |
jlvillal | lucasagomes: :) Oh did you try vim-fugitive? I'm curious... | 16:38 |
BadCub | lucasagomes: and I would say "I am hungry" LOL | 16:38 |
lucasagomes | jlvillal, I haven't yet... lemme download it now | 16:38 |
jlvillal | lucasagomes: Gblame is the best part of it, IMHO | 16:38 |
NobodyCam | lol: funny friday image: http://data.daryo.net/assets/daryo.com/1kWG_Lj4L.jpg | 16:39 |
lucasagomes | BadCub, yeah, it's all different. In english it's like a state of being where in portuguese hunger is something you possess | 16:39 |
BadCub | NobodyCam: lol | 16:40 |
lucasagomes | jlvillal, will check that out in a sec! thakns for the tip again :) | 16:40 |
*** Marga_ has quit IRC | 16:40 | |
BadCub | lucasagomes: absolutely. I got in trouble in school as a kid because I typically speak and write in Queen's English versus American English. :( | 16:41 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Removed hardcoded IDs from "chassis" test resources https://review.openstack.org/133205 | 16:41 |
NobodyCam | I might say "I'm having hunger pangs, and need food" | 16:41 |
*** EmilienM is now known as EmilienM|afk | 16:41 | |
NobodyCam | it is wrong that I just found this page? https://wiki.openstack.org/wiki/Ironic/CoreTeam | 16:42 |
*** andreykurilin_ has joined #openstack-ironic | 16:42 | |
jroll | lol | 16:42 |
jroll | NobodyCam: that's pretty new | 16:42 |
TheJulia | BadCub: Why is it not surprising to hear that you've always been a troublemaker.... | 16:42 |
NobodyCam | and sould we list all the core nic's / email addys on it? | 16:42 |
jroll | we made it when ramesh joined | 16:42 |
jroll | NobodyCam: no, we'll forget to update it, the list is already in gerrit :P{ | 16:43 |
NobodyCam | ahh | 16:43 |
jroll | s/{// | 16:43 |
BadCub | TheJulia: I never make trouble... Trouble already exists when I come into the situation! LOL | 16:43 |
lucasagomes | BadCub, hah damn | 16:43 |
NobodyCam | lol | 16:43 |
lucasagomes | not sure what they would do to you if you did speak and write in Irish english | 16:43 |
*** viktors has quit IRC | 16:43 | |
* BadCub prefers the term "Trouble Facilitator" | 16:43 | |
*** ifarkas has quit IRC | 16:44 | |
jroll | lucasagomes: probably give him a whiskey | 16:44 |
BadCub | lucasagomes: LOL jroll: whiskey good... Gray goose better! | 16:44 |
jroll | BadCub: if you're speaking in irish english it has to be whiskey :P | 16:45 |
NobodyCam | lol the only Irish english /me knows is "I like a round of Irish car bombs" | 16:45 |
BadCub | jroll: True that. :-P | 16:45 |
*** dprince has quit IRC | 16:46 | |
*** dprince has joined #openstack-ironic | 16:46 | |
BadCub | NobodyCam: there was a shot called that. baileys and whiskey with a topper of 151 set on fire when served | 16:47 |
jroll | oh? | 16:48 |
NobodyCam | yep! | 16:48 |
jroll | I've always known carbomb as baileys and whiskey dropped into a guiness | 16:48 |
BadCub | yep | 16:48 |
BadCub | that was another one! | 16:48 |
jroll | http://en.wikipedia.org/wiki/Irish_Car_Bomb | 16:48 |
jroll | yeah | 16:48 |
* NobodyCam would deffer to lucasagomes as he is actually there | 16:48 | |
jroll | never heard of the 151 version | 16:49 |
jroll | lol | 16:49 |
* BadCub use to make them quite often in his bartender days | 16:49 | |
*** kkoski has quit IRC | 16:50 | |
lucasagomes | NobodyCam, :) the problem is when Irish words comes into english | 16:50 |
lucasagomes | like craic to say fun | 16:50 |
NobodyCam | lol - on our internal "what you do this week" report we have a suggestion to improve your team question. last week I suggested they snail mail a drink to everyone. | 16:51 |
*** harlowja_away is now known as harlowja_ | 16:51 | |
*** kkoski has joined #openstack-ironic | 16:51 | |
lucasagomes | \o/ | 16:52 |
dtantsur | people discussion baileys and whiskey clear means I should go home to enjoy the weekend | 16:55 |
dtantsur | aaaaaaaaaand | 16:55 |
dtantsur | that's what I do! | 16:55 |
NobodyCam | have a great weekend dtantsur | 16:55 |
dtantsur | g'night, have a nice weekend | 16:55 |
*** dtantsur is now known as dtantsur|afk | 16:55 | |
BadCub | dtantsur: have a good (and fun) weekend! | 16:55 |
NobodyCam | brb | 16:56 |
lucasagomes | dtantsur|afk, hah enjoy have a good night! | 16:57 |
adam_g | devananda, not a way to trigger that as is, we'd need to add a new job for it | 17:03 |
*** Marga_ has joined #openstack-ironic | 17:13 | |
*** achanda has joined #openstack-ironic | 17:15 | |
*** jistr has quit IRC | 17:19 | |
*** openstackgerrit has quit IRC | 17:21 | |
*** openstackgerrit has joined #openstack-ironic | 17:22 | |
*** mjturek1 has quit IRC | 17:26 | |
*** pelix has joined #openstack-ironic | 17:35 | |
tiagogomes_ | hi, I am trying to run `ironic node-set-provision-state` in a node, but I am getting an "FailedToUpdateDHCPOptOnPort: No VIFs found for node 55ebdccb-58ff-4cd0-b564-cc40a1c9483f when attempting to update DHCP BOOT options" error. Anyone has a clue about this? | 17:36 |
*** pelix has quit IRC | 17:38 | |
*** pelix has joined #openstack-ironic | 17:39 | |
NobodyCam | tiagogomes_: do you have ports created for your node. | 17:39 |
NobodyCam | where / how are you testing? | 17:39 |
NobodyCam | sounds like the the ports didn;t get a vif id attached to them | 17:40 |
tiagogomes_ | NobodyCam, I believe I have ports: http://paste.baserock.org/izokowudey | 17:40 |
tiagogomes_ | I am trying to integrate Ironic in Baserock. The branch that I am using is stable/juno | 17:41 |
jroll | tiagogomes_: you need to create a port in neutron, and put that in your ironic port object | 17:42 |
NobodyCam | tiagogomes_: what jroll said... | 17:42 |
jroll | one sec | 17:42 |
NobodyCam | oh brb | 17:42 |
*** humble_ has quit IRC | 17:43 | |
jroll | tiagogomes_: ironic port-update add extra/vif_port_id=$neutron_port_id | 17:44 |
tiagogomes_ | jroll, it may be a silly question, but how do I find the neutron_port_id? I am following these instructions http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-neutron-to-communicate-with-the-bare-metal-server, and doesn't mention that | 17:48 |
jroll | tiagogomes_: neutron port-create | 17:49 |
jroll | so let's back up a bit | 17:49 |
jroll | do you plan to use nova with this? | 17:49 |
*** kkoski has quit IRC | 17:49 | |
* NobodyCam is back | 17:49 | |
tiagogomes_ | yes, but know I am trying without nova first | 17:49 |
jroll | ok | 17:50 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation and script https://review.openstack.org/166201 | 17:50 |
jroll | so nova does this for you, a deployment is actually a few steps | 17:50 |
jroll | so you need to have a network in neutron, which is covered by those docs | 17:51 |
jroll | then do: neutron port-create --network $network_id | 17:51 |
jroll | or something like that | 17:51 |
jroll | that will give you a port id to use | 17:51 |
*** coolsvap_ is now known as coolsvap|afk | 17:52 | |
NobodyCam | tiagogomes_: have you seen this: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#using-ironic-as-a-standalone-service | 17:52 |
*** kkoski has joined #openstack-ironic | 17:53 | |
jroll | NobodyCam: that assumes no neutron :| | 17:53 |
*** kkoski has quit IRC | 17:53 | |
jroll | (which may be useful for now, idk) | 17:53 |
* NobodyCam also notes that stable juno will not work | 17:53 | |
*** kkoski has joined #openstack-ironic | 17:53 | |
tiagogomes_ | NobodyCam it will not work? | 17:54 |
*** andreykurilin_ has quit IRC | 17:54 | |
NobodyCam | standalone support was introduced in K | 17:54 |
tiagogomes_ | ok, but can I use it with neutron and glance, but not nova? | 17:55 |
NobodyCam | I not sure but keystone may be required there too | 17:56 |
NobodyCam | at least I think so | 17:56 |
lucasagomes | you can pass noauth to avoid keystone | 17:56 |
lucasagomes | but tiagogomes_ is using stable/juno | 17:56 |
NobodyCam | with a fake token? | 17:56 |
lucasagomes | so glance still needed | 17:56 |
tiagogomes_ | I am using keystone as well | 17:56 |
lucasagomes | right, try passing noauth to the auth_strategy config option | 17:57 |
lucasagomes | https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L84-L86 | 17:58 |
tiagogomes_ | lucasagomes but what for? I am having authentication problems | 17:58 |
jroll | guys | 17:59 |
jroll | it sounds like tiagogomes_ doesn't care about standalone | 17:59 |
jroll | but is just getting things working one by one | 17:59 |
tiagogomes_ | yep | 17:59 |
lucasagomes | tiagogomes_, oh, I thought u want to use ironic standalone... but I'm my be wrong | 17:59 |
lucasagomes | gotcha | 17:59 |
lucasagomes | sorry | 17:59 |
NobodyCam | sorry I think I started that side track :( | 18:00 |
tiagogomes_ | lucasagomes completely standalone will not work because of glance | 18:00 |
tiagogomes_ | Now I am trying to use everything but Nova | 18:00 |
tiagogomes_ | ok, adding a port changed the provision state to wait 'call-back' | 18:00 |
NobodyCam | :) | 18:01 |
jroll | yep, so now ironic should be booting the deploy ramdisk, which calls back to ironic to begin the deployment | 18:01 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation and script https://review.openstack.org/166201 | 18:01 |
lucasagomes | aight folks, I will call it a day :) | 18:02 |
lucasagomes | have a good night everyone. Enjoy the weekend | 18:02 |
jlvillal | lucasagomes: Good night! And I guess I added another comment... :( | 18:02 |
jroll | night lucas :) | 18:02 |
BadCub | have a good weekend lucasagomes | 18:03 |
lucasagomes | jlvillal, hah it's all good! thanks! | 18:03 |
jlvillal | lucasagomes: adeus | 18:03 |
lucasagomes | jlvillal, até mais! | 18:04 |
*** lucasagomes is now known as lucas-beer | 18:04 | |
tiagogomes_ | lucasagomes, bom fim de semana! | 18:04 |
tiagogomes_ | mmm node-show shows that the power is on, but IPMI says the opposite | 18:04 |
lucas-beer | tiagogomes_, pra você também! Até mais | 18:05 |
* jroll looks at names and suddenly wondering if y'all are siblings | 18:06 | |
TheJulia | that would make the world even smaller I think | 18:06 |
*** spandhe has joined #openstack-ironic | 18:09 | |
*** derekh has quit IRC | 18:11 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation and script https://review.openstack.org/166201 | 18:11 |
lucas-beer | jlvillal, ^ :) | 18:11 |
* lucas-beer gone now! see y'all | 18:11 | |
TheJulia | :) | 18:11 |
jlvillal | lucas-beer: Leave! :) | 18:11 |
*** Sukhdev has joined #openstack-ironic | 18:11 | |
NobodyCam | night lucas-beer | 18:12 |
*** absubram has quit IRC | 18:24 | |
*** absubram has joined #openstack-ironic | 18:26 | |
NobodyCam | hummm | 18:29 |
NobodyCam | I think we may have some broken tests | 18:29 |
NobodyCam | try this on master branch: grep -r -i called ironic/tests|grep -v assert_called_once_with|grep -v [.]called | 18:29 |
*** tiagogomes_ has quit IRC | 18:36 | |
* BadCub will be back in a while | 18:36 | |
*** BadCub is now known as BadCub_Away | 18:36 | |
*** penick has joined #openstack-ironic | 18:41 | |
Shrews | NobodyCam: what are you saying is incorrect? | 18:45 |
Shrews | not sure what i should be looking for in that output | 18:46 |
jroll | Shrews: I think he's looking for things like assert_called_once() that don't actually exist | 18:47 |
NobodyCam | Shrews: looked like: assert_called_once is not a valid check | 18:47 |
NobodyCam | ironic/tests/test_images.py: umount_mock.assert_called_once_wth('mountdir') is bad | 18:47 |
NobodyCam | ironic/tests/drivers/test_ssh.py: exec_ssh_mock.assert_not_called() | 18:47 |
jroll | wth | 18:48 |
NobodyCam | yep | 18:48 |
NobodyCam | its there | 18:48 |
jroll | that was a quote, not "what the hell" | 18:48 |
jroll | :P | 18:48 |
Shrews | NobodyCam: ah. yepeers. | 18:48 |
NobodyCam | lol | 18:48 |
Shrews | git grep "assert_called_once(" | 18:48 |
Shrews | 5 of those | 18:48 |
Shrews | ooh, 8 | 18:49 |
Shrews | we suck | 18:49 |
NobodyCam | there are a couple of assert not called | 18:49 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update installation service overview https://review.openstack.org/166346 | 18:49 |
NobodyCam | which is not a real assert | 18:49 |
jroll | so using autospec=True on mocks will make those break | 18:49 |
jroll | we should do that for most/all mocks | 18:49 |
*** Marga_ has quit IRC | 18:50 | |
jroll | probably doesn't work great with magic stuff though | 18:50 |
jroll | like sqlalchemy | 18:50 |
*** Marga_ has joined #openstack-ironic | 18:50 | |
NobodyCam | jroll: I'd be happy with the correct assert syntax | 18:51 |
NobodyCam | :-p | 18:51 |
jroll | NobodyCam: it's just going to happen again. | 18:51 |
jroll | if we make sure people are putting a spec on their mocks, we can be reasonable sure they're using valid asserts | 18:51 |
*** ijw has quit IRC | 18:51 | |
NobodyCam | I wounder if we could come up with a flake 8 test for invalid asserts? | 18:52 |
jroll | that would work too | 18:52 |
jroll | but we won't be able to catch all typos | 18:52 |
jroll | unless it can introspect the mock api | 18:53 |
jroll | someone will end up with assrt_called_once_with etc | 18:53 |
jroll | brb | 18:53 |
Shrews | NobodyCam: i'm pretty sure that I recall a ML discussion for just that error, but it was turned down, iirc, for the same reason that jroll pointed out | 18:54 |
NobodyCam | ahh | 18:54 |
*** mjturek1 has joined #openstack-ironic | 18:58 | |
*** Marga_ has quit IRC | 18:59 | |
*** achanda has quit IRC | 19:07 | |
*** EmilienM|afk is now known as EmilienM | 19:10 | |
*** BadCub_Away is now known as BadCub | 19:14 | |
* BadCub is back | 19:14 | |
NobodyCam | snyone know off the tops of their head if this one is valid: assert_any_call(port_dict1) | 19:18 |
jroll | I believe so but not 100% | 19:18 |
*** achanda has joined #openstack-ironic | 19:21 | |
*** pelix has quit IRC | 19:22 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: fix invalid asserts in tests https://review.openstack.org/166352 | 19:28 |
NobodyCam | take a look and let me know ^^^^ | 19:28 |
jroll | +2 with a comment about my dreams for the future | 19:30 |
JoshNang | blah, some of those were mine. nice catch NobodyCam | 19:31 |
NobodyCam | :-p | 19:35 |
NobodyCam | there will be more | 19:36 |
*** andreykurilin_ has joined #openstack-ironic | 19:41 | |
*** athomas has quit IRC | 19:42 | |
openstackgerrit | Merged openstack/ironic: Fail deploy if root uuid or disk id isn't available https://review.openstack.org/143919 | 19:52 |
*** Marga_ has joined #openstack-ironic | 20:00 | |
*** andreykurilin_ has quit IRC | 20:01 | |
rfchapman | NobodyCam: I have my first BareMetal Deployment.. | 20:02 |
*** lucas-beer has quit IRC | 20:03 | |
NobodyCam | rfchapman: w00t awesome !!!! | 20:03 |
devananda | rfchapman: grats! | 20:03 |
*** Marga_ has quit IRC | 20:05 | |
rfchapman | :-) Yes, the problems that I was having was with the antiquated dl365g1 servers. I now have a dl360g6 and it is working much better. | 20:05 |
*** ChuckC has quit IRC | 20:05 | |
*** ijw has joined #openstack-ironic | 20:05 | |
rfchapman | Only one problem at this time, I cannot log into the machines because the deployment is not getting the ssh keys that I set up. | 20:06 |
*** Sukhdev has quit IRC | 20:06 | |
rfchapman | I'm now looking into fixing that issue | 20:06 |
* BadCub wanders off to eat food-like stuff | 20:06 | |
rfchapman | Thank you to both NobodyCam and Devananda for helping me over come the many issues I had getting to my first deployment. | 20:08 |
*** Marga_ has joined #openstack-ironic | 20:08 | |
NobodyCam | :) awesome news and you really did the work :-p | 20:09 |
*** kkoski has quit IRC | 20:09 | |
* BadCub wonders if https://review.openstack.org/#/c/161066/ is lost in the gate again, cuz it is sitting at the same place it was last night | 20:16 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: fix invalid asserts in tests https://review.openstack.org/166352 | 20:20 |
NobodyCam | jroll: JoshNang: devananda: ^^^^^^^^ | 20:20 |
NobodyCam | :-p | 20:20 |
* NobodyCam goes back to eating his pizza | 20:20 | |
*** kkoski has joined #openstack-ironic | 20:20 | |
jroll | get_node_cap_mock.assert_not_called(task.node, 'boot_mode') | 20:21 |
jroll | wat | 20:21 |
NobodyCam | lol | 20:21 |
jroll | NobodyCam: I have a feeling that's supposed to be assert_called_once_with | 20:22 |
NobodyCam | one I had to put a note block on so I'd understand what I did | 20:22 |
jroll | there's also get_node_cap_mock.return_value = | 20:22 |
jlvillal | jroll: NobodyCam: autospec=True caught the one case I tried it on. | 20:22 |
jroll | jlvillal: yeah. that's what it does. | 20:22 |
jroll | won't let you access attributes that don't exist on the real thing | 20:22 |
jlvillal | jroll: Seems to work :) | 20:22 |
NobodyCam | test all pass here | 20:23 |
*** ndipanov has quit IRC | 20:23 | |
jroll | NobodyCam: orly | 20:24 |
jlvillal | NobodyCam: So what does the Feature Freeze mean in regards to patches? | 20:24 |
jroll | NobodyCam: I feel like that might be a broken test | 20:24 |
jroll | jlvillal: bugs and docs | 20:24 |
* jroll finds a link | 20:24 | |
jlvillal | jroll: Okay, so bug fixes are okay to submit. Good :) | 20:24 |
NobodyCam | y aI should file a bug | 20:25 |
BadCub | https://wiki.openstack.org/wiki/FeatureFreeze | 20:25 |
jroll | jlvillal: https://wiki.openstack.org/wiki/FeatureFreeze https://wiki.openstack.org/wiki/DepFreeze https://wiki.openstack.org/wiki/StringFreeze | 20:25 |
jlvillal | jroll: BadCub: Thanks | 20:25 |
jroll | NobodyCam: meh :P | 20:25 |
BadCub | yw :-0 | 20:25 |
BadCub | :-) | 20:25 |
*** dprince has quit IRC | 20:26 | |
NobodyCam | jroll: are looking in to that one test? | 20:26 |
*** Marga_ has quit IRC | 20:26 | |
jroll | NobodyCam: no, I'm not | 20:26 |
jroll | :/ | 20:26 |
NobodyCam | ack. I can after i finish eating | 20:26 |
NobodyCam | and going out for a smoke | 20:26 |
BadCub | mmmmm.. smoke....... brb | 20:27 |
*** andreykurilin_ has joined #openstack-ironic | 20:34 | |
*** gabrielbezerra has quit IRC | 20:34 | |
*** pas-ha has quit IRC | 20:44 | |
*** gabrielbezerra has joined #openstack-ironic | 20:44 | |
*** hyakuhei has joined #openstack-ironic | 20:46 | |
*** enikanorov has quit IRC | 20:50 | |
*** enikanorov has joined #openstack-ironic | 20:51 | |
NobodyCam | random thought: this line seem strange to me: Make sure all your change has thorough unit tests, especially if your patch touches an area of the code that currently is not well-tested | 20:59 |
NobodyCam | for several reasons :-p | 21:00 |
*** trown is now known as trown|outttypeww | 21:00 | |
BadCub | what seems strange about that? | 21:03 |
*** kkoski has quit IRC | 21:04 | |
NobodyCam | the bits after especially | 21:04 |
BadCub | it kinda seems like an "enter at own risk" type warning to me | 21:04 |
* NobodyCam translates as be sure to write good test as we have not done so yet | 21:04 | |
jroll | I mean | 21:05 |
jroll | there are bits that are not well-tested | 21:05 |
jroll | especially when you consider all of openstack | 21:05 |
BadCub | I can see that too.... Or a forewarning that "well, yeah, we got some tests going. But check yo'sht bro | 21:05 |
NobodyCam | that cut and pasted from the https://wiki.openstack.org/wiki/FeatureFreeze doc | 21:05 |
jroll | right | 21:05 |
jroll | there are parts of openstack not well-tested | 21:05 |
jroll | which is why there's tons of bugs everywhere | 21:06 |
NobodyCam | :-p | 21:06 |
*** ijw has quit IRC | 21:16 | |
*** ijw has joined #openstack-ironic | 21:17 | |
NobodyCam | jroll: unit test pass heheheh https://jenkins01.openstack.org/job/gate-ironic-python27/2445/ | 21:19 |
*** ijw_ has joined #openstack-ironic | 21:19 | |
jroll | NobodyCam: hrm, I wonder if that's expectedf | 21:20 |
jroll | I guess it's feasible | 21:20 |
jroll | but that looks like a bad test to me, need to at least remove the thing setting return_value | 21:20 |
NobodyCam | hehehe I do need to look closer | 21:20 |
NobodyCam | but I think I somehow come down with : http://www.healthdictionary.info/Fridayitis.htm | 21:22 |
jroll | ^ | 21:22 |
BadCub | yip! | 21:23 |
jroll | great link | 21:23 |
*** ijw has quit IRC | 21:23 | |
NobodyCam | lol | 21:23 |
NobodyCam | looks so offical | 21:23 |
*** hyakuhei has quit IRC | 21:24 | |
*** Marga_ has joined #openstack-ironic | 21:30 | |
adam_g | https://review.openstack.org/#/c/166386/ | 21:37 |
adam_g | devananda, jroll etc ^ updated the whiteboard with a plan for microversion testing in tempest | 21:37 |
devananda | adam_g: ooh neat. /me reads | 21:39 |
devananda | adam_g: "all" vs "latest" ? | 21:39 |
adam_g | devananda, all is every API version up to and including latest | 21:40 |
adam_g | latest is its own test | 21:41 |
adam_g | (ie, sending 'latest' vs '1.6' in the headers) | 21:41 |
*** EmilienM is now known as EmilienM|PTO | 21:44 | |
devananda | adam_g: ah, right | 21:45 |
devananda | adam_g: also, v1.0 vs. v1.1 is still an open debate, and a somewhat frustrating one -- https://review.openstack.org/#/c/164369/ | 21:46 |
devananda | adam_g: right now, min version supported by trunk is (no version) or v1.1 | 21:47 |
adam_g | devananda, ah.. that tempest patch just translates v1.0 to no version now. we can change it based on whats decided | 21:47 |
devananda | k k | 21:48 |
*** jamielennox|away is now known as jamielennox | 21:49 | |
*** enikanorov has quit IRC | 21:53 | |
*** enikanorov has joined #openstack-ironic | 21:54 | |
*** kkoski has joined #openstack-ironic | 21:59 | |
*** igordcard_ has joined #openstack-ironic | 22:04 | |
*** jamielennox is now known as jamielennox|away | 22:05 | |
*** kkoski has quit IRC | 22:06 | |
*** Marga_ has quit IRC | 22:10 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Support the equivalent of the Juno API https://review.openstack.org/164369 | 22:24 |
*** Marga_ has joined #openstack-ironic | 22:24 | |
*** ijw has joined #openstack-ironic | 22:26 | |
*** ijw_ has quit IRC | 22:30 | |
* devananda pokes the gate for https://review.openstack.org/#/c/161066/ again | 22:43 | |
* BadCub thinks the gate is evil | 22:45 | |
*** zer0c00l has quit IRC | 22:45 | |
*** zer0c00l has joined #openstack-ironic | 22:45 | |
JoshNang | the gate really doesn't like that patch.. | 22:45 |
BadCub | No, it does not | 22:46 |
BadCub | or perhaps, it likes it too much...... | 22:46 |
JoshNang | heh | 22:46 |
*** kkoski has joined #openstack-ironic | 22:49 | |
* NobodyCam thinks it would be nice to : git clone fridge://diet.coke.com /desktop/* | 22:53 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: Fix typos in ironic-specs https://review.openstack.org/166412 | 22:54 |
*** enikanorov has quit IRC | 22:55 | |
*** enikanorov has joined #openstack-ironic | 22:56 | |
*** andreykurilin_ has quit IRC | 22:58 | |
devananda | ironic node-create -d coffee_driver -p size=large_mug -n cup_number_three | 22:59 |
BadCub | devananda: LOL | 22:59 |
TheJulia | devananda: win! | 22:59 |
devananda | ironic node-set-provision-state cup_number_three provide | 23:00 |
NobodyCam | +++ | 23:01 |
NobodyCam | humm have to dig up the plm driver stuff | 23:01 |
BadCub | but instead of cup_number, should it not be pot_number ? | 23:02 |
* BadCub is thinking coffe should be tested in HA env | 23:02 | |
NobodyCam | there is a starbucks API | 23:03 |
NobodyCam | and it has a twitter acount | 23:03 |
TheJulia | NobodyCam: is this why you were talking about raw ethernet frames earlier?!? | 23:03 |
NobodyCam | https://twitter.com/starbucksapi | 23:04 |
devananda | https://gist.github.com/anonymous/993d7d2c685273a26f98 | 23:04 |
NobodyCam | roflmfao | 23:04 |
devananda | BadCub: oh. good point. it should be -p type=espresso | 23:04 |
* BadCub thinks devananda is possibly going slightly insane and backs away slowly | 23:04 | |
TheJulia | shouldn't there also be a beer_dirver ? | 23:05 |
devananda | TheJulia: yes! | 23:05 |
BadCub | gray_goose_driver | 23:05 |
TheJulia | BadCub: that would be a feature of the liquor_driver | 23:05 |
* NobodyCam points to: https://github.com/NoBodyCam/ironic | 23:06 | |
BadCub | liquor_driver_type=gray_goose | 23:06 |
devananda | TheJulia: i'm sure we can create a power driver for a beer tap | 23:06 |
TheJulia | devananda: I would soooo buy the beer tap | 23:06 |
NobodyCam | teh PLM driver would do it | 23:07 |
NobodyCam | read the read me for that link ^^^^^ | 23:07 |
*** ijw_ has joined #openstack-ironic | 23:07 | |
*** mtanino has quit IRC | 23:08 | |
devananda | NobodyCam: lol, 2384 commits behind master | 23:09 |
devananda | we, um, might have written some code .... | 23:09 |
NobodyCam | lol yes and all the ip checking we have will break it but it itself should run and work | 23:10 |
BadCub | ya'think? lol | 23:10 |
*** ijw__ has joined #openstack-ironic | 23:10 | |
*** ijw has quit IRC | 23:11 | |
devananda | NobodyCam: downsides to that: serial line, $80, bulky, doesn't run on low voltage | 23:11 |
devananda | NobodyCam: i want something like that for a 19V line that I can control with TCP/IP | 23:12 |
devananda | NobodyCam: so I can control one of these: www.amazon.com/Intel-Computing-BOXDC53427HYE-Desktop-Black/dp/B00C5K8FRI/ | 23:13 |
TheJulia | you can get 5v off of a parallel port | 23:13 |
devananda | er, I mean, so I can control something that is NOT one of those | 23:13 |
devananda | but like it | 23:13 |
*** ijw_ has quit IRC | 23:13 | |
devananda | clearly my brain has stopped working when i'm pasting links to the opposite of what I meant to | 23:14 |
NobodyCam | humm ok :) | 23:14 |
devananda | also, that should work with the AMT driver | 23:15 |
*** ijw__ has quit IRC | 23:17 | |
openstackgerrit | Merged openstack/ironic-python-agent: Add cleaning/zapping support to IPA https://review.openstack.org/161066 | 23:19 |
JoshNang | \o/ finally | 23:20 |
NobodyCam | lol | 23:20 |
NobodyCam | ok At 4:20 on a friday it lands. you just know ..... | 23:21 |
devananda | NobodyCam: LOL | 23:21 |
*** enikanorov has quit IRC | 23:22 | |
*** kkoski has quit IRC | 23:22 | |
JoshNang | :P | 23:22 |
NobodyCam | devananda: this is what I got. http://www.amazon.com/gp/product/B00HZDLNWO/ref=s9_simh_gw_p147_d0_i2?pf_rd_m=ATVPDKIKX0DER&pf_rd_s=desktop-1&pf_rd_r=06VXW21TZXBBM8YN2KKH&pf_rd_t=36701&pf_rd_p=1970559082&pf_rd_i=desktop | 23:22 |
devananda | NobodyCam: that does not have AMT / vPro, unfortunately. | 23:23 |
devananda | NobodyCam: you could set the BIOS to default power state ON, then use the liteon thing | 23:23 |
devananda | none of the 4th gen NUC had AMT support | 23:23 |
*** enikanorov has joined #openstack-ironic | 23:23 | |
NobodyCam | lol it's for my tv | 23:23 |
NobodyCam | :( | 23:23 |
devananda | there is a new 5th gen that does ... | 23:23 |
*** achanda has quit IRC | 23:24 | |
NobodyCam | ahh will get one for testing as long as it doesn't come in the same box as the nuc... omg the most annoying packaging ever | 23:25 |
NobodyCam | https://www.youtube.com/watch?v=X3DQ5mWvqQY | 23:26 |
*** achanda has joined #openstack-ironic | 23:26 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Fix typos in ironic/ironic/drivers/modules https://review.openstack.org/166418 | 23:28 |
*** Marga_ has quit IRC | 23:29 | |
NobodyCam | and devananda you know the commit message on 164369 is longer then the patch itself .. almost including tests! | 23:29 |
devananda | yes, I know! | 23:29 |
NobodyCam | :-p | 23:29 |
devananda | also - time for drinks. because it's friday .... | 23:30 |
devananda | have a good night (and weekend) ya'll! | 23:30 |
NobodyCam | have a good weekend and flight devananda | 23:30 |
TheJulia | goodnight | 23:31 |
BadCub | have a good trip devananda | 23:36 |
* BadCub is thinking it is cocktail time! | 23:37 | |
* NobodyCam digs up rum to flavor is diet coke, saying something about 5:00 o'clock somewhere... | 23:38 | |
BadCub | Good weekend to all! | 23:38 |
*** BadCub is now known as BadCub_Hiding | 23:39 | |
*** ijw has joined #openstack-ironic | 23:40 | |
*** harlowja_ has quit IRC | 23:41 | |
*** ijw has quit IRC | 23:42 | |
*** Marga_ has joined #openstack-ironic | 23:42 | |
*** harlowja has joined #openstack-ironic | 23:42 | |
*** ijw has joined #openstack-ironic | 23:42 | |
*** kkoski has joined #openstack-ironic | 23:46 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!