| *** MattMan has quit IRC | 00:01 | |
| *** MattMan has joined #openstack-ironic | 00:01 | |
| *** amotoki is now known as amotoki_away | 00:14 | |
| *** baoli has quit IRC | 00:16 | |
| *** baoli has joined #openstack-ironic | 00:18 | |
| *** rama_y has joined #openstack-ironic | 00:18 | |
| *** Goneri has quit IRC | 00:34 | |
| *** SerenaFeng has joined #openstack-ironic | 00:39 | |
| *** amotoki_away is now known as amotoki | 00:41 | |
| *** yolanda has quit IRC | 00:48 | |
| *** SerenaFeng has quit IRC | 00:58 | |
| *** litao__ has joined #openstack-ironic | 01:00 | |
| *** amotoki is now known as amotoki_away | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** chlong has quit IRC | 01:04 | |
| *** SerenaFeng has joined #openstack-ironic | 01:09 | |
| *** amotoki_away is now known as amotoki | 01:11 | |
| *** SerenaFeng has quit IRC | 01:11 | |
| *** chlong has joined #openstack-ironic | 01:20 | |
| *** SerenaFeng has joined #openstack-ironic | 01:23 | |
| *** pmannidi has quit IRC | 01:43 | |
| *** pmannidi has joined #openstack-ironic | 01:48 | |
| *** SerenaFeng has quit IRC | 01:52 | |
| openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 01:52 |
|---|---|---|
| *** tuanluong has joined #openstack-ironic | 01:54 | |
| *** SerenaFeng has joined #openstack-ironic | 01:54 | |
| litao__ | : hi guys, I summit a bp spec about ironic conductor service management, so any suggestions? | 01:55 |
| litao__ | https://review.openstack.org/#/c/471217/ | 01:55 |
| patchbot | patch 471217 - ironic-specs - Add service management API for ironic | 01:55 |
| *** zhurong has joined #openstack-ironic | 01:56 | |
| *** SerenaFeng has quit IRC | 02:06 | |
| *** SerenaFeng has joined #openstack-ironic | 02:07 | |
| *** cuongnq has joined #openstack-ironic | 02:08 | |
| *** baoli has quit IRC | 02:35 | |
| *** crushil has joined #openstack-ironic | 02:53 | |
| *** yolanda has joined #openstack-ironic | 03:11 | |
| *** mtanino_ has quit IRC | 03:11 | |
| *** crushil has quit IRC | 03:12 | |
| *** SerenaFeng has quit IRC | 03:20 | |
| *** SerenaFeng has joined #openstack-ironic | 03:23 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Make ip address of socat console configurable https://review.openstack.org/473201 | 03:40 |
| *** tenobreg has quit IRC | 03:46 | |
| *** rbudden has quit IRC | 04:00 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 04:10 | |
| *** rama_y_ has joined #openstack-ironic | 04:11 | |
| *** rama_y has quit IRC | 04:11 | |
| *** rama_y_ has quit IRC | 04:14 | |
| *** amarao has joined #openstack-ironic | 04:15 | |
| *** amarao has quit IRC | 04:19 | |
| *** Dinesh_Bhor has quit IRC | 04:24 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 04:25 | |
| *** zhurong has quit IRC | 04:26 | |
| *** SerenaFeng has quit IRC | 04:28 | |
| *** zhurong has joined #openstack-ironic | 04:35 | |
| *** yolanda has quit IRC | 04:35 | |
| *** adisky__ has joined #openstack-ironic | 04:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:58 | |
| Nisha_Agarwal | morning Ironic | 05:06 |
| *** zhurong has quit IRC | 05:06 | |
| *** zhurong has joined #openstack-ironic | 05:07 | |
| *** rbartal has joined #openstack-ironic | 05:07 | |
| *** rbartal has quit IRC | 05:15 | |
| *** mvpnitesh has joined #openstack-ironic | 05:25 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adding 'get_current_boot_mode' method using redfish https://review.openstack.org/466697 | 05:31 |
| *** ChubYann has quit IRC | 05:36 | |
| *** cuongnq has quit IRC | 05:38 | |
| *** pmannidi has quit IRC | 05:56 | |
| *** pmannidi has joined #openstack-ironic | 05:57 | |
| *** e0ne has joined #openstack-ironic | 06:01 | |
| *** rbartal has joined #openstack-ironic | 06:02 | |
| *** mvpnitesh has quit IRC | 06:03 | |
| *** mvpnitesh has joined #openstack-ironic | 06:03 | |
| *** e0ne has quit IRC | 06:09 | |
| *** e0ne has joined #openstack-ironic | 06:16 | |
| *** rcernin has joined #openstack-ironic | 06:19 | |
| *** mjura has joined #openstack-ironic | 06:22 | |
| *** yolanda has joined #openstack-ironic | 06:28 | |
| *** jaganathan has joined #openstack-ironic | 06:31 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps: Add 'make' to the required packages https://review.openstack.org/473225 | 06:31 |
| *** SerenaFeng has joined #openstack-ironic | 06:36 | |
| *** rcernin has quit IRC | 06:36 | |
| *** rcernin has joined #openstack-ironic | 06:40 | |
| *** gcb has joined #openstack-ironic | 06:41 | |
| *** e0ne has quit IRC | 06:50 | |
| *** SerenaFeng has quit IRC | 06:52 | |
| *** pcaruana has joined #openstack-ironic | 06:52 | |
| openstackgerrit | Miles Fu proposed openstack/ironic-inspector master: ironic-inspector-dbsync on postgre error. https://review.openstack.org/473233 | 06:52 |
| *** e0ne has joined #openstack-ironic | 06:54 | |
| *** mvpnitesh has quit IRC | 06:58 | |
| *** mvpnitesh has joined #openstack-ironic | 06:58 | |
| *** SerenaFeng has joined #openstack-ironic | 07:07 | |
| *** aarefiev has quit IRC | 07:08 | |
| openstackgerrit | Miles Fu proposed openstack/ironic-inspector master: ironic-inspector-dbsync on postgre error. https://review.openstack.org/473233 | 07:10 |
| *** aarefiev has joined #openstack-ironic | 07:11 | |
| *** ralonsoh has joined #openstack-ironic | 07:13 | |
| *** e0ne has quit IRC | 07:17 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Discover iscsi_boot attribute https://review.openstack.org/341653 | 07:18 |
| *** Nisha_Agarwal has quit IRC | 07:19 | |
| *** tesseract has joined #openstack-ironic | 07:19 | |
| *** yolanda has quit IRC | 07:22 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 07:31 | |
| *** ccamacho has joined #openstack-ironic | 07:32 | |
| *** racedo has joined #openstack-ironic | 07:40 | |
| *** yolanda has joined #openstack-ironic | 07:43 | |
| *** karts has joined #openstack-ironic | 07:46 | |
| *** fragatina has joined #openstack-ironic | 07:47 | |
| *** karthiks_afk has quit IRC | 07:48 | |
| aarefiev | morning all! | 07:48 |
| *** jaganathan has quit IRC | 07:48 | |
| *** SerenaFeng has quit IRC | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** hjensas has joined #openstack-ironic | 08:02 | |
| *** jaganathan has joined #openstack-ironic | 08:02 | |
| *** yolanda has quit IRC | 08:03 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Make default_boot_option configurable in devstack https://review.openstack.org/473294 | 08:07 |
| *** Nisha_Agarwal has quit IRC | 08:09 | |
| pas-ha | morning ironicers :) | 08:09 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:13 | |
| *** SerenaFeng has joined #openstack-ironic | 08:14 | |
| *** lennyb has joined #openstack-ironic | 08:15 | |
| *** jaosorior has joined #openstack-ironic | 08:16 | |
| *** mvpnitesh has quit IRC | 08:17 | |
| *** lucas-afk is now known as lucasagomes | 08:19 | |
| lucasagomes | morning all | 08:19 |
| *** dtantsur has joined #openstack-ironic | 08:22 | |
| dtantsur | morning ironic | 08:23 |
| dtantsur | I've lost my IRC bouncer over the weekend, please repeat any pings | 08:24 |
| *** SerenaFeng has quit IRC | 08:26 | |
| *** mgoddard has joined #openstack-ironic | 08:28 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/472913 | 08:29 |
| dtantsur | vsaienk0: hi! do you happen to know if QFX 5100 is/will be supported by n-g-s? | 08:30 |
| *** SerenaFeng has joined #openstack-ironic | 08:31 | |
| litao__ | Hi guys, I summited a bp spec about ironic conductor service management, so any suggestions? | 08:31 |
| litao__ | https://review.openstack.org/#/c/471217/ | 08:31 |
| patchbot | patch 471217 - ironic-specs - Add service management API for ironic | 08:31 |
| *** SerenaFeng has quit IRC | 08:31 | |
| *** milan has joined #openstack-ironic | 08:33 | |
| *** SerenaFeng has joined #openstack-ironic | 08:37 | |
| *** derekh has joined #openstack-ironic | 08:39 | |
| *** racedo has quit IRC | 08:39 | |
| vdrok | good morning ironic, pas-ha lucasagomes dtantsur litao__ | 08:44 |
| dtantsur | o/ | 08:44 |
| vsaienk0 | dtantsur: It is not supported at the moment by n-g-s, and I don't know if netmiko juniper ssh connector supports QFX hardware https://github.com/ktbyers/netmiko/blob/develop/netmiko/juniper/juniper_ssh.py (probably it should. In this case you just need to build command sets in n-g-s) | 08:45 |
| dtantsur | ack, thanks | 08:45 |
| *** mvpnitesh has joined #openstack-ironic | 08:48 | |
| *** athomas has joined #openstack-ironic | 08:52 | |
| *** e0ne has joined #openstack-ironic | 08:55 | |
| *** mvpnitesh has quit IRC | 08:56 | |
| *** mvpnitesh has joined #openstack-ironic | 08:57 | |
| *** racedo has joined #openstack-ironic | 09:04 | |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/472934 | 09:06 |
| *** Nisha_Agarwal has quit IRC | 09:10 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/472935 | 09:10 |
| *** FrankJ has joined #openstack-ironic | 09:16 | |
| FrankJ | hi everyone. I have a problem about conductor tables | 09:17 |
| *** amotoki is now known as amotoki_away | 09:17 | |
| FrankJ | mysql> select * from conductors; | 09:17 |
| FrankJ | +---------------------+---------------------+----+-------------------+-------------+--------+ | 09:17 |
| FrankJ | | created_at | updated_at | id | hostname | drivers | online | | 09:17 |
| FrankJ | +---------------------+---------------------+----+-------------------+-------------+--------+ | 09:17 |
| FrankJ | | 2016-12-21 08:40:54 | 2016-12-23 23:46:16 | 1 | node-1.domain.tld | ["pxe_ssh"] | 0 | | 09:17 |
| FrankJ | +---------------------+---------------------+----+-------------------+-------------+--------+ | 09:17 |
| FrankJ | 1 row in set (0.00 sec) | 09:17 |
| *** e0ne has quit IRC | 09:18 | |
| FrankJ | the online column is 0 and I restart openstack-ironic-conductor, it still remiands 0 | 09:18 |
| FrankJ | why would this happned? Does anyone meet this issue? | 09:19 |
| *** e0ne has joined #openstack-ironic | 09:20 | |
| *** amotoki_away is now known as amotoki | 09:23 | |
| *** yolanda has joined #openstack-ironic | 09:23 | |
| openstackgerrit | shaofeng cheng proposed openstack/ironic master: Update multitenancy file address https://review.openstack.org/473330 | 09:24 |
| pas-ha | FrankJ: any errors in conductor logs? like failure to connect to DB? | 09:25 |
| FrankJ | I found the reason. some thing wrong with driver - -!, Thank you | 09:28 |
| FrankJ | this is no error in ironc-conductor.log, but some error appears in "systemctl status openstack-ironic-conductor" | 09:29 |
| FrankJ | [root@node-1 ironic]# systemctl status openstack-ironic-conductor -l | 09:29 |
| FrankJ | ● openstack-ironic-conductor.service - OpenStack Ironic Conductor service | 09:29 |
| FrankJ | Loaded: loaded (/usr/lib/systemd/system/openstack-ironic-conductor.service; enabled; vendor preset: disabled) | 09:29 |
| FrankJ | Active: active (running) since 六 2016-12-24 08:02:28 CST; 3min 41s ago | 09:29 |
| FrankJ | Main PID: 2126501 (ironic-conducto) | 09:29 |
| FrankJ | CGroup: /system.slice/openstack-ironic-conductor.service | 09:29 |
| FrankJ | └─2126501 /usr/bin/python2 /usr/bin/ironic-conductor | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: verify_requirements=verify_requirements, | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: File "/usr/lib/python2.7/site-packages/stevedore/enabled.py", line 60, in __init__ | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: verify_requirements=verify_requirements, | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: File "/usr/lib/python2.7/site-packages/stevedore/extension.py", line 92, in __init__ | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: verify_requirements) | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: File "/usr/lib/python2.7/site-packages/stevedore/extension.py", line 170, in _load_plugins | 09:29 |
| openstackgerrit | Miles Fu proposed openstack/ironic-inspector master: ironic-inspector-dbsync on postgre error. https://review.openstack.org/473233 | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: self._on_load_failure_callback(self, ep, err) | 09:29 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: File "/usr/lib/python2.7/site-packages/ironic/common/driver_factory.py", line 116, in _catch_driver_not_found | 09:30 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: raise exception.DriverLoadError(driver=ep.name, reason=exc) | 09:30 |
| FrankJ | 12月 24 08:02:29 node-1.domain.tld ironic-conductor[2126501]: DriverLoadError: Driver pxe_ssh could not be loaded. Reason: expected an indented block (iscsi_deploy.py, line 731). | 09:30 |
| FrankJ | now the env is fine. Thank godness | 09:31 |
| *** ralonsoh has quit IRC | 09:34 | |
| *** wagiel has joined #openstack-ironic | 09:36 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Moving redfish_version attribute to Field https://review.openstack.org/473339 | 09:37 |
| aarefiev | dtantsur: hey, one question regarding version https://review.openstack.org/#/c/462861/5/ironic_inspector/worker/rpc_api.py | 09:39 |
| patchbot | patch 462861 - ironic-inspector - Introduce oslo.messaging RPC | 09:39 |
| *** mvpnitesh has quit IRC | 09:41 | |
| *** mvpnitesh has joined #openstack-ironic | 09:41 | |
| *** amarao has joined #openstack-ironic | 09:43 | |
| dtantsur | sure | 09:52 |
| *** deray has joined #openstack-ironic | 09:52 | |
| *** _milan_ has joined #openstack-ironic | 09:53 | |
| *** milan has quit IRC | 09:54 | |
| *** ralonsoh has joined #openstack-ironic | 09:55 | |
| *** amotoki is now known as amotoki_away | 09:55 | |
| openstackgerrit | Merged openstack/ironic master: Replace get_transport with get_rpc_transport https://review.openstack.org/471603 | 09:55 |
| *** amotoki_away is now known as amotoki | 09:57 | |
| openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 10:01 |
| aarefiev | dtantsur: do you see some issues adding rpc version later? | 10:02 |
| dtantsur | aarefiev: then it will be unclear which version is initial | 10:03 |
| aarefiev | dtantsur: why, you will add version with initial value 1.0 | 10:05 |
| *** pmannidi has quit IRC | 10:05 | |
| *** pmannidi has joined #openstack-ironic | 10:05 | |
| dtantsur | aarefiev: yes, but it's not 1.0 right now. so e.g. on upgrade it won't work. | 10:05 |
| dtantsur | if you request version 1.0, it won't exist | 10:06 |
| dtantsur | opposite question: what's the reason NOT to add it? | 10:06 |
| *** SerenaFeng has quit IRC | 10:06 | |
| aarefiev | dtantsur: I don't understand what are you expecting to see, just variable with version | 10:07 |
| aarefiev | afaik oslo.messaging assumes 1.0 by default | 10:07 |
| dtantsur | aarefiev: the state of ironic before rolling upgrades work. if oslo.msg can handle it - fine, but I'm still not sure why not add it right away.. | 10:07 |
| *** amotoki is now known as amotoki_away | 10:09 | |
| aarefiev | dtantsur: I'm trying to understand your concerns, so you want to add version variable with initial value | 10:11 |
| aarefiev | and thats all | 10:11 |
| dtantsur | aarefiev: yep (and make sure we tell the version to oslo.msg, obviously) | 10:12 |
| *** zhurong has quit IRC | 10:12 | |
| aarefiev | ok, thanks for clarification | 10:12 |
| dtantsur | np, sorry for confusion, I'm still waking up | 10:13 |
| aarefiev | np, I just thought there is some known issue, just to be sure | 10:14 |
| *** amotoki_away is now known as amotoki | 10:16 | |
| *** wagiel has quit IRC | 10:19 | |
| *** wagiel has joined #openstack-ironic | 10:25 | |
| *** vmud213 has joined #openstack-ironic | 10:25 | |
| wagiel | Hi All, I need some guidance please, does ironic/ipa supports DHCP only environments? Trying to boot ipa and set ip,netmask,gw static, tinycore nor coreos seem not to take kernel parameters as arguments thus no network is configured | 10:29 |
| dtantsur | wagiel: this is related to how you build the ramdisk, not to IPA itself. e.g. ramdisks built with DIB use dracut, hence they understand ip,netmask,gw. | 10:30 |
| wagiel | true | 10:30 |
| wagiel | I will try DIB | 10:30 |
| wagiel | thanks | 10:31 |
| *** root2 has quit IRC | 10:31 | |
| *** root2 has joined #openstack-ironic | 10:31 | |
| *** tuanluong has quit IRC | 10:36 | |
| *** zhurong has joined #openstack-ironic | 10:39 | |
| *** amotoki is now known as amotoki_away | 10:42 | |
| *** _milan_ is now known as milan | 10:43 | |
| milan | morning Ironic! :) | 10:43 |
| *** gcb has quit IRC | 10:44 | |
| *** jkilpatr has quit IRC | 10:45 | |
| *** FrankJ has quit IRC | 10:45 | |
| *** yolanda has quit IRC | 10:48 | |
| openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 10:49 |
| dtantsur | hi milan | 10:51 |
| milan | dtantsur, morning! :) | 10:51 |
| *** ralonsoh has quit IRC | 10:53 | |
| *** lucasagomes has quit IRC | 10:55 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add mention about desirec config_drive_format on nova-compute https://review.openstack.org/473370 | 10:59 |
| *** wagiel has quit IRC | 11:00 | |
| *** mtanino_ has joined #openstack-ironic | 11:00 | |
| *** lucasagomes has joined #openstack-ironic | 11:01 | |
| *** lucasagomes is now known as lucas-hungry | 11:01 | |
| *** milan is now known as milan|lunch | 11:04 | |
| *** jkilpatr has joined #openstack-ironic | 11:05 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Adds clean step 'reset_bios_config' to iRMC drivers https://review.openstack.org/416403 | 11:06 |
| *** lucasagomes has joined #openstack-ironic | 11:06 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Make default_boot_option configurable in devstack https://review.openstack.org/473294 | 11:08 |
| *** jaypipes has joined #openstack-ironic | 11:10 | |
| *** ralonsoh has joined #openstack-ironic | 11:12 | |
| *** deray has quit IRC | 11:16 | |
| *** amotoki_away is now known as amotoki | 11:17 | |
| *** jaianshu has joined #openstack-ironic | 11:18 | |
| phuongnh | Hi TheJulia, | 11:21 |
| *** rbudden has joined #openstack-ironic | 11:23 | |
| phuongnh | TheJulia, I am writing tempest test for BFV, should I add a bug description as portgroup? | 11:28 |
| phuongnh | TheJulia, as in https://bugs.launchpad.net/ironic/+bug/1666858 | 11:28 |
| openstack | Launchpad bug 1666858 in Ironic "Add tests for portgroups to the ironic tempest plugin" [Low,Fix released] - Assigned to Kyrylo Romanenko (kromanenko) | 11:28 |
| phuongnh | TheJulia, or should I use the same bug number as the RFE of BFV? | 11:28 |
| dtantsur | phuongnh: please use the same number as the BFV RFE | 11:29 |
| phuongnh | dtantsur, I got it, thanks | 11:29 |
| dtantsur | generally we consider tests to be a part of the same RFE as the feature | 11:29 |
| phuongnh | yes, I see | 11:29 |
| phuongnh | dtantsur, I am leaving now, have a nice day | 11:30 |
| dtantsur | have a nice day | 11:30 |
| *** bfournie has quit IRC | 11:41 | |
| *** dprince has joined #openstack-ironic | 11:43 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 11:47 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds get_ilo_firmware_version method using redfish https://review.openstack.org/468852 | 11:51 |
| *** jaianshu has quit IRC | 11:54 | |
| *** fellypefca has joined #openstack-ironic | 11:54 | |
| fellypefca | Good morning, Ironic :) | 11:55 |
| TheJulia | Good morning everyone | 11:56 |
| *** tenobreg has joined #openstack-ironic | 11:56 | |
| *** tenobreg is now known as tellesnobrega | 11:57 | |
| *** phuongnh has quit IRC | 11:57 | |
| *** gcb has joined #openstack-ironic | 11:58 | |
| *** litao__ has quit IRC | 11:59 | |
| dtantsur | morning TheJulia, fellypefca | 12:02 |
| *** amotoki is now known as amotoki_away | 12:04 | |
| *** ccamacho has quit IRC | 12:14 | |
| *** lucas-hungry has quit IRC | 12:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/472806 | 12:15 |
| *** zhurong has quit IRC | 12:16 | |
| *** milan|lunch is now known as milan | 12:18 | |
| *** bfournie has joined #openstack-ironic | 12:19 | |
| *** skazi has quit IRC | 12:20 | |
| *** skazi has joined #openstack-ironic | 12:20 | |
| *** bfournie1 has joined #openstack-ironic | 12:21 | |
| *** bfournie has quit IRC | 12:23 | |
| *** yolanda has joined #openstack-ironic | 12:24 | |
| *** bcornec1 has joined #openstack-ironic | 12:25 | |
| fellypefca | vdrok Hi! Did you had some time to look at the answer that Xavier gave to you about the flag on driver_internal_info, on Onetime Boot patch? (https://review.openstack.org/#/c/436469/) | 12:26 |
| patchbot | patch 436469 - ironic - Adds onetime boot in oneview drivers | 12:26 |
| *** skazi_ has joined #openstack-ironic | 12:26 | |
| *** baoli has joined #openstack-ironic | 12:28 | |
| *** fellypefca has quit IRC | 12:31 | |
| *** skazi has quit IRC | 12:32 | |
| *** skazi has joined #openstack-ironic | 12:32 | |
| *** fellypefca has joined #openstack-ironic | 12:32 | |
| *** jcoufal has joined #openstack-ironic | 12:34 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 12:34 |
| openstackgerrit | Aparna proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 12:35 |
| *** skarthik has joined #openstack-ironic | 12:36 | |
| *** jaganathan_ has joined #openstack-ironic | 12:36 | |
| *** fellypefca has quit IRC | 12:37 | |
| *** fellypefca has joined #openstack-ironic | 12:37 | |
| *** karts has quit IRC | 12:38 | |
| *** jaganathan has quit IRC | 12:39 | |
| openstackgerrit | liuxiaoyang proposed openstack/ironic master: Replace http with https in openstack.org. https://review.openstack.org/473395 | 12:39 |
| *** baoli has quit IRC | 12:41 | |
| milan | aarefiev, hey, thanks for the review! w/r the context manager, it might be a bit tricky, I've posted an answer, please, let me know wdyt | 12:42 |
| *** skazi_ has quit IRC | 12:43 | |
| aarefiev | milan: not sure I get it, you have initialized and locked filter, who is able to reset it | 12:45 |
| milan | aarefiev, I check before I lock so it's racy | 12:45 |
| milan | aarefiev, I could check after I lock but then I have to release the lock | 12:46 |
| milan | aarefiev, so the except *NotInitialized as e: is IMO sort of inevitable :-/ | 12:46 |
| *** baoli has joined #openstack-ironic | 12:47 | |
| *** sasha2 has joined #openstack-ironic | 12:48 | |
| aarefiev | milan: ok, didn't think about init in concurrency context | 12:51 |
| milan | aarefiev, yeah it's a bit tricky, I had a (local only) version with greenlet.event passing the "it's time to quit" message which was more explicit maybe but also more code-heavy | 12:53 |
| *** vmud213 has quit IRC | 12:53 | |
| milan | aarefiev, so if this turns out insufficient we'd refactor? | 12:53 |
| aarefiev | yeah, it doesn't hurt anyway | 12:55 |
| milan | aarefiev, cool | 12:55 |
| aarefiev | thx | 12:55 |
| milan | aarefiev, thanks as well! It's big, I know O:-) | 12:56 |
| vdrok | fellypefca: sorry, I have not, and not sure if I will have time today. might be worth asking others to look at that. My -1 is not a -2, so a patch can be merged with it :) | 12:56 |
| openstackgerrit | Aparna proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 12:56 |
| openstackgerrit | Aparna proposed openstack/ironic master: WIP Rescue mode tempest tests https://review.openstack.org/452308 | 12:58 |
| *** rbartal has quit IRC | 12:59 | |
| *** baoli has quit IRC | 13:00 | |
| milan | aarefiev, so +2 you say? ;) | 13:03 |
| milan | * modulo the broken link | 13:03 |
| *** trown|outtypewww is now known as trown | 13:04 | |
| *** root2 has quit IRC | 13:05 | |
| *** root2 has joined #openstack-ironic | 13:05 | |
| aarefiev | milan: yeah, it's lgtm, I'd update log msg also, saying about race | 13:07 |
| aarefiev | that driver is non initialized | 13:08 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:10 | |
| *** rbartal has joined #openstack-ironic | 13:11 | |
| fellypefca | vdrok thats ok! Thanks :) | 13:16 |
| *** rloo has joined #openstack-ironic | 13:20 | |
| fellypefca | dtantsur My next plan was to ask if you would take a look at the full answer and explanation that Ricardo and I prepared and posted about the doubts you had in this patch. I hope it's clearer now. :) | 13:22 |
| dtantsur | ack (in a meeting now) | 13:23 |
| fellypefca | ok :) | 13:23 |
| *** krtaylor has joined #openstack-ironic | 13:26 | |
| *** Nisha_ has joined #openstack-ironic | 13:28 | |
| *** baoli has joined #openstack-ironic | 13:30 | |
| *** Nisha_Agarwal has quit IRC | 13:31 | |
| TheJulia | ppiela: I think 469459 could use a release note or something doc wise. not treating that as a blocker though since it augments/improves the existing js testing. | 13:32 |
| *** amotoki_away is now known as amotoki | 13:33 | |
| *** Nisha_brb has joined #openstack-ironic | 13:34 | |
| ppiela | TheJulia: agreed. I will write-up a release note, and work on some developer-oriented doc | 13:36 |
| *** Nisha_brb has quit IRC | 13:36 | |
| *** Nisha_brb has joined #openstack-ironic | 13:36 | |
| *** Nisha_ has quit IRC | 13:37 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: Include IPA Version during heartbeat https://review.openstack.org/364834 | 13:39 |
| TheJulia | ppiela: Thanks! | 13:40 |
| *** mvpnitesh has quit IRC | 13:42 | |
| *** amotoki is now known as amotoki_away | 13:42 | |
| *** fragatina has quit IRC | 13:47 | |
| *** fragatina has joined #openstack-ironic | 13:47 | |
| *** crushil has joined #openstack-ironic | 13:51 | |
| milan | aarefiev, ack, will update | 13:53 |
| *** fellypefca has quit IRC | 13:53 | |
| *** beekneemech is now known as bnemec | 13:54 | |
| *** superdan is now known as dansmith | 13:56 | |
| *** amotoki_away is now known as amotoki | 13:58 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps: Add 'make' to the required packages https://review.openstack.org/473225 | 14:01 |
| *** jo_______ has joined #openstack-ironic | 14:01 | |
| *** makowals has quit IRC | 14:04 | |
| *** joe___ has quit IRC | 14:04 | |
| *** jaganathan_ has quit IRC | 14:06 | |
| *** makowals has joined #openstack-ironic | 14:07 | |
| sambetts | dtantsur: ironic always includes the version header in responses right?? | 14:09 |
| dtantsur | sambetts: IIRC yes | 14:09 |
| sambetts | cool, just updating the IPA code for the versioning stuff, and I want IPA to check the API version before it sends its own version because if the API version is too low including the new field results in a badrequest | 14:11 |
| dtantsur | sambetts: or you can check the root endpoint / | 14:12 |
| sambetts | dtantsur: yeah, was trying to avoid the second API call by making IPA cache the version on response from lookup, although TBH a longer term solution would be to hit / first to ge the version in case we change anything in lookup | 14:13 |
| dtantsur | yep, this is how proper versioning has to be done :) | 14:13 |
| *** amotoki is now known as amotoki_away | 14:13 | |
| *** pmannidi has quit IRC | 14:14 | |
| *** amotoki_away is now known as amotoki | 14:14 | |
| *** pmannidi has joined #openstack-ironic | 14:14 | |
| *** root2 has quit IRC | 14:18 | |
| *** root2 has joined #openstack-ironic | 14:18 | |
| *** amotoki is now known as amotoki_away | 14:20 | |
| *** Nisha_brb has quit IRC | 14:21 | |
| *** rama_y has joined #openstack-ironic | 14:22 | |
| *** amotoki_away is now known as amotoki | 14:24 | |
| *** hjensas has quit IRC | 14:29 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver raid property list' cmd https://review.openstack.org/362047 | 14:34 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver property list' command https://review.openstack.org/381153 | 14:35 |
| openstackgerrit | Sean Dague proposed openstack/ironic master: WIP: pass global_request_id to cinder/neutron https://review.openstack.org/472666 | 14:36 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver raid property list' cmd https://review.openstack.org/362047 | 14:36 |
| *** makowals has quit IRC | 14:37 | |
| *** jlvacation is now known as jlvillal | 14:41 | |
| *** amotoki is now known as amotoki_away | 14:41 | |
| dtantsur | rloo: morning! any objections to merging https://review.openstack.org/#/c/471830/ ? | 14:43 |
| patchbot | patch 471830 - ironic-python-agent (stable/ocata) - [imagebuild] support requirements referring to /op... | 14:43 |
| rloo | dtantsur: i was going to ask you about that. what happens when we only have two stable cores. i think we only need one +2 then? | 14:44 |
| dtantsur | rloo: by the policy, backports proposed by one of stable cores require only 1x +2 | 14:44 |
| rloo | dtantsur: ok, will +A then. | 14:44 |
| rloo | dtantsur: done! | 14:45 |
| *** yolanda has quit IRC | 14:45 | |
| dtantsur | 4th paragraph of https://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines | 14:45 |
| dtantsur | thanks rloo | 14:45 |
| jtanner | it's really tricky to do impi management on a libvirt domain who's on_poweroff value is "destroy" =) | 14:46 |
| TheJulia | jtanner: well, destroy != undefine | 14:46 |
| jtanner | in tools/create_vm_nodes.sh ... | 14:48 |
| jtanner | - <on_poweroff>destroy</on_poweroff> | 14:48 |
| jtanner | + <on_poweroff>preserve</on_poweroff> | 14:48 |
| jtanner | not sure if i'm going against the grain on the intent of that script | 14:48 |
| TheJulia | I don't think so | 14:49 |
| *** makowals has joined #openstack-ironic | 14:50 | |
| *** rama_y has quit IRC | 14:52 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Wait until iDRAC is ready before out-of-band cleaning https://review.openstack.org/466086 | 14:52 |
| *** rpioso has joined #openstack-ironic | 14:55 | |
| *** makowals has quit IRC | 14:56 | |
| *** fellypefca has joined #openstack-ironic | 14:56 | |
| *** dprince has quit IRC | 15:00 | |
| *** SerenaFeng has joined #openstack-ironic | 15:00 | |
| *** mjura has quit IRC | 15:01 | |
| *** makowals has joined #openstack-ironic | 15:03 | |
| *** bcornec1 has quit IRC | 15:05 | |
| *** makowals has quit IRC | 15:05 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC https://review.openstack.org/462861 | 15:06 |
| aarefiev | dtantsur: please, check when you have time https://review.openstack.org/#/c/453057/10 | 15:12 |
| patchbot | patch 453057 - ironic-inspector - Use new oslo db EngineFacade | 15:12 |
| dtantsur | sure (a bit later) | 15:12 |
| aarefiev | thanks | 15:12 |
| *** makowals has joined #openstack-ironic | 15:13 | |
| *** aarefiev is now known as aarefiev_afk | 15:15 | |
| TheJulia | jtanner: feel free to submit a patch for that diff :) | 15:23 |
| jtanner | i don't think it's quite right yet | 15:23 |
| jtanner | still experimenting | 15:23 |
| *** bcornec1 has joined #openstack-ironic | 15:23 | |
| TheJulia | okay | 15:24 |
| jtanner | i think it needs to be "virsh define" instead of "virsh create" | 15:28 |
| jtanner | been a long time since i've dug into libvirt | 15:28 |
| NobodyCam | Good Morning Ironic'ers | 15:28 |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: Include IPA Version during heartbeat https://review.openstack.org/364834 | 15:29 |
| TheJulia | jtanner: that could be, but I really don't remember | 15:29 |
| *** pcaruana has quit IRC | 15:31 | |
| sambetts | rloo: hi, I'm a little lost to why we're pinning the RPC API on the API service side if all conductors are upgraded before APIs | 15:34 |
| rloo | sambetts: conductors upgraded and *pinned* | 15:35 |
| sambetts | rloo: I wasn't a large part of the design for rolling upgrade, but I thought the plan would have been pin RPC version on conductors, upgrade and restart all conductors, unpin conductors, move on to API services | 15:35 |
| rloo | sambetts: we need to get all services upgraded and pinned first before we unpin | 15:35 |
| rloo | sambetts: good question. i wasn't part of the initial thing; that was the proposed process. let me think about it. | 15:36 |
| rloo | sambetts: cannot unpin conductors until all api is upgraded cuz we cannot save objects in DB in latest version before all code can understand it | 15:37 |
| rloo | sambetts: so if we unpin conductor, it will save objects in latest version. but there could be api services running old code and no idea how to handle new objects in db. | 15:38 |
| *** ijw has joined #openstack-ironic | 15:38 | |
| sambetts | can we pin object version but not RPC API version? | 15:39 |
| rloo | sambetts: well, we might be able to, i dunno, but do you really want to complicate things? | 15:39 |
| sambetts | also everything should keep working until we do the DB contract operation right? | 15:39 |
| rloo | sambetts: yes, by 'DB contract' you mean eg delete a column? | 15:40 |
| *** rama_y has joined #openstack-ironic | 15:42 | |
| sambetts | rloo: IMO it simplifies things, because new API services don't need to check if they can use an RPC version or not, new ones can just start using their features as soon as they come up | 15:44 |
| *** rcernin has quit IRC | 15:45 | |
| sambetts | rloo: unfortunatly because both the API and conductor read the DB, the object layer is really the sync point we need to pin | 15:46 |
| *** SerenaFeng has quit IRC | 15:46 | |
| *** ijw has quit IRC | 15:47 | |
| sambetts | rloo: also when you say unpin, does that require a service restart? or are we automagically detecting it? | 15:47 |
| rloo | sambetts: unpin requires service restart until some sighup feature is done and i need to find out what the status is with that feature. | 15:48 |
| *** krtaylor has quit IRC | 15:48 | |
| sambetts | rloo: I thought the original plan was that conductors would detect other conductor versions and run at the lowest of the other conductors in the cluster, so that once all conductors are upgrade they all automatically unpin | 15:48 |
| sambetts | themselves without a restart | 15:49 |
| rloo | sambetts: honestly, that doesn't ring a bell for me. when was that discussed? written down? | 15:49 |
| rloo | sambetts: again, it doesn't work cuz API accesses the DB. | 15:49 |
| rloo | sambetts: but I hadn't heard of that conductor plan. | 15:49 |
| rloo | sambetts: i don't recall it being in Tan Lin's original spec, or in Greg's updated spec. I haven't veered far from those specs. | 15:50 |
| *** jroll has quit IRC | 15:52 | |
| sambetts | yeah, I remember it being discussed before, not sure if it was at a summit or not, the reason I thought we needed to automatically do it based on sync between the conductors, is because if you have a service restart in there to unpin the service then you end up in a situation where some of your services are pinned and some aren't, which is exactly the same as being in the situation of upgrading | 15:52 |
| sambetts | them with no pin | 15:52 |
| *** jroll has joined #openstack-ironic | 15:53 | |
| sambetts | where some services are on old version and some are on new | 15:53 |
| rloo | sambetts: have you read the spec? (just don't want to go over what is already described there) | 15:53 |
| rloo | sambetts: again, even if we sync the conductors, we still have old API running. | 15:54 |
| sambetts | rloo: yeah, I read through the spec and I struggled with it TBH | 15:54 |
| sambetts | rloo: IMO, I thought it would work like this: upgrade all the conductors, upgrade all the APIs, then run the migrations | 15:55 |
| rloo | sambetts: i guess i don't understand your comment above, 'some of your services are pinned and some aren't, which is exactly the same as being in the situation of upgrading'. the pinning/unpinning is part of the upgrading? | 15:55 |
| sambetts | rloo: I meant upgrading without pinning | 15:55 |
| rloo | sambetts: but it doesn't work that way. how can upgraded conductors work with old APIs? | 15:55 |
| *** srart has joined #openstack-ironic | 15:56 | |
| rloo | sambetts: it could only work as you suggest if API did not go directly to DB. | 15:56 |
| rloo | sambetts: in your scenario, upgraded conductors would update the DB with new objects that old API will barf on. | 15:56 |
| rloo | sambetts: actually, old API would not barf on but would interpret incorrectly. | 15:56 |
| *** e0ne has quit IRC | 15:57 | |
| sambetts | rloo: thats why I would run the migrations last, because the services could detect the object version from the DB itself | 15:57 |
| sambetts | this is why I was struggling with the spec | 15:58 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/471830 | 15:58 |
| rloo | sambetts:to be honest, i don't doubt that there are multiple ways to do the rolling upgrades. | 15:58 |
| rloo | sambetts: the spec has been around for many months if not more than a year. i wish you had brought this up a lot sooner. | 15:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/newton: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/473518 | 15:58 |
| dtantsur | ok, the last backport ^^^ | 15:59 |
| rloo | sambetts: i am totally fine if we put the existing rolling upgrades on hole while you look into and propose another solution. | 15:59 |
| rloo | sambetts: s/hole/hold/ | 15:59 |
| dtantsur | rloo: does it mean reverting the version column patch? | 15:59 |
| rloo | dtantsur: dunno, it might mean reverting the patch before that where we added pinning. | 16:00 |
| rloo | dtantsur: it depends on what sambetts is thinking of. | 16:00 |
| dtantsur | I'd continue with the current plan, unless the new plan is really, really better | 16:00 |
| *** ijw has joined #openstack-ironic | 16:01 | |
| rloo | dtantsur: i don't know what the new plan is, and i don't know whether it is better or not. just that i am not volunteering to look into it. | 16:01 |
| dtantsur | I'm just noting that we must have really good reasons to change the direction at this stage.. | 16:02 |
| rloo | dtantsur, sambetts: i just want to be open to new ideas. honestly, i've spent so much time trying to understand the existing spec, how nova does it, etc, that i don't think i could (or want to?) fairly look outside the box for a different solution. | 16:02 |
| dtantsur | a different (principally) solution would be to use an extrean orchestrator like etcd | 16:02 |
| rloo | dtantsur: right | 16:03 |
| sambetts | rloo: I'm happy to continue with the current plan I was just struggling to work out how the current plan dealt nicely with new RPC API features without a bunch of if statements everywhere disabling them when we're pinned | 16:04 |
| dtantsur | I'm pretty sure the number of "if" statements is going to explode | 16:05 |
| dtantsur | but I'm not sure any option will avoid it. old code is old code, we can't magically make it understand the new code. | 16:05 |
| rloo | sambetts: i suspect we will not end up with IF code based on RPC version; more likely with object version, but it really depends on the feature. | 16:05 |
| rloo | sambetts: as dtantsur sez ^^. The POC I put up with a fake new feature has an if on REST API version and on object (node) version. | 16:06 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Unit test framework for Ironic-UI API service https://review.openstack.org/469459 | 16:06 |
| *** krtaylor has joined #openstack-ironic | 16:07 | |
| rloo | sambetts: i think when I looked at the physical network address work, that the port object version (instead of rpc API version) could be 'iffed' on. | 16:07 |
| *** kjw3 has joined #openstack-ironic | 16:09 | |
| *** root4 has joined #openstack-ironic | 16:10 | |
| sambetts | rloo: yeah, sounds good, I'm sure the flow in the spec has all the edge cases worked out better than I've thought them through, I'm not going to try to rewrite the world :) | 16:12 |
| *** root2 has quit IRC | 16:14 | |
| sambetts | rloo: question though, during unpinning, how do we deal with the fact that some services will be on new versions and some aren't? | 16:15 |
| rloo | sambetts: ok. i was just thinking of your migration-after-upgrade. that won't work unless you add code at the db layer, cuz if the db hasn't been updated, the new code will barf cuz it expects the updates. i guess i don't want to touch the code at the db layer with ifs... | 16:15 |
| rloo | sambetts: unpinning of conductor or api or both? | 16:15 |
| rloo | sambetts: i always have to refer to this: http://specs.openstack.org/openstack/ironic-specs/specs/approved/support-rolling-upgrade.html#rolling-upgrade-process | 16:16 |
| rloo | sambetts: that table there. | 16:16 |
| rloo | sambetts: i tested all these cases... so you can see there are some steps where the conductor or api have mixed versions | 16:16 |
| sambetts | rloo: API I guess, because with conductor the APIs won't request a new version until their unpinned right?? | 16:17 |
| rloo | sambetts: during unpinning, i mean, when we first unpin, we know that *all* services are running new code | 16:17 |
| rloo | sambetts: by the time we unpin API, all conductors are unpinned | 16:17 |
| rloo | sambetts: so we have some pinned, some unpinned API. | 16:17 |
| rloo | sambetts: the APIs can access DB and new objects, no problem. the only diff is that if you/user gets a pinned API, won't be able to access new features. | 16:18 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:18 | |
| *** lucasagomes is now known as lucas-afk | 16:21 | |
| *** dprince has joined #openstack-ironic | 16:25 | |
| *** ricardoas has joined #openstack-ironic | 16:26 | |
| *** ijw has quit IRC | 16:29 | |
| *** mtanino_ has quit IRC | 16:31 | |
| sambetts | ah ok I think I'm getting it now... so basically the reason we pin is just to get everything running the new object code | 16:32 |
| sambetts | and then once everything has the new object code we can start enabling the new features etc | 16:33 |
| sambetts | thanks for helping me understand it, I was struggling with the spec | 16:35 |
| rloo | sambetts: yup, you got it. sorry about the spec -- guess it needs to be written better :-( | 16:37 |
| *** makowals has quit IRC | 16:38 | |
| *** hjensas has joined #openstack-ironic | 16:40 | |
| *** hjensas has quit IRC | 16:41 | |
| *** hjensas has joined #openstack-ironic | 16:41 | |
| mgoddard | rloo, sambetts: I realise this work is still in progress, but it might help to have a list of important points /assumptions to consider when working with rolling upgrades | 16:44 |
| mgoddard | rloo, sambetts: e.g. "as a pinned conductor, I cannot assume that API services are upgraded or pinned" | 16:45 |
| *** makowals has joined #openstack-ironic | 16:45 | |
| *** kaifeng has joined #openstack-ironic | 16:45 | |
| mgoddard | rloo, sambetts: "as a pinned API service, I cannot make pre-upgrade RPC calls to a conductor" | 16:46 |
| rloo | mgoddard: good point. i think there is patch for dev doc update, but cannot recall what it sez, will check and add if not there. | 16:46 |
| mgoddard | rloo, sambetts: also, what expectations are there about the level of support for existing/new features at various points during the upgrade? | 16:47 |
| *** bcornec1 has quit IRC | 16:47 | |
| rloo | mgoddard: in the rolling upgrades spec, i think it mentions that no new features should be made avail (to user) until after the upgrade is done. let me see... | 16:48 |
| mgoddard | rloo: I don't see how we could sensibly do that without also pinning the API microversion | 16:48 |
| sambetts | yeah the biggest point for me is that the API service touches the DB, if it didn't do that then you could upgrade all the conductors and the APIs wouldn't care because they just keep talking an older RPC API version and the conductors deal with that when they receive the request | 16:48 |
| *** stendulker has joined #openstack-ironic | 16:49 | |
| mgoddard | sambetts: although if they didn't touch the DB then either you wouldn't need versioned objects, or you'd need a nova-conductor like broker service | 16:49 |
| rloo | sambetts: ++, easier if only conductor (or one service) touched DB only | 16:50 |
| sambetts | mgoddard: you'd still need versioned objects, because the conductor would have to convert the objects before responding to the RPC request | 16:50 |
| rloo | mgoddard: woudl still need versioning, eg, old API would talk to conductor to get object and would need to tell it what version it understands | 16:50 |
| *** kaifeng has quit IRC | 16:51 | |
| *** ccamacho has joined #openstack-ironic | 16:51 | |
| mgoddard | sambetts, rloo: perhaps but not necessarily - could just pass through the API call to an RPC. no objects required | 16:51 |
| mgoddard | anyway, we're off in fantasy land now :) | 16:51 |
| *** kaifeng has joined #openstack-ironic | 16:52 | |
| sambetts | heh | 16:52 |
| mgoddard | argh, it's meeting time and I'm about to get kicked out of my office. I'll be back shortly... | 16:52 |
| sambetts | but the object code reading/writing the DB that is the key reason we have to pin from what I can tell and I feel that point is missed in the spec, TBH not sure if we even need to pin RPC because the RPC versioning should handle that | 16:54 |
| rloo | mgoddard: wrt new features, typically they are guarded by new REST API version, so don't use new version until after upgrade | 16:55 |
| sambetts | rloo: the Ironic API service will advertise that it is avavible straight away unless we hide it on our side | 16:56 |
| sambetts | and you can't stop users hitting it unless we hide it | 16:56 |
| rloo | sambetts: true, that's a good point. if pinned, we could hide the new microversions. | 16:57 |
| *** mgoddard has quit IRC | 16:57 | |
| rloo | sambetts: but i think that would require an RFE. | 16:57 |
| *** ralonsoh has quit IRC | 16:57 | |
| * dtantsur would be happy to approve such an RFE | 16:57 | |
| jtanner | what's the next place to look for some sort of logging related to "deploy failed" on a node? | 16:58 |
| sambetts | rloo: would it not be part of the rolling upgrade spec? | 16:58 |
| *** derekh has quit IRC | 16:59 | |
| rloo | sambetts: it wasn't | 16:59 |
| rloo | sambetts: i don't know if it needs to be part of it... mostly cuz i don't want to update, and i think it can be done separately. let me think about it. | 17:00 |
| sambetts | rloo: its acutally mentions in there as an additional enhancement http://specs.openstack.org/openstack/ironic-specs/specs/approved/support-rolling-upgrade.html#rest-api-impact | 17:01 |
| rloo | sambetts: aha! | 17:01 |
| *** dsneddon has quit IRC | 17:02 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall https://review.openstack.org/471831 | 17:03 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 17:03 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 17:03 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 17:03 |
| milan | boom! O:-) | 17:04 |
| sambetts | rloo: this sentence "Therefore, until the upgrade is completed, API requests should not be issued for new functionality (i.e., with new API micro versions) since there is no guarantee that they will work properly." worries me because how can you stop that happening without the additional peice, e.g. people using the CLI with latest or equiv set (I think the default for OSC soon right?) will get | 17:05 |
| sambetts | weird unexpected invalid API version on some requests but not others | 17:05 |
| *** dprince has quit IRC | 17:06 | |
| sambetts | because we'll ask for the latest API version we can use from ironic and negotiate it and use it for everything even if we don't have too | 17:06 |
| *** xavierrr has joined #openstack-ironic | 17:08 | |
| *** stendulker has quit IRC | 17:08 | |
| *** xavierrr has left #openstack-ironic | 17:08 | |
| *** stendulker has joined #openstack-ironic | 17:08 | |
| *** skarthik has quit IRC | 17:10 | |
| *** karts has joined #openstack-ironic | 17:11 | |
| *** racedo has quit IRC | 17:11 | |
| *** mgoddard has joined #openstack-ironic | 17:11 | |
| *** ChubYann has joined #openstack-ironic | 17:14 | |
| dtantsur | a trivial patch to help move forward python 3: https://review.openstack.org/#/c/471072/ | 17:15 |
| patchbot | patch 471072 - ironic - Runs the script configure_vm.py in py3.5 | 17:15 |
| *** aparnav has joined #openstack-ironic | 17:16 | |
| *** trown is now known as trown|lunch | 17:20 | |
| *** racedo has joined #openstack-ironic | 17:20 | |
| sambetts | dtantsur: +A | 17:21 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Extend backend mock with port operations https://review.openstack.org/473539 | 17:22 |
| rajinir | krtaylor: Have a question on molten iron. I have a requirement to run our CI with previous generation servers and current ones. Is there a way to allocate one set of servers to a build type? | 17:23 |
| rajinir | mjturek: Have a question on molten iron. I have a requirement to run our CI with previous generation servers and current ones. Is there a way to allocate one set of servers to a build type? | 17:23 |
| sambetts | dtantsur: I've got to run, so I won't be around for the rest of the meeting, if you mention me I'll try to follow up on anything tomorrrow if needed | 17:27 |
| dtantsur | ok, thanks. see you | 17:27 |
| *** leetpy has joined #openstack-ironic | 17:27 | |
| *** sambetts is now known as sambetts|afk | 17:27 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat https://review.openstack.org/364861 | 17:28 |
| *** dprince has joined #openstack-ironic | 17:30 | |
| *** dsneddon has joined #openstack-ironic | 17:32 | |
| *** racedo has quit IRC | 17:32 | |
| *** e0ne has joined #openstack-ironic | 17:36 | |
| *** racedo has joined #openstack-ironic | 17:37 | |
| *** racedo has quit IRC | 17:38 | |
| *** amarao has quit IRC | 17:39 | |
| krtaylor | rajinir, I am assuming from one pool? but that would be in how molten iron would be used - also mmedvede might be able to help with how to set up | 17:40 |
| *** jaosorior is now known as jaosorior_away | 17:41 | |
| *** leetpy has quit IRC | 17:45 | |
| *** racedo has joined #openstack-ironic | 17:46 | |
| *** racedo has quit IRC | 17:49 | |
| mmedvede | rajinir: iirc, molteniron does not allow to specify any filters which are used to allocate a node. So I would say there is no easy way to do it the way it is implemented | 17:49 |
| *** fragatina has quit IRC | 17:54 | |
| *** tesseract has quit IRC | 17:54 | |
| *** racedo has joined #openstack-ironic | 17:54 | |
| krtaylor | mmedvede, rajinir it would have to be handled outside of molteniron | 17:55 |
| *** racedo has quit IRC | 17:58 | |
| mmedvede | krtaylor: if you mean _without_ molteniron, then yes. molteniron would give you the first node it sees as available. If it is not what you want, you can keep asking for more until you get the one you wnat. But that defeats the purpose of molteniron. So either molteniron needs to have that added, or something else used instead of it | 17:59 |
| krtaylor | yeah, thats why I was asking about in the same pool | 18:00 |
| *** aparnav has quit IRC | 18:01 | |
| *** kaifeng has left #openstack-ironic | 18:01 | |
| * milan relocates | 18:01 | |
| milan | Good night Ironic! :) | 18:01 |
| Nisha_Agarwal | guys, can u atleast leave your opinion on the RFE? | 18:01 |
| Nisha_Agarwal | milan, ^^^^ | 18:02 |
| milan | Nisha_Agarwal, yeah I need to read through it tho | 18:02 |
| Nisha_Agarwal | milan, good night | 18:02 |
| milan | so I guess tomorrow | 18:02 |
| Nisha_Agarwal | milan, sure | 18:02 |
| milan | Nisha_Agarwal night! :) | 18:02 |
| Nisha_Agarwal | milan, Thanks | 18:02 |
| *** stendulker has quit IRC | 18:02 | |
| pas-ha | rajinir: take a look at mogan | 18:02 |
| mmedvede | krtaylor, rajinir : ah, you can use a separate deployment of molteniron to track two pools. But it is just a workaround | 18:03 |
| *** milan has quit IRC | 18:03 | |
| pas-ha | rajinir: afaik mogan was created a kind of nova-only-for-baremetal, with scheduling etc | 18:04 |
| *** dprince has quit IRC | 18:08 | |
| openstackgerrit | Sean Dague proposed openstack/ironic master: WIP: pass global_request_id to cinder/neutron https://review.openstack.org/472666 | 18:14 |
| mgoddard | could anyone comment on whether tempest tests are normally added with the API changes they are testing or not? | 18:20 |
| mgoddard | seems like if the tempest plugin is being factored out then separate might be better? | 18:21 |
| *** Nisha_ has joined #openstack-ironic | 18:22 | |
| *** Nisha_Agarwal has quit IRC | 18:22 | |
| *** dprince has joined #openstack-ironic | 18:24 | |
| *** cinerama` is now known as cinerama | 18:24 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:27 | |
| *** Nisha_ has quit IRC | 18:27 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Physical network aware VIF attachment https://review.openstack.org/470406 | 18:29 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add tempest tests for physical networks https://review.openstack.org/470915 | 18:29 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Move port object creation to conductor https://review.openstack.org/469931 | 18:29 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate portgroup physical network consistency https://review.openstack.org/469932 | 18:29 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Expose ports' physical network attribute in API https://review.openstack.org/469933 | 18:29 |
| rajinir | mmedvede, pas-ha: thanks | 18:33 |
| *** SerenaFeng has joined #openstack-ironic | 18:33 | |
| openstackgerrit | Merged openstack/ironic master: Runs the script configure_vm.py in py3.5 https://review.openstack.org/471072 | 18:39 |
| *** mgoddard has quit IRC | 18:40 | |
| *** dyasny_ has joined #openstack-ironic | 18:44 | |
| *** rajinir has quit IRC | 18:46 | |
| *** rajinir has joined #openstack-ironic | 18:46 | |
| *** dyasny has quit IRC | 18:47 | |
| *** SerenaFeng has quit IRC | 18:48 | |
| *** Nisha_ has joined #openstack-ironic | 18:54 | |
| *** Nisha_Agarwal has quit IRC | 18:54 | |
| *** dyasny_ has quit IRC | 18:55 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Add OSC 'baremetal driver property list' command https://review.openstack.org/381153 | 18:55 |
| *** trown|lunch is now known as trown | 18:56 | |
| *** ElCoyote_ has joined #openstack-ironic | 18:58 | |
| *** fellypefca has quit IRC | 19:02 | |
| *** dtantsur is now known as dtantsur|afk | 19:03 | |
| *** adisky__ has quit IRC | 19:03 | |
| *** Nisha_ has quit IRC | 19:11 | |
| *** [1]rpioso has joined #openstack-ironic | 19:15 | |
| *** tellesnobrega has quit IRC | 19:16 | |
| *** pmannidi has quit IRC | 19:17 | |
| *** tellesnobrega has joined #openstack-ironic | 19:20 | |
| *** rpioso has quit IRC | 19:20 | |
| *** dyasny has joined #openstack-ironic | 19:20 | |
| *** [1]rpioso has quit IRC | 19:21 | |
| *** ijw has joined #openstack-ironic | 19:24 | |
| *** ijw has quit IRC | 19:28 | |
| *** mgoddard has joined #openstack-ironic | 19:29 | |
| *** pmannidi has joined #openstack-ironic | 19:30 | |
| *** mgoddard has quit IRC | 19:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/472806 | 19:39 |
| *** dyasny has quit IRC | 19:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/472914 | 19:48 |
| *** [1]rpioso has joined #openstack-ironic | 19:53 | |
| *** leetpy has joined #openstack-ironic | 19:59 | |
| *** dprince has quit IRC | 20:05 | |
| *** pmannidi has quit IRC | 20:10 | |
| *** pmannidi has joined #openstack-ironic | 20:11 | |
| *** amotoki_away is now known as amotoki | 20:13 | |
| *** crushil has quit IRC | 20:14 | |
| *** amotoki is now known as amotoki_away | 20:22 | |
| *** crushil has joined #openstack-ironic | 20:22 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Put tests in correct location for ironic/api/controllers/v1/ https://review.openstack.org/462658 | 20:25 |
| *** e0ne has quit IRC | 20:30 | |
| *** e0ne has joined #openstack-ironic | 20:30 | |
| *** e0ne has quit IRC | 20:31 | |
| *** e0ne has joined #openstack-ironic | 20:31 | |
| *** e0ne has quit IRC | 20:32 | |
| *** fragatina has joined #openstack-ironic | 20:32 | |
| *** e0ne has joined #openstack-ironic | 20:32 | |
| *** e0ne has quit IRC | 20:32 | |
| anupn | TheJulia: i got the stickers, yay! thanks | 20:33 |
| *** e0ne has joined #openstack-ironic | 20:33 | |
| *** e0ne has quit IRC | 20:33 | |
| *** e0ne has joined #openstack-ironic | 20:34 | |
| *** e0ne has quit IRC | 20:34 | |
| *** leetpy has quit IRC | 20:41 | |
| *** leetpy has joined #openstack-ironic | 20:42 | |
| *** jcoufal has quit IRC | 20:43 | |
| *** SerenaFeng has joined #openstack-ironic | 20:49 | |
| rajinir | mjturek: Timed out waiting for Nova hypervisor-stats vcpus >= 4? DO you have the launchpad bug on it? | 20:51 |
| rajinir | krtaylor: Timed out waiting for Nova hypervisor-stats vcpus >= 4? DO you have the launchpad bug on it? I remember IBM driver was seeing this problem | 20:52 |
| *** trown is now known as trown|outtypewww | 20:56 | |
| *** leetpy has quit IRC | 20:58 | |
| *** kjw3 has quit IRC | 20:58 | |
| *** dyasny has joined #openstack-ironic | 21:01 | |
| *** harlowja has quit IRC | 21:02 | |
| *** leetpy has joined #openstack-ironic | 21:02 | |
| *** crushil has quit IRC | 21:03 | |
| *** crushil has joined #openstack-ironic | 21:03 | |
| *** dprince has joined #openstack-ironic | 21:05 | |
| *** leetpy has quit IRC | 21:05 | |
| krtaylor | rajinir, where are you seeing that? it doesn't ring a bell for me off the top of my head | 21:06 |
| *** crushil has quit IRC | 21:18 | |
| *** crushil has joined #openstack-ironic | 21:18 | |
| *** baoli has quit IRC | 21:19 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add options for osc 'port set' command https://review.openstack.org/472457 | 21:20 |
| *** crushil has quit IRC | 21:20 | |
| *** crushil has joined #openstack-ironic | 21:21 | |
| *** shobhar_ has joined #openstack-ironic | 21:24 | |
| *** jkilpatr has quit IRC | 21:30 | |
| *** dprince has quit IRC | 21:33 | |
| *** SerenaFeng has quit IRC | 21:36 | |
| *** shobhar_ has quit IRC | 21:48 | |
| *** shobhar_ has joined #openstack-ironic | 21:48 | |
| *** crushil has quit IRC | 21:50 | |
| *** crushil has joined #openstack-ironic | 21:50 | |
| *** shobhar__ has joined #openstack-ironic | 21:50 | |
| *** shobhar_ has quit IRC | 21:54 | |
| *** harlowja has joined #openstack-ironic | 22:06 | |
| *** bfournie1 has quit IRC | 22:12 | |
| *** crushil has quit IRC | 22:13 | |
| *** crushil has joined #openstack-ironic | 22:14 | |
| *** fragatina has quit IRC | 22:17 | |
| *** boris-42_ has joined #openstack-ironic | 22:20 | |
| jtanner | 2017-06-12 22:29:08.765 15908 ERROR ironic.conductor.manager [req-044fae3a-99c7-4e72-ae29-4d7fd844a894 - - - - -] Error while preparing to deploy to node 97a14e0e-8e82-456b-9a9e-134e9d1990fe: Cannot validate driver deploy. Some parameters were missing in node's instance_info. Missing are: ['root_gb']: MissingParameterValue: Cannot validate driver deploy. Some parameters were missing in node's instance_info. Missing | 22:30 |
| jtanner | are: ['root_gb'] | 22:30 |
| jtanner | is root_gb supposed to come from the inspection process? | 22:31 |
| jtanner | local_gb was set, but not root_rb | 22:31 |
| jtanner | err, root_gb* | 22:31 |
| *** karts has quit IRC | 22:31 | |
| *** karts has joined #openstack-ironic | 22:31 | |
| *** yolanda has joined #openstack-ironic | 22:34 | |
| *** crushil has quit IRC | 22:38 | |
| *** crushil has joined #openstack-ironic | 22:39 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/472806 | 22:46 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/473073 | 23:05 |
| *** crushil has quit IRC | 23:12 | |
| *** crushil has joined #openstack-ironic | 23:13 | |
| *** crushil has quit IRC | 23:15 | |
| *** crushil has joined #openstack-ironic | 23:16 | |
| *** jaypipes has quit IRC | 23:18 | |
| *** [1]rpioso has quit IRC | 23:25 | |
| *** crushil has quit IRC | 23:33 | |
| *** crushil has joined #openstack-ironic | 23:34 | |
| *** crushil has quit IRC | 23:51 | |
| *** crushil has joined #openstack-ironic | 23:52 | |
| *** sasha2 has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!