| *** rook-ugh is now known as rook | 00:03 | |
| *** ijw has joined #openstack-ironic | 00:05 | |
| *** rama_y has quit IRC | 00:10 | |
| *** mat128 has joined #openstack-ironic | 00:10 | |
| *** mat128 has quit IRC | 00:11 | |
| *** mat128 has joined #openstack-ironic | 00:12 | |
| *** mtanino has joined #openstack-ironic | 00:14 | |
| *** baoli has joined #openstack-ironic | 00:22 | |
| *** Syed__ has quit IRC | 00:25 | |
| *** baoli has quit IRC | 00:26 | |
| *** amotoki has joined #openstack-ironic | 00:26 | |
| *** IridiumScaffold has quit IRC | 00:29 | |
| *** gcb_ has quit IRC | 00:35 | |
| *** mtanino has quit IRC | 00:39 | |
| *** wrouesnel has joined #openstack-ironic | 00:39 | |
| *** mat128 has quit IRC | 00:41 | |
| *** ppiela has quit IRC | 00:46 | |
| *** ppiela has joined #openstack-ironic | 00:47 | |
| *** sayantani01 has joined #openstack-ironic | 00:50 | |
| *** pmannidi has quit IRC | 00:51 | |
| *** pmannidi has joined #openstack-ironic | 00:52 | |
| *** mat128 has joined #openstack-ironic | 00:54 | |
| *** ppiela has quit IRC | 00:54 | |
| *** ppiela has joined #openstack-ironic | 00:55 | |
| *** sasha2 has quit IRC | 00:57 | |
| *** rloo has quit IRC | 00:58 | |
| *** phuongnh has joined #openstack-ironic | 01:05 | |
| *** krtaylor has joined #openstack-ironic | 01:06 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:09 | |
| *** IridiumScaffold has joined #openstack-ironic | 01:14 | |
| *** wrouesnel has quit IRC | 01:16 | |
| *** wrouesnel has joined #openstack-ironic | 01:18 | |
| *** IridiumScaffold has quit IRC | 01:20 | |
| *** dsneddon has joined #openstack-ironic | 01:22 | |
| *** SerenaFeng has joined #openstack-ironic | 01:24 | |
| *** Sukhdev_ has quit IRC | 01:31 | |
| *** gcb has joined #openstack-ironic | 01:49 | |
| *** alexpilotti has quit IRC | 01:50 | |
| *** crushil has joined #openstack-ironic | 01:51 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-ui master: Adds Ironic-UI Devstack plugin https://review.openstack.org/446298 | 01:57 |
|---|---|---|
| *** zhurong has joined #openstack-ironic | 01:58 | |
| *** causten has joined #openstack-ironic | 02:00 | |
| *** R08209 has joined #openstack-ironic | 02:01 | |
| R08209 | 213456 | 02:01 |
| *** R08209 has quit IRC | 02:05 | |
| *** ijw has quit IRC | 02:06 | |
| *** linggao has quit IRC | 02:09 | |
| *** rbudden has quit IRC | 02:48 | |
| *** alexpilotti has joined #openstack-ironic | 02:50 | |
| *** gcb has quit IRC | 02:58 | |
| *** alexpilotti has quit IRC | 03:04 | |
| *** zhurong has quit IRC | 03:05 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-ui master: Adds Ironic-UI Devstack plugin https://review.openstack.org/446298 | 03:08 |
| *** causten has quit IRC | 03:13 | |
| *** gcb has joined #openstack-ironic | 03:14 | |
| *** gcb has quit IRC | 03:14 | |
| *** gcb has joined #openstack-ironic | 03:15 | |
| *** zhurong has joined #openstack-ironic | 03:22 | |
| *** SerenaFeng has quit IRC | 03:33 | |
| *** fragatin_ has joined #openstack-ironic | 03:34 | |
| *** fragatina has quit IRC | 03:38 | |
| *** fragatin_ has quit IRC | 03:39 | |
| *** fragatina has joined #openstack-ironic | 03:40 | |
| *** fragatin_ has joined #openstack-ironic | 03:43 | |
| *** sayantan_ has joined #openstack-ironic | 03:44 | |
| *** links has joined #openstack-ironic | 03:44 | |
| *** fragatina has quit IRC | 03:45 | |
| *** sayantani01 has quit IRC | 03:47 | |
| *** fragatin_ has quit IRC | 03:47 | |
| *** crushil has quit IRC | 03:50 | |
| *** mat128 has quit IRC | 03:57 | |
| *** fragatina has joined #openstack-ironic | 04:03 | |
| *** fragatina has quit IRC | 04:03 | |
| *** fragatina has joined #openstack-ironic | 04:04 | |
| *** zhurong has quit IRC | 04:12 | |
| *** d0ugal has quit IRC | 04:16 | |
| *** d0ugal has joined #openstack-ironic | 04:17 | |
| *** Guest55273 has joined #openstack-ironic | 04:27 | |
| *** SerenaFeng has joined #openstack-ironic | 04:34 | |
| openstackgerrit | Tao Li proposed openstack/ironic-inspector master: Remove log translations https://review.openstack.org/447791 | 04:42 |
| *** Guest55273 has quit IRC | 04:50 | |
| *** udesale has joined #openstack-ironic | 04:50 | |
| *** jaypipes has quit IRC | 04:57 | |
| *** jaypipes has joined #openstack-ironic | 04:57 | |
| *** zhurong has joined #openstack-ironic | 05:00 | |
| *** pmannidi has quit IRC | 05:00 | |
| *** pmannidi has joined #openstack-ironic | 05:01 | |
| *** alexpilotti has joined #openstack-ironic | 05:01 | |
| openstackgerrit | Tao Li proposed openstack/python-ironic-inspector-client master: Remove log translations https://review.openstack.org/447801 | 05:01 |
| *** SerenaFeng has quit IRC | 05:01 | |
| *** udesale has quit IRC | 05:03 | |
| *** alexpilotti has quit IRC | 05:05 | |
| *** udesale has joined #openstack-ironic | 05:06 | |
| *** chuckC_ has joined #openstack-ironic | 05:06 | |
| openstackgerrit | Tao Li proposed openstack/python-ironicclient master: Remove log translations https://review.openstack.org/447806 | 05:06 |
| *** udesale has quit IRC | 05:09 | |
| *** udesale has joined #openstack-ironic | 05:09 | |
| *** causten has joined #openstack-ironic | 05:14 | |
| *** aileronajay has joined #openstack-ironic | 05:15 | |
| *** causten has quit IRC | 05:18 | |
| *** vmud213 has joined #openstack-ironic | 05:19 | |
| *** dsneddon has quit IRC | 05:23 | |
| *** aparnav has joined #openstack-ironic | 05:27 | |
| *** zhurong has quit IRC | 05:40 | |
| *** vikrant has joined #openstack-ironic | 05:41 | |
| *** rcernin has joined #openstack-ironic | 05:43 | |
| *** SerenaFeng has joined #openstack-ironic | 05:46 | |
| *** rcernin has quit IRC | 05:57 | |
| *** alexpilotti has joined #openstack-ironic | 06:02 | |
| *** alexpilotti has quit IRC | 06:06 | |
| *** udesale has quit IRC | 06:06 | |
| *** udesale has joined #openstack-ironic | 06:07 | |
| *** zackf has joined #openstack-ironic | 06:07 | |
| *** mvpnitesh has joined #openstack-ironic | 06:11 | |
| *** zackf has quit IRC | 06:11 | |
| *** vsaienko has joined #openstack-ironic | 06:13 | |
| *** vsaienko has quit IRC | 06:18 | |
| *** mjura has joined #openstack-ironic | 06:29 | |
| *** jaosorior has joined #openstack-ironic | 06:31 | |
| openstackgerrit | Merged openstack/ironic master: Update test requirement https://review.openstack.org/440226 | 06:31 |
| *** yolanda has quit IRC | 06:32 | |
| openstackgerrit | Tao Li proposed openstack/ironic-inspector master: Remove log translations https://review.openstack.org/447791 | 06:36 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Rmove translation of log messages from ironic-inspector https://review.openstack.org/447884 | 06:39 |
| *** aileronajay has quit IRC | 06:39 | |
| openstackgerrit | Tao Li proposed openstack/python-ironic-inspector-client master: Remove log translations https://review.openstack.org/447801 | 06:44 |
| *** yolanda has joined #openstack-ironic | 06:46 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:54 | |
| *** fragatina has quit IRC | 06:56 | |
| openstackgerrit | Tao Li proposed openstack/python-ironicclient master: Remove log translations https://review.openstack.org/447806 | 06:59 |
| *** amotoki_ has joined #openstack-ironic | 07:00 | |
| *** tuanluong has joined #openstack-ironic | 07:00 | |
| *** zackf has joined #openstack-ironic | 07:02 | |
| *** amotoki has quit IRC | 07:03 | |
| *** alexpilotti has joined #openstack-ironic | 07:03 | |
| *** zackf has quit IRC | 07:06 | |
| *** ChubYann has quit IRC | 07:07 | |
| *** alexpilotti has quit IRC | 07:07 | |
| *** sayantan_ has quit IRC | 07:08 | |
| *** Serlex has joined #openstack-ironic | 07:11 | |
| *** amotoki has joined #openstack-ironic | 07:18 | |
| *** amotoki_ has quit IRC | 07:21 | |
| *** e0ne has joined #openstack-ironic | 07:22 | |
| *** jtomasek has quit IRC | 07:23 | |
| openstackgerrit | Tao Li proposed openstack/python-ironic-inspector-client master: Remove log translations https://review.openstack.org/447801 | 07:31 |
| *** tesseract has joined #openstack-ironic | 07:35 | |
| *** rbartal has joined #openstack-ironic | 07:35 | |
| *** alexpilotti has joined #openstack-ironic | 07:37 | |
| *** Lambert has joined #openstack-ironic | 07:41 | |
| *** amarao has quit IRC | 07:41 | |
| *** alexpilotti has quit IRC | 07:42 | |
| Lambert | text_transparent | 07:43 |
| *** e0ne has quit IRC | 07:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add standalone tests using direct HTTP links https://review.openstack.org/437549 | 07:46 |
| pas-ha | first failure due to local copy of tempest scenario manager :( https://bugs.launchpad.net/ironic/+bug/1674597 | 07:47 |
| openstack | Launchpad bug 1674597 in Ironic "gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial is broken due to excessive pruning of local scenario manager" [Undecided,New] | 07:47 |
| pas-ha | and good morning ironic | 07:48 |
| *** ralonsoh has joined #openstack-ironic | 07:49 | |
| *** moshele has joined #openstack-ironic | 07:50 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 07:52 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** alexpilotti has joined #openstack-ironic | 08:02 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Fix local copy of scenario manager https://review.openstack.org/447930 | 08:03 |
| *** Nisha_Agarwal has quit IRC | 08:04 | |
| *** venki has joined #openstack-ironic | 08:05 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic stable/ocata: DNM test stable gates with tempest plugin fix https://review.openstack.org/447933 | 08:06 |
| *** alexpilotti has quit IRC | 08:06 | |
| *** alexpilotti has joined #openstack-ironic | 08:08 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 08:10 |
| *** pcaruana has joined #openstack-ironic | 08:11 | |
| *** alexpilotti has quit IRC | 08:13 | |
| *** snecklifter has quit IRC | 08:13 | |
| *** alexpilotti has joined #openstack-ironic | 08:18 | |
| *** alexpilotti has quit IRC | 08:23 | |
| Lambert | i have a question, anybody knows how to insert ASCII draw in reStructer doc | 08:27 |
| *** alexpilotti has joined #openstack-ironic | 08:27 | |
| *** e0ne has joined #openstack-ironic | 08:30 | |
| -openstackstatus- NOTICE: Wiki is broken with database problems, we are working to resolve it | 08:31 | |
| *** ChanServ changes topic to "Wiki is broken with database problems, we are working to resolve it" | 08:31 | |
| *** alexpilotti has quit IRC | 08:32 | |
| *** zhurong has joined #openstack-ironic | 08:34 | |
| openstackgerrit | Tao Li proposed openstack/ironic-inspector master: Remove log translations https://review.openstack.org/447791 | 08:35 |
| *** mvpnitesh has quit IRC | 08:38 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 08:39 | |
| -openstackstatus- NOTICE: Wiki problems have been fixed, it's up and running | 08:39 | |
| *** amoralej|off is now known as amoralej | 08:41 | |
| *** mgoddard has joined #openstack-ironic | 08:44 | |
| *** amarao has joined #openstack-ironic | 08:46 | |
| openstackgerrit | Tao Li proposed openstack/python-ironicclient master: Remove log translations https://review.openstack.org/447806 | 08:48 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:50 | |
| *** racedo has joined #openstack-ironic | 08:51 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: Follow up to 447435 https://review.openstack.org/447555 | 08:52 |
| *** zackf has joined #openstack-ironic | 08:52 | |
| *** zackf has quit IRC | 08:57 | |
| *** mvpnitesh has joined #openstack-ironic | 08:59 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 08:59 |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** athomas has joined #openstack-ironic | 09:06 | |
| *** openstackgerrit has joined #openstack-ironic | 09:07 | |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-ironic-install: Retry failed pip installation https://review.openstack.org/447444 | 09:07 |
| *** mjura has quit IRC | 09:08 | |
| *** pmannidi has quit IRC | 09:09 | |
| *** pmannidi has joined #openstack-ironic | 09:10 | |
| *** dtantsur|afk is now known as dtantsur | 09:13 | |
| lucasagomes | morning all | 09:19 |
| *** kromanenko has joined #openstack-ironic | 09:30 | |
| *** stendulker has joined #openstack-ironic | 09:36 | |
| *** derekh has joined #openstack-ironic | 09:36 | |
| openstackgerrit | Béla Vancsics proposed openstack/ironic master: Use more specific asserts in tests https://review.openstack.org/432941 | 09:41 |
| *** jtomasek has joined #openstack-ironic | 09:42 | |
| openstackgerrit | Tao Li proposed openstack/python-ironicclient master: Remove log translations https://review.openstack.org/447806 | 09:46 |
| *** udesale has quit IRC | 09:57 | |
| *** Serlex has quit IRC | 09:59 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Support to perform erase on SSD drives https://review.openstack.org/443542 | 10:00 |
| vdrok | morning ironic! | 10:04 |
| *** zhurong has quit IRC | 10:05 | |
| *** MattMan has quit IRC | 10:11 | |
| *** MattMan has joined #openstack-ironic | 10:11 | |
| pas-ha | morning vdrok :) | 10:12 |
| pas-ha | morning lucasagomes :) | 10:12 |
| vdrok | o/ | 10:12 |
| lucasagomes | o/ | 10:12 |
| *** SerenaFe_ has joined #openstack-ironic | 10:14 | |
| *** SerenaFe_ has quit IRC | 10:14 | |
| *** jaosorior is now known as jaosorior_brb | 10:14 | |
| *** SerenaFeng has quit IRC | 10:15 | |
| *** SerenaFe_ has joined #openstack-ironic | 10:15 | |
| *** amotoki has quit IRC | 10:17 | |
| *** udesale has joined #openstack-ironic | 10:21 | |
| *** zhurong has joined #openstack-ironic | 10:24 | |
| *** gcb has quit IRC | 10:26 | |
| *** ccamacho has joined #openstack-ironic | 10:27 | |
| *** venki has quit IRC | 10:28 | |
| *** gcb has joined #openstack-ironic | 10:28 | |
| *** venki has joined #openstack-ironic | 10:28 | |
| *** phuongnh has quit IRC | 10:31 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Support to perform erase on SSD drives https://review.openstack.org/443542 | 10:31 |
| Nisha_Agarwal | lucasagomes, hi | 10:35 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec https://review.openstack.org/445478 | 10:36 |
| lucasagomes | Nisha_Agarwal, hi there | 10:36 |
| lucasagomes | dtantsur, rloo, mariojv mgould|afk ^ uploaded a new version of the redfish spec follow-up... Hope I have addressed all the comments properly | 10:36 |
| Nisha_Agarwal | I was looking at sushy ....and i see some problems with the design | 10:36 |
| *** gcb has quit IRC | 10:37 | |
| Nisha_Agarwal | Like when we collect the Systems data, do we expect that the user explicitly extracts the 'identity' for each uri applicable under systems | 10:38 |
| Nisha_Agarwal | like : /redfish/v1/Systems/1/Processors | 10:38 |
| Nisha_Agarwal | like : /redfish/v1/Systems/1/Storage | 10:39 |
| Nisha_Agarwal | like : /redfish/v1/Systems/1/Storage/1 | 10:39 |
| Nisha_Agarwal | like : /redfish/v1/Systems/1/SimpleStorage/1 | 10:39 |
| Nisha_Agarwal | the identifiers in aove example as "1" vary across the systems | 10:39 |
| Nisha_Agarwal | the current implementation enforces user to first extract the identifier from the collection and then make the corresponding URI call | 10:40 |
| Nisha_Agarwal | while you may just get the complete required URI from the Systems() data or Chassis() data or Managers() data | 10:41 |
| Nisha_Agarwal | lucasagomes, ^^^^ | 10:41 |
| *** zackf has joined #openstack-ironic | 10:42 | |
| *** e0ne has quit IRC | 10:42 | |
| lucasagomes | Nisha_Agarwal, I don't think we expect that, it's implicit ins't it ? I mean, once you have an instance of system and you want to get the manager (or any other resource) related to that I think we just need a get_manager() method within the System class | 10:43 |
| lucasagomes | or in case of multiples, get_storages() (which returns all storages related to that system) | 10:44 |
| *** jkilpatr has quit IRC | 10:44 | |
| *** e0ne has joined #openstack-ironic | 10:44 | |
| lucasagomes | Nisha_Agarwal, the current implementation allows both, you can either do a get_systems() from the root resource which will return all systems (no id needed) | 10:45 |
| lucasagomes | or get_system() which returns a specific one | 10:45 |
| lucasagomes | this is to optimize things in Ironic where each node is represented by a unique system in the redfish controller | 10:45 |
| Nisha_Agarwal | lucasagomes, so when we get_storage(), we expect user to provide the identifier for each level? | 10:47 |
| *** zackf has quit IRC | 10:47 | |
| Nisha_Agarwal | lucasagomes, example we want to extract data for this | 10:47 |
| Nisha_Agarwal | lucasagomes, "/redfish/v1/Systems/1/Storage/2/Volumes/1" | 10:48 |
| lucasagomes | Nisha_Agarwal, sushy needs is suppose to be optimzed for ironic use, cause that's the goal of the library it's not a generic purposed library | 10:48 |
| lucasagomes | if there's a need to get a specific storage for our use case then yeah maybe a get_storage(<id>) is needed in the System's class | 10:49 |
| Nisha_Agarwal | lucasagomes, i understand that but it looks like on ironic side we will need to extarct the identifier at each level | 10:49 |
| lucasagomes | but, if we always want to get all storages (for inspection use-case I suppose) then a get_storages() (no id) that returns all storages related to that System seems enough | 10:49 |
| lucasagomes | Nisha_Agarwal, ok, why we need to extract the identifier at each level ? | 10:50 |
| Nisha_Agarwal | lucasagomes, this is one eample but if the level of uri increases for any other case also it has to e done that many times | 10:50 |
| Nisha_Agarwal | lucasagomes, for Systems the identifier is 1 in above example | 10:50 |
| Nisha_Agarwal | for stotage there may be 1, 2, 3 etc | 10:51 |
| Nisha_Agarwal | and then for volumes it may be 1 or any oter | 10:51 |
| lucasagomes | Nisha_Agarwal, sure... but why we need to extract it at each level as you are saying ? | 10:51 |
| Nisha_Agarwal | it may not be necessarily 1 always | 10:51 |
| *** moshele has quit IRC | 10:51 | |
| *** moshele has joined #openstack-ironic | 10:52 | |
| lucasagomes | you can extract the id's of each object if you want after a get_storages() for example | 10:52 |
| Nisha_Agarwal | because the uri can be "/redfish/v1/Systems/1/Storage/3/Volumes/3" | 10:52 |
| lucasagomes | Nisha_Agarwal, sure, I get that... because the "/redfish/v1/Systems/1/Storage" URI is related to the System with id "1" | 10:53 |
| lucasagomes | for other system's it will be a different ID | 10:53 |
| Nisha_Agarwal | yes | 10:53 |
| lucasagomes | so ? | 10:53 |
| Nisha_Agarwal | so we expect the "id" has to be passed by ironic always? | 10:54 |
| *** udesale has quit IRC | 10:54 | |
| lucasagomes | Nisha_Agarwal, yes, if you mean the id to the system | 10:55 |
| Nisha_Agarwal | i mean id of the storage/Volumes/etc | 10:55 |
| lucasagomes | cause the relation here is a Node in Ironic is represented by a specific System in the Redfish controller, therefore we need to know the id | 10:55 |
| Nisha_Agarwal | Systems is the first level | 10:55 |
| lucasagomes | Nisha_Agarwal, I don't think we need ID for those in Ironic | 10:55 |
| *** gcb has joined #openstack-ironic | 10:55 | |
| lucasagomes | Nisha_Agarwal, once you have the system id | 10:56 |
| *** e0ne has quit IRC | 10:56 | |
| lucasagomes | you can figure out the ID's off all storage, managers, volumes etc... related to that system | 10:56 |
| lucasagomes | via the URI as you just shown | 10:56 |
| *** faizy has joined #openstack-ironic | 10:56 | |
| lucasagomes | it's scoped | 10:56 |
| *** gcb has quit IRC | 10:57 | |
| thomas_oneill | submitted a bifrost bug ... https://bugs.launchpad.net/bifrost/+bug/1674649 ... sorry haven't really done much further investigation, looks like it's looking for some dict key 'bootstrap' and not finding it... this key isn't anywhere in the vars for this role | 10:57 |
| openstack | Launchpad bug 1674649 in bifrost "AnsibleUndefinedVariable bootstrap" [Undecided,New] | 10:57 |
| *** gcb has joined #openstack-ironic | 10:57 | |
| thomas_oneill | I'm not sure what it's looking for and why, so I don't know what to do | 10:58 |
| *** Serlex has joined #openstack-ironic | 10:59 | |
| *** faizy has quit IRC | 10:59 | |
| *** faizy has joined #openstack-ironic | 11:00 | |
| openstackgerrit | Ji Chengke proposed openstack/ironic master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/448002 | 11:00 |
| *** milan has joined #openstack-ironic | 11:01 | |
| *** udesale has joined #openstack-ironic | 11:02 | |
| *** jkilpatr has joined #openstack-ironic | 11:02 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups https://review.openstack.org/436878 | 11:02 |
| *** e0ne has joined #openstack-ironic | 11:03 | |
| *** stendulker has quit IRC | 11:04 | |
| *** gcb has quit IRC | 11:06 | |
| *** venki has quit IRC | 11:08 | |
| *** venki has joined #openstack-ironic | 11:08 | |
| *** mgould|afk is now known as mgould | 11:09 | |
| mgould | morning ironic | 11:09 |
| pas-ha | morning mgould | 11:10 |
| mgould | morning pas-ha thomas_oneill lucasagomes Nisha_Agarwal | 11:10 |
| thomas_oneill | morning :) | 11:15 |
| lucasagomes | mgould, hi there o/ | 11:16 |
| Nisha_Agarwal | mgould, o/ | 11:16 |
| pas-ha | thomas_oneill: could you add to the bug the exact ansible command you've used to run install.yml? with all extra-vars etc... | 11:17 |
| pas-ha | from the first look it seems it tries to register ironic endpoints in keystone and is missing credentials for that | 11:17 |
| *** udesale has quit IRC | 11:18 | |
| *** udesale has joined #openstack-ironic | 11:18 | |
| thomas_oneill | pas-ha: I just edited the bug with my cli, I did modify it from the doc actually, I noticed there is -K in there now which I didn't like so I took that out and added -e ansible_become=true | 11:18 |
| thomas_oneill | I'll try with it as it is in the doc, sec | 11:19 |
| * pas-ha kind of suspects it might be due to his https://review.openstack.org/#/c/444820/2 but the gates do pass... | 11:19 | |
| patchbot | patch 444820 - bifrost - Always write clouds.yaml (MERGED) | 11:19 |
| *** gcb has joined #openstack-ironic | 11:20 | |
| *** faizy_ has joined #openstack-ironic | 11:24 | |
| thomas_oneill | pas-ha: same error using the command-line specified on the readme, updated the bug. | 11:24 |
| *** faizy has quit IRC | 11:25 | |
| thomas_oneill | I find these situations with ansible really difficult to approach debugging | 11:26 |
| thomas_oneill | I mean what's it even asking for? attribute bootstrap? does it mean keystone.bootstrap? | 11:26 |
| *** moshele has quit IRC | 11:26 | |
| thomas_oneill | and when and why is it asking for this? if you look at the play itself https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-keystone-client-config/tasks/main.yml#L41-L46 it's not asking for this attribute, so wtf D: | 11:27 |
| pas-ha | I think I understand... it needs those on the play level as it tries to pass it to the role... my bad, will fix | 11:29 |
| pas-ha | although still strange that gates are passing as they do not install keystone either... | 11:30 |
| thomas_oneill | pas-ha: but there's no mention of any 'bootstrap' in any of the files in the role directory | 11:31 |
| pas-ha | it is in the install.yml itself | 11:31 |
| pas-ha | https://github.com/openstack/bifrost/blob/master/playbooks/install.yaml#L12-L24 | 11:32 |
| *** moshele has joined #openstack-ironic | 11:32 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec https://review.openstack.org/445478 | 11:33 |
| lucasagomes | syntax errors :-( | 11:33 |
| thomas_oneill | pas-ha: oh look! https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-keystone-install/defaults/main.yml#L39 | 11:34 |
| thomas_oneill | oh that's a dict haha sorry | 11:35 |
| thomas_oneill | thought there was a thing called bootstrap and it was blank, but yeah I guess it shouldn't even be checking this because keystone is disabled by default | 11:35 |
| *** amotoki has joined #openstack-ironic | 11:36 | |
| *** hshiina has joined #openstack-ironic | 11:37 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Support to perform erase on SSD drives https://review.openstack.org/443542 | 11:37 |
| *** Nisha_Agarwal has quit IRC | 11:37 | |
| thomas_oneill | pas-ha: these kolla gates are failing fwiw: https://bugs.launchpad.net/kolla/+bug/1674483 | 11:38 |
| openstack | Launchpad bug 1674483 in kolla "Bifrost failing because of missing SUDO_USER" [Undecided,New] | 11:38 |
| pas-ha | well that is strange as the whole playbook is running under become unless it is overridden somewhere up https://github.com/openstack/bifrost/blob/36e32ecdc26035ecd853f830b30d42827662a68a/playbooks/install.yaml#L4 | 11:40 |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:42 | |
| *** bfournie has quit IRC | 11:43 | |
| *** athomas has quit IRC | 11:44 | |
| *** Nisha_Agarwal has quit IRC | 11:46 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Performs IPA erase_devices after Sanitize disk erase https://review.openstack.org/436450 | 11:47 |
| *** venki has quit IRC | 11:48 | |
| *** venki has joined #openstack-ironic | 11:48 | |
| *** faizy_ has quit IRC | 11:48 | |
| *** lucasagomes is now known as lucas-brb | 11:50 | |
| pas-ha | thomas_oneill: left a comment in that kolla bug | 11:50 |
| *** yolanda has quit IRC | 11:52 | |
| dtantsur | jroll, hey. seems like our new get_inventory could break tripleo: https://bugs.launchpad.net/tripleo/+bug/1674236 | 11:53 |
| openstack | Launchpad bug 1674236 in tripleo "CI / promotion: Nova isn't aware of the nodes that were registered with Ironic" [Critical,Triaged] | 11:53 |
| *** yolanda has joined #openstack-ironic | 11:54 | |
| *** vsaienko has joined #openstack-ironic | 11:57 | |
| *** tuanluong has quit IRC | 12:03 | |
| rook | lucas-brb: dtantsur pxe_ipmitool vs agent_ipmitool? | 12:05 |
| vdrok | dtantsur: oh, I have seen this actually. in my case, one of two nodes I've had was not picked up by nova during scheduling. | 12:05 |
| rook | what is the biggest difference here? | 12:06 |
| vdrok | could not figure out what's wrong with that | 12:06 |
| vdrok | all the records in placement/nova dbs were correct | 12:06 |
| vdrok | but, that was not right from the beginning, but after 4-5 times I've booted it | 12:06 |
| rook | jroll maybe you can answer the agent vs pxe driver differences? | 12:07 |
| dtantsur | rook, different ways to flash images | 12:07 |
| rook | ah -- so not so much how things are managed | 12:07 |
| dtantsur | rook, with pxe_* conductor flashes an image on an iscsi share published from the ramdisk, while in agent_* the agent fetches the image from an HTTP URL (e.g. swift tempurl) and flashes it | 12:07 |
| rook | dtantsur: roger, so the *_ipmi is just how things get flashed.. where ipmi is still used for both. | 12:09 |
| dtantsur | yep | 12:09 |
| dtantsur | vdrok, hmm | 12:09 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 12:09 |
| thomas_oneill | hey you know openstack gate checks, I know every time I push changes a zuul check is triggered, but how do I just check the latest gate stuff on master branches of projects? | 12:11 |
| thomas_oneill | or on other branches too? | 12:11 |
| dtantsur | thomas_oneill, what exactly are you trying to achieve? | 12:11 |
| thomas_oneill | dtantsur: it was just when pas-ha said that bifrost was passing the gates fine, just wanted to see it for myslef | 12:11 |
| thomas_oneill | re: the bug I mentioned a buffer or two up, I've just tested on a fresh box (I think) and having same issue. Testing with a previous commit now, prior to the one pas-ha mentioned. | 12:12 |
| thomas_oneill | But if it is the case that this commit is bad, then like he said it is strange that it made the gates. So I just wanted to look at that. | 12:12 |
| pas-ha | thomas_oneill: just make a dummy patch changing some string/comment to trigger the gate | 12:12 |
| thomas_oneill | oh okay lol | 12:13 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 12:13 |
| *** vmud213 has quit IRC | 12:16 | |
| thomas_oneill | pas-ha: confirmed running commit 7164cbfbd85b93e89fc0036a2d7c6428bcf4311e (one prior to the change you mentioned) has no issue, so yeah you're right | 12:16 |
| *** SerenaFe_ has quit IRC | 12:19 | |
| jroll | morning | 12:19 |
| jroll | dtantsur: yeah, was trying to help with that yesterday in nova channel | 12:19 |
| nicodemos | morning jroll, and Ironic | 12:19 |
| dtantsur | morning jroll. and what was the outcome? | 12:20 |
| jroll | dtantsur: some of the folks that know nova better were looking, too | 12:20 |
| *** bfournie has joined #openstack-ironic | 12:20 | |
| jroll | dtantsur: the outcome was mostly "wtf" | 12:20 |
| jroll | so far | 12:20 |
| dtantsur | ugh | 12:20 |
| *** gcb has quit IRC | 12:20 | |
| *** gcb has joined #openstack-ironic | 12:21 | |
| vdrok | morning jroll | 12:21 |
| jroll | morning vdrok | 12:22 |
| *** mgoddard has quit IRC | 12:22 | |
| jroll | dtantsur: dan or mriedem may have continued looking, I left a bit early yesterday | 12:22 |
| jroll | see also https://review.openstack.org/#/c/447520/ where they're debugging | 12:24 |
| patchbot | patch 447520 - nova - DNM: Log placement result in scheduler | 12:24 |
| dtantsur | fun | 12:24 |
| *** venki has quit IRC | 12:28 | |
| *** venki has joined #openstack-ironic | 12:28 | |
| *** kjw3 has quit IRC | 12:29 | |
| *** kjw3 has joined #openstack-ironic | 12:30 | |
| *** Lambert has quit IRC | 12:31 | |
| *** aparnav has quit IRC | 12:34 | |
| *** alexpilotti has joined #openstack-ironic | 12:37 | |
| *** jaosorior_brb is now known as jaosorior | 12:38 | |
| *** moshele has quit IRC | 12:38 | |
| *** zhurong has quit IRC | 12:39 | |
| *** moshele has joined #openstack-ironic | 12:41 | |
| *** trown|outtypewww is now known as trown | 12:45 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death https://review.openstack.org/446571 | 12:46 |
| *** dprince has joined #openstack-ironic | 12:48 | |
| thomas_oneill | hmmm apparently an inspection is occurring but the baremetal host itself is just on a shell prompt with tc@box:~$ and nothing seems to be happening :S | 12:51 |
| thomas_oneill | nothing in my logs either | 12:51 |
| thomas_oneill | apart from loads of 200s and periodic callback things... | 12:52 |
| thomas_oneill | I guess it'll time out and then tell me what the error is... it's been like 10 minutes already -.- | 12:52 |
| *** kencjohnston_ has quit IRC | 12:56 | |
| *** jcoufal has joined #openstack-ironic | 12:58 | |
| *** dyasny has quit IRC | 13:01 | |
| thomas_oneill | oh it's doing stuff again :D | 13:03 |
| thomas_oneill | maybe cleaning is accidentally enabled somewhere, taking ages | 13:03 |
| milan | morning Ironic! :) | 13:04 |
| dtantsur | lucas-brb, sushy is under our governance, yay! | 13:05 |
| dtantsur | hey milan, it's great that you survived Iceland, as we assigned everything to you! | 13:05 |
| *** sasha2 has joined #openstack-ironic | 13:05 | |
| *** baoli has joined #openstack-ironic | 13:06 | |
| vdrok | good morning milan :) | 13:06 |
| *** baoli has quit IRC | 13:06 | |
| *** Goneri has joined #openstack-ironic | 13:06 | |
| milan | dtantsur, lol :D thank you guys, how very thoughtful of you :P | 13:07 |
| milan | hey vdrok! :) | 13:07 |
| mariojv | good morning ironic | 13:07 |
| milan | morning mariojv! :) | 13:07 |
| mariojv | thanks for spec update lucas-brb, will look again today | 13:08 |
| mariojv | hi milan | 13:08 |
| *** baoli has joined #openstack-ironic | 13:08 | |
| * milan -> coffee :P | 13:08 | |
| *** baoli has quit IRC | 13:09 | |
| vdrok | so, it seems like the issue is in the filter sql statement here -- https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L657-L667 | 13:09 |
| thomas_oneill | damn, yeah I'm getting ""Inspection of node 5975770a-3495-419f-b86c-cf176ddd58b4 failed, last error: timeout reached while inspecting the node" | 13:09 |
| *** baoli has joined #openstack-ironic | 13:09 | |
| vdrok | it filters out the available resource provider | 13:09 |
| vdrok | or the next one :) | 13:10 |
| *** mat128 has joined #openstack-ironic | 13:11 | |
| *** rloo has joined #openstack-ironic | 13:11 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:14 | |
| *** lucas-brb is now known as lucasagomes | 13:14 | |
| lucasagomes | dtantsur, w00t! | 13:15 |
| lucasagomes | mariojv, thanks for the comments there, no rush! | 13:15 |
| vdrok | actually, this one https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L671-L672 | 13:15 |
| jroll | vdrok: https://bugs.launchpad.net/nova/+bug/1674236 | 13:15 |
| openstack | Launchpad bug 1674236 in tripleo "CI / promotion: Nova isn't aware of the nodes that were registered with Ironic" [Critical,Triaged] | 13:15 |
| jroll | dtantsur: ^^ tracked it down | 13:16 |
| sbezverk | mgoddard: ping | 13:16 |
| dtantsur | jroll, oh, nice catch! | 13:16 |
| vdrok | jroll: cool, thanks! | 13:17 |
| *** pmannidi has quit IRC | 13:18 | |
| *** pmannidi has joined #openstack-ironic | 13:19 | |
| *** zackf has joined #openstack-ironic | 13:19 | |
| *** dtantsur is now known as dtantsur|bbl | 13:20 | |
| *** amoralej is now known as amoralej|lunch | 13:20 | |
| *** zackf has quit IRC | 13:21 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Remove logging translation calls from ironic.common https://review.openstack.org/448089 | 13:22 |
| *** mvpnitesh has quit IRC | 13:22 | |
| *** links has quit IRC | 13:24 | |
| *** cdearborn has joined #openstack-ironic | 13:25 | |
| *** venki has quit IRC | 13:25 | |
| *** venki has joined #openstack-ironic | 13:25 | |
| *** crushil has joined #openstack-ironic | 13:27 | |
| *** jtomasek has quit IRC | 13:29 | |
| *** faizy_ has joined #openstack-ironic | 13:31 | |
| *** mgoddard has joined #openstack-ironic | 13:31 | |
| *** [1]cdearborn has quit IRC | 13:32 | |
| *** rbudden has joined #openstack-ironic | 13:32 | |
| *** faizy__ has joined #openstack-ironic | 13:33 | |
| *** faizy_ has quit IRC | 13:36 | |
| *** jtomasek has joined #openstack-ironic | 13:43 | |
| *** mvpnitesh has joined #openstack-ironic | 13:43 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Remove logging translation calls from ironic.common https://review.openstack.org/448089 | 13:45 |
| *** hoangcx has joined #openstack-ironic | 13:49 | |
| *** faizy__ has quit IRC | 13:51 | |
| *** amoralej|lunch is now known as amoralej | 13:53 | |
| lucasagomes | rloo, jroll fwiw, raising DriverLoadError in the constructor of the interfaces if the requirements are not installed does work | 13:53 |
| *** dyasny has joined #openstack-ironic | 13:53 | |
| jroll | lucasagomes: woot, I figured as much | 13:54 |
| rloo | lucasagomes: nice! :) | 13:54 |
| lucasagomes | yeah, forgot to update you yesterday so I'm saying now as I make the changes in the patch :-) | 13:54 |
| *** mat128 has quit IRC | 13:55 | |
| rloo | lucasagomes: heh. i assume the commit will mention it too :) | 13:55 |
| lucasagomes | rloo, yeah | 13:55 |
| *** mtanino has joined #openstack-ironic | 13:59 | |
| lucasagomes | rloo, JayF btw, re i18n of log messages... is there a new guide for the expected changes ? | 14:00 |
| lucasagomes | e.g, if I have an error message that is going to be raised in an exception an also logged that should have the _() translation thingy, right ? | 14:00 |
| rloo | lucasagomes: no guide. i was hoping someone would update the bug associated with it, to indicate what to do. | 14:01 |
| rloo | lucasagomes: that is a good question. joanna asked that yesterday. i don't know the answer. | 14:01 |
| lucasagomes | hmm | 14:01 |
| rloo | lucasagomes: it seems that the, or one of the, reasons for no i18n in logs, is cuz they want them in english so they can search on the internet. | 14:01 |
| rloo | lucasagomes: which makes me think we should not _() them. | 14:02 |
| *** moshele has quit IRC | 14:02 | |
| rloo | lucasagomes: i guess it is ugly, but it is just a bit more work to do two versions of the msg, _() and not? but i haven't thought much about it. | 14:02 |
| lucasagomes | rloo, right yeah idk as well. I would prefer to have a guide or more info on how things are expected before doing any blind changes anyway | 14:03 |
| rloo | might be worth asking the rest of the community. am hoping someone else will do it.i'm trying to focus and i'd like people who sign up for things to take more responsibility/stretch/ask/find answer... | 14:03 |
| mariojv | i think dmitry mentioned that _() had another purpose besides translation, as well | 14:03 |
| * mariojv checks meeting logs | 14:03 | |
| rloo | lucasagomes: i agree with you about guidelines. | 14:03 |
| mariojv | http://eavesdrop.openstack.org/irclogs/%23openstack-meeting-3/%23openstack-meeting-3.2017-03-20.log.html#t2017-03-20T17:56:22 | 14:04 |
| rloo | lucasagomes: (but i'm not taking responsibility for that) | 14:04 |
| lucasagomes | mariojv, ++ yeah, so things like exception makes sense to be translated | 14:04 |
| lucasagomes | since the user can see it | 14:04 |
| mariojv | lucasagomes: yeah, i think so | 14:04 |
| mariojv | well, i don' | 14:04 |
| lucasagomes | yeah hmm | 14:05 |
| rloo | mariojv: i think he means all the other i18n wrappers should disappear, and we should use _() for anything userfacing, eg Exceptions | 14:05 |
| mariojv | *don't know if that necessarily means "translated" | 14:05 |
| lucasagomes | thing is that exceptions are also logged | 14:05 |
| lucasagomes | yeah it's confusing /me waits for a guide | 14:05 |
| rloo | lucasagomes: the actual Exception isn't logged, we do that by LOG.exception(). that's separate. | 14:06 |
| rloo | lucasagomes: it is fine to wait for a guide. i think we said it is ok to have i18n until we remove them all from the code or something like that. | 14:06 |
| lucasagomes | rloo, yeah, I was just thinking on doing it now since I'll change the code anyway | 14:07 |
| lucasagomes | but I assumed there was more info about it | 14:07 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups https://review.openstack.org/436878 | 14:07 |
| mariojv | it'd be very confusing if some logs are in english and other logs are in another language | 14:07 |
| mariojv | although, i guess that's the case today if there's not a translation for something yet | 14:07 |
| rloo | lucasagomes: gotcha. and ++ mariojv. | 14:07 |
| *** zackf has joined #openstack-ironic | 14:07 | |
| *** causten has joined #openstack-ironic | 14:08 | |
| rloo | mariojv: that is a good reason to make sure nothing is translated. | 14:08 |
| rloo | nothing in logs i mean. | 14:08 |
| mariojv | i agree | 14:08 |
| mariojv | i also wonder what people who use deployments in other languages think | 14:09 |
| rloo | i honestly think we should leave all that stuff in and operators should just install openstack using english/whatever locale setting. but anyway. | 14:09 |
| mariojv | but i'm sure oslo or whoever's coming up with these new guidelines have thought about it | 14:09 |
| rloo | mariojv: i suggest you read the email thread. it was NOT oslo. it was i18n team. | 14:09 |
| mariojv | ack | 14:09 |
| *** mvpnitesh has quit IRC | 14:11 | |
| jroll | lucasagomes: rloo: re guides, this may be somewhat helpful https://review.openstack.org/#/c/446762/2 | 14:12 |
| patchbot | patch 446762 - oslo.i18n - add notes about skipping log translation setup | 14:12 |
| *** zackf has quit IRC | 14:12 | |
| lucasagomes | cheers, will take a look | 14:12 |
| rloo | jroll: i took a look at that. i don't think it sez what to do 'in the future'. | 14:14 |
| jroll | rloo: it says what not to do | 14:15 |
| rloo | jroll: well, it doesn't address lucasagomes ' question :) | 14:16 |
| jroll | rloo: we continue to _() exceptions that are raised, as they may be returned to users | 14:17 |
| jroll | for lucasagomes question, I guess we would do | 14:17 |
| jroll | msg = 'foo' | 14:17 |
| jroll | LOG.exception(msg) | 14:17 |
| jroll | raise FooException(_(foo)) | 14:17 |
| *** chlong has joined #openstack-ironic | 14:18 | |
| rloo | jroll: right. but for complicated msgs, msg = 'foo %(bar)s and chocolate %(bars)s'. | 14:18 |
| rloo | jroll: sometimes the message string is generated first. | 14:18 |
| vdrok | good morning mariojv and rloo | 14:18 |
| rloo | hi vdrok! | 14:19 |
| jroll | rloo: if you want to be strict on it, then interpolate twice | 14:19 |
| lucasagomes | jroll, right, so we should not reuse the same variable for both log and exceptions... It's ok, but might duplicate some of the messages in the code | 14:19 |
| jroll | or interpolate once and sometimes log it translated | 14:19 |
| rloo | jroll: that's what i think too. | 14:19 |
| * jroll doesn't really care, we have bigger problems to think about | 14:19 | |
| mariojv | hi vdrok | 14:19 |
| lucasagomes | jroll, ++ | 14:19 |
| rloo | ++ jroll. that's why i'm trying to stay away from it. | 14:20 |
| lucasagomes | honestly, translation for projects like ironic or things that runs in the background that only operators will see is a waste of time IMHO | 14:20 |
| lucasagomes | as a non-english native, I would never run it in my mother language | 14:20 |
| lucasagomes | I agree for UX that trnaslation helps but other than that is just a waste of time, money and effort | 14:21 |
| rloo | jroll, dtantsur|bbl: btw, i just -1'd the client patch for the dynamic drivers. some of the options are incorrect. do we need two of us to 'approve' what they ought to be? | 14:21 |
| jroll | lucasagomes: en_US.UTF-8 or bust | 14:21 |
| rloo | jroll, dtantsur|bbl: https://review.openstack.org/#/c/419274/17 | 14:21 |
| patchbot | patch 419274 - python-ironicclient - Extends driver-list, driver-show supporting new ha... | 14:21 |
| lucasagomes | jroll, pretty much | 14:21 |
| *** jkilpatr has quit IRC | 14:21 | |
| jroll | rloo: if you say they're incorrect, you've already decided, right? :) | 14:22 |
| rloo | jroll: yes, but who made me the dictator? :) | 14:22 |
| jroll | rloo: is there something other than the --long / --detail issues? | 14:22 |
| rloo | jroll: --fields addition. | 14:22 |
| jroll | rloo: I don't think driver APIs support passing fields | 14:22 |
| rloo | jroll: i could comment explicitly and you could ack? or i could update spec? or ? | 14:23 |
| rloo | jroll: oh, right. so a future thing then. | 14:23 |
| jroll | rloo: I'll just comment | 14:23 |
| rloo | jroll: so just --long vs --detail, and no --detail for driver-show. | 14:23 |
| jroll | yep | 14:23 |
| rloo | thx | 14:23 |
| jroll | those are unwritten standards, no need for quorum :) | 14:23 |
| rloo | to be honest, sometimes people that don't know us, don't know who to listen to. which is understandable. | 14:24 |
| *** chlong has quit IRC | 14:24 | |
| jroll | rloo: sure, if there's a disagreement, we should fix that :) | 14:24 |
| rloo | jroll: definitely. it isn't disagreement, it is giving direction to people. i think we wanted that patch done soon. i think there is another thing missing too, need to check... | 14:25 |
| jroll | nod | 14:26 |
| rloo | ok it is good. the latest version is 31. why did we up that and not get this feature in. oh well. | 14:26 |
| rloo | hey lucasagomes, reviewing the redfish spec changes. when/if it is changed to use the python-redfish library, will there need to be code changes to this driver? | 14:29 |
| rloo | lucasagomes: i mean is the idea that both can/could be used, or that only one? | 14:30 |
| lucasagomes | rloo, oh yes, but for the user of the driver it will be transparently | 14:30 |
| lucasagomes | that's what I tried to say there | 14:30 |
| lucasagomes | rloo, the idea is only one to be used | 14:30 |
| lucasagomes | tho it's all open for suggestion | 14:31 |
| lucasagomes | I mean, we can not use python-redfish as for now | 14:31 |
| rloo | lucasagomes: ok. so you think we'd have this one driver, but have some configuration/new code to support both of the libraries? | 14:31 |
| lucasagomes | so I'm trying not to speculate much on it | 14:31 |
| lucasagomes | rloo, yeah new code, I honestly I avoid any configuration for that | 14:32 |
| lucasagomes | but that's personal, again, we don't even know when that might happen | 14:32 |
| rloo | lucasagomes: well, if the 'API' from the user's point of view won't change, then it will have to be the same 'named' (entrypoint) driver, and some code changes will need to be done. the spec makes/made it sound like you could just replace one library with the other, so i wondered. | 14:32 |
| lucasagomes | so I would postpone that conversation when time comes | 14:32 |
| rloo | lucasagomes: that is fine, but we should clarify it so people don't assume that it is just a replacement. | 14:33 |
| lucasagomes | rloo, oh right, so I need to reword it | 14:33 |
| lucasagomes | if that's how it sounds | 14:33 |
| rloo | lucasagomes: am thinking. will propose something now that i understand. | 14:33 |
| lucasagomes | rloo, awesome right on | 14:33 |
| lucasagomes | I will update it as soon as you finish the review then | 14:34 |
| *** jkilpatr has joined #openstack-ironic | 14:34 | |
| *** vikrant has quit IRC | 14:36 | |
| *** vishwanathj has joined #openstack-ironic | 14:36 | |
| *** e0ne has quit IRC | 14:37 | |
| TheJulia | Good morning | 14:40 |
| mjturek | gm TheJulia! | 14:40 |
| *** alexpilotti has quit IRC | 14:40 | |
| *** Syed__ has joined #openstack-ironic | 14:40 | |
| *** mat128 has joined #openstack-ironic | 14:42 | |
| rloo | lucasagomes: one more question. or two. wrt the two fields, redfish_root_uri and redfish_system_id. how does the library know how to create the full URI? does it call some public API of the redfish controller? | 14:42 |
| lucasagomes | rloo, the root URI is the "entry point" for it | 14:43 |
| lucasagomes | from there the library should be able to find the resources and their links based on the version of redfish (or another OEM identifier) | 14:44 |
| rloo | lucasagomes: so with the root URI, you're assuming there is only one 'system' thing? | 14:44 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd https://review.openstack.org/447642 | 14:44 |
| rloo | lucasagomes: i mean, will a controller *only* have '/systems', or '/Nodes' but not both? | 14:44 |
| lucasagomes | rloo, I assume there's links to the collection of resources (managers, systems, chassis etc...) | 14:44 |
| lucasagomes | rloo, AFAIUI yeah either /Systems or /Nodes | 14:44 |
| TheJulia | So.. Moonshots | 14:45 |
| rloo | lucasagomes: or are you assuming the redfish_system_id is unique throughout. | 14:45 |
| lucasagomes | but that's an assumption based on what I know about redfish | 14:45 |
| lucasagomes | and the mockups etc... | 14:45 |
| TheJulia | are /Systems/1 /Systems/2 /Systems/3 | 14:45 |
| lucasagomes | schema, I don't know if someone will have both | 14:45 |
| lucasagomes | TheJulia, ++ yeah Systems/'s a collection | 14:45 |
| rloo | lucasagomes: is there any way to find out about the assumption. cuz TheJulia just invalidated that assumption... | 14:45 |
| TheJulia | rloo: it entirely depends on how the firmware was written, and how the hardware it controls works | 14:46 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP Fix options logging https://review.openstack.org/448132 | 14:46 |
| TheJulia | All we can do is be as generic as possible and make _no_ assumptions | 14:46 |
| lucasagomes | rloo, I think the best resource to find that is look at the schemas provided by the DMTF liek: http://redfish.dmtf.org/schemas/v1/ServiceRoot.v1_0_0.json | 14:46 |
| rloo | TheJulia: so we cannot assume then. | 14:46 |
| lucasagomes | there you can see all the collections | 14:46 |
| lucasagomes | TheJulia, yeah :-/ | 14:47 |
| lucasagomes | rloo, and there you can see it's called "Systems" that's why "Nodes" came as a surprise to me | 14:47 |
| rloo | lucasagomes: reading schemas is torture... | 14:47 |
| jroll | so much for a standard :D | 14:47 |
| TheJulia | full URL, provide some doc examples, and cross our fingers? | 14:47 |
| lucasagomes | rloo, I know, but... I honestly don't have a better source | 14:47 |
| rloo | lucasagomes: who used Nodes? they are wrong then... should we support it? | 14:47 |
| lucasagomes | rloo, there was a comment from waynen about it in the original spec | 14:48 |
| TheJulia | jroll: s/standard/standard recommendation/ ;) | 14:48 |
| *** e0ne has joined #openstack-ironic | 14:48 | |
| jroll | TheJulia: standard framework? :P | 14:48 |
| lucasagomes | rloo, I personally have never seem /Nodes, but I don't have much experience with different hardware that supports redfish anyway | 14:48 |
| TheJulia | jroll: i like! | 14:48 |
| openstackgerrit | Xavier proposed openstack/ironic master: Introduces onetime boot flag in oneview drivers https://review.openstack.org/436469 | 14:48 |
| jroll | TheJulia: it's like neutron for managing your hardware | 14:49 |
| TheJulia | rloo: I _think_ nodes is more for hardware managers, kind of like oneview | 14:49 |
| lucasagomes | rloo, that's the reason to make that split btw, these differences I think would be better handled in the library and exposed to the users in a consistent form | 14:49 |
| lucasagomes | than actually having the ironic driver to keep trying to handle all that differences by itself | 14:49 |
| Serlex | Hi, im trying to set " provision_state" from error to deleted, the command throws no errors but state doesn't seem to change | 14:50 |
| TheJulia | jroll: exactly! | 14:50 |
| rloo | lucasagomes: %#@$#$ I really was hoping to review your patch and approve, and at the same time get up to speed with the redfish driver. i guess that is happening except i can't yet approve... | 14:50 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-lib master: Test that configdrive disk exist with test -e https://review.openstack.org/446950 | 14:50 |
| rloo | lucasagomes: i have to go to a meeting, so will need to look into it more later and read the comments in the code patch... | 14:50 |
| lucasagomes | rloo, I know right ? No worries | 14:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/448137 | 14:51 |
| rloo | lucasagomes: heh. | 14:51 |
| lucasagomes | I mean, I'm trying to make design it the best way as I understand but knowing that I don't understand it much heh | 14:51 |
| lucasagomes | so ¯\_(ツ)_/¯ | 14:51 |
| lucasagomes | s/make// | 14:55 |
| TheJulia | All any of us can do really | 14:56 |
| *** faizy has joined #openstack-ironic | 14:57 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 14:58 |
| lucasagomes | yeah :-/ | 14:58 |
| *** baha has joined #openstack-ironic | 14:58 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP Fix options logging https://review.openstack.org/448132 | 14:59 |
| NobodyCam | Good Morning Ironic'ers | 15:01 |
| NobodyCam | hey hey TheJulia: any hi priority reviews that need a looking at? | 15:01 |
| thomas_oneill | dib_os_element should go in the baremetal group vars, right? rather than the localhost/target? | 15:02 |
| TheJulia | NobodyCam: Just one you could look at on bifrost, stable/ocata lock rev. I think it is top of the queue right now | 15:02 |
| TheJulia | thomas_oneill: that is a super super good question | 15:03 |
| thomas_oneill | lol thanks :D | 15:03 |
| *** vsaienko has quit IRC | 15:03 | |
| TheJulia | one which I need to look up :) | 15:03 |
| TheJulia | one moment | 15:03 |
| TheJulia | target | 15:04 |
| thomas_oneill | configdrive failed to do the ssh pub key thing on my last deployment too, I noticed there was a comment that said about putting it in both the baremetal and the localhost/target so I'm going to try that now, if that doesn't work though I need to work out what's going wrong there too | 15:04 |
| *** hoangcx has quit IRC | 15:04 | |
| thomas_oneill | oh really? oh yeah I suppose the dib will be image-building on the target | 15:04 |
| TheJulia | Yeah | 15:04 |
| jlvillal | Good morning Ironic :) | 15:04 |
| thomas_oneill | I wonder if there's any harm in putting these kinds of vars in both places. And I wonder why there can't just be a single group_vars file for this project. | 15:04 |
| TheJulia | It should be building on target, you'd have to make a custom playbook if you wanted to build a special dib image per parameters known about each host from inventory. wouldn't be difficult :) | 15:05 |
| TheJulia | thomas_oneill: There should be no harm in that | 15:05 |
| TheJulia | jlvillal: good morning | 15:05 |
| jlvillal | :) | 15:05 |
| *** venki has quit IRC | 15:05 | |
| lucasagomes | NobodyCam, morning o/ | 15:05 |
| NobodyCam | morning lucasagomes :) | 15:05 |
| NobodyCam | o/ | 15:06 |
| joanna | morning TheJulia, lucasagomes, jlvillal, NobodyCam | 15:06 |
| *** venki has joined #openstack-ironic | 15:06 | |
| lucasagomes | joanna, morning o/ | 15:06 |
| NobodyCam | morning joanna | 15:06 |
| jlvillal | :) | 15:06 |
| *** vsaienko has joined #openstack-ironic | 15:07 | |
| NobodyCam | morning jlvillal :) | 15:09 |
| jlvillal | :) | 15:09 |
| NobodyCam | TheJulia: LGTM +A | 15:09 |
| *** faizy has quit IRC | 15:10 | |
| TheJulia | NobodyCam: thank you, that should hopefully save headache for down the road :) | 15:11 |
| *** faizy has joined #openstack-ironic | 15:11 | |
| NobodyCam | :) | 15:11 |
| *** hshiina has quit IRC | 15:12 | |
| *** dtantsur|bbl is now known as dtantsur | 15:14 | |
| dtantsur | morning TheJulia, NobodyCam, joanna, jlvillal, rloo | 15:14 |
| jlvillal | :) | 15:14 |
| NobodyCam | Morning dtantsur :) | 15:14 |
| *** faizy_ has joined #openstack-ironic | 15:14 | |
| rloo | hi dtantsur, NobodyCam, jlvillal, joanna | 15:15 |
| jlvillal | :) | 15:15 |
| NobodyCam | Morning rloo :) | 15:15 |
| joanna | I think I found 1 issue with log translations removal, I asked yesterday, but go only 1 response, so maybe I'll be more lucky today :) | 15:16 |
| joanna | we sometimes have sth like msg=_(<>), LOG.error(msg), raise Exception(msg) | 15:16 |
| joanna | morning rloo! | 15:16 |
| rloo | joanna: we had a short discussion about it earlier today | 15:16 |
| *** baoli has quit IRC | 15:17 | |
| rloo | joanna: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-03-21.log.html#t2017-03-21T14:00:49 | 15:17 |
| *** faizy__ has joined #openstack-ironic | 15:17 | |
| rloo | joanna: no definitive solution but... | 15:17 |
| joanna | rloo: awesome, thanks! looking... | 15:17 |
| *** faizy has quit IRC | 15:18 | |
| * jlvillal wonders if anyone else got spam email about a "github t-shirt" | 15:20 | |
| *** faizy_ has quit IRC | 15:20 | |
| *** faizy__ has quit IRC | 15:21 | |
| *** faizy__ has joined #openstack-ironic | 15:21 | |
| *** faizy__ has quit IRC | 15:24 | |
| *** rpioso has joined #openstack-ironic | 15:24 | |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Set branches to stable/ocata for ocata branch https://review.openstack.org/441232 | 15:24 |
| *** faizy__ has joined #openstack-ironic | 15:24 | |
| *** vsaienko has quit IRC | 15:25 | |
| *** vsaienko has joined #openstack-ironic | 15:25 | |
| joanna | rloo: thanks, will duplicate the messages. | 15:25 |
| *** faizy__ has quit IRC | 15:26 | |
| *** sayantani01 has joined #openstack-ironic | 15:27 | |
| *** faizy__ has joined #openstack-ironic | 15:27 | |
| joanna | I'm wondering if there's some sort of unique error id/timestamp shown in Horizon? | 15:28 |
| *** sayantani01 has quit IRC | 15:29 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:29 | |
| *** sayantani01 has joined #openstack-ironic | 15:29 | |
| *** faizy__ has quit IRC | 15:29 | |
| *** faizy__ has joined #openstack-ironic | 15:30 | |
| venki | Hi, i'm beginner.. I want to do ironic booting in DELL server (PowerEdge T430)... | 15:32 |
| venki | Anyone has previous knowledge ... Pls respond | 15:32 |
| dtantsur | venki, hi and welcome. You may want to start with the install guide: https://docs.openstack.org/project-install-guide/baremetal/draft/ | 15:33 |
| *** jrist has quit IRC | 15:33 | |
| venki | @dtantsur : ok ... | 15:36 |
| venki | @dtantsur : can i use the devstack for ironic setup ? | 15:36 |
| dtantsur | venki, certainly: https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 15:37 |
| venki | dtantsur : But i tried that configuration file , it is for fake baremetal right ? | 15:38 |
| dtantsur | ah, yeah, you want devstack + dell | 15:39 |
| dtantsur | hmm | 15:39 |
| *** rama_y_ has joined #openstack-ironic | 15:39 | |
| * dtantsur wonders if somebody here remembers how to use devstack with bare metal | 15:40 | |
| dtantsur | rpioso, around re ^^^? | 15:40 |
| *** jrist has joined #openstack-ironic | 15:40 | |
| venki | dtantsur: yes ... but i'm struggling for the same... | 15:41 |
| venki | ***dtantsur: Finally, a Big quesion ,.. Is it possible to do the devstack with baremetal ? | 15:42 |
| dtantsur | venki, yes, I mentioned rpioso because they (dell) run dell CI with devstack, I think | 15:43 |
| dtantsur | venki, looking at devstack source code, you may need to populate https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L80-L105 | 15:44 |
| dtantsur | and set IRONIC_DEPLOY_DRIVER to pxe_drac | 15:44 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Disable guessing content type from extension in resource names. https://review.openstack.org/408597 | 15:44 |
| dtantsur | mgould, hey, did you succeed in using devstack with dell boxes? maybe you can help venki | 15:44 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Disable guessing content type from extension in resource names. https://review.openstack.org/408597 | 15:44 |
| *** udesale has quit IRC | 15:46 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Disable guessing content type from extension in resource names. https://review.openstack.org/408597 | 15:46 |
| venki | dtantsur: I will look into that ... :) thanks for the help :) | 15:47 |
| dtantsur | np | 15:47 |
| aNuposic | Hi dtantsur, wrt https://review.openstack.org/#/c/444610/ so do you think sqlite headers won't be required while installing Py35 on Fedora? | 15:48 |
| patchbot | patch 444610 - ironic - Updates documentation to install PySqlite3 | 15:48 |
| dtantsur | aNuposic, why would you need headers for a binary package that is already built? | 15:49 |
| dtantsur | I think the package will pull everything it needs | 15:49 |
| dtantsur | you need the headers on ubuntu, as we're building a package for it from source | 15:49 |
| mgould | dtantsur: no, I never got devstack to work with dell hardware | 15:49 |
| *** pbandark has joined #openstack-ironic | 15:49 | |
| mgould | I ended up installing via tripleo | 15:50 |
| aNuposic | dtantsur, okay i see, i did not tried on Fedora but on Ubuntu where they are required, so thought to add under fedora. Cool, got it will update that | 15:50 |
| dtantsur | thnx | 15:51 |
| mgould | I also remember devstack-on-fedora being a world of pain; my devstack VM is centos | 15:52 |
| *** vsaienko has quit IRC | 15:52 | |
| *** hamzy has quit IRC | 15:53 | |
| mgould | venki: here's the local.conf sambetts_ uses for devstack-controlling-baremetal: http://paste.openstack.org/show/516174/ | 15:53 |
| *** pbandark has quit IRC | 15:53 | |
| *** vsaienko has joined #openstack-ironic | 15:54 | |
| mgould | you then need to populate $IRONIC_HWINFO_FILE with hardware info, in a format that's only documented in comments in the ironic devstack plugin | 15:54 |
| *** rbartal has quit IRC | 15:54 | |
| mgould | also, any instructions that tell you to boot a Cirros-based image on baremetal are full of lies | 15:56 |
| mgould | this does not work | 15:56 |
| openstackgerrit | Merged openstack/bifrost master: Use env lookup to determine HOME for SSH pubkey https://review.openstack.org/435503 | 15:57 |
| TheJulia | mostly full of lies..... You might be able to boot, but you likely won't be able to login or have network drivers :) | 15:57 |
| * mgould tries to make sense of his increasingly deranged notes from early February | 15:59 | |
| mgould | venki: hwinfo file format is described at https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L80 | 16:00 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages from ironic-inspector https://review.openstack.org/447884 | 16:01 |
| rama_y_ | dtantsur: this dmidecode patch has a +2 and a +1: https://review.openstack.org/#/c/406790/ ; I was wondering if this could get another review; thanks. | 16:01 |
| patchbot | patch 406790 - ironic-inspector-specs - Collect Bios, Memory and Processor Info from Dmide... | 16:01 |
| mgould | venki: pxe_ipmitool and pxe_drac drivers ought to work for you | 16:02 |
| mgould | maybe agent_ipmitool too, but I've never tried it | 16:02 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revert the move of the logger setup https://review.openstack.org/448132 | 16:02 |
| jroll | there's no reason agent_ipmitool shouldn't work on any hardware that pxe_ipmitool works on :) | 16:03 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revert the move of the logger setup https://review.openstack.org/448132 | 16:04 |
| vdrok | rloo: vsaienk01 apparently move of the logger setup is not that easy ^^ :( | 16:05 |
| rloo | vdrok: too bad. i liked it in conf/. I did wonder but figured it worked :) | 16:06 |
| vdrok | rloo: yeah. the tests will be still complaining about the missing logger, but, yeah. will need to dig more into this | 16:07 |
| mgould | venki: AFAICT you'll need to build your own ramdisk image, because devstack uses a cirros-based one by default: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L540 | 16:08 |
| *** vsaienko has quit IRC | 16:09 | |
| mgould | TheJulia: I can hardly find any unit tests for the Django side of ironic-ui, where ought I to be looking? | 16:09 |
| venki | mgould: I'll give a hard try... thanks for all the guide :) | 16:10 |
| mgould | venki: good luck! | 16:11 |
| *** vsaienko has joined #openstack-ironic | 16:11 | |
| mgould | as I say, I ended up giving up on devstack and installing openstack via tripleo via rpm | 16:11 |
| *** Nisha_Agarwal has quit IRC | 16:12 | |
| * mgould tries to find upstream docs for that | 16:13 | |
| TheJulia | mgould: so, it is more angularjs panels that call a very thin api overlay. I suspect why you've found is the very thin overlay | 16:13 |
| *** amotoki has quit IRC | 16:13 | |
| *** rama_y_ has quit IRC | 16:13 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative tests for VIF attach/detach operations https://review.openstack.org/435952 | 16:13 |
| mgould | TheJulia: yeah, here's where I've got to: https://review.openstack.org/#/c/447085/ | 16:13 |
| patchbot | patch 447085 - ironic-ui - Show boot device in Node Details/Configuration | 16:13 |
| *** gfhellma has joined #openstack-ironic | 16:14 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-ui master: Adds Ironic-UI Devstack plugin https://review.openstack.org/446298 | 16:16 |
| *** soliosg1 has quit IRC | 16:16 | |
| venki | mgould: Is there anything i need to configure on Dell Baremetal machine ? | 16:17 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:18 | |
| mgould | venki: on the node you're deploying onto? | 16:18 |
| mgould | set it to boot in legacy BIOS mode, I think | 16:18 |
| venki | mgould: yes | 16:18 |
| venki | okey ... (y) :) | 16:18 |
| venki | i'll try that... | 16:19 |
| *** zackf has joined #openstack-ironic | 16:19 | |
| mgould | ironic ought to do the rest | 16:19 |
| *** e0ne has quit IRC | 16:20 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 16:21 |
| *** udesale has joined #openstack-ironic | 16:21 | |
| *** zackf has quit IRC | 16:26 | |
| venki | mgould: okey | 16:26 |
| *** jaosorior has quit IRC | 16:29 | |
| *** soliosg has left #openstack-ironic | 16:33 | |
| *** milan has quit IRC | 16:34 | |
| *** fragatina has joined #openstack-ironic | 16:35 | |
| rloo | hey lucasagomes, you still around? am trying to understand (w/o spending too much time on your driver code). what parts of the URI does your code need? 1. some base URI to connect to the redfish controller? which part is this, the <scheme>://<authority>/ or <scheme>://<authority>/<root service and version>/ ? | 16:36 |
| rloo | 2. the URI of the resource (BM node, right)? | 16:36 |
| dtantsur | 2. s/URI/ID/ I think | 16:36 |
| *** baha has quit IRC | 16:38 | |
| rloo | dtantsur: oh. is the ID mentioned in the dmtf somewhere besides the URI section: http://redfish.dmtf.org/schemas/DSP0266_1.1.html#uris. that's the only part of the dmtf i've read so far. | 16:39 |
| lucasagomes | rloo, yeah around | 16:39 |
| *** faizy__ has quit IRC | 16:40 | |
| lucasagomes | rloo, as dtantsur said... /me checks that doc | 16:40 |
| *** soliosg has joined #openstack-ironic | 16:41 | |
| dtantsur | rloo, well, reading that /redfish/v1/Systems/2 is a unique resource ID, but it may be quite inconvenient to use for our purpose. What they also might have it a system ID, which could be handy for us. | 16:41 |
| alezil | https://www.irccloud.com/pastebin/69yOkRi4/ | 16:41 |
| alezil | This is what I found while reading. | 16:41 |
| rloo | dtantsur, lucasagomes: so they have three parts to the URI, that's why i wanted to understand what you need for the 'base' URI. | 16:41 |
| dtantsur | rloo, lucasagomes, oh, fun: http://redfish.dmtf.org/schemas/DSP0266_1.1.html#resource-identifier-property | 16:41 |
| dtantsur | it can be /redfish/v1/Systems/2 or even //another-service.com/redfish/v1/Systems/2 | 16:42 |
| mgould | dtantsur: what's would be the problem with using that string as an identifier on our side? | 16:42 |
| dtantsur | so, certain my assumptions are apparently wrong | 16:42 |
| rloo | alezil: yes, that's from the dmtf, right? (links are above) | 16:42 |
| dtantsur | mgould, probably it's not a problem, except for duplication between root_uri and resource_uri in this case | 16:43 |
| lucasagomes | dtantsur, irgh | 16:43 |
| mgould | alezil: yeah, it then goes on to say that parts 2 + 3 form the unique identifier | 16:43 |
| lucasagomes | dtantsur, probably we need to handle the whole @odata.id then when creating the objects | 16:43 |
| dtantsur | so we'll have root_uri=http://example.com/redfish/v1 and resource_id=/redfish/v1/Systems/1 | 16:43 |
| *** jcoufal has quit IRC | 16:43 | |
| dtantsur | maybe it's fine to duplicate them, dunno.... | 16:43 |
| mgould | riiiiight | 16:44 |
| dtantsur | rloo, mgould ^^^ | 16:44 |
| mgould | and http://example.com/redfish/v1 is the thing we need to hit to start enumerating resources? | 16:44 |
| dtantsur | what is really fun (and I suspected that already) is that a system can be on another server... | 16:44 |
| rloo | so i'm fine if we need to duplicate part of the uris. as long as it is clear, what those URIs are. | 16:44 |
| dtantsur | so we have have root_uri=http://example.com/redfish/v1 and resource_id=//another.example.com/redfish/v1/Systems/1 | 16:44 |
| dtantsur | infinite awesomeness | 16:44 |
| dtantsur | s/have have/can have/ | 16:45 |
| rloo | dtantsur, lucasagomes: so you need the root uri as <scheme>://<authority>/<root service and version> then, not just <scheme>..<authority>, right? | 16:45 |
| lucasagomes | rloo, ++ | 16:46 |
| dtantsur | dunno if we can rely on /redfish/v1 always to be there | 16:46 |
| mgould | dtantsur: where do you get that? AFAICT it says you shouldn treat the scheme+authority portion as part of the identifier, because it might be redirected | 16:46 |
| rloo | dtantsur, lucasagomes, et all. And then we need the unique URL for a resource (baremetal machine presumably), which is a resource path. | 16:47 |
| lucasagomes | dtantsur, I think we can, see 6.2:The root URI for this version of the Redfish protocol shall be "/redfish/v1/". | 16:47 |
| dtantsur | mgould, http://redfish.dmtf.org/schemas/DSP0266_1.1.html#resource-identifier-property "Resources within the same authority as the request URI shall be represented according to the rules of path-absolute defined by that specification. That is, they shall always start with a single forward slash ("/"). Resources within a different authority as the request URI shall start with a double-slash ("//") followed by the authority and | 16:47 |
| dtantsur | path to the resource." | 16:47 |
| rloo | so yeah, i think dtantsur is right. | 16:47 |
| lucasagomes | (and shall actually means MUST in the dmtf spec see appendix) | 16:47 |
| dtantsur | aha | 16:47 |
| dtantsur | so we can have redfish_address=example.com, redfish_port=80, redfish_resource_id=/redfish/v1/Systems/1 | 16:48 |
| mgould | dtantsur: oh, right | 16:48 |
| dtantsur | (I'm using variables similar to other drivers, i.e. address and port) | 16:48 |
| mgould | dtantsur: LGTM | 16:48 |
| dtantsur | lucasagomes, rloo, if we do that ^^^ we even avoid duplication | 16:48 |
| *** Daviey_ is now known as Daviey | 16:49 | |
| rloo | dtantsur, lucasagomes: i thought you needed the base URL to have the root service+version too | 16:49 |
| mgould | and the root resource /redfish/v1 is just another resource | 16:49 |
| dtantsur | rloo, lucasagomes just quoted 6.2:The root URI for this version of the Redfish protocol shall be "/redfish/v1/". | 16:49 |
| *** jcoufal has joined #openstack-ironic | 16:49 | |
| dtantsur | rloo, meaning, we can derive that from host+port | 16:49 |
| lucasagomes | dtantsur, hmm perhaps call it redfish_system_id ? I don't really like resource_id because there are multiple resources | 16:49 |
| lucasagomes | that's actually pointing to a system | 16:49 |
| dtantsur | lucasagomes, whatever, it was just an example | 16:49 |
| lucasagomes | right, I think some of that changes should be included in sushy as well | 16:51 |
| lucasagomes | the way we handle such id's | 16:51 |
| *** crushil has quit IRC | 16:51 | |
| *** jkilpatr_ has joined #openstack-ironic | 16:51 | |
| * lucasagomes looks into it | 16:51 | |
| rloo | dtantsur, lucasagomes: so i am starting to get a headache, trying to understand the dmtf 6.2 & 6.3. but if you can parse that, i think it will make it easier to figure out what the driver_info fields should be. | 16:53 |
| *** jkilpatr has quit IRC | 16:54 | |
| dtantsur | yeah, my proposal is above | 16:54 |
| Serlex | anyone with a working /tftpboot/? I'm getting permission denied when launching an instance, not sure what I missed | 16:54 |
| dtantsur | (module s/resource/system/) | 16:54 |
| dtantsur | Serlex, acl? selinux? | 16:54 |
| *** trown is now known as trown|lunch | 16:55 | |
| *** jaosorior has joined #openstack-ironic | 16:55 | |
| Serlex | its in a container | 16:55 |
| lucasagomes | rloo, dtantsur mgould right on, yeah I will take a look at this changes... luckily we haven't released any version of sushy still so we can accommodate those changes more easily | 16:55 |
| dtantsur | yep | 16:56 |
| Serlex | dtantsur http://pastebin.com/2jEz3wWh | 16:56 |
| lucasagomes | Serlex, try to run the tftp CLI and get a file from that tftp server | 16:56 |
| *** udesale has quit IRC | 16:57 | |
| lucasagomes | Serlex, also try ls -laZ on that folder to see the selinux labels (if you are running it) | 16:57 |
| Serlex | its disabled | 16:57 |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 16:59 |
| Serlex | "Failed to prepare to deploy. Error: [Errno 13] Permission denied: '/tftpboot/9cb7f801-ee89-4d6c-bb1b-f25fd5901c97'" | 17:00 |
| jlvillal | dtantsur: lucasagomes rloo jroll TheJulia and everyone else. If the config file for the conductor has incompatible options set. Should the conductor die early, as in on start-up. Or just log a message and then fail deploys later on? | 17:01 |
| dtantsur | jlvillal, if it's obvious that a deployment cannot succeed, the conductor should fail | 17:01 |
| jroll | ++ | 17:01 |
| jroll | fail as early as possible imo | 17:01 |
| dtantsur | if it may succeed depending on user input or non-input, then we should probably only log | 17:02 |
| jlvillal | This is in regards to aNuposic patch for Ceph radosgw support. If they set it to radosgw but don't set the credentials. | 17:02 |
| lucasagomes | jlvillal, fail to start yeah, I think oslo.config does some of that for us | 17:02 |
| dtantsur | but the latter is arguable | 17:02 |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Remove translator assignments from i18n https://review.openstack.org/448196 | 17:02 |
| mgould | jlvillal: fail early, definitely | 17:02 |
| jlvillal | That is my feeling to. To die early. | 17:02 |
| dtantsur | jlvillal, if it cannot work without credentials, we should fail | 17:02 |
| jroll | radosgw support for what, btw? | 17:02 |
| lucasagomes | Serlex, oh, it's not permission denied when trying to download a file from it but actually ir-conductor writing to it | 17:02 |
| jlvillal | configdrive support. Instead of swift | 17:02 |
| jroll | ah | 17:02 |
| lucasagomes | Serlex, is the ir-cond and tftpd running in the same container ? | 17:02 |
| jroll | jlvillal: yeah, that will fail all builds, kill it at startup | 17:02 |
| Serlex | no its sliced up into different containers like a turkey at christmas | 17:03 |
| jlvillal | Thanks. | 17:03 |
| jlvillal | aNuposic: ^^^^^ | 17:03 |
| lucasagomes | Serlex, if not, I think it may be somehow related of how you pass that bind mount to the ir-conductor container | 17:03 |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Remove logging translation calls from ironic.common https://review.openstack.org/448089 | 17:03 |
| lucasagomes | by bind mount I mean the /tftproot directory | 17:03 |
| * lucasagomes dunno much about container | 17:03 | |
| Serlex | I see, ok thanks | 17:04 |
| lucasagomes | Serlex, I would suggest you to ask it in #openstack-kolla | 17:04 |
| lucasagomes | cause they have ironic containirized so, idk, maybe there's a trick there | 17:04 |
| Serlex | I will, thanks for replies | 17:04 |
| lucasagomes | no worries | 17:04 |
| *** dsneddon has joined #openstack-ironic | 17:05 | |
| *** yee379 has quit IRC | 17:06 | |
| *** yee379 has joined #openstack-ironic | 17:06 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Remove translator assignments from i18n https://review.openstack.org/448196 | 17:06 |
| *** vsaienko has quit IRC | 17:09 | |
| aNuposic | jlvillal, jroll lucasagomes ok cool, right now i am just throwing an exception because I did not see earlier in Ironic we are making conductor fail upon missing required conf options like 'swift_endpoint_url' | 17:09 |
| aNuposic | which is required by glance if agent_ipmitool is used | 17:10 |
| *** crushil has joined #openstack-ironic | 17:10 | |
| *** baoli has joined #openstack-ironic | 17:12 | |
| *** vsaienko has joined #openstack-ironic | 17:12 | |
| *** artom has joined #openstack-ironic | 17:13 | |
| *** baha has joined #openstack-ironic | 17:13 | |
| Serlex | lucasagomes | 17:14 |
| lucasagomes | ? | 17:14 |
| Serlex | Sorry dude, as my node is now in error state due to tfptboot problem, I tried to remove it by changing the state to deleted | 17:15 |
| Serlex | is that right? doesn't seem to want to change from error | 17:16 |
| Serlex | or there is another way to detach the failed instance | 17:16 |
| *** kromanenko has quit IRC | 17:16 | |
| lucasagomes | Serlex, yeah deleted is the right verb, anything from the ironic conductor logs ? | 17:17 |
| *** baoli has quit IRC | 17:19 | |
| *** artom has quit IRC | 17:19 | |
| *** artom has joined #openstack-ironic | 17:19 | |
| Serlex | nothing -_- | 17:20 |
| *** vsaienko has quit IRC | 17:22 | |
| lucasagomes | Serlex, anything in the node's last_error property ? | 17:24 |
| *** vsaienko has joined #openstack-ironic | 17:24 | |
| *** baoli has joined #openstack-ironic | 17:24 | |
| soliosg | jlvillal: do you have an idea why some gate jobs are failing? | 17:25 |
| Serlex | Sure, its the permissioned denied error regarding tfpboot | 17:25 |
| artom | Heya - we're trying to do some integration testing with Nova for the serial console feature. The test environment was setup by a Jenkins job that I'm not intimately familiar with, but it looks like it's TripleO with a virtual baremetal overcloud. For now I'm just trying to manually enable the console on one of the nodes that were provisionned for us, and I get: http://paste.openstack.org/show/603645/ | 17:25 |
| soliosg | jlvillal: for instance, in this patch https://review.openstack.org/#/c/447642/ | 17:25 |
| patchbot | patch 447642 - ironic - Remove translation of log messages from ironic/dhc... | 17:25 |
| Nisha_Agarwal | Serlex, you are trying deploy as a root user? | 17:26 |
| soliosg | jlvillal: gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial | 17:26 |
| Serlex | "Failed to tear down. Error: [Errno 13] Permission denied: '/tftpboot/master_images'" | 17:26 |
| artom | https://review.openstack.org/#/c/241453/14 means that consoles should be supported with virtual baremetal, right? | 17:26 |
| patchbot | patch 241453 - ironic - Add console feature to ssh driver (MERGED) | 17:26 |
| *** hamzy has joined #openstack-ironic | 17:26 | |
| Nisha_Agarwal | Serlex, the paste shows that the tftp directory all have root permissions, while you may not be running as root | 17:27 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revert the move of the logger setup https://review.openstack.org/448132 | 17:27 |
| *** fragatina has quit IRC | 17:28 | |
| jlvillal | soliosg: Not off the top of my head. | 17:28 |
| jlvillal | soliosg: Sometimes things fail intermittently. | 17:29 |
| *** chuckC_ has quit IRC | 17:32 | |
| jlvillal | soliosg: It died due to the job taking too long. At least that is what it looks like to me. | 17:33 |
| *** artom has quit IRC | 17:35 | |
| rloo | artom: the error is Missing 'ssh_terminal_port' parameter in node's 'driver_info'. Did you set that value? in ironic-0's driver_info['ssh_terminal_port']. | 17:36 |
| thomas_oneill | my dib image is being built as debian every time... I thought default was ubuntu? even so I've specified dib_os_element: "ubuntu" in the baremetal and in the localhost/target group vars... wtf | 17:37 |
| thomas_oneill | respect my dib_os_element bifrost you... | 17:37 |
| *** artom has joined #openstack-ironic | 17:37 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 17:37 |
| lucasagomes | folks, calling it a day! rloo dtantsur I will look at the redfish thingy tomorrow morning | 17:38 |
| lucasagomes | have a good evening all! | 17:38 |
| dtantsur | g'night | 17:38 |
| rloo | bye lucasagomes! | 17:38 |
| *** lucasagomes is now known as lucas-afk | 17:38 | |
| rloo | soliosg: did you see https://review.openstack.org/#/c/447930/ | 17:43 |
| patchbot | patch 447930 - ironic - Fix local copy of scenario manager | 17:43 |
| rloo | soliosg: that's the file that you copied, right? | 17:43 |
| *** jtomasek has quit IRC | 17:43 | |
| soliosg | jlvillal: thanks, yeah job is timing out | 17:44 |
| rloo | soliosg: i guess it is related to this failure: https://review.openstack.org/443800 | 17:44 |
| patchbot | patch 443800 - ironic (stable/ocata) - Updated from global requirements | 17:44 |
| *** Serlex has quit IRC | 17:46 | |
| *** trown|lunch is now known as trown | 17:47 | |
| soliosg | rloo: oh, that's bad, when pruning manager.py I should have stable/newton, stable/mitaka and stable/ocata | 17:50 |
| soliosg | *should have checked | 17:50 |
| *** derekh has quit IRC | 17:50 | |
| *** crushil has quit IRC | 17:51 | |
| thomas_oneill | the DIB image creation is in the install playbook? gerughghgh | 17:51 |
| rloo | soliosg: hindsight. no worries, no one else thought to check either. that's why we have tests :) | 17:51 |
| *** mgould is now known as mgould|afk | 17:52 | |
| *** ralonsoh has quit IRC | 17:52 | |
| *** crushil has joined #openstack-ironic | 17:53 | |
| TheJulia | thomas_oneill: hmmmm..sounds like there is a bad variable pass :( | 17:54 |
| *** adreznec has quit IRC | 17:55 | |
| *** fragatina has joined #openstack-ironic | 17:55 | |
| thomas_oneill | oh I need to transform boot image or something maybe | 17:56 |
| *** sayantani01 has quit IRC | 17:56 | |
| thomas_oneill | oh no not that | 17:57 |
| *** sayantani01 has joined #openstack-ironic | 17:57 | |
| *** adreznec has joined #openstack-ironic | 17:57 | |
| *** jaosorior has quit IRC | 17:57 | |
| dtantsur | going now, good night | 17:58 |
| *** dtantsur is now known as dtantsur|afk | 17:58 | |
| *** mtanino has quit IRC | 17:58 | |
| thomas_oneill | I've reduced the install.yaml playbook which calls the dib image creation role stuff down to this: http://paste.openstack.org/show/603657/ | 17:58 |
| thomas_oneill | I'm running with: ansible-playbook -vvvv -i inventory/bifrost_inventory.py install.yaml | 17:59 |
| TheJulia | goodnight dtantsur|afk | 17:59 |
| thomas_oneill | I've exported the inventory but I'm not sure if that matters... anyway it just barely does anything, here is the output: http://paste.openstack.org/show/603659/ | 17:59 |
| TheJulia | heh, looks like dib_os_element is not passed in for it | 18:00 |
| TheJulia | for the user image | 18:00 |
| TheJulia | anyway, meeting time | 18:00 |
| *** mgoddard has quit IRC | 18:01 | |
| *** racedo has quit IRC | 18:05 | |
| *** artom_ has joined #openstack-ironic | 18:06 | |
| *** artom has quit IRC | 18:09 | |
| *** dprince has quit IRC | 18:10 | |
| *** mtanino has joined #openstack-ironic | 18:11 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages from ironic-inspector https://review.openstack.org/447884 | 18:14 |
| *** artom__ has joined #openstack-ironic | 18:14 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages from ironic-inspector https://review.openstack.org/448221 | 18:16 |
| *** artom_ has quit IRC | 18:17 | |
| *** racedo has joined #openstack-ironic | 18:17 | |
| *** racedo has quit IRC | 18:23 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient master: Add validation for connections https://review.openstack.org/377103 | 18:30 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-1 https://review.openstack.org/447884 | 18:30 |
| *** artom__ has quit IRC | 18:31 | |
| *** tesseract has quit IRC | 18:31 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-2 https://review.openstack.org/448221 | 18:31 |
| *** artom has joined #openstack-ironic | 18:32 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-1 https://review.openstack.org/447884 | 18:32 |
| *** baoli has quit IRC | 18:35 | |
| *** baoli has joined #openstack-ironic | 18:36 | |
| xavierr | TheJulia: hey Julia :) Could you tell me the diference between serial and web console? both are accessible via horizon? | 18:36 |
| *** mgoddard has joined #openstack-ironic | 18:37 | |
| *** artom_ has joined #openstack-ironic | 18:37 | |
| *** Sukhdev has joined #openstack-ironic | 18:38 | |
| *** artom has quit IRC | 18:39 | |
| *** pcaruana has quit IRC | 18:39 | |
| *** ChubYann has joined #openstack-ironic | 18:42 | |
| *** dprince has joined #openstack-ironic | 18:43 | |
| *** rama_y has joined #openstack-ironic | 18:54 | |
| *** causten has quit IRC | 18:55 | |
| *** alexpilotti has joined #openstack-ironic | 18:57 | |
| *** amarao has quit IRC | 18:59 | |
| *** mat128 has quit IRC | 18:59 | |
| *** alexpilotti has quit IRC | 19:00 | |
| *** alexpilotti has joined #openstack-ironic | 19:01 | |
| *** linggao has joined #openstack-ironic | 19:04 | |
| *** rama_ has joined #openstack-ironic | 19:05 | |
| *** rama_ is now known as Guest20447 | 19:05 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 19:13 |
| *** chlong has joined #openstack-ironic | 19:13 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 19:15 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 19:16 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 19:16 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 19:16 |
| mariojv | API/conductor and rescuewait patches are ready for re-review, agent driver one is just a rebase for now | 19:18 |
| *** rloo has quit IRC | 19:28 | |
| *** rloo has joined #openstack-ironic | 19:29 | |
| *** Nisha_ has joined #openstack-ironic | 19:35 | |
| *** Nisha_Agarwal has quit IRC | 19:38 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 19:40 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-1 https://review.openstack.org/447884 | 19:42 |
| *** amoralej is now known as amoralej|off | 19:44 | |
| *** mgoddard has quit IRC | 19:44 | |
| *** mat128 has joined #openstack-ironic | 19:57 | |
| *** Nisha_ has quit IRC | 19:57 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Add tests for node list and show with specific fields https://review.openstack.org/384923 | 19:59 |
| *** [1]rpioso has joined #openstack-ironic | 20:01 | |
| *** rpioso has quit IRC | 20:03 | |
| *** e0ne has joined #openstack-ironic | 20:04 | |
| *** artom__ has joined #openstack-ironic | 20:04 | |
| *** artom_ has quit IRC | 20:07 | |
| *** zackf has joined #openstack-ironic | 20:08 | |
| *** zackf has quit IRC | 20:12 | |
| rloo | jroll: wrt the ironic version, it is 7.0.1 for ocata. for master, ironic.version.version_info also shows 7.0.1. will it always show 7.0.1 until we do a release? | 20:15 |
| jroll | rloo: it will show 7.0.1.devXXX until we do a release, yes. why? | 20:15 |
| rloo | jroll: it shows 7.0.1 now, i just tried from master. | 20:16 |
| rloo | jroll: cuz i wanted to programmatically determine what version of ironic was being used. | 20:16 |
| jroll | rloo: for details on why, see: https://docs.openstack.org/developer/pbr/#version | 20:16 |
| jroll | rloo: why? :D | 20:16 |
| rloo | jroll: cuz someone asked if it was possible | 20:16 |
| *** vsaienko has quit IRC | 20:17 | |
| *** dsneddon has quit IRC | 20:17 | |
| rloo | jroll: what sets the version to .devXXX part? do we need to do something to get that to happen? | 20:17 |
| jroll | pbr / setuptools does | 20:18 |
| jroll | version_info is the wrong thing, let me see here | 20:18 |
| *** priteau has joined #openstack-ironic | 20:18 | |
| rajinir | Ironic Thirdparty Ci builds: Our HW nodes are not receiving the DHCP offers since March14 pm, nothing changed on our side. Did something merge upstream to affect this? | 20:19 |
| jroll | rloo: this might be a question for #openstack-release, though | 20:19 |
| *** zackf has joined #openstack-ironic | 20:20 | |
| rloo | jroll: ok. do you know if it might have anything to do with this 'create an empty commit with a Sem-Ver tag to bump the generated minor version.' (from https://docs.openstack.org/developer/ironic/dev/releasing.html) | 20:20 |
| *** chlong has quit IRC | 20:21 | |
| rloo | jroll: i think dtantsur|afk said that needed to be done after cutting a release, in master | 20:21 |
| jroll | rloo: yes, that's how you can affect it other than each commit incrementing the .devXXX part | 20:21 |
| jroll | rloo: ah, found it | 20:22 |
| jroll | >>> version_info.release_string() | 20:22 |
| jroll | '7.0.1.dev120' | 20:22 |
| jroll | that's what pip will report, bte | 20:22 |
| jroll | btw* | 20:22 |
| rloo | jroll: that's what I want! thx! | 20:22 |
| jroll | yep, np | 20:22 |
| *** zackf has quit IRC | 20:24 | |
| *** clenimar has joined #openstack-ironic | 20:26 | |
| *** crushil has quit IRC | 20:27 | |
| *** jkilpatr_ has quit IRC | 20:28 | |
| soliosg | jroll: why is version_info.release_string() different that python setup.py --version? | 20:33 |
| soliosg | jroll: for no special purpose, just trying to understand | 20:33 |
| *** dsneddon has joined #openstack-ironic | 20:33 | |
| jroll | soliosg: I don't try to understand setuptools/pbr/etc | 20:34 |
| jroll | except that we aren't supposed to use setup.py in openstack | 20:34 |
| jroll | 'pip freeze' should show the same as release_string(), fwiw | 20:34 |
| soliosg | ack, the first one gives me 7.0.1.dev115, second one gives 7.0.1.dev120 | 20:35 |
| jroll | O_o | 20:35 |
| jroll | even weirder | 20:36 |
| soliosg | jroll: thanks for the trick to get the version with release_string(); yikes, I always used python setup.py --version | 20:36 |
| jroll | soliosg: `pip freeze` ftw :) | 20:36 |
| TheJulia | xavierr: ironic-ui ? web would be shellinabox and serial would be a tcp port you can connect to | 20:36 |
| *** jkilpatr has joined #openstack-ironic | 20:44 | |
| jroll | TheJulia: this is up for forum sessions: http://forumtopics.openstack.org | 20:45 |
| openstackgerrit | Adam Kimball proposed openstack/ironic master: Remove log translation function calls from ironic.db https://review.openstack.org/448275 | 20:46 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Remove translation of log messages from ironic/conductor https://review.openstack.org/447727 | 20:47 |
| *** chlong has joined #openstack-ironic | 20:48 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 20:48 |
| *** zackf has joined #openstack-ironic | 20:49 | |
| *** e0ne has quit IRC | 20:53 | |
| *** trown is now known as trown|outtypewww | 20:54 | |
| *** zackf has quit IRC | 20:54 | |
| *** mjturek has quit IRC | 20:55 | |
| *** priteau_ has joined #openstack-ironic | 20:55 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-2 https://review.openstack.org/448221 | 20:56 |
| *** rpioso has joined #openstack-ironic | 20:57 | |
| *** [1]rpioso has quit IRC | 20:58 | |
| *** zackf has joined #openstack-ironic | 20:58 | |
| *** priteau has quit IRC | 20:59 | |
| *** zackf has quit IRC | 21:02 | |
| *** zackf has joined #openstack-ironic | 21:03 | |
| *** athomas has joined #openstack-ironic | 21:05 | |
| *** zackf has quit IRC | 21:07 | |
| *** crushil has joined #openstack-ironic | 21:09 | |
| *** Goneri has quit IRC | 21:09 | |
| joanna | if sb is bored: https://review.openstack.org/#/c/403910/ | 21:14 |
| patchbot | patch 403910 - ironic - Added UUID validation to port and portgroup PATCH ... | 21:14 |
| openstackgerrit | Adam Kimball proposed openstack/ironic master: Remove log translation function calls from ironic.db https://review.openstack.org/448275 | 21:17 |
| *** baha has quit IRC | 21:22 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 21:23 |
| *** bfournie has quit IRC | 21:25 | |
| *** zackf has joined #openstack-ironic | 21:26 | |
| *** causten has joined #openstack-ironic | 21:27 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 21:28 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-2 https://review.openstack.org/448221 | 21:29 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 21:30 |
| *** zackf has quit IRC | 21:31 | |
| *** crushil has quit IRC | 21:36 | |
| *** alex_xu has quit IRC | 21:37 | |
| *** ijw has joined #openstack-ironic | 21:38 | |
| *** alex_xu has joined #openstack-ironic | 21:38 | |
| *** causten has quit IRC | 21:38 | |
| *** causten has joined #openstack-ironic | 21:39 | |
| *** causten has quit IRC | 21:43 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 21:44 |
| jlvillal | joanna, On the localization string stuff. | 21:46 |
| jlvillal | joanna, Maybe we ignore error H701 | 21:46 |
| jlvillal | joanna, Or maybe start an email discussion about the best way to handle the case of a log message and then a raised exception. | 21:47 |
| jlvillal | I'm leaning towards that last one. | 21:47 |
| jlvillal | We may end up updating the PEP8 test which is stopping us | 21:47 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 21:47 |
| jroll | didn't we discuss this once or twice already today? what's the latest problem? | 21:47 |
| jroll | or better yet, can we just ignore this whole "remove the translations" thing | 21:48 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 21:49 |
| joanna | jroll: there's another issue... | 21:54 |
| joanna | I'll send an email explaining the problem. | 21:54 |
| jroll | zomg | 21:54 |
| jroll | this shouldn't be hard for us to agree on :( | 21:54 |
| *** dprince has quit IRC | 21:54 | |
| * jroll ragequits for the day | 21:55 | |
| *** jcoufal has quit IRC | 21:55 | |
| joanna | jroll: everything is going to be fine! :) | 21:56 |
| rloo | have a great evening jroll. best thing to do is ignore it; let others figure it out. | 21:56 |
| jroll | rloo: I'd rather people did more productive things than talk about removing translations | 21:56 |
| jroll | we think we move slow because review bandwidth is low. that isn't the only thing. | 21:57 |
| rloo | jroll: i know what you mean. but i see it as a learning experience for the others. as long as i minimize my involvement :) | 21:58 |
| soliosg | aNuposic: posted a comment in your configdrive+ceph patch | 21:58 |
| *** ijw has quit IRC | 21:58 | |
| *** ijw has joined #openstack-ironic | 21:59 | |
| *** mat128 has quit IRC | 22:00 | |
| aNuposic | soliosg, checking | 22:00 |
| aNuposic | soliosg, oh yeah calling Connection outside of if/else | 22:03 |
| soliosg | aNuposic: imo, it's not that big change having the connection outside | 22:03 |
| aNuposic | soliosg, i feel better to keep it as the suggestion | 22:03 |
| soliosg | aNuposic: i'm ok with either way, the nit is the dict at the beginning is not needed | 22:04 |
| aNuposic | soliosg, there was no dict earlier but changed as galyna1 suggestion | 22:06 |
| *** ijw has quit IRC | 22:08 | |
| aNuposic | soliosg, but i can remove that | 22:09 |
| *** zackf has joined #openstack-ironic | 22:10 | |
| soliosg | aNuposic: or maybe wait for more reviews (although not sure if people would review with cr-1) | 22:13 |
| *** zackf has quit IRC | 22:15 | |
| *** athomas has quit IRC | 22:16 | |
| *** baoli has quit IRC | 22:18 | |
| *** crushil has joined #openstack-ironic | 22:18 | |
| *** jaypipes has quit IRC | 22:22 | |
| *** chlong has quit IRC | 22:27 | |
| *** wrouesnel has quit IRC | 22:31 | |
| sbezverk | jroll: ping | 22:42 |
| *** crushil has quit IRC | 22:42 | |
| jroll | sbezverk: hi | 22:43 |
| jroll | disclaimer: I may leave soon | 22:43 |
| sbezverk | jroll: can you check this log and maybe some hints where to look further? | 22:43 |
| sbezverk | http://logs.openstack.org/50/441550/104/experimental/gate-kolla-kubernetes-deploy-centos-source-3-ironic-nv/9940aa3/console.html#_2017-03-21_22_10_11_247633 | 22:43 |
| sbezverk | jroll: ^^^ | 22:43 |
| sbezverk | jroll: funny thing the same code works fine in my test bed | 22:44 |
| jroll | sbezverk: seems like something wrong with tftp configuration | 22:45 |
| sbezverk | jroll: well, baremetal vm gets pxelinux.o via tftp | 22:45 |
| jroll | is tftp and ironic-conductor in separate containers? | 22:45 |
| jroll | true | 22:45 |
| *** maratoid_ has joined #openstack-ironic | 22:46 | |
| sbezverk | jroll: conductor and pxe are two containers running in the same kuebrnetes pod | 22:46 |
| sbezverk | they both have access to /tftboot | 22:46 |
| sbezverk | who gets images from glance and push it to the baremetal? | 22:47 |
| jroll | either conductor or agent, depends on the driver | 22:47 |
| sbezverk | ipmi driver | 22:48 |
| jroll | pxe_ipmitool | 22:48 |
| jroll | so the conductor | 22:48 |
| sbezverk | yep that one | 22:48 |
| jroll | yeah, I don't see anything super off in the logs | 22:49 |
| jroll | oh wait | 22:50 |
| jroll | 2017-03-21 22:04:29.926 7 WARNING ironic.common.utils [req-4bdb9026-ca81-498b-b355-9d4275d04bb5 51114c39dde04f0083d9c28930a7b74d d9aa71f2482d46139cafba9412773096 - - -] Failed to create symlink from ../db8f3988-b2bc-4c8a-910b-d4f270f67cc6/config to /tftpboot/pxelinux.cfg/01-00-01-de-ad-be-ef, error: [Errno 13] Permission denied | 22:50 |
| jroll | that feels... relevant | 22:50 |
| sbezverk | jroll: but it uses default at the end | 22:50 |
| sbezverk | jroll: could you quickly check this log | 22:50 |
| sbezverk | http://logs.openstack.org/50/441550/104/experimental/gate-kolla-kubernetes-deploy-centos-source-3-ironic-nv/9940aa3/logs/openstack/kolla-ironic-conductor-bns2f.txt | 22:50 |
| jroll | that's what I'm reading | 22:51 |
| sbezverk | jroll: ah got it | 22:51 |
| jroll | this very much feels like a misconfiguration, somewhere between the conductor and tftp | 22:51 |
| jroll | maybe how the volumes are bind mounted | 22:51 |
| jroll | I feel like someone was here earlier today with a similar problem | 22:52 |
| jroll | containers, sigh | 22:52 |
| jroll | I bet fixing that permissions issue fixes things right up | 22:52 |
| jroll | I gotta go | 22:52 |
| jroll | good luck | 22:52 |
| sbezverk | jroll: ik thanks for your help | 22:52 |
| *** sayantani01 has quit IRC | 23:02 | |
| *** sayantani01 has joined #openstack-ironic | 23:03 | |
| *** sayantani01 has quit IRC | 23:03 | |
| *** patchbot has quit IRC | 23:03 | |
| *** sayantani01 has joined #openstack-ironic | 23:04 | |
| *** eglute_s has joined #openstack-ironic | 23:04 | |
| *** ElCoyote_ has quit IRC | 23:06 | |
| *** sayantani01 has quit IRC | 23:06 | |
| *** harlowja has quit IRC | 23:06 | |
| *** sayantani01 has joined #openstack-ironic | 23:06 | |
| *** eglute_s has quit IRC | 23:07 | |
| *** bfournie has joined #openstack-ironic | 23:13 | |
| *** priteau_ has quit IRC | 23:15 | |
| *** priteau has joined #openstack-ironic | 23:15 | |
| *** sayantani01 has quit IRC | 23:20 | |
| *** sayantani01 has joined #openstack-ironic | 23:21 | |
| *** gfhellma has quit IRC | 23:26 | |
| *** rbudden has quit IRC | 23:31 | |
| *** eglute_s has joined #openstack-ironic | 23:34 | |
| *** priteau has quit IRC | 23:34 | |
| *** mtanino has quit IRC | 23:38 | |
| *** rpioso has quit IRC | 23:41 | |
| *** priteau has joined #openstack-ironic | 23:41 | |
| *** priteau has quit IRC | 23:46 | |
| *** eglute_s has quit IRC | 23:46 | |
| *** eglute_s has joined #openstack-ironic | 23:47 | |
| *** eglute has quit IRC | 23:50 | |
| *** rama_y has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!