*** baoli has quit IRC | 00:00 | |
*** jistr has quit IRC | 00:01 | |
*** jistr has joined #openstack-ironic | 00:02 | |
*** hshiina has joined #openstack-ironic | 00:19 | |
*** zackf has joined #openstack-ironic | 00:23 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: API for hardware interface properties https://review.openstack.org/471174 | 00:27 |
---|---|---|
*** zackf has quit IRC | 00:28 | |
*** mtanino has joined #openstack-ironic | 00:28 | |
*** rloo has quit IRC | 00:32 | |
*** tuanluong has joined #openstack-ironic | 00:35 | |
*** shuyingya has joined #openstack-ironic | 00:38 | |
*** bfournie has joined #openstack-ironic | 00:38 | |
*** bfournie has quit IRC | 00:39 | |
*** bfournie has joined #openstack-ironic | 00:40 | |
*** bfournie has quit IRC | 00:40 | |
* aNuposic anupn | 00:40 | |
*** aNuposic is now known as anupn | 00:40 | |
*** rama_y has quit IRC | 00:42 | |
*** shuyingya has quit IRC | 00:43 | |
*** korekhov_ has joined #openstack-ironic | 00:50 | |
*** mtanino has quit IRC | 00:55 | |
*** SerenaFeng has joined #openstack-ironic | 01:01 | |
*** phuongnh has joined #openstack-ironic | 01:02 | |
*** korekhov has joined #openstack-ironic | 01:14 | |
*** korekhov_ has quit IRC | 01:17 | |
*** shuyingya has joined #openstack-ironic | 01:31 | |
*** pmannidi has quit IRC | 01:33 | |
*** litao__ has joined #openstack-ironic | 01:41 | |
*** armax has quit IRC | 01:44 | |
*** causten has joined #openstack-ironic | 01:52 | |
*** shuyingya has quit IRC | 02:11 | |
*** shuyingya has joined #openstack-ironic | 02:12 | |
*** pmannidi has joined #openstack-ironic | 02:20 | |
*** links has joined #openstack-ironic | 02:27 | |
*** Nisha_Agarwal has joined #openstack-ironic | 02:28 | |
*** links has quit IRC | 02:30 | |
*** links has joined #openstack-ironic | 02:31 | |
*** jaganathan has joined #openstack-ironic | 02:42 | |
*** tiendc has joined #openstack-ironic | 02:47 | |
*** ccamacho has quit IRC | 02:48 | |
*** mat128 has quit IRC | 02:56 | |
*** shuyingya has quit IRC | 03:02 | |
*** shuyingya has joined #openstack-ironic | 03:02 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: DNM: Experimental change to see if python3 gate passes https://review.openstack.org/471072 | 03:07 |
*** rbudden has quit IRC | 03:18 | |
*** shuyingy_ has joined #openstack-ironic | 03:20 | |
*** shuyingya has quit IRC | 03:24 | |
*** Nisha_Agarwal has quit IRC | 03:26 | |
*** zhurong has joined #openstack-ironic | 03:28 | |
*** SerenaFeng has quit IRC | 03:43 | |
*** korekhov has left #openstack-ironic | 03:45 | |
*** vikrant has joined #openstack-ironic | 03:51 | |
*** vikrant is now known as vikrant|brb | 03:53 | |
*** shuyingy_ has quit IRC | 03:58 | |
*** hshiina has quit IRC | 03:58 | |
*** shuyingya has joined #openstack-ironic | 03:59 | |
openstackgerrit | Aparna proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 03:59 |
*** hshiina has joined #openstack-ironic | 04:01 | |
*** zhurong has quit IRC | 04:27 | |
*** hshiina has quit IRC | 04:32 | |
*** armax has joined #openstack-ironic | 04:44 | |
*** zhurong has joined #openstack-ironic | 04:54 | |
*** zackf has joined #openstack-ironic | 04:55 | |
*** mvpnitesh has joined #openstack-ironic | 04:59 | |
*** zackf has quit IRC | 04:59 | |
*** tuanluong has quit IRC | 05:08 | |
*** shuyingy_ has joined #openstack-ironic | 05:11 | |
*** shuyingya has quit IRC | 05:15 | |
*** mjura has joined #openstack-ironic | 05:39 | |
*** udesale has joined #openstack-ironic | 05:41 | |
*** jaosorior_away is now known as jaosorior | 05:44 | |
openstackgerrit | vinay kumar muddu proposed openstack/proliantutils master: Add HPE extensions to redfish connection library https://review.openstack.org/467912 | 05:44 |
*** Nisha_Agarwal has joined #openstack-ironic | 05:46 | |
*** SerenaFeng has joined #openstack-ironic | 05:46 | |
*** hw_wutianwei has quit IRC | 05:55 | |
*** hw_wutianwei has joined #openstack-ironic | 05:56 | |
pas-ha | morning ironic :) | 05:58 |
*** tuanluong has joined #openstack-ironic | 06:04 | |
*** causten has quit IRC | 06:06 | |
*** rcernin has joined #openstack-ironic | 06:10 | |
*** korekhov_ has joined #openstack-ironic | 06:10 | |
*** jaganathan has quit IRC | 06:14 | |
*** ChubYann has quit IRC | 06:15 | |
*** korekhov_ has quit IRC | 06:16 | |
openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 06:27 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: DNM: Experimental change to see if python3 gate passes https://review.openstack.org/471072 | 06:31 |
*** zhurong has quit IRC | 06:32 | |
openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 06:33 |
*** udesale has quit IRC | 06:37 | |
*** pcaruana has joined #openstack-ironic | 06:39 | |
openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 06:39 |
*** zhurong has joined #openstack-ironic | 06:40 | |
*** jaosorior is now known as jaosorior_away | 06:41 | |
*** hw_wutianwei has quit IRC | 06:42 | |
*** udesale has joined #openstack-ironic | 06:42 | |
*** hw_wutianwei has joined #openstack-ironic | 06:43 | |
*** ppiela has quit IRC | 06:47 | |
*** ppiela_ has joined #openstack-ironic | 06:47 | |
*** patriciadomin has quit IRC | 06:49 | |
*** jrist has quit IRC | 06:50 | |
*** patriciadomin has joined #openstack-ironic | 06:51 | |
*** rbartal has joined #openstack-ironic | 06:52 | |
*** udesale__ has joined #openstack-ironic | 06:54 | |
*** udesale has quit IRC | 06:56 | |
*** hw_wutianwei has quit IRC | 07:00 | |
*** hw_wutianwei has joined #openstack-ironic | 07:01 | |
*** aarefiev has joined #openstack-ironic | 07:03 | |
*** jaganathan has joined #openstack-ironic | 07:05 | |
*** GheRivero has left #openstack-ironic | 07:05 | |
*** Nisha_Agarwal has quit IRC | 07:08 | |
openstackgerrit | Merged openstack/bifrost master: Revert "Remove useless paramiko requirements" https://review.openstack.org/471136 | 07:11 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Remove legacy auth loading https://review.openstack.org/469626 | 07:12 |
*** athomas has joined #openstack-ironic | 07:13 | |
*** tesseract has joined #openstack-ironic | 07:13 | |
openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 07:24 |
*** amarao has joined #openstack-ironic | 07:26 | |
*** armax has quit IRC | 07:29 | |
*** mgoddard_ has joined #openstack-ironic | 07:31 | |
*** root4 has joined #openstack-ironic | 07:32 | |
*** mgoddard_ has quit IRC | 07:36 | |
*** Guest46818 has quit IRC | 07:36 | |
*** jrist has joined #openstack-ironic | 07:36 | |
*** jrist has quit IRC | 07:36 | |
*** jrist has joined #openstack-ironic | 07:36 | |
*** dtantsur|afk is now known as dtantsur | 07:38 | |
dtantsur | morning ironic | 07:38 |
*** l_jacob has quit IRC | 07:39 | |
*** Nisha_Agarwal has joined #openstack-ironic | 07:40 | |
vdrok | good morning ironic, pas-ha and dtantsur ! | 07:41 |
dtantsur | o/ | 07:42 |
pas-ha | morning dtantsur :) | 07:42 |
dtantsur | hey pas-ha! so your glance clean up started having progress :) | 07:42 |
*** ccamacho has joined #openstack-ironic | 07:46 | |
pas-ha | yep, thanks :) nex step is to rework auth, for glance in particular. there's a hacked-up patch, but I'm splitting it to more smaller independent ones | 07:48 |
pas-ha | and it will be kind of urgent, as the moment glance API goes under apache vhost (and I've seen such patches are up already) we'll be broken big time | 07:49 |
*** Nisha_Agarwal has quit IRC | 07:49 | |
*** mvpnitesh has quit IRC | 07:54 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** Serlex has joined #openstack-ironic | 08:01 | |
*** links has quit IRC | 08:02 | |
dtantsur | ok¸ I'll keep an eye on them (feel free to add me to reviews) | 08:05 |
*** causten has joined #openstack-ironic | 08:07 | |
*** causten has quit IRC | 08:11 | |
*** e0ne has joined #openstack-ironic | 08:13 | |
*** yuriyz has joined #openstack-ironic | 08:19 | |
*** links has joined #openstack-ironic | 08:19 | |
*** stendulker has joined #openstack-ironic | 08:22 | |
*** dtantsur is now known as dtantsur|bbl | 08:22 | |
openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Improve driver_info/redfish_verify_ca value validation https://review.openstack.org/462982 | 08:25 |
*** lucas-afk is now known as lucasagomes | 08:26 | |
lucasagomes | morning all | 08:26 |
*** mgoddard_ has joined #openstack-ironic | 08:29 | |
*** Serlex has quit IRC | 08:33 | |
openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Add a note for specifying octal value of permission https://review.openstack.org/468778 | 08:34 |
*** milan has joined #openstack-ironic | 08:34 | |
*** zhurong has quit IRC | 08:35 | |
*** zhurong has joined #openstack-ironic | 08:41 | |
openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/469565 | 08:42 |
openstackgerrit | Aparna proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 08:43 |
*** derekh has joined #openstack-ironic | 08:43 | |
*** fragatina has joined #openstack-ironic | 08:45 | |
openstackgerrit | fupingxie proposed openstack/ironic master: Add Error Codes https://review.openstack.org/471261 | 08:46 |
*** priteau has joined #openstack-ironic | 08:49 | |
*** hoonetorg has quit IRC | 08:51 | |
*** jaosorior_away is now known as jaosorior | 08:53 | |
*** rbartal has quit IRC | 09:00 | |
openstackgerrit | paresh sao proposed openstack/ironic master: Generic implementation for UEFI iSCSI boot for vendors https://review.openstack.org/308425 | 09:01 |
openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 09:01 |
*** rbartal has joined #openstack-ironic | 09:02 | |
*** jaosorior has quit IRC | 09:04 | |
*** hoonetorg has joined #openstack-ironic | 09:09 | |
*** zackf has joined #openstack-ironic | 09:16 | |
*** zackf has quit IRC | 09:21 | |
openstackgerrit | Tao Li proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 09:24 |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: Cannot delete port of a node whose provision state is active https://review.openstack.org/470273 | 09:24 |
*** sambetts|afk is now known as sambetts | 09:24 | |
sambetts | Morning all | 09:25 |
*** pmannidi has quit IRC | 09:25 | |
*** links has quit IRC | 09:25 | |
*** mvpnitesh has joined #openstack-ironic | 09:27 | |
*** hshiina has joined #openstack-ironic | 09:31 | |
*** zhurong has quit IRC | 09:33 | |
*** l_jacob has joined #openstack-ironic | 09:33 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Use new oslo db EngineFacade https://review.openstack.org/453057 | 09:41 |
*** links has joined #openstack-ironic | 09:43 | |
*** udesale has joined #openstack-ironic | 09:53 | |
*** udesale__ has quit IRC | 09:54 | |
*** zackf has joined #openstack-ironic | 09:54 | |
*** fragatina has quit IRC | 09:58 | |
*** fragatina has joined #openstack-ironic | 09:59 | |
*** zackf has quit IRC | 09:59 | |
*** SerenaFeng has quit IRC | 10:01 | |
*** dtantsur|bbl is now known as dtantsur | 10:05 | |
*** causten has joined #openstack-ironic | 10:08 | |
*** hjensas has quit IRC | 10:09 | |
*** hjensas has joined #openstack-ironic | 10:09 | |
*** hjensas has joined #openstack-ironic | 10:09 | |
*** mgoddard_ has quit IRC | 10:11 | |
*** mgoddard_ has joined #openstack-ironic | 10:12 | |
*** causten has quit IRC | 10:13 | |
*** SerenaFeng has joined #openstack-ironic | 10:21 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Remove legacy auth loading https://review.openstack.org/469626 | 10:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 10:39 |
*** jkilpatr has quit IRC | 10:41 | |
*** shvepsy has joined #openstack-ironic | 10:45 | |
shvepsy | Hi there! | 10:46 |
shvepsy | Can u help me to find config issue? | 10:46 |
sambetts | shvepsy: ask away and maybe someone will be able to help | 10:48 |
*** mvpnitesh has quit IRC | 10:53 | |
*** mvpnitesh has joined #openstack-ironic | 10:53 | |
shvepsy | I have problems with ironic-inspector: Periodic update of firewall rules failed : AmbiguousAuthSystem: Must provide Keystone credentials or user-defined endpoint and token | 10:53 |
shvepsy | but other ironic tasks working without problems. | 10:53 |
shvepsy | Config and log - http://paste.openstack.org/show/611485/ | 10:53 |
shvepsy | ironic-inspector/jessie-backports,jessie-backports,now 3.2.0-2~bpo8+1 all [installed] | 10:53 |
shvepsy | * but other ironic-inspector tasks working without problems. | 10:54 |
openstackgerrit | vinay kumar muddu proposed openstack/proliantutils master: Add secure boot APIs support through redfish https://review.openstack.org/471314 | 10:57 |
sambetts | shvepsy: during that periodic task inspector needs to make ironic API calls, it appears that your auth_url in the Ironic section is wrong, auth_url should always be the keystone endpoint | 10:58 |
*** jkilpatr has joined #openstack-ironic | 10:59 | |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: Cannot delete port of a node whose provision state is active https://review.openstack.org/470273 | 10:59 |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 11:00 |
shvepsy | sambetts: Thank you, i will check. | 11:00 |
*** stendulker has quit IRC | 11:02 | |
*** phuongnh has quit IRC | 11:07 | |
*** tellesnobrega has quit IRC | 11:12 | |
*** lucasagomes is now known as lucas-hungry | 11:16 | |
*** Nisha_Agarwal has joined #openstack-ironic | 11:20 | |
TheJulia | good morning everyone | 11:21 |
*** Nisha_ has joined #openstack-ironic | 11:24 | |
Nisha_ | Hi Ironic'iers!!! | 11:25 |
*** Nisha_Agarwal has quit IRC | 11:25 | |
Nisha_ | dtantsur, regarding py35 issue...you were right....the devstack is trying to import the module jinja2 in py27 while the module is there in py35.... http://logs.openstack.org/72/471072/3/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/90eac25/logs/pip3-freeze.txt.gz | 11:27 |
TheJulia | Nisha_: That was what I was trying to put in the bug you filed | 11:28 |
Nisha_ | TheJulia, yes | 11:28 |
Nisha_ | TheJulia, i am not sure how to enforce devstack plugin to use py35 | 11:29 |
Nisha_ | may be USE_PYTHON3 variable may help | 11:29 |
Nisha_ | TheJulia, just will put the experimental patch for it | 11:29 |
TheJulia | Offhand, changing the #!/usr/bin/env python to #!/usr/bin/python3 comes to mind... Although that doesn't seem craceful | 11:31 |
TheJulia | err, graceful | 11:31 |
*** zhurong has joined #openstack-ironic | 11:31 | |
Nisha_ | TheJulia, it shouldnt be that way | 11:31 |
Nisha_ | the devstack needs to work in both env | 11:31 |
Nisha_ | in py2.7 and py3.5 | 11:31 |
TheJulia | Well, this is the risk of writing random scripts in python... | 11:32 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: DNM: Experimental change to see if python3 gate passes https://review.openstack.org/471072 | 11:33 |
*** udesale has quit IRC | 11:34 | |
*** root4 has quit IRC | 11:35 | |
*** root4 has joined #openstack-ironic | 11:36 | |
*** mat128 has joined #openstack-ironic | 11:36 | |
*** udesale has joined #openstack-ironic | 11:37 | |
dtantsur | Nisha_, TheJulia, we can change the place where this script is called from just ./path/to/script.py to $PYTHON ./path/to/script.py | 11:47 |
dtantsur | and then figure out $PYTHON based on which python openstack runs with | 11:47 |
TheJulia | ++++ | 11:47 |
TheJulia | we should... ideally... strip the #!/usr/bin/env part out of the file at that point. FWIW | 11:48 |
dtantsur | well, it still can be run on its own, but the caller has to be sure to install the dependencies | 11:48 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 11:48 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 11:48 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 11:48 |
jtanner | morning | 11:48 |
dtantsur | morning jtanner | 11:48 |
TheJulia | dtantsur: Eh.... true. I think there is doc for that someplace | 11:48 |
milan | morning Ironic! :) | 11:48 |
TheJulia | which mean the #!... part would need to remain :( | 11:49 |
dtantsur | yep | 11:49 |
TheJulia | good morning jtanner | 11:49 |
jtanner | i had bit more success last night by following rcarrillocruz's pathway | 11:49 |
jtanner | now i'm at the point where the tests are unable to inspect the vm | 11:50 |
jtanner | so i get to go code diving and learn what "inspect" means | 11:50 |
*** rcernin has quit IRC | 11:52 | |
dtantsur | sambetts: do you know if there is a way to remove a package from tinyipa? i.e. the opposite of tce-load? | 11:52 |
dtantsur | jtanner: this may help https://docs.openstack.org/developer/ironic/deploy/inspection.html | 11:53 |
jtanner | maybe | 11:53 |
*** pcaruana has quit IRC | 11:53 | |
jtanner | i think it's some sort of issue with virtualbmc, but i dunno really | 11:54 |
sambetts | dtantsur: which package do you want to remove? the tinycorelinux base image effectivly starts with no packages installed only whats built into the OS and we just add what we need at build time | 11:54 |
dtantsur | sambetts: I need to add package for building (in both reqs), then remove it as the very end | 11:55 |
*** jaosorior has joined #openstack-ironic | 11:56 | |
*** litao__ has quit IRC | 11:58 | |
TheJulia | jtanner: by chance do you see anything in the console log for the vm? | 11:58 |
TheJulia | jtanner: First thing would be to identify if IPA booted, really... | 11:59 |
jtanner | ... i was gonna ask about that | 11:59 |
jtanner | what is IPA's role here? | 11:59 |
sambetts | dtantsur: no requirements for build should be in finalreqs, the reason tinyipa has two steps to the build process is to deal with this, buildreqs should be for building, and finalreqs should only be the requirements needed at run time | 12:00 |
TheJulia | IPA is the deployment agent that we also use for cleaning/inspecting nodes. So when we perform those actions, we boot our IPA ramdisk | 12:00 |
jtanner | so this is not "IPA" as in freeipa? | 12:00 |
rcarrillocruz | ironic python agent | 12:01 |
jtanner | k, that makes more sense now | 12:01 |
sambetts | jtanner: IPA is a peice of software we run on the baremetal machine using a ramdisk, during inspection it can query the kernel to find out information about the baremetal system itself | 12:01 |
rcarrillocruz | it's a little program that communicated with conductor to perform actions server<->metal | 12:01 |
TheJulia | jtanner: IPA then reports to the inspector service, which takes action for the ironic api | 12:01 |
rcarrillocruz | communicateS | 12:01 |
jtanner | k, i'm gonna clean up and run again to inspect the console log. I did some manual startups on it, so the log is "dirty" | 12:01 |
sambetts | dtantsur: why do you need a build requirement in the finalreqa? | 12:01 |
TheJulia | rcarrillocruz: well, conductor connects to it, but not with inspection :) | 12:02 |
rcarrillocruz | jtanner: that role does introspection, but not sure if you're even interested in that | 12:02 |
rcarrillocruz | you just care about being able to provision things with virtualbmc | 12:02 |
TheJulia | jtanner: if you have hardware inventory, you very likely don't need inspection | 12:02 |
rcarrillocruz | ? | 12:02 |
TheJulia | err, a hardware inventory | 12:02 |
jtanner | TheJulia: i'm just following the goatpath, or trying to with virtualbmc+libvirt | 12:02 |
TheJulia | and if your doing it manually, you _really_ don't need to know everything about the node that inspector is going to look for | 12:03 |
* TheJulia wonders if we could "borrow" some sort of road paving equipment to widen the path. | 12:03 | |
* milan likes heavy machinery, nothing like a bit of a steam punk ;) | 12:04 | |
* TheJulia wonders if this could be combined with obtaining a tasty IPA | 12:04 | |
jtanner | i like the visual ... but i don't get it =P | 12:04 |
TheJulia | jtanner: your trying to follow a narrow path to achieve something very specific that is a side effect of the testing. Perhaps just use the pieces from entire sequence that test-bifrost.sh is executing to get you exactly what you want. ? | 12:05 |
TheJulia | jtanner: As rcarrillocruz, you likely don't care about inspection. I'm truly curious why it is not working for you, but I can spend time troubleshooting that and you can proceed forth to your goal | 12:06 |
TheJulia | as rcarrillocruz said | 12:07 |
TheJulia | gah, I really need more coffee | 12:07 |
jtanner | me too | 12:07 |
*** rcernin has joined #openstack-ironic | 12:07 | |
TheJulia | Or I just need to go to the store and pickup some IPA | 12:07 |
*** baoli has joined #openstack-ironic | 12:08 | |
jtanner | yuck | 12:08 |
jtanner | i prefer ambers/lagers/hefs | 12:08 |
*** baoli_ has joined #openstack-ironic | 12:09 | |
* TheJulia wonders if the store might still have some pilot mountain pale ale | 12:09 | |
rcarrillocruz | big fan of amber, so hard to find in Spain | 12:09 |
rcarrillocruz | :/ | 12:09 |
*** pcaruana has joined #openstack-ironic | 12:09 | |
TheJulia | I can't drink lagers at all. Something about the fermentation | 12:10 |
TheJulia | whee, looks like I just found myself another bug :( | 12:10 |
jtanner | yay for job security? | 12:10 |
*** zhurong has quit IRC | 12:12 | |
*** mvpnitesh has quit IRC | 12:13 | |
*** mvpnitesh has joined #openstack-ironic | 12:13 | |
*** baoli has quit IRC | 12:13 | |
*** tellesnobrega has joined #openstack-ironic | 12:13 | |
*** udesale__ has joined #openstack-ironic | 12:14 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost master: Ensure ~/.config exists https://review.openstack.org/471328 | 12:14 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 12:15 |
dtantsur | sambetts: well, it was to work around the way pbr works (it needs git all the time). I think I've found another way though ^^^ | 12:16 |
*** udesale has quit IRC | 12:16 | |
*** lucas-hungry is now known as lucasagomes | 12:16 | |
*** tuanluong has quit IRC | 12:17 | |
sambetts | dtantsur: hmmm so the upper constraints file was ending up with a bunch of file:// entries in it which is what was causing problems?? | 12:19 |
*** SerenaFeng has quit IRC | 12:19 | |
dtantsur | sambetts: yep | 12:21 |
dtantsur | and we can't get them from chroot.. and installing them requires git because of pbr | 12:21 |
jtanner | ok, ignoring the failing inspection ... i have a "node" registered to ironic. What's the quick way to "provision" it with something? | 12:22 |
jtanner | virtualbmc based of course | 12:22 |
sambetts | dtantsur: I'm surprised that we actually need upperconstraints for the finalise step at all, because all the requirements are built into wheels and we tell it not to look at anywhere else | 12:23 |
dtantsur | sambetts: hmm, maybe we should drop u-c from the final step indeed | 12:23 |
dtantsur | lemme try it | 12:23 |
sambetts | jtanner: if you've provided all the information you need to provide to the ironic node to make it validate (except the image_source because that'll be provided by nova) you can check using node validate <uuid> | 12:24 |
sambetts | then you need to make sure its in state available, and not in maintaince, then you can issue nova boot with the flavor that matches your baremetal, an image and the baremetal network and ironic will provision it | 12:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 12:25 |
TheJulia | sambetts: jtanner is running standalone. jtanner: you'll need to supply instance_info/image_source | 12:26 |
TheJulia | and instance_info/image_checksum | 12:26 |
sambetts | ah ok | 12:26 |
sambetts | +10000 for using the bifrost deploy playbook for that | 12:27 |
jtanner | Missing are: ['ramdisk', 'kernel', 'image_source'] | 12:27 |
TheJulia | jtanner: what did you do to get that? | 12:28 |
sambetts | node validate <uuid> | 12:28 |
jtanner | ^ that | 12:28 |
TheJulia | you should be able to go from enroll to manageable and available without those parameters | 12:29 |
jtanner | not familiar with those concepts (yet) | 12:30 |
*** Nisha_ has quit IRC | 12:30 | |
*** vikrant|brb has quit IRC | 12:31 | |
rcarrillocruz | i believe you could get going by removing the node | 12:31 |
rcarrillocruz | then modifying the test-bifrost.sh to not do inspect | 12:32 |
rcarrillocruz | there's a flag envvar for that | 12:32 |
rcarrillocruz | that way you get past the inspect which was failing for you | 12:32 |
rcarrillocruz | and go over to the deploy | 12:32 |
rcarrillocruz | ? | 12:32 |
rcarrillocruz | in essence | 12:33 |
rcarrillocruz | modify test-bifrost.sh | 12:33 |
rcarrillocruz | change INSPECT_NODES to false | 12:33 |
rcarrillocruz | then | 12:33 |
*** hshiina has quit IRC | 12:33 | |
*** trown|outtypewww is now known as trown | 12:33 | |
rcarrillocruz | ironic node-set-provision-state testvm1 manage | 12:33 |
rcarrillocruz | ironic node-delete testvm1 | 12:33 |
rcarrillocruz | i think that will delete the node | 12:33 |
rcarrillocruz | then run the script again | 12:33 |
rcarrillocruz | which will re-enroll it | 12:33 |
rcarrillocruz | get past the inspect | 12:33 |
rcarrillocruz | go straight to deploy | 12:34 |
*** bfournie has joined #openstack-ironic | 12:34 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost master: Provide default for test-bifrost.yaml for clouds.yaml https://review.openstack.org/471333 | 12:35 |
openstackgerrit | fupingxie proposed openstack/ironic master: Add Error Codes https://review.openstack.org/471261 | 12:35 |
TheJulia | jtanner: what rcarrillocruz said, basically. :) | 12:36 |
*** shuyingy_ has quit IRC | 12:36 | |
*** hw_wutianwei has quit IRC | 12:37 | |
mvpnitesh | hi all , i'm install ironic devstack , the installation is exiting with error "[ERROR] /home/ubuntu/devstack/functions-common:211 Failure creating NET_ID for private 5d7dc8211ea54d308b4f8e4868a224ee" | 12:48 |
TheJulia | mvpnitesh: any clue in the neutron log? | 12:50 |
*** Goneri has joined #openstack-ironic | 12:50 | |
*** rbudden has joined #openstack-ironic | 12:53 | |
mvpnitesh | TheJulia: no , but when it is executing "openstack --os-cloud devstack-admin --os-region RegionOne network creat$ | 12:53 |
mvpnitesh | --project 5d7dc8211ea54d308b4f8e4868a224ee private" and i'm getting this error "Error while executing command: Service Unavailable (HTTP 503) (Request-ID: req-cb7590c2-c5f1-420f-a680-781e6b256b12)" | 12:53 |
TheJulia | Seems like a neutron issue then | 12:53 |
sambetts | mvpnitesh: I've seen this when you've not got enough resources to allocate a new network, if you re-run the command with --debug flag then you'll see the error more clearly | 12:54 |
TheJulia | I've also seen it whens something was broken in the background, I just don't remember what it was exactly | 12:54 |
TheJulia | but basically, looking at the neutron log gave me insight into what was broken very quickly | 12:55 |
TheJulia | I think it was something like it couldn't access the database or message queue | 12:55 |
mvpnitesh | Sambetts: but actually i'm getting this error , when i'm running ./stack.sh | 12:55 |
*** links has quit IRC | 12:56 | |
mvpnitesh | TheJulia , Sambetts: Thanks , i'll look into it | 12:56 |
jtanner | "go straight to deploy" ... what's the subcommand to "deploy"? | 12:59 |
*** shuyingya has joined #openstack-ironic | 13:00 | |
*** sasha21 has joined #openstack-ironic | 13:01 | |
*** shuyingya has quit IRC | 13:02 | |
*** shuyingya has joined #openstack-ironic | 13:02 | |
TheJulia | jtanner: via the ironic command line, if image_source/image_checksum is defined? | 13:02 |
TheJulia | Hmm... something does feel broken with inspection | 13:03 |
* TheJulia waits for it to timeout while reviewing code | 13:03 | |
jtanner | TheJulia: no image source defined ... suggestions? | 13:06 |
rcarrillocruz | jtanner: sudo scripts/test-bifrost.sh | 13:06 |
rcarrillocruz | that includes creating vms for test | 13:06 |
rcarrillocruz | then deploy | 13:06 |
jtanner | that's how i got to current state | 13:06 |
*** jrist has quit IRC | 13:07 | |
rcarrillocruz | following that i'm currently at "TASK [bifrost-deploy-nodes-dynamic : Deploy to hardware - bifrost default] ***** | 13:07 |
rcarrillocruz | " | 13:07 |
*** mvpnitesh has quit IRC | 13:09 | |
*** jrist has joined #openstack-ironic | 13:10 | |
openstackgerrit | Debayan Ray proposed openstack/sushy master: Add manager https://review.openstack.org/441236 | 13:11 |
*** rloo has joined #openstack-ironic | 13:14 | |
TheJulia | ppiela_: I took a look at 469459 and replied with feedback. I <3 the direction. | 13:15 |
openstackgerrit | Merged openstack/ironic master: Add a note for specifying octal value of permission https://review.openstack.org/468778 | 13:20 |
rloo | hi and good morning TheJulia and everyone else! | 13:22 |
TheJulia | good morning rloo | 13:22 |
rloo | dtantsur: if you're around, wrt 460523, I'm concerned (and maybe i shouldn't be). if someone puts the node in maintenance, they may know that the power state is fine but putting in maintenance for some other reason. changing to None may have some other ramifications for them. should we sent email out to the list first? | 13:24 |
dtantsur | given the amount of feedback we get from the ML each time, I'd prefer not to bother with it | 13:25 |
dtantsur | I mean, a post to the ML is not an approval to do anything, right? | 13:25 |
dtantsur | I'd like to have None there because it's the only way to say "we don't know the power state" | 13:25 |
rloo | dtantsur: it isn't an approval, it is an 'we're planning on doing this unless you have objections' | 13:25 |
rloo | dtantsur: but we do know the power state at the time it is set to maintenance. | 13:25 |
dtantsur | rloo: yes, and it may become invalid 1 second after that | 13:26 |
rloo | dtantsur: we don't know the power state at the time of the power sync | 13:26 |
dtantsur | sorry, I don't get it. I'm just saying that displaying any power state for nodes in maintenance mode may be wrong | 13:26 |
rloo | dtantsur: i'm not sure what the right answer is. i agree this fixes the issue your customer reported. i'm just concerned there may be usecases out there that we are unaware of, and that this breaks them. | 13:27 |
dtantsur | well, if they expect the power state to never be None - sure | 13:27 |
dtantsur | however, we do set the power state to None at random times already | 13:27 |
rloo | dtantsur: wrt 'it may become invalid 1 sec after that'. This holds true for any node/power state, we don't know what the state is 1 sec after we get the power state from the server. that's why we have the sync'ing. | 13:27 |
dtantsur | well, yes, but we may expect it to become true quite soon, unlike for the maintenance mode | 13:28 |
dtantsur | so we at least have eventual consistency | 13:28 |
sambetts | dtantsur: it depends if Ironic is the source of truth for the nodes power state | 13:28 |
rloo | dtantsur: ? i doubt that we set the power state to None at RANDOM times. we have a reason I hope. | 13:28 |
*** jcoufal has joined #openstack-ironic | 13:28 | |
dtantsur | sambetts: not for maintenance mode | 13:28 |
rloo | sambetts: we're talking about https://review.openstack.org/#/c/460523/ | 13:28 |
*** root4 has quit IRC | 13:28 | |
patchbot | patch 460523 - ironic - Set power_state to None on entering maintenance mode | 13:28 |
dtantsur | rloo: under "random" I mean "unpredicatable by an operator". | 13:28 |
sambetts | dtantsur: if ironic is the source of truth and then you clear the power state, then you un-set maintenance ironic won't assert the power state | 13:28 |
*** root4 has joined #openstack-ironic | 13:29 | |
rloo | sambetts: if you unset maintenance, ironic will get the power state when syncing it. | 13:29 |
dtantsur | sambetts: fair. maybe we should set power_state=None at API level when returning the node info | 13:29 |
sambetts | rloo: depends if your sync power state is configure to only reads the power state or set it | 13:30 |
rloo | sambetts: true. i think (i've already forgotten how it works) | 13:30 |
sambetts | rloo: normally my ironic config means that Ironic forces the power state it knows onto the node | 13:30 |
*** baoli_ has quit IRC | 13:31 | |
sambetts | and if you clear the power state then ironic has to re-learn it, instead of forcing it to its known state | 13:31 |
sambetts | so in my world Ironic node power state is never invalid, because it defines the state that Ironic *wants* the node to be in | 13:32 |
dtantsur | .. except for in mainenance mode | 13:33 |
dtantsur | which is the source of the problem in question | 13:33 |
sambetts | no, in maintenace mode it still the state that ironic wants the node to be in | 13:33 |
sambetts | its just we've told ironic to be hands off for a minute | 13:33 |
TheJulia | ironic is just not acting upon it. | 13:33 |
TheJulia | IMHO, this is a human understanding issue, we shouldn't be changing behavior because of a contextual issue. | 13:34 |
*** pcaruana has quit IRC | 13:34 | |
rloo | you're going to hate me, add a config to set to None or leave it? | 13:35 |
TheJulia | Ironic is the source of truth, an operator's would know if they are re-asserting the power state or not, the break in the chain is what the user expects after doing out of band actions | 13:36 |
dtantsur | well, we need a way to signal power_state = 'power on' (but not really, I don't know which state is) | 13:36 |
ppiela_ | TheJulia: Thank you for the feedback, good comments. I will have updates for you to look at shortly. | 13:36 |
dtantsur | rloo: mmm, I'd prefer to not add more config options for essential behaviours | 13:36 |
*** baoli has joined #openstack-ironic | 13:36 | |
TheJulia | rloo: I think hat is reasonable. It allows the operator to choose their behavior, and they can then communicate that to their users | 13:36 |
*** rcernin has quit IRC | 13:36 | |
TheJulia | s/hat/that/ | 13:37 |
rloo | dtantsur: i agree, but i don't know what else. seems like TheJulia and sambetts don't agree with your change as-is. | 13:37 |
*** udesale__ has quit IRC | 13:37 | |
sambetts | dtantsur: you kind of need a curent_power_state and wanted_power_state fields to do that | 13:37 |
dtantsur | we have target_power_state already | 13:38 |
rloo | sambetts: you mean target_power_state? :-) | 13:38 |
sambetts | maybe we should set target_power_state = power_state and power_state= none?? | 13:38 |
sambetts | when force_power_state_during_sync = true | 13:40 |
TheJulia | curious.... how the hell is it getting power_off when in maintenance when it should be in power_on | 13:40 |
sambetts | yeah the bug report is weir | 13:40 |
sambetts | yeah the bug report is weird | 13:40 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 13:40 |
dtantsur | it is, yeah | 13:41 |
sambetts | dtantsur: has the bugzilla got the reason for why cleaning failed? | 13:42 |
* dtantsur checks | 13:42 | |
sambetts | dtantsur: could it have failed because it couldn't set the power state correctly? | 13:42 |
*** jaganathan has quit IRC | 13:42 | |
dtantsur | sambetts: or simply missing cleaning_network configuration | 13:42 |
rloo | just reading the bug report too. their expected results -- "Power state of clean failed node should be reported accurately/set to None to hint the operator to look at this node" | 13:43 |
rloo | the node is in maintenance, that should be enough to hint the operator to look at the node... | 13:43 |
TheJulia | Also, the real issue they are trying to solve is network addressing | 13:44 |
sambetts | +1 node in maintenace is the big red flag here | 13:44 |
TheJulia | in flat networking | 13:44 |
dtantsur | rloo: well, you know that ironic does not update power_state in maintenance, but an operator may not (and they don't, from my experience) | 13:44 |
TheJulia | That sees like a documentation issue to me | 13:44 |
dtantsur | I wish we called it "Disabled" or something more scary | 13:44 |
rloo | dtantsur: right. but if you had nodes that clean-failed, would you try to reuse them again? | 13:44 |
dtantsur | rloo: if it was missing cleaning_network configuration? easily | 13:45 |
rloo | dtantsur: i mean, reuse them again w/o operator looking at it? | 13:45 |
dtantsur | or e.g. a network glitch prevented proper PXE booting | 13:45 |
dtantsur | rloo: no, but that's not about it, right? | 13:45 |
jtanner | TheJulia: did you get past the inspection phase? | 13:45 |
* dtantsur has to disappear for roughly 30 minutes | 13:45 | |
rloo | dtantsur: the 'it' to me is that it is in maintenance, so the operator should look at the node? | 13:45 |
dtantsur | rloo: right | 13:46 |
TheJulia | jtanner: doesn't look like it | 13:46 |
dtantsur | which, in my experience, operators are not always aware of | 13:46 |
rloo | i agree with TheJulia, wrt docs, maybe we need to add more docs about maintenance. | 13:46 |
dtantsur | I had to write a troubleshooting guide teaching people to look at maintenance as source of "no valid host found" errors | 13:46 |
sambetts | +100 I thought we documented it, but yeah maybe we need better docs there | 13:47 |
dtantsur | rloo: I agree. but if people don't realize that maintenance is a red flag, they won't bother reading the docs | 13:47 |
jtanner | TheJulia: what's your stack like? virtualbox + centos w/ nested virt? | 13:47 |
rloo | sambetts: well, eg to add that the power state could be different... | 13:47 |
dtantsur | (which is sad, but is my constant real life experience) | 13:47 |
* dtantsur really disappears, brb | 13:47 | |
TheJulia | jtanner: I'm disabling inspection for now. This would be libvirt on centos7.3. | 13:49 |
*** rcernin has joined #openstack-ironic | 13:49 | |
jtanner | TheJulia: what's your underlying hypervisor .. or is it centos7 on metal | 13:49 |
jtanner | ? | 13:49 |
TheJulia | oh, virtual box hosting centos7 inside with vms running inside of centos7 for guest machines. | 13:50 |
TheJulia | my base OS on this machine is OSX | 13:50 |
*** pcaruana has joined #openstack-ironic | 13:51 | |
rloo | sambetts: btw, i added you to this spec, cuz it tries to address the property info in machine-readable view. let me know if it reflects your thinking. | 13:51 |
rloo | sambetts: https://review.openstack.org/#/c/471174/ | 13:52 |
patchbot | patch 471174 - ironic-specs - API for hardware interface properties | 13:52 |
rloo | sambetts: i'm not in any hurry, just that i said i'd write something up so i did before i forgot. i think we have too much stuff to do in pike so not sure this will get far unless people want it... | 13:52 |
*** cdearborn__ has joined #openstack-ironic | 13:56 | |
TheJulia | jtanner: I got to deploy \o/ Then the previous test vm in my database caused the deploy to fail | 13:58 |
jtanner | hah | 13:59 |
TheJulia | since the state machine does not let you go from 'inspect fail' to 'deploy' | 13:59 |
sambetts | thanks rloo! I'll try to get some cycles to review it | 13:59 |
*** pcaruana has quit IRC | 13:59 | |
jtanner | i cleaned up the vm from virt-manager, vbmc and ironic before rerunning the tests | 13:59 |
TheJulia | jtanner: you also need to remove the port and node from ironic | 14:00 |
*** pcaruana has joined #openstack-ironic | 14:00 | |
jtanner | oh ... | 14:00 |
TheJulia | jtanner: alternatively, since your doing a local test with the test script, you could just drop your entire ironic db. test-bifrost.sh will re-create it for you | 14:00 |
*** cdearborn__ is now known as cdearborn | 14:00 | |
* dtantsur is back | 14:02 | |
*** fellypefca has joined #openstack-ironic | 14:02 | |
*** mat128 has quit IRC | 14:04 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint_type handling https://review.openstack.org/469627 | 14:04 |
*** rcernin has quit IRC | 14:04 | |
*** rcernin has joined #openstack-ironic | 14:05 | |
TheJulia | jtanner: pip install sushy | 14:07 |
*** rcernin has quit IRC | 14:07 | |
TheJulia | somehow we are missing that as a dependency | 14:07 |
*** rcernin has joined #openstack-ironic | 14:07 | |
TheJulia | my power actions were failing because of it | 14:07 |
*** causten_ has joined #openstack-ironic | 14:09 | |
*** jaypipes has quit IRC | 14:10 | |
*** tuanla_fujitsu has joined #openstack-ironic | 14:11 | |
*** links has joined #openstack-ironic | 14:12 | |
*** r-daneel has joined #openstack-ironic | 14:14 | |
*** armax has joined #openstack-ironic | 14:15 | |
* jtanner is waiting on inspection phase again | 14:16 | |
*** mat128 has joined #openstack-ironic | 14:17 | |
TheJulia | jtanner: it will fail | 14:17 |
TheJulia | looks like there is also a permission issue | 14:17 |
*** tiendc has quit IRC | 14:17 | |
jtanner | on the console log? | 14:17 |
*** links has quit IRC | 14:17 | |
TheJulia | https://usercontent.irccloud-cdn.com/file/XQsO4DNm/ohjoy.png | 14:18 |
*** rbartal has quit IRC | 14:19 | |
jtanner | yeah, i read in a bz that was related to a previous power failure | 14:19 |
jtanner | so i chmod -R 777 /var/log/libvirt | 14:19 |
TheJulia | okay, looks like my file was also owned by root, nuked it, told the instance to rebuild again | 14:20 |
* TheJulia tries that | 14:20 | |
jtanner | Inspection of node 4e41df61-84b1-5856-bfb6-6b5f2cd3dd11 failed, last error: ironic-inspector inspection failed: Failed to power on the node, check it's power management configuration: %s | 14:21 |
jtanner | so sushy didn't seem to affect that | 14:21 |
TheJulia | I had to restart vbmc | 14:22 |
*** mtanino has joined #openstack-ironic | 14:23 | |
TheJulia | ugh... now some sort of ipxe issue | 14:24 |
* TheJulia pulls down tcpdump to troubleshoot networking | 14:26 | |
jtanner | fun times | 14:26 |
TheJulia | whatever binary the vm is firing up is not trying to actually network boot | 14:27 |
*** tellesnobrega has quit IRC | 14:31 | |
openstackgerrit | Xavier proposed openstack/ironic master: [WIP] Migrate oneview driver to use python-hpOneView https://review.openstack.org/466872 | 14:31 |
*** mrtenio has joined #openstack-ironic | 14:34 | |
jtanner | TheJulia: i'm gonna spin up a vm from iso and see if network is even functional in the nested env | 14:36 |
*** crushil has joined #openstack-ironic | 14:36 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs master: Collect Bios, Memory and Processor Info from Dmidecode - follow-up https://review.openstack.org/464259 | 14:38 |
TheJulia | likely a good idea, looks like my sushy thing was a complete red herring based on unrelated errors :( | 14:40 |
*** baoli has quit IRC | 14:41 | |
dtantsur | sambetts: what was the reason behind such complex python package installation for tinyipa: first create wheels, then install them? why not install things right away? the current approach is nearly impossible to do right.. | 14:43 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 14:46 |
NobodyCam | Good Morning Ironic'ers | 14:47 |
dtantsur | morning NobodyCam | 14:48 |
NobodyCam | :) | 14:48 |
*** Nisha_ has joined #openstack-ironic | 14:49 | |
jtanner | TheJulia: nope, not getting any dhcp response ... i'll keep digging after lunch | 14:51 |
TheJulia | are you seeing the request? | 14:52 |
TheJulia | because I wasn't even seeing packets being sent by ipxe | 14:52 |
jtanner | haven't looked yet ... need to get out of the house for a bit | 14:52 |
TheJulia | understood, I feel like doing the same honestly | 14:52 |
NobodyCam | :) | 14:53 |
*** rcernin has quit IRC | 14:54 | |
*** tuanla_fujitsu has quit IRC | 14:55 | |
TheJulia | jtanner: Yeah, 0 rx packets on vnet0, 0 tx/rx on virbr0. Sniffing vnet0 I see stp being broadcast to what should be the VM, but nothing back :( | 14:56 |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add ldlinux.c32 to boot ISO for virtual media https://review.openstack.org/470883 | 14:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [refactoring] simplify signature of _finish function https://review.openstack.org/471393 | 14:56 |
ricardoas | morning, NobodyCam! | 14:56 |
NobodyCam | hey hey ricardoas Good morning ... how are you doing today :) | 14:57 |
*** armax has quit IRC | 14:58 | |
*** armax has joined #openstack-ironic | 15:00 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Physical network aware VIF attachment https://review.openstack.org/470406 | 15:01 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: [WIP] Add tempest tests for physical networks https://review.openstack.org/470915 | 15:01 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Move port object creation to conductor https://review.openstack.org/469931 | 15:01 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate portgroup physical network consistency https://review.openstack.org/469932 | 15:01 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Expose ports' physical network attribute in API https://review.openstack.org/469933 | 15:01 |
*** mjura has quit IRC | 15:02 | |
*** rpioso has joined #openstack-ironic | 15:09 | |
dtantsur | folks, this is the last call to review https://review.openstack.org/#/c/439907/ I plan on approving it by EOD (EU time) | 15:11 |
patchbot | patch 439907 - ironic-specs - OSC commands for ironic driver-related commands | 15:11 |
rloo | dtantsur: I approve :-) | 15:12 |
rloo | dtantsur: any more thoughts about maintenance/power-state? | 15:12 |
dtantsur | I don't know.. I don't like any of the option, including my patch :) | 15:12 |
rloo | dtantsur: ok, i'm happy that you aren't happy too :D | 15:13 |
dtantsur | rloo: the rolling upgrade spec can be simpler, I guess, if we start doing all db writes through conductor | 15:15 |
rloo | dtantsur: ++ | 15:15 |
dtantsur | maybe we should do it? actually, only chassis and portgroups are left | 15:15 |
dtantsur | then we should only care about read-only access | 15:15 |
rloo | dtantsur: i was wondering about making that change. | 15:15 |
rloo | dtantsur: i would need to think about what read-access means. for whatever reason, i think that if we have read access, we may have a similar problem, but i don't think i went down that route. | 15:16 |
dtantsur | yeah, we may | 15:16 |
rloo | dtantsur: i am looking at https://review.openstack.org/#/c/469931/ wrt rolling upgrades. after that, i'll think about read-only access... | 15:17 |
patchbot | patch 469931 - ironic - Move port object creation to conductor | 15:17 |
dtantsur | rloo: right, I saw your comment, and then got to your spec update | 15:17 |
rloo | dtantsur: and my spec update simplified (to me) what had been proposed before :-) | 15:17 |
dtantsur | rloo: I'm reading "New DB model change policy", and it seems to me that read-only access is fine | 15:18 |
*** korekhov has joined #openstack-ironic | 15:18 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: DNM: Experimental change to see if python3 gate passes https://review.openstack.org/471072 | 15:19 |
rloo | dtantsur: well, it is fine. i mean, i just need to make sure that we save objs in versions that old services can still understand... | 15:19 |
dtantsur | right.. I guess I need to check the nova's spec | 15:19 |
*** tellesnobrega has joined #openstack-ironic | 15:20 | |
*** penick has joined #openstack-ironic | 15:22 | |
dtantsur | rloo: to be honest, I don't understand why "ironic's solution is different". it looks the same to me at first glance.. | 15:22 |
*** korekhov has quit IRC | 15:23 | |
rloo | dtantsur: cuz for nova, all db requests (read and write) go via nova-conductor | 15:24 |
dtantsur | yeah, but what does it change wrt upgrades? | 15:24 |
rloo | dtantsur: and based on what i read, for an upgrade, they take down all conductors, upgrade, start up conductors. *then* they do rolling of other services. | 15:24 |
rloo | dtantsur: so if you update the db with a newer version of an object, the other conductors will understand it. | 15:25 |
rloo | dtantsur: so no need to worry about old services. | 15:25 |
dtantsur | rloo: well, they still have to worry about e.g. API not understanding the new version yet. and they also use version pinning, etc | 15:25 |
dtantsur | what I wonder if we can simplify anything by doing writes on conductors. I'm not sure yet.. | 15:26 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint_type handling https://review.openstack.org/469627 | 15:26 |
rloo | dtantsur: AND, if a compute (older) doesn't understand new version of an object, when it gets a new object, it just sends it back to conductor saying i understand older version X of object, give it back to me in target version | 15:26 |
rloo | dtantsur: so basically, conductor handles all conversions of object versions | 15:26 |
dtantsur | I think I understand, thanks | 15:27 |
*** flaviodsr_ has quit IRC | 15:28 | |
rloo | dtantsur: i cannot recall now, if nova uses version pinning for objects; i think they use the object VERSION they are aware of. the version pinning might be rpc pinning (i didn't go into detail wrt nova and it is starting to fade now, heh) | 15:28 |
* dtantsur head starts to boil | 15:29 | |
rloo | dtantsur: and wrt nova pinning, they have some way to detect internally what the pinnings are, so operator doesn't need to explicitly set config. i suspect it has to do with the services knowing which versions they support and then ... ok, forget i said anything :-) | 15:29 |
*** chlong has joined #openstack-ironic | 15:29 | |
dtantsur | rloo: "In Nova, we (currently) handle this by allowing versions for each service to be pinned in the config file. | 15:29 |
dtantsur | (this is from the blog post linked in the spec) | 15:30 |
rloo | dtantsur: right. let me find where it says that it can autodetect or whatever... | 15:30 |
rloo | dtantsur: https://docs.openstack.org/developer/nova/upgrade.html | 15:31 |
rloo | dtantsur: se #2 in Rolling upgrade process. The config is set to 'auto'. | 15:31 |
* TheJulia hands dtantsur a cool beer to hopefully reduce brain boiling :) | 15:32 | |
dtantsur | cool beer! | 15:32 |
*** e0ne has quit IRC | 15:32 | |
dtantsur | rloo: aha, I see. we should probably link to this spec, the blog post is outdated | 15:32 |
rloo | dtantsur: i can do. thought it was already linked but guess not. | 15:33 |
dtantsur | hah, they have a cheat: all services are stored in the database | 15:33 |
rloo | dtantsur: it is linked in the spec, #2. maybe i need to reference it somewhere else in that spec. | 15:34 |
dtantsur | I wonder if we can use some remote synchronization stuff (etcd?) for services to be aware of the upgrade process | 15:34 |
rloo | dtantsur: yeah, that too. i think at some point, i stopped trying to figure out the details in nova | 15:34 |
rloo | dtantsur: what i really want to know, is if there is some 'standard' way to do this with openstack services. instead of reinventing the wheel. | 15:35 |
dtantsur | standard = what nova does ;) | 15:35 |
rloo | dtantsur: given that the projects are different i realize they can't all implement the same way but... | 15:35 |
rloo | dtantsur: so i don't know the history behind our decisions. i recall deva wanting api to access (at least for reads) DB w/o going via conductor. | 15:37 |
rloo | dtantsur: probably to reduce load on conductor, api doesn't do much. | 15:37 |
dtantsur | right, this is because of speed | 15:37 |
dtantsur | I do agree with reads, less with writes | 15:37 |
rloo | dtantsur: i agree wrt the writes. | 15:38 |
*** shuyingya has quit IRC | 15:39 | |
*** amarao has quit IRC | 15:41 | |
*** tellesnobrega has quit IRC | 15:46 | |
*** jaosorior is now known as jaosorior_away | 15:47 | |
*** hamzy has quit IRC | 15:52 | |
*** crushil has quit IRC | 15:54 | |
*** penick has quit IRC | 15:56 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint_type handling https://review.openstack.org/469627 | 15:57 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 15:57 |
*** lucasagomes is now known as lucas-afk | 15:58 | |
*** penick has joined #openstack-ironic | 16:00 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-inspector master: add disabled option to VALID_ADD_PORTS_VALUES https://review.openstack.org/471156 | 16:17 |
anupn | Good morning Ironic'ers | 16:17 |
NobodyCam | good morning anupn | 16:18 |
anupn | morning NobodyCam :) | 16:18 |
TheJulia | good morning anupn | 16:18 |
rloo | good morning anupn, NobodyCam! | 16:19 |
anupn | morning TheJulia, rloo :) | 16:19 |
*** trown is now known as trown|lunch | 16:20 | |
anupn | TheJulia: did you got my mailing address i had sent for ironic stickers? | 16:20 |
TheJulia | Oh! you sent it to my ibm address | 16:20 |
anupn | yeah | 16:21 |
TheJulia | Sorry, it got lost in the shuffle/travel | 16:21 |
TheJulia | I'll try to send some out tomorrow for you | 16:21 |
anupn | ah no problem, sure | 16:22 |
anupn | thanks | 16:22 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance https://review.openstack.org/467728 | 16:22 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint_type handling https://review.openstack.org/469627 | 16:22 |
anupn | it would be delivered fast | 16:22 |
*** rama_y has joined #openstack-ironic | 16:24 | |
*** jaganathan has joined #openstack-ironic | 16:27 | |
*** clenimar_ has joined #openstack-ironic | 16:28 | |
*** clenimar_ has quit IRC | 16:28 | |
*** penick has quit IRC | 16:28 | |
*** baoli has joined #openstack-ironic | 16:29 | |
*** tesseract has quit IRC | 16:32 | |
*** flaviodsr has joined #openstack-ironic | 16:32 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-inspector master: add disabled option to VALID_ADD_PORTS_VALUES https://review.openstack.org/471156 | 16:34 |
*** pcaruana has quit IRC | 16:34 | |
*** armax has quit IRC | 16:35 | |
TheJulia | jtanner: heh, setuptools on fc25 too while taking what should be the happiest path | 16:37 |
*** armax has joined #openstack-ironic | 16:38 | |
NobodyCam | Good Morning rloo :) | 16:41 |
rloo | NobodyCam: :) | 16:41 |
NobodyCam | :) | 16:41 |
*** amarao has joined #openstack-ironic | 16:48 | |
joanna | morning NobodyCam, TheJulia, anupn, rloo :) | 16:49 |
NobodyCam | Good Morning joanna :) | 16:50 |
rloo | hi joanna! | 16:50 |
TheJulia | greetings joanna | 16:50 |
anupn | Hi joanna! | 16:50 |
openstackgerrit | Merged openstack/ironic master: Add physical network to port data model https://review.openstack.org/461301 | 16:51 |
joanna | well that was enthusiastic :) | 16:51 |
NobodyCam | :) | 16:52 |
*** derekh has quit IRC | 16:54 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add docs for node.resource_class and flavor creation https://review.openstack.org/345081 | 16:54 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 16:56 |
*** mrtenio has quit IRC | 16:57 | |
*** baoli has quit IRC | 16:58 | |
*** baoli has joined #openstack-ironic | 16:59 | |
*** mgoddard_ has quit IRC | 17:03 | |
*** zackf has joined #openstack-ironic | 17:06 | |
dtantsur | morning NobodyCam, anupn, joanna | 17:07 |
dtantsur | and good night to everyone :) | 17:07 |
joanna | night, dtantsur | 17:07 |
NobodyCam | lol have a good night dtantsur | 17:07 |
*** dtantsur is now known as dtantsur|afk | 17:07 | |
anupn | dtantsur|afk, good night | 17:08 |
rloo | good night dtantsur|afk! | 17:08 |
*** korekhov has joined #openstack-ironic | 17:09 | |
*** jlvillal is now known as jlvacation | 17:10 | |
*** penick has joined #openstack-ironic | 17:10 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance https://review.openstack.org/467728 | 17:11 |
TheJulia | wow, bifrost on fc25 is... ugh | 17:15 |
* sambetts -> home | 17:16 | |
sambetts | night all | 17:16 |
*** sambetts is now known as sambetts|afk | 17:16 | |
jtanner | TheJulia: hah ... did you get stuck at starting mysql? | 17:17 |
TheJulia | not yet | 17:18 |
jtanner | that's where i decided to switch to centos | 17:18 |
jtanner | i had to comment out a few of the default packages being installed | 17:18 |
jtanner | i think virt related | 17:18 |
*** hamzy has joined #openstack-ironic | 17:19 | |
TheJulia | yeah, looks like one of the package names changed, and others are outdated | 17:20 |
TheJulia | a few other things seemingly missing pip dependency wise | 17:20 |
*** baoli has quit IRC | 17:21 | |
*** baoli has joined #openstack-ironic | 17:22 | |
TheJulia | jtanner: looks like mysql fired right up | 17:26 |
jtanner | argh, you cheated | 17:26 |
*** penick has quit IRC | 17:28 | |
*** korekhov has quit IRC | 17:34 | |
TheJulia | jtanner: nope :) | 17:36 |
TheJulia | jtanner: on a plus side, looks like networking is good on fc25 | 17:36 |
TheJulia | I'm past inspection | 17:36 |
jtanner | nice | 17:36 |
TheJulia | oh wait | 17:36 |
TheJulia | it is running introspection now | 17:36 |
*** milan has quit IRC | 17:37 | |
*** zackf has quit IRC | 17:40 | |
TheJulia | jtanner: introspection and deployment completed | 17:42 |
*** mgoddard_ has joined #openstack-ironic | 17:44 | |
*** tellesnobrega has joined #openstack-ironic | 17:47 | |
*** ricardoas1 has joined #openstack-ironic | 17:54 | |
*** ricardoas has quit IRC | 17:56 | |
*** baoli has quit IRC | 17:58 | |
*** crushil has joined #openstack-ironic | 17:58 | |
*** fellypefca has left #openstack-ironic | 18:02 | |
*** trown|lunch is now known as trown | 18:02 | |
*** Nisha_ has quit IRC | 18:02 | |
*** fragatina has quit IRC | 18:08 | |
*** baoli has joined #openstack-ironic | 18:08 | |
*** penick has joined #openstack-ironic | 18:13 | |
*** baoli has quit IRC | 18:23 | |
*** baoli has joined #openstack-ironic | 18:24 | |
*** nicodemos has quit IRC | 18:25 | |
*** xavierr has quit IRC | 18:28 | |
*** ricardoas1 has quit IRC | 18:28 | |
*** clenimar has quit IRC | 18:29 | |
*** zackf has joined #openstack-ironic | 18:34 | |
*** zackf has quit IRC | 18:38 | |
*** mgoddard_ has quit IRC | 18:39 | |
NobodyCam | TheJulia: :) | 18:39 |
*** causten_ has quit IRC | 18:39 | |
*** causten has joined #openstack-ironic | 18:40 | |
*** causten has quit IRC | 18:44 | |
*** korekhov has joined #openstack-ironic | 18:45 | |
*** causten has joined #openstack-ironic | 18:50 | |
jtanner | TheJulia: ok, cycling back to the same question | 18:50 |
jtanner | what's your stack? | 18:50 |
jtanner | f25 on top of what? | 18:50 |
*** tenobreg has joined #openstack-ironic | 18:50 | |
*** tellesnobrega has quit IRC | 18:52 | |
TheJulia | jtanner: fc25 running in virtual box, fired up via vagrant | 18:53 |
jtanner | what mode is the underlying nic in ? | 18:54 |
jtanner | nat/hostonly/etc | 18:54 |
TheJulia | nat | 18:54 |
jtanner | kthnx | 18:54 |
*** yee37921 has joined #openstack-ironic | 18:56 | |
*** pcaruana has joined #openstack-ironic | 18:56 | |
*** yee379 has quit IRC | 19:00 | |
*** jkilpatr has quit IRC | 19:03 | |
*** cdearborn_ has joined #openstack-ironic | 19:15 | |
*** trown is now known as trown|outtypewww | 19:22 | |
openstackgerrit | Merged openstack/ironic master: Remove legacy auth loading https://review.openstack.org/469626 | 19:26 |
*** mgoddard_ has joined #openstack-ironic | 19:28 | |
*** cdearborn has quit IRC | 19:29 | |
*** harlowja has quit IRC | 19:29 | |
*** jaganathan has quit IRC | 19:32 | |
*** pcaruana has quit IRC | 19:33 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 19:34 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 19:35 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 19:37 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 19:38 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 19:38 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 19:38 |
*** korekhov has left #openstack-ironic | 19:41 | |
openstackgerrit | Michael Turek proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 19:43 |
*** Goneri has quit IRC | 19:50 | |
*** harlowja has joined #openstack-ironic | 19:50 | |
anupn | Hi TheJulia: so even while enrolling the nodes would you like to see default interfaces or enabled interfaces? | 19:50 |
anupn | in Ironic-ui | 19:51 |
*** causten has quit IRC | 20:00 | |
*** causten has joined #openstack-ironic | 20:01 | |
TheJulia | If a user can select upon enrolling, that is awesome, but that is also in the advanced use territory. It would be better to allow them to edit it after the fact, and that simplifies the amount of things they see when enrolling | 20:04 |
TheJulia | I could see a case for showing defaults, but I doubt most users are going to enroll and not change it if they have a very specific need | 20:05 |
*** causten has quit IRC | 20:05 | |
*** armax has quit IRC | 20:07 | |
*** priteau has quit IRC | 20:07 | |
*** mgoddard_ has quit IRC | 20:07 | |
*** korekhov has joined #openstack-ironic | 20:11 | |
*** fragatina has joined #openstack-ironic | 20:15 | |
*** korekhov has quit IRC | 20:16 | |
*** korekhov has joined #openstack-ironic | 20:17 | |
*** fragatina has quit IRC | 20:20 | |
*** fragatina has joined #openstack-ironic | 20:22 | |
*** causten has joined #openstack-ironic | 20:22 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: Rolling upgrades: different object versions https://review.openstack.org/469940 | 20:22 |
*** dtantsur|afk has quit IRC | 20:23 | |
*** dtantsur has joined #openstack-ironic | 20:26 | |
*** racedo has joined #openstack-ironic | 20:35 | |
*** jkilpatr has joined #openstack-ironic | 20:38 | |
anupn | TheJulia: hmm, i can show defaults then for now and can discuss on the patch once uploaded, i was just concerned so likewise i will write a method to fetch default or enabled interfaces | 20:39 |
*** causten has quit IRC | 20:41 | |
*** causten has joined #openstack-ironic | 20:42 | |
jtanner | TheJulia: progress on centos7 host | 20:43 |
TheJulia | networking progress? | 20:44 |
jtanner | eth0 is 192.168.122.0/24, virbr0 is 192.168.124.0/24 ... dnsmasq was configured to serve 192.168.122.2-254 | 20:44 |
jtanner | meaning dnsmasq was trying to serve on the wrong nic | 20:44 |
jtanner | nic / device | 20:44 |
jtanner | i bumped the third octect to 124 in the dnsmasq.conf dhcp range, restarted the service and then my testvm was able to get a lease from it | 20:45 |
TheJulia | but are you seeing packets crossing the virtual wire? | 20:45 |
TheJulia | wow, awesome | 20:45 |
jtanner | yeah, i see packets coming in | 20:45 |
TheJulia | so it was just my vm it seems | 20:45 |
jtanner | but dnsmasq was logging that it didn't have a range available | 20:46 |
jtanner | based on past experience iwth it, i know it uses the configured range to determine the nic to listen on | 20:46 |
jtanner | so i mapped it all out and noticed the third octets | 20:46 |
TheJulia | Yeah, and that makes sense. It was just fun that I wasn't even seeing packets come off the vm. :( | 20:47 |
*** crushil has quit IRC | 20:47 | |
TheJulia | I'm pretty much a the end of my day, but I'll post the fixes for fedora in the morning | 20:48 |
TheJulia | its not a horrible list, it shouldn't take very long for me to put together | 20:48 |
*** armax has joined #openstack-ironic | 20:48 | |
*** jcoufal has quit IRC | 20:51 | |
jtanner | thanks! have a good one | 20:59 |
*** korekhov has quit IRC | 21:03 | |
*** korekhov has joined #openstack-ironic | 21:06 | |
*** korekhov has quit IRC | 21:10 | |
*** fragatina has quit IRC | 21:15 | |
*** athomas has quit IRC | 21:25 | |
*** mat128 has quit IRC | 21:26 | |
*** korekhov has joined #openstack-ironic | 21:33 | |
*** baoli has quit IRC | 21:41 | |
*** racedo has quit IRC | 21:57 | |
*** penick has quit IRC | 22:00 | |
*** racedo has joined #openstack-ironic | 22:21 | |
*** racedo has quit IRC | 22:22 | |
*** causten has quit IRC | 22:24 | |
*** mat128 has joined #openstack-ironic | 22:29 | |
*** causten has joined #openstack-ironic | 22:29 | |
*** causten has quit IRC | 22:32 | |
*** causten has joined #openstack-ironic | 22:35 | |
*** zackf has joined #openstack-ironic | 22:42 | |
*** zackf has quit IRC | 22:46 | |
*** rpioso has quit IRC | 22:51 | |
*** bfournie has quit IRC | 23:01 | |
*** rbudden has quit IRC | 23:01 | |
*** pmannidi has joined #openstack-ironic | 23:04 | |
*** causten has quit IRC | 23:08 | |
*** dsneddon has quit IRC | 23:10 | |
*** sasha21 has quit IRC | 23:27 | |
*** korekhov has quit IRC | 23:40 | |
*** jkilpatr has quit IRC | 23:42 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 23:45 |
*** jkilpatr has joined #openstack-ironic | 23:47 | |
*** jkilpatr has quit IRC | 23:50 | |
*** jkilpatr has joined #openstack-ironic | 23:53 | |
*** rama_y has quit IRC | 23:54 | |
*** jkilpatr has quit IRC | 23:57 | |
*** MattMan has quit IRC | 23:58 | |
*** MattMan has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!