*** crushil has quit IRC | 00:03 | |
*** gcb has quit IRC | 00:18 | |
*** oomichi has quit IRC | 00:19 | |
*** oomichi has joined #openstack-ironic | 00:23 | |
*** mtanino_ has joined #openstack-ironic | 00:26 | |
*** mtanino has quit IRC | 00:27 | |
*** fragatin_ has quit IRC | 00:32 | |
*** fragatina has joined #openstack-ironic | 00:32 | |
*** sayantan_ has quit IRC | 00:42 | |
openstackgerrit | Julian Edwards proposed openstack/ironic master: Update install guide to explain Nova integration https://review.openstack.org/450492 | 00:51 |
---|---|---|
*** tovin07 has joined #openstack-ironic | 00:52 | |
*** jjung has quit IRC | 00:59 | |
*** amotoki has joined #openstack-ironic | 01:08 | |
*** amotoki has quit IRC | 01:14 | |
*** tuanluong has joined #openstack-ironic | 01:20 | |
*** sasha2 has joined #openstack-ironic | 01:22 | |
*** MarginHu has joined #openstack-ironic | 01:27 | |
*** tovin07 has quit IRC | 01:28 | |
*** tovin07 has joined #openstack-ironic | 01:29 | |
*** SerenaFeng has joined #openstack-ironic | 01:31 | |
*** sasha2 has quit IRC | 01:34 | |
*** SerenaFeng has quit IRC | 01:36 | |
*** SerenaFeng has joined #openstack-ironic | 01:40 | |
*** phuongnh has joined #openstack-ironic | 01:46 | |
*** SerenaFe_ has joined #openstack-ironic | 01:47 | |
*** SerenaFeng has quit IRC | 01:49 | |
*** causten has joined #openstack-ironic | 01:50 | |
*** causten has quit IRC | 01:50 | |
*** causten has joined #openstack-ironic | 01:50 | |
*** zhurong has joined #openstack-ironic | 01:53 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-2 https://review.openstack.org/448221 | 01:53 |
openstackgerrit | klyang proposed openstack/ironic master: Fix some api field lists in docstrings https://review.openstack.org/450503 | 01:59 |
*** rloo has quit IRC | 02:07 | |
*** gcb has joined #openstack-ironic | 02:07 | |
*** crushil has joined #openstack-ironic | 02:07 | |
*** mtanino_ has quit IRC | 02:12 | |
*** oomichi has quit IRC | 02:28 | |
*** oomichi has joined #openstack-ironic | 02:32 | |
*** jaganathan has joined #openstack-ironic | 02:33 | |
*** oomichi has quit IRC | 02:39 | |
*** oomichi has joined #openstack-ironic | 02:42 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Make terminal timeout value configurable https://review.openstack.org/449134 | 02:43 |
*** fragatina has quit IRC | 02:44 | |
*** litao has joined #openstack-ironic | 02:52 | |
*** MarginHu has quit IRC | 02:53 | |
*** UForgotten_ has quit IRC | 03:01 | |
*** UForgotten_ has joined #openstack-ironic | 03:01 | |
*** UForgotten_ is now known as UForgotten | 03:01 | |
*** dimak has quit IRC | 03:20 | |
*** dimak has joined #openstack-ironic | 03:21 | |
*** g3ek has quit IRC | 03:27 | |
*** SerenaFe_ has quit IRC | 03:30 | |
*** links has joined #openstack-ironic | 03:36 | |
*** g3ek has joined #openstack-ironic | 03:36 | |
*** crushil has quit IRC | 03:39 | |
*** jkilpatr has quit IRC | 03:50 | |
*** joanna has joined #openstack-ironic | 03:51 | |
*** aparnav has joined #openstack-ironic | 03:52 | |
*** tovin07 has quit IRC | 03:53 | |
*** zhurong has quit IRC | 03:57 | |
*** sasha2 has joined #openstack-ironic | 04:01 | |
*** aparna has joined #openstack-ironic | 04:05 | |
*** nmathew has joined #openstack-ironic | 04:06 | |
*** aparnav has quit IRC | 04:08 | |
*** vmud213 has joined #openstack-ironic | 04:12 | |
*** fragatina has joined #openstack-ironic | 04:13 | |
*** aparna has quit IRC | 04:17 | |
*** fragatina has quit IRC | 04:17 | |
*** jaganathan has quit IRC | 04:23 | |
*** SerenaFeng has joined #openstack-ironic | 04:32 | |
*** jaganathan has joined #openstack-ironic | 04:35 | |
*** fragatina has joined #openstack-ironic | 04:40 | |
*** fragatina has quit IRC | 04:42 | |
*** fragatina has joined #openstack-ironic | 04:42 | |
*** SerenaFeng has quit IRC | 04:55 | |
*** zhurong has joined #openstack-ironic | 04:55 | |
*** aparnav has joined #openstack-ironic | 05:03 | |
*** makowals_ has joined #openstack-ironic | 05:04 | |
*** aNuposic has quit IRC | 05:05 | |
*** sasha2 has quit IRC | 05:07 | |
*** makowals has quit IRC | 05:07 | |
*** makowals has joined #openstack-ironic | 05:09 | |
*** makowals_ has quit IRC | 05:11 | |
*** aparnav has quit IRC | 05:17 | |
*** aparnav has joined #openstack-ironic | 05:20 | |
*** vmud213 has quit IRC | 05:27 | |
*** vmud213 has joined #openstack-ironic | 05:28 | |
openstackgerrit | klyang proposed openstack/ironic master: Add Error Codes to ironic api references https://review.openstack.org/449495 | 05:32 |
*** SerenaFeng has joined #openstack-ironic | 05:34 | |
*** vishwanathj has quit IRC | 05:35 | |
*** rcernin has joined #openstack-ironic | 05:38 | |
*** vikrant has joined #openstack-ironic | 05:39 | |
*** garyk has joined #openstack-ironic | 05:54 | |
*** ChubYann has quit IRC | 05:54 | |
*** aparnav has quit IRC | 06:09 | |
*** rbartal has joined #openstack-ironic | 06:20 | |
*** mjura has joined #openstack-ironic | 06:20 | |
*** yogi has joined #openstack-ironic | 06:21 | |
*** niks has joined #openstack-ironic | 06:27 | |
niks | Failed to set boot device to PXE: Driver fake_pxe does not support management (disabled or not implemented). | 06:31 |
niks | any idea | 06:31 |
vsaienk0 | vsaienk0 | 06:39 |
vsaienk0 | Morning Ironic'ers! | 06:40 |
tuanluong | Morning Ironic'ers!, vsaienk0 niks | 06:44 |
openstackgerrit | hnyang proposed openstack/ironic master: Add Error Codes to ironic api references https://review.openstack.org/449495 | 06:44 |
niks | Morning | 06:44 |
niks | any idea on : Failed to set boot device to PXE: Driver fake_pxe does not support management (disabled or not implemented). | 06:44 |
niks | i am trying to configure rdo10 using director with vms | 06:45 |
niks | and using fake_pxe | 06:45 |
niks | as the driver | 06:45 |
*** SerenaFeng has quit IRC | 06:47 | |
niks | http://paste.openstack.org/show/604423/ | 06:47 |
niks | tuanluong: any idea | 06:47 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 06:48 |
*** oomichi has quit IRC | 06:48 | |
*** sambetts|afk has quit IRC | 06:48 | |
tuanluong | niks, do you deloy baremental on hardware or VM | 06:49 |
*** oomichi has joined #openstack-ironic | 06:52 | |
*** tovin07 has joined #openstack-ironic | 06:52 | |
niks | tuanluong: i created 3 VMs on a Virtual Box(Virtual Box is installed on a RHEL server), out of these 3 VMs, i am using one VM for director and other two VMs for controller and compute | 06:52 |
*** sambetts_ has joined #openstack-ironic | 06:52 | |
niks | tuanluong: it is just for POC | 06:52 |
niks | i do not have enough hardware now | 06:53 |
tuanluong | niks, I suggest you should try https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 06:54 |
tuanluong | niks, you should try agent_ipmitool instead of fake_pxe because it often use for testing | 06:56 |
*** mtreinish has quit IRC | 06:57 | |
*** mtreinish has joined #openstack-ironic | 06:58 | |
*** hshiina has joined #openstack-ironic | 06:58 | |
*** amotoki has joined #openstack-ironic | 07:13 | |
*** pcaruana has joined #openstack-ironic | 07:13 | |
*** tesseract has joined #openstack-ironic | 07:18 | |
*** amotoki has quit IRC | 07:18 | |
*** aarefiev_afk is now known as aarefiev | 07:24 | |
aarefiev | morning all! | 07:24 |
*** jaosorior has joined #openstack-ironic | 07:29 | |
*** rcernin has quit IRC | 07:29 | |
*** SerenaFeng has joined #openstack-ironic | 07:31 | |
*** jaganathan is now known as jaganathan_lunch | 07:34 | |
*** makowals has quit IRC | 07:37 | |
*** jaganathan_lunch is now known as jaganathan | 07:44 | |
*** amarao has joined #openstack-ironic | 07:45 | |
*** rcernin has joined #openstack-ironic | 07:47 | |
*** SerenaFeng has quit IRC | 07:49 | |
*** makowals has joined #openstack-ironic | 07:49 | |
*** amoralej|off is now known as amoralej | 07:53 | |
*** racedo has joined #openstack-ironic | 07:55 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
openstackgerrit | Merged openstack/bifrost master: Trivial spacing fix. https://review.openstack.org/450338 | 08:01 |
*** SerenaFeng has joined #openstack-ironic | 08:03 | |
*** rcernin has quit IRC | 08:06 | |
*** rcernin has joined #openstack-ironic | 08:07 | |
*** rcernin has quit IRC | 08:10 | |
*** rcernin has joined #openstack-ironic | 08:10 | |
*** amotoki has joined #openstack-ironic | 08:14 | |
*** ccamacho has joined #openstack-ironic | 08:14 | |
*** ralonsoh has joined #openstack-ironic | 08:16 | |
*** fdegir has joined #openstack-ironic | 08:16 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 08:16 |
*** amotoki has quit IRC | 08:19 | |
*** mgoddard has joined #openstack-ironic | 08:20 | |
*** derekh has joined #openstack-ironic | 08:34 | |
*** e0ne has joined #openstack-ironic | 08:35 | |
*** lucas-afk is now known as lucasagomes | 08:35 | |
lucasagomes | morning all | 08:35 |
fdegir | hi | 08:36 |
fdegir | we recently started seeing issues with ironic-dbsync | 08:36 |
fdegir | here is the log to most recent attempt | 08:37 |
fdegir | https://build.opnfv.org/ci/job/xci-provision-virtual-xenial-daily-ocata/3/console | 08:37 |
fdegir | anyone is aware of any issues? | 08:37 |
*** aparnav has joined #openstack-ironic | 08:40 | |
*** rcernin has quit IRC | 08:46 | |
*** rcernin has joined #openstack-ironic | 08:46 | |
*** rcernin_ has joined #openstack-ironic | 08:48 | |
*** g3ek has quit IRC | 08:48 | |
*** aparnav has quit IRC | 08:49 | |
*** aparnav has joined #openstack-ironic | 08:49 | |
*** g3ek has joined #openstack-ironic | 08:51 | |
*** aparna has joined #openstack-ironic | 08:52 | |
*** aparnav has quit IRC | 08:52 | |
*** tovin07 has quit IRC | 08:55 | |
thomas_oneill | fdegir: yes I'm getting the same thing actually | 08:58 |
thomas_oneill | fdegir: I don't suppose you've tried cleaning up and then re-installing using the install.yaml playbook? | 09:01 |
thomas_oneill | I've been working on a cleanup playbook, which I've ran, so if it was just me I'd suspect that my DB needs wiping/purging properly as part of the cleanup and that I'm not doing it properly | 09:01 |
fdegir | thomas_oneill: did that but didn't work | 09:01 |
thomas_oneill | the fact that you're also getting this same error this morning makes me wonder if there's a bug in some other patch in the tree D: | 09:02 |
fdegir | and it seems it hits both master and ocata | 09:02 |
thomas_oneill | dang | 09:02 |
*** aparna has quit IRC | 09:03 | |
thomas_oneill | fdegir: database completely destroyed and re-installed, and ironic.conf completely removed and re-generated? | 09:12 |
*** amotoki has joined #openstack-ironic | 09:15 | |
fdegir | thomas_oneill: yes, we use bifrost which does ironic stuff | 09:15 |
fdegir | thomas_oneill: and we wipe the database before we run bifrost | 09:16 |
thomas_oneill | yeah I'm using bifrost too... | 09:16 |
*** Serlex has joined #openstack-ironic | 09:16 | |
fdegir | thomas_oneill: this is what we do for cleanup | 09:17 |
fdegir | thomas_oneill: https://gerrit.opnfv.org/gerrit/gitweb?p=releng.git;a=blob;f=prototypes/bifrost/scripts/destroy-env.sh;hb=HEAD#l35 | 09:17 |
*** jaganathan has quit IRC | 09:18 | |
*** amotoki has quit IRC | 09:19 | |
*** yatinkarel has quit IRC | 09:20 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Fixes iLO drivers third party CI https://review.openstack.org/450670 | 09:20 |
*** milan has joined #openstack-ironic | 09:25 | |
milan | morning Ironic! :) | 09:25 |
thomas_oneill | fdegir: thanks, so you drop the database using the drop command... wonder if the database could be destroyed more destructively | 09:27 |
fdegir | thomas_oneill: I'm onto something | 09:28 |
fdegir | thomas_oneill: will come back once I try this | 09:28 |
thomas_oneill | ok :D | 09:29 |
*** jaganathan has joined #openstack-ironic | 09:30 | |
*** yatinkarel has joined #openstack-ironic | 09:33 | |
*** fragatina has quit IRC | 09:37 | |
fdegir | thomas_oneill: most likely a false hope :/ | 09:38 |
*** niks has quit IRC | 09:42 | |
openstackgerrit | Fatih Degirmenci proposed openstack/bifrost master: Set database pool_size in ironic.conf and inspector.conf templates https://review.openstack.org/450681 | 09:48 |
fdegir | thomas_oneill: ^ | 09:48 |
thomas_oneill | oh nice, ty | 09:48 |
vdrok | good morning ironic and all! | 09:50 |
*** jaganathan has quit IRC | 09:50 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Trivial: don't create unused temporary directory https://review.openstack.org/450683 | 09:53 |
milan | morning vdrok! :) | 09:54 |
vdrok | hey milan :) | 09:54 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Trivial: don't create unused temporary directory https://review.openstack.org/450683 | 09:54 |
fdegir | thomas_oneill: our verification jobs are passing that point where it fails | 09:58 |
lucasagomes | vdrok, milan morning! | 09:59 |
fdegir | thomas_oneill: so the patch will hopefully be merged soon | 09:59 |
lucasagomes | folks, I'm creating a new repository to add the tools to help with the development and test of sushy (the libvirt simulator and the static server): https://review.openstack.org/#/c/450685/ | 10:00 |
openstackgerrit | Fatih Degirmenci proposed openstack/bifrost stable/ocata: Set database pool_size in ironic.conf and inspector.conf templates https://review.openstack.org/450686 | 10:00 |
lucasagomes | jroll, dtantsur ^ | 10:00 |
milan | morning lucasagomes! :) | 10:01 |
* milan runs lunch :P | 10:01 | |
*** hshiina has quit IRC | 10:02 | |
*** yatinkarel has quit IRC | 10:02 | |
*** mkrai has quit IRC | 10:04 | |
*** retr0h has quit IRC | 10:04 | |
*** csmart has quit IRC | 10:04 | |
*** snecklifter has joined #openstack-ironic | 10:05 | |
*** jaganathan has joined #openstack-ironic | 10:08 | |
*** phuongnh has quit IRC | 10:09 | |
*** tuanluong has quit IRC | 10:09 | |
*** athomas has quit IRC | 10:10 | |
*** athomas has joined #openstack-ironic | 10:11 | |
openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 10:11 |
*** tuanluong has joined #openstack-ironic | 10:13 | |
*** MattMan has quit IRC | 10:14 | |
vdrok | morning lucasagomes , oh , cool | 10:14 |
*** MattMan has joined #openstack-ironic | 10:14 | |
*** mjura has quit IRC | 10:15 | |
lucasagomes | yeah, it's just so we can manage things like dependencies separated... If they all live in the same repo we will need to mix the tools dependencies with the library itself in order to get them checked by g-r :-( | 10:16 |
lucasagomes | which sucks | 10:16 |
*** amotoki has joined #openstack-ironic | 10:16 | |
*** athomas has quit IRC | 10:16 | |
*** yatinkarel has joined #openstack-ironic | 10:16 | |
*** SerenaFeng has quit IRC | 10:17 | |
*** SerenaFeng has joined #openstack-ironic | 10:20 | |
*** amotoki has quit IRC | 10:20 | |
*** zhurong has quit IRC | 10:20 | |
*** mkrai has joined #openstack-ironic | 10:21 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Deduplicate method in OSC functional tests https://review.openstack.org/450702 | 10:21 |
openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 10:24 |
vdrok | lucasagomes: btw, I thought it's possible to have a sub project inside some other project, like https://github.com/openstack/fuel-agent and https://github.com/openstack/fuel-agent/tree/master/contrib/fuel_bootstrap/fuel_bootstrap_cli. I'm not sure how this works, and not opposed to a separate project, just pointing out :) | 10:26 |
*** zhurong has joined #openstack-ironic | 10:28 | |
lucasagomes | vdrok, oh lemme check | 10:28 |
lucasagomes | vdrok, that's cool, does the g-r check it's dependencies and all ? | 10:30 |
vdrok | lucasagomes: dunno, it appears that fuel-agent does not have a requirements job | 10:31 |
lucasagomes | and I assume they can be installed separately right ? | 10:31 |
lucasagomes | oh | 10:31 |
lucasagomes | I see | 10:31 |
vdrok | yup, if g-r cannot understand it, better to have a separate project :) also asked yuriyz , it seems to bring troubles with versioning in pbr | 10:35 |
*** aparnav has joined #openstack-ironic | 10:35 | |
*** cdearborn has joined #openstack-ironic | 10:36 | |
openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 10:38 |
*** aparnav has quit IRC | 10:39 | |
*** mgould|afk is now known as mgould | 10:39 | |
mgould | hi everyone! | 10:39 |
*** aparnav has joined #openstack-ironic | 10:39 | |
vdrok | morning mgould | 10:40 |
*** aparnav has quit IRC | 10:40 | |
*** aparna has joined #openstack-ironic | 10:40 | |
mgould | morning vdrok lucasagomes milan and everyone else | 10:41 |
mgould | lucasagomes: nooooooo not more repos... | 10:41 |
*** fragatina has joined #openstack-ironic | 10:41 | |
mgould | https://danluu.com/monorepo/ | 10:41 |
*** athomas has joined #openstack-ironic | 10:41 | |
mgould | OTOH, we're kinda locked into the "zillions of repos" (anti)pattern now | 10:42 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/450711 | 10:42 |
*** appu103 has joined #openstack-ironic | 10:43 | |
*** aparna has quit IRC | 10:43 | |
*** aparna has joined #openstack-ironic | 10:49 | |
*** appu103 has quit IRC | 10:49 | |
lucasagomes | mgould, heh yeah I was trying to avoid it but... How to deal with the dependencies on a single repo ? | 10:53 |
lucasagomes | also installation, for example in devstack what I'm doing now is to clone the sushy repo and then run the simulator | 10:54 |
lucasagomes | but that's a bit dangerous cause it will always run from master | 10:54 |
*** vills has joined #openstack-ironic | 10:55 | |
*** alexpilotti has quit IRC | 10:55 | |
* mgould nods | 10:57 | |
*** jkilpatr has joined #openstack-ironic | 11:00 | |
*** jjung has joined #openstack-ironic | 11:01 | |
*** sambetts_ is now known as sambetts | 11:04 | |
sambetts | Morning all | 11:05 |
TheJulia | Good morning | 11:05 |
mgould | morning sambetts TheJulia | 11:05 |
lucasagomes | TheJulia, sambetts morning o/ | 11:06 |
sambetts | Hey TheJulia, mgould | 11:06 |
*** litao has quit IRC | 11:09 | |
*** oomichi has quit IRC | 11:09 | |
*** athomas has quit IRC | 11:09 | |
openstackgerrit | Jaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector https://review.openstack.org/424729 | 11:10 |
*** oomichi has joined #openstack-ironic | 11:13 | |
*** fragatina has quit IRC | 11:16 | |
openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 11:16 |
*** aparna has quit IRC | 11:17 | |
*** amotoki has joined #openstack-ironic | 11:17 | |
TheJulia | thomas_oneill: I replied to your change in bifrost with some thoughts, and have asked a couple other reviewers to take a look. I think it makes sense fwiw :) | 11:17 |
*** aparna has joined #openstack-ironic | 11:17 | |
*** athomas has joined #openstack-ironic | 11:17 | |
vsaienk0 | sambetts: do you think the sentence should be rephrased? https://review.openstack.org/#/c/415003/12/specs/approved/dynamic-portgroups.rst@17 | 11:20 |
*** amotoki has quit IRC | 11:21 | |
thomas_oneill | TheJulia: yeah I saw, thanksf or the feedbcak | 11:25 |
*** zhurong has quit IRC | 11:25 | |
TheJulia | vsaienk0: By chance are you working bug 16768809 ? | 11:25 |
thomas_oneill | oops can't type but yeah useful stuff, been trying to get on with it this morning, these things are always more complicated than they first seem though... bloody packages and distro's argh lol | 11:25 |
*** aparna has quit IRC | 11:26 | |
*** appu103 has joined #openstack-ironic | 11:26 | |
vsaienk0 | TheJulia: yeah, I'm and vdrok looking onto it https://bugs.launchpad.net/ironic/+bug/1676809 | 11:26 |
openstack | Launchpad bug 1676809 in Ironic "ironic-dbsync failed on CI" [Undecided,Confirmed] | 11:26 |
TheJulia | vsaienk0: okay, I just confirmed it since it is being seen on bifrost's OPNFV ci as well | 11:27 |
vdrok | morning sambetts TheJulia | 11:27 |
TheJulia | marking critical since, while it should be an easy type fix for the default, it seems kind of bad.... | 11:28 |
*** lucasagomes is now known as lucas-hungry | 11:28 | |
*** openstackgerrit has quit IRC | 11:33 | |
*** nmathew has quit IRC | 11:33 | |
*** mat128 has joined #openstack-ironic | 11:35 | |
*** aparna has joined #openstack-ironic | 11:39 | |
*** appu103 has quit IRC | 11:39 | |
*** rbartal_ has joined #openstack-ironic | 11:39 | |
*** racedo has quit IRC | 11:42 | |
*** appu103 has joined #openstack-ironic | 11:42 | |
*** aparna has quit IRC | 11:42 | |
*** openstackgerrit has joined #openstack-ironic | 11:42 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Don't pass sqlite_db in db_options.set_defaults() https://review.openstack.org/450733 | 11:42 |
vsaienk0 | TheJulia: dtantsur vdrok and all other ironic cores patch to unblock ci https://review.openstack.org/450733 | 11:43 |
TheJulia | vsaienk0: Awesome, that looks... fun :) | 11:44 |
garyk | mgould: maybe you can help me for a sec. I am getting "SSH connect failed: Invalid private key" | 11:44 |
*** yogi has quit IRC | 11:45 | |
*** yogi has joined #openstack-ironic | 11:45 | |
TheJulia | I guess CI will tell us for sure, but it looks good to me :) | 11:45 |
garyk | i am trying to use pxe_ssh. maybe using wrong information here | 11:46 |
*** racedo has joined #openstack-ironic | 11:47 | |
*** aparna has joined #openstack-ironic | 11:54 | |
*** appu103 has quit IRC | 11:54 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector master: Don't pass sqlite_db in db_options.set_defaults() https://review.openstack.org/450737 | 11:55 |
*** jcoufal has joined #openstack-ironic | 11:55 | |
*** bfournie has quit IRC | 11:55 | |
milan | aarefiev, hey, wanted to have another philosophical discussion w/ you ;) w/r the multiple filtering backends | 11:56 |
*** jaypipes has joined #openstack-ironic | 11:56 | |
aarefiev | milan: hey | 11:56 |
milan | aarefiev, I started to update my POC last night but ran into the issue that all the filters will have to figure out MAC address (including Infiniband MACs) | 11:57 |
milan | aarefiev, so maybe we should amend the spec w/ [<mac>, <mac>, ...] | 11:57 |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add names to portgroup api tempest tests https://review.openstack.org/450739 | 11:58 |
milan | aarefiev, also we need to track the mapping of node-id -- mac as once the Ironic "drops" the node, we won't know (or the filters ) what macs to remove | 11:58 |
milan | aarefiev, wdyt? | 11:58 |
aarefiev | milan: yeah, sounds good to me | 11:59 |
milan | aarefiev, also a tricky thing w/ IB | 11:59 |
milan | we need to accumulate the MAC-IB mapping from all the nodes in HA env | 11:59 |
milan | into that table | 11:59 |
milan | *table == node_id, mac mapping | 11:59 |
aarefiev | milan: why you don't get macs from node_info object | 12:00 |
milan | aarefiev, hmmmm | 12:00 |
* milan looks | 12:01 | |
*** SerenaFe_ has joined #openstack-ironic | 12:03 | |
*** SerenaFe_ has quit IRC | 12:03 | |
milan | aarefiev, unless I missed something we don't persist macs, rather we poll ironic all the time: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L374 | 12:04 |
milan | aarefiev, or you mean the introspection data? | 12:05 |
*** SerenaFeng has quit IRC | 12:05 | |
aarefiev | milan: no, I mean our cache | 12:05 |
milan | aarefiev, but we don't store the mac in the DB, do we? | 12:06 |
aarefiev | milan: I don't know :) | 12:06 |
aarefiev | yeah, you are right | 12:07 |
milan | aarefiev, right, we create them and delegate to ironic | 12:07 |
milan | once we drop a node cache item, we poll ironic to fetch | 12:08 |
milan | the ports again | 12:08 |
* milan wonders whether it's sane | 12:08 | |
*** SerenaFeng has joined #openstack-ironic | 12:08 | |
milan | aarefiev, so if a node was removed in ironic, we won't be able to figure out what mac addresses to remove from the filters unless we track this, correct? | 12:09 |
aarefiev | milan: maybe it make sense if new port was added | 12:09 |
aarefiev | milan: removing node on introspection ? | 12:09 |
* milan looks whether we don't have this as a bug | 12:10 | |
milan | aarefiev, after | 12:10 |
aarefiev | milan: if you remove node, inspector will drop it from cache | 12:10 |
aarefiev | ? | 12:10 |
aarefiev | w/o update filters | 12:11 |
milan | aarefiev, yup but will it drop the mac addresses from iptables? | 12:11 |
*** vills has quit IRC | 12:11 | |
milan | maybe we've got an iptables rules leak ;) | 12:11 |
*** tuanluong has quit IRC | 12:11 | |
aarefiev | milan: hmm, sounds like we should store macs in db | 12:11 |
milan | aarefiev, I think we might have a bug even right now ;) | 12:12 |
aarefiev | ok :) | 12:12 |
*** chlong has joined #openstack-ironic | 12:17 | |
milan | aarefiev, we don't have a bug right now, we rebuild a chain all the time and replace current w/ new https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/firewall.py#L187 | 12:17 |
* milan needs to think this through | 12:17 | |
*** appu103 has joined #openstack-ironic | 12:18 | |
*** aparna has quit IRC | 12:18 | |
*** vmud213 has quit IRC | 12:20 | |
aarefiev | milan: yeah, could be optimized | 12:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic-lib master: Fix disk size check output handling https://review.openstack.org/450754 | 12:21 |
*** appu103 has quit IRC | 12:22 | |
*** hshiina has joined #openstack-ironic | 12:22 | |
*** appu103 has joined #openstack-ironic | 12:22 | |
milan | aarefiev, or if we want to stick to the current implementation, we have to remove the remove_node_ids/macs from the driver spec | 12:22 |
milan | aarefiev, and change the meaning of the whitelist&blacklist methods to be "absolute" so what you feed them that is applied (rather than added) | 12:24 |
aarefiev | milan: I guess filter's behavior will be driver specific on remove | 12:24 |
*** dprince has joined #openstack-ironic | 12:24 | |
milan | aarefiev, right. I've anyways not specified that the whitelist&blacklist logic was additive | 12:25 |
milan | as we have the remove_node_ids there | 12:25 |
milan | :-/ | 12:25 |
*** gcb has quit IRC | 12:25 | |
*** jaganathan has quit IRC | 12:26 | |
*** gcb has joined #openstack-ironic | 12:26 | |
*** Goneri has joined #openstack-ironic | 12:27 | |
aarefiev | milan: so if node was removed and we cache macs we call filter.remove(nodes) | 12:27 |
*** bfournie has joined #openstack-ironic | 12:27 | |
aarefiev | and drop nodes from cache after all | 12:27 |
aarefiev | right? | 12:27 |
milan | aarefiev, correct | 12:27 |
milan | aarefiev, or if we don't introduce the remove logic, we have to have an absolute blacklist/whitelist parameter | 12:28 |
*** appu103 has quit IRC | 12:28 | |
*** lucas-hungry is now known as lucasagomes | 12:28 | |
*** appu103 has joined #openstack-ironic | 12:28 | |
*** SerenaFeng has quit IRC | 12:28 | |
milan | so filter.blacklist == node_list after filter.blacklist_nodes(node_list) | 12:29 |
milan | ditto w/ whitelist | 12:29 |
milan | I guess both ways will have drawbacks | 12:29 |
milan | second way (absolute lists) is in addition always working with all nodes currently tracked by Ironic, which may be bad in bigger deployments | 12:30 |
aarefiev | milan: what do you mean by `working with all nodes`? | 12:31 |
milan | aarefiev, btw we also agreed to keep the node cache in inspector in sync w/ ironic w/o purging items | 12:32 |
aarefiev | yeah | 12:32 |
milan | aarefiev, ironic.node_list() or ironic.port_list() | 12:32 |
milan | is called all the time | 12:32 |
milan | and then all the items are always passed on to the filter | 12:32 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Call clean_up_instance() during node teardown for Agent deploy https://review.openstack.org/450146 | 12:33 |
*** jjung has quit IRC | 12:33 | |
aarefiev | milan: we could pass only nodes to be updated | 12:33 |
milan | aarefiev, that was my original idea and it requires the remove_node_ids method to be consistent | 12:34 |
aarefiev | milan: so the problem is how iptables driver works now | 12:35 |
milan | aarefiev, correct but also you can't get mac address from ironic once the node was removed | 12:35 |
aarefiev | milan: if we don't cache it | 12:36 |
milan | aarefiev, exactly so we either cache or always rebuild as we do currently | 12:36 |
aarefiev | what if some driver requires macs for clean pu | 12:36 |
milan | aarefiev, I guess almost all the nodes will work with MACs | 12:36 |
* milan can't imagine other criteria to blacklist on as we're talking about DHCP in the end | 12:37 | |
aarefiev | milan: we could hide rebuild chain in iptables impl for now | 12:38 |
milan | aarefiev, so keep the remove_node_ids in the spec? | 12:39 |
aarefiev | milan: spec is only words :) | 12:40 |
*** appu103 has quit IRC | 12:40 | |
*** appu103 has joined #openstack-ironic | 12:40 | |
milan | aarefiev, unless someone (ehhhm dtantsur ) -2 it based on a spec :P | 12:40 |
*** nmathew has joined #openstack-ironic | 12:40 | |
*** nmathew has quit IRC | 12:41 | |
aarefiev | milan: we always could sync spec with code | 12:41 |
*** trown|outtypewww is now known as trown | 12:41 | |
*** csmart has joined #openstack-ironic | 12:41 | |
*** retr0h has joined #openstack-ironic | 12:41 | |
aarefiev | we could miss some details on design stage | 12:42 |
milan | aarefiev, other thing, if the iptables driver calculates the items to remove, then it's both inspector and iptables driver calculating the same thing (based on the spec) :D | 12:42 |
milan | aarefiev, right, so my brain dump so far: | 12:43 |
milan | * all the drivers will use MAC | 12:43 |
milan | * we have issues mapping MAC to node ids | 12:43 |
milan | * we'd better use the remove_node_ids semantics (additive lists) | 12:43 |
aarefiev | milan: you will pass result node list, why it will calculate twice | 12:44 |
milan | == let's cache MACs in inspector and change the interface to use MACs instead of node_ids? | 12:44 |
milan | aarefiev, yeah, if it's calculated by inspector rather than by the driver then once | 12:45 |
aarefiev | milan: I'm also wondering if some driver requires some other id than macs | 12:45 |
aarefiev | port uuid by neutron | 12:45 |
milan | aarefiev, right :-/ port uuids make sense | 12:46 |
milan | so a cache would have to contain that too? | 12:46 |
milan | it's getting hairy I guess | 12:46 |
milan | node_id x port_id x MAC x optional IB id? | 12:47 |
jroll | lucasagomes: \o/ for sushy-tools, +1 there | 12:47 |
aarefiev | milan: I guess it would be some new port object | 12:48 |
lucasagomes | jroll, o/ cheers! | 12:48 |
milan | aarefiev, hmmm, yeah | 12:48 |
aarefiev | milan: ok, I guess we should go step by step | 12:48 |
*** jjung has joined #openstack-ironic | 12:49 | |
aarefiev | milan: lets start from macs | 12:49 |
jroll | morning btw | 12:49 |
milan | morning jroll! :) | 12:49 |
aarefiev | morning jroll | 12:50 |
milan | aarefiev, also, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L729 worries me a bit | 12:50 |
milan | we keep the MACs in attributes until the introspection finishes | 12:51 |
milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L290 | 12:52 |
milan | but the purpose is node lookup so maybe it's OK to duplicate the data in a mac cache table | 12:54 |
aarefiev | milan: it's temporary data, so i'd not worry a lot | 12:55 |
*** sbezverk has quit IRC | 12:55 | |
*** appu103 has quit IRC | 12:55 | |
*** appu103 has joined #openstack-ironic | 12:55 | |
*** mat128 has quit IRC | 12:57 | |
milan | aarefiev, lemme maybe update the POC w/ a mac cache, we can update the spec later? | 12:58 |
*** mjura has joined #openstack-ironic | 12:58 | |
aarefiev | milan: we can update spec once code would be ready | 12:58 |
milan | alright | 12:59 |
milan | aarefiev, thx for the discussion :) | 12:59 |
aarefiev | np :) | 12:59 |
*** appu103 has quit IRC | 12:59 | |
*** appu103 has joined #openstack-ironic | 12:59 | |
* milan .raise(CaffeineLevelsDepleted('productive discussion')) | 13:00 | |
*** rbudden has joined #openstack-ironic | 13:03 | |
*** aparna has joined #openstack-ironic | 13:04 | |
*** appu103 has quit IRC | 13:04 | |
*** sasha2 has joined #openstack-ironic | 13:05 | |
*** gcb has quit IRC | 13:07 | |
*** amoralej is now known as amoralej|lunch | 13:08 | |
*** garyk has quit IRC | 13:12 | |
*** baoli has joined #openstack-ironic | 13:13 | |
*** baoli has quit IRC | 13:14 | |
*** baoli has joined #openstack-ironic | 13:14 | |
*** rbartal__ has joined #openstack-ironic | 13:16 | |
*** yogi has quit IRC | 13:17 | |
jroll | TheJulia: with some of the defaults changing, looks like bifrost would be a major version bump, you good with that? | 13:17 |
* TheJulia goes back and double checks the history | 13:18 | |
*** amotoki has joined #openstack-ironic | 13:18 | |
jroll | TheJulia: I'm looking at https://docs.openstack.org/releasenotes/bifrost/unreleased.html fwiw | 13:18 |
TheJulia | which would be master branch I guess | 13:19 |
jroll | yeah, I assume so, though bugs in reno have proved me wrong in the past | 13:19 |
TheJulia | jroll: I was just meaning release a cut of stable/ocata since we have a few nasty bug fixes in there | 13:20 |
jroll | TheJulia: oh! gotcha | 13:20 |
TheJulia | master branch can continue rolling for now :) | 13:20 |
jroll | sounds good | 13:21 |
* TheJulia may or may not be plotting a major feature | 13:21 | |
*** jkilpatr has quit IRC | 13:22 | |
jroll | well this is odd. harumph. | 13:22 |
*** jkilpatr has joined #openstack-ironic | 13:22 | |
jroll | https://github.com/openstack/ironic-lib/blob/master/tox.ini#L8 | 13:22 |
jroll | https://github.com/openstack/ironic-lib/blob/stable/ocata/tox.ini#L8 | 13:22 |
* jroll fixes | 13:22 | |
*** links has quit IRC | 13:22 | |
*** amotoki has quit IRC | 13:23 | |
TheJulia | jroll: You may want to review https://review.openstack.org/#/c/450733/ | 13:23 |
TheJulia | funky | 13:23 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib master: Revert "Update UPPER_CONSTRAINTS_FILE for stable/ocata" https://review.openstack.org/450777 | 13:24 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib stable/ocata: Update UPPER_CONSTRAINTS_FILE for stable/ocata https://review.openstack.org/450778 | 13:24 |
*** jvgrant_ has quit IRC | 13:24 | |
jroll | ^ could I have reviews on those please? :) | 13:24 |
*** jvgrant_ has joined #openstack-ironic | 13:24 | |
jroll | JayF: ^ one is stable | 13:24 |
jroll | TheJulia: +A, will wait for that one for ironic release | 13:25 |
TheJulia | jroll: awesome, since otherwise everything is broken without it at the moment ;) | 13:27 |
jroll | TheJulia: just db migrations, right? | 13:27 |
jroll | "just" | 13:27 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector master: Don't pass sqlite_db in db_options.set_defaults() https://review.openstack.org/450737 | 13:29 |
TheJulia | I think creations as well, but I didn't dig too deeply into it since vsaienk0 was looking at it | 13:29 |
jroll | sure | 13:29 |
*** rloo has joined #openstack-ironic | 13:31 | |
*** crushil has joined #openstack-ironic | 13:31 | |
*** alexpilotti has joined #openstack-ironic | 13:31 | |
*** amotoki has joined #openstack-ironic | 13:34 | |
jroll | TheJulia: ironic-ui has network_interface and manual cleaning support, shall we release it as well? | 13:36 |
TheJulia | That does sound like a very good idea :) | 13:37 |
jroll | alright, will do | 13:38 |
* jroll has things lined up but waiting on some patches | 13:39 | |
TheJulia | story of our lives? :) | 13:39 |
jroll | heh | 13:39 |
*** causten has quit IRC | 13:40 | |
*** sbezverk has joined #openstack-ironic | 13:40 | |
openstackgerrit | Merged openstack/ironic-lib master: Revert "Update UPPER_CONSTRAINTS_FILE for stable/ocata" https://review.openstack.org/450777 | 13:42 |
jroll | oh, that oslo.db bug killed our entire gate, fun! | 13:43 |
openstackgerrit | Miles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration https://review.openstack.org/447085 | 13:43 |
jroll | folks, please don't recheck or approve things until this merges: https://review.openstack.org/#/c/450733/ | 13:43 |
jlvillal | jroll, Is that in the whiteboard? | 13:45 |
* jroll looks | 13:45 | |
jroll | jlvillal: the bug was, but status was still 'sporadic' | 13:46 |
jlvillal | jroll, Thanks | 13:46 |
* jroll fixed | 13:46 | |
rloo | morning all, jlvillal, jroll, TheJulia | 13:46 |
jlvillal | Good morning :) | 13:46 |
jroll | morning rloo | 13:47 |
mariojv | morning everyone o/ | 13:47 |
rloo | jroll: wrt the releases waiting on patches -- do you need any reviews, or just waiting to merge? | 13:47 |
jroll | rloo: all patches are +W | 13:47 |
rloo | jlvillal: you working EDT hours starting today? :) | 13:47 |
rloo | jroll: cool. | 13:48 |
jlvillal | rloo, I have to take my wife to the airport today. I figure that will take 2-4 hours. | 13:48 |
jlvillal | rloo, So I started work early :) | 13:48 |
rloo | jlvillal: :) | 13:48 |
mgould | morning rloo mariojv | 13:49 |
*** hoangcx has joined #openstack-ironic | 13:53 | |
mgould | I'm getting "AH00112: Warning: DocumentRoot [/opt/stack/data/ironic/httpboot] does not exist" errors when I run stack.sh, does anyone know what that's about? | 13:53 |
mgould | /opt/stack/data/ironic doesn't exist; if I create it manually it gets deleted | 13:53 |
*** hamzy has quit IRC | 13:54 | |
mgould | more output: http://paste.openstack.org/show/604484/ | 13:55 |
*** krtaylor has quit IRC | 13:56 | |
jlvillal | mgould, I'd be more worried about the unable to bind messages. | 13:58 |
*** rbartal__ has quit IRC | 13:58 | |
*** krtaylor has joined #openstack-ironic | 14:00 | |
*** ElCoyote_ has joined #openstack-ironic | 14:02 | |
TheJulia | mgould: 447085 needs a minor tweak to the release note. It fails validation presently :( | 14:02 |
*** amoralej|lunch is now known as amoralej | 14:03 | |
*** aparna has quit IRC | 14:03 | |
*** aparnav has joined #openstack-ironic | 14:04 | |
mgould | jlvillal: according to netstat, there's nothing listening on those ports :-( | 14:04 |
jlvillal | mgould, I don't know. Seems like Apache is saying there is something. :( | 14:05 |
mgould | TheJulia: thanks | 14:05 |
jroll | apache loves to restart slowly | 14:05 |
jroll | I think devstack even does a stop; sleep X; start | 14:05 |
jroll | on ubuntu anyway | 14:05 |
* jlvillal is sure mgould only uses Ubuntu :P | 14:06 | |
* mgould drags desks around to debug whatever's stopping him from sconnecting to his other devstack VM | 14:06 | |
mgould | [A[B | 14:06 |
mgould | jlvillal: this VM's centos | 14:06 |
*** _milan_ has joined #openstack-ironic | 14:07 | |
jlvillal | mgould, I was joking on Ubuntu. Based on where you work I doubt you use Ubuntu much :) | 14:07 |
* mgould still has a couple of Ubuntu machines at home | 14:08 | |
mgould | speaking of which: I've handed in my notice at Red Hat, so I won't be around after April 12th | 14:08 |
jlvillal | mgould, Inconcievable! | 14:08 |
*** milan has quit IRC | 14:08 | |
jlvillal | mgould, Oh :( Does that mean no more Ironic for you??? | 14:09 |
TheJulia | mgould: Sad to hear, but good luck on your next venture! | 14:09 |
* jlvillal sad.. | 14:09 | |
mgould | TheJulia, jlvillal: thanks! | 14:09 |
* TheJulia sad too | 14:09 | |
mariojv | mgould: :( gl though | 14:09 |
mgould | mariojv: thanks! | 14:10 |
jroll | mgould: sad for us, congrats to you :) | 14:10 |
jlvillal | mgould, Willing to share yet what you will be doing next? :) | 14:10 |
* mgould is going to do machine-learning at a local security startup | 14:10 | |
mgould | http://zonefox.com | 14:10 |
jlvillal | Cool :) | 14:10 |
jroll | nice | 14:11 |
mgould | I'll miss you all, but remote work was doing my head in | 14:11 |
vdrok | morning jroll rloo and jlvillal ! | 14:11 |
openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Abstraction of distribution variables and cleanup role. https://review.openstack.org/450347 | 14:11 |
jlvillal | mgould, Oh you are changing jobs because you don't want to work remote? I can kind of understand. | 14:11 |
rloo | hi mgould, vdrok! | 14:12 |
vdrok | oh, mgould , good luck with your new project! :) | 14:12 |
jroll | \o vdrok | 14:12 |
mgould | hi rloo | 14:12 |
mgould | jroll, vdrok: thanks! | 14:12 |
openstackgerrit | Mario Villaplana proposed openstack/python-ironicclient master: Log warning when API version unspecified https://review.openstack.org/442153 | 14:12 |
*** jaosorior has quit IRC | 14:12 | |
TheJulia | mgould: very cool! Have fun with the new job. | 14:12 |
jlvillal | mgould, I did remote for two years as a single person. I did feel like I lacked human contact at times since I was home all day by myself. | 14:13 |
rloo | oh mgould, congrats!!! remote work isn't for everyone; glad you realized what works for you :) | 14:13 |
*** tuan_luong has joined #openstack-ironic | 14:13 | |
*** tuanla_ has joined #openstack-ironic | 14:14 | |
jlvillal | Hi vdrok :) | 14:14 |
thomas_oneill | TheJulia: hey you mentioned about knowing operators who have (already?) "split the roles apart" - are they on IRC/gerrit? | 14:14 |
mgould | rloo, TheJulia: thanks! | 14:15 |
vdrok | jroll: so, after some more thinking on the versioning , we still need the first patch from rolling upgrades. having old api and new conductor, we'll be able to do rpcapi requests in a backwards compatible way, but the old api won't be able to deserialize new objects still | 14:15 |
vdrok | *new versions of objects | 14:15 |
jroll | vdrok: fair point, yeah. I think we tend to bump RPC version more than object version, so I think fixing the CI will still reduce the number of patches that kill that job :) | 14:16 |
vdrok | yup | 14:16 |
TheJulia | thomas_oneill: I mean more along the lines of picking/choosing specific roles. One of those operators, I know I'm never going to be able to remember aside from the city and company they work for. The others... I think it was more so an entirely in passing "hey, we did this to fit our needs" thing | 14:16 |
TheJulia | thomas_oneill: not broken into multiple git repos | 14:16 |
vdrok | also, the changes we need to the job appear to be not so straight-forward, as discussed with vsaienk0 | 14:16 |
mgould | yay, I can get into my other virthost! | 14:17 |
mgould | turns out our subnet changed this morning before I got in | 14:17 |
mgould | *obviously* :-/ | 14:17 |
*** rbartal has quit IRC | 14:17 | |
jroll | vdrok: :( let me know if there's any way I can help | 14:17 |
*** rbartal_ has quit IRC | 14:17 | |
vsaienk0 | vdrok: jroll so easiest way is to seems not restart ir-api on primary node after upgrade, it should work... but need to check | 14:18 |
*** causten has joined #openstack-ironic | 14:18 | |
jroll | vsaienk0: what does moving the api to the subnode require? changing service catalog/configs, anything else? | 14:19 |
vdrok | jroll: we'll need to skip enrolling the nodes, and do that during subnode setup instead | 14:20 |
jroll | aha, I see | 14:20 |
vsaienk0 | jroll: yeah, service catalog/configs, and we should enroll ironic nodes when subnode is ready | 14:20 |
* jroll needs to read more about how multinode stuff is ordered and such | 14:22 | |
* jroll wonders if we could do a separate "install and run" phase and "configure resources" phase | 14:23 | |
jroll | that could be longer term though :) | 14:23 |
*** Goneri has quit IRC | 14:24 | |
openstackgerrit | Miles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration https://review.openstack.org/447085 | 14:26 |
*** hamzy has joined #openstack-ironic | 14:27 | |
* mgould plays the game called "can I get all my in-progress patches merged before I leave?" | 14:27 | |
*** aparnav has quit IRC | 14:27 | |
TheJulia | heh | 14:30 |
*** Goneri has joined #openstack-ironic | 14:30 | |
mgould | if anyone has core privileges on diskimage-builder, I'd *love* a review on https://review.openstack.org/#/c/344214/ | 14:30 |
*** gcb has joined #openstack-ironic | 14:33 | |
TheJulia | cinerama: ^^^ | 14:34 |
vsaienk0 | mgould: we allow more characters in logical names here https://review.openstack.org/#/c/447085/8/ironic_ui/api/ironic_rest_api.py@230 | 14:35 |
TheJulia | fun :( | 14:37 |
TheJulia | That is going to be a number of lines changed | 14:37 |
*** crushil has quit IRC | 14:45 | |
*** vikrant has quit IRC | 14:48 | |
NobodyCam | good morning Ironic'ers | 14:52 |
vdrok | good morning NobodyCam :) | 14:55 |
NobodyCam | Morning vdrok :) | 14:55 |
*** jtaryma has joined #openstack-ironic | 14:58 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 15:00 |
*** hoangcx has quit IRC | 15:00 | |
*** crushil has joined #openstack-ironic | 15:00 | |
openstackgerrit | Michael Turek proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd https://review.openstack.org/447642 | 15:01 |
openstackgerrit | Michael Turek proposed openstack/ironic master: Remove logging translation calls from ironic.common https://review.openstack.org/448089 | 15:01 |
openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Abstraction of distribution variables and cleanup role. https://review.openstack.org/450347 | 15:01 |
mjturek | whoops vgadiraj, I rebased your patch... sorry about that! | 15:02 |
*** jtaryma has quit IRC | 15:03 | |
openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role. https://review.openstack.org/450347 | 15:03 |
thomas_oneill | how's the test coverage of the keystone install in bifrost? | 15:04 |
*** jjung has quit IRC | 15:08 | |
*** dyasny has joined #openstack-ironic | 15:08 | |
*** mjura has quit IRC | 15:08 | |
thomas_oneill | hmmm 'FATAL: Unable to delete script file /tmp/hudson3781404264779685742.sh' - http://artifacts.opnfv.org/cross-community-ci/openstack/bifrost/review.openstack.org/450347/3/openstack-bifrost-verify-trusty-virtual-master/build_log.txt | 15:09 |
*** rpioso has joined #openstack-ironic | 15:09 | |
TheJulia | thomas_oneill: there is not a specific job that exercises it yet afaik | 15:10 |
thomas_oneill | oh it's failing at different parts but with this same failure in places. The ci server must be barfing or something, I presume | 15:10 |
* TheJulia wonders if someone is clicking the abort button | 15:10 | |
thomas_oneill | it says ABORTED on the 3 tests... where is said abort button? :o | 15:11 |
TheJulia | in Jenkins console | 15:11 |
mgould | vsaienk0: thanks! | 15:12 |
TheJulia | for an authenticated and authorized user most likely | 15:12 |
* mgould has got his devstack VM to fail in a different way! | 15:12 | |
mgould | there were two "Listen 5000" lines in /etc/httpd | 15:12 |
mgould | did we move that setting? | 15:12 |
TheJulia | thomas_oneill: fwiw, bifrost's jobs on opnfv ci are broken pending a fix in ironic | 15:13 |
mgould | anyway, yum remove httpd && rm -rf /etc/httpd made the error go away | 15:13 |
thomas_oneill | oh yeah I saw about that this morning | 15:13 |
TheJulia | wheeeee http://logs.openstack.org/33/450733/1/gate/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-ubuntu-xenial/dac3e06/logs/screen-ir-cond.txt.gz#_2017-03-28_14_13_27_958 | 15:15 |
TheJulia | http://logs.openstack.org/33/450733/1/gate/gate-grenade-dsvm-ironic-ubuntu-xenial/33d53f6/logs/grenade.sh.txt.gz#_2017-03-28_14_28_25_597 <-- also fun..... | 15:18 |
* TheJulia sees jroll has already issued a recheck | 15:19 | |
*** jjung has joined #openstack-ironic | 15:19 | |
*** jtaryma has joined #openstack-ironic | 15:20 | |
jroll | indeed, I have a feeling something is just slow there judging by the sleep 5 before the command that returns RTNETLINK answers: Network is unreachable | 15:20 |
*** sbezverk has quit IRC | 15:20 | |
*** _milan_ has quit IRC | 15:24 | |
*** aarefiev is now known as aarefiev_afk | 15:24 | |
TheJulia | Yeah, I wonder if it could be a little longer or retry once or something. :( | 15:25 |
jroll | or poll | 15:26 |
*** sayantan_ has joined #openstack-ironic | 15:28 | |
*** baoli has quit IRC | 15:29 | |
*** rcernin has quit IRC | 15:31 | |
*** rcernin_ has quit IRC | 15:31 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update https://review.openstack.org/450855 | 15:32 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing converting BaseDriver.*_interfaces to tuples https://review.openstack.org/450856 | 15:32 |
jlvillal | jroll, ^^^ what I mentioned before | 15:33 |
jroll | cool | 15:34 |
jroll | thanks! | 15:34 |
*** Serlex has quit IRC | 15:35 | |
jroll | jlvillal: do you have a link handy to the patch that fixed that memory bloat bug btw? | 15:35 |
jroll | don't go searching, I can do that, just curious :) | 15:36 |
*** alexpilotti has quit IRC | 15:36 | |
* jroll wants to backport that | 15:36 | |
jlvillal | jroll: It is the one I depend on | 15:36 |
jlvillal | directly | 15:36 |
jroll | ah, duh | 15:36 |
jroll | I thought it had merged but realized I'm waiting for merge to release :D | 15:36 |
jlvillal | jroll, https://review.openstack.org/#/c/449577/ | 15:36 |
jroll | yep | 15:36 |
jroll | thanks! | 15:36 |
jlvillal | You're welcome | 15:36 |
*** alexpilotti has joined #openstack-ironic | 15:39 | |
* jroll steps out for lunch/errands | 15:39 | |
*** amarao has quit IRC | 15:41 | |
*** alexpilotti has quit IRC | 15:46 | |
*** e0ne has quit IRC | 15:46 | |
mgould | apparently my pool_size is configured to "ironic.sqlite" | 15:51 |
mgould | yeah, I can see how that wouldn't work | 15:52 |
*** fragatina has joined #openstack-ironic | 15:56 | |
*** crushil has quit IRC | 16:06 | |
vdrok | mgould: fix for that is in merge queue | 16:08 |
*** bfournie has quit IRC | 16:12 | |
mgould | vdrok: awesome! | 16:13 |
openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 16:13 |
mgould | vdrok: in ironic? | 16:14 |
vdrok | mgould: yup, https://review.openstack.org/450733 | 16:15 |
mgould | excellent | 16:15 |
*** fragatina has quit IRC | 16:15 | |
*** alexpilotti has joined #openstack-ironic | 16:15 | |
* mgould rebases his patch on top of it | 16:16 | |
openstackgerrit | Miles Gould proposed openstack/ironic master: devstack: install python-dracclient if DRAC enabled https://review.openstack.org/425326 | 16:16 |
*** jaganathan has joined #openstack-ironic | 16:18 | |
mgould | vsaienk0, TheJulia, thanks for reviews | 16:23 |
*** Nisha_Agarwal has joined #openstack-ironic | 16:23 | |
JayF | dtantsur: jroll: WDYT about https://review.openstack.org/#/c/449504/1 -- it technically violates stable policy but it seems like something we need to land anyway. | 16:24 |
*** alexpilotti has quit IRC | 16:24 | |
*** causten has quit IRC | 16:25 | |
*** pcaruana has quit IRC | 16:26 | |
*** causten has joined #openstack-ironic | 16:26 | |
alezil | milan: thanks for reviewing the patch. | 16:26 |
*** baha has joined #openstack-ironic | 16:26 | |
*** aNuposic has joined #openstack-ironic | 16:28 | |
*** causten_ has joined #openstack-ironic | 16:29 | |
*** causten has quit IRC | 16:30 | |
openstackgerrit | Merged openstack/ironic-lib stable/ocata: Update UPPER_CONSTRAINTS_FILE for stable/ocata https://review.openstack.org/450778 | 16:31 |
JayF | jroll: dtantsur: Stable patch that should be simple to land: https://review.openstack.org/#/c/448632/ | 16:31 |
*** jaganathan has quit IRC | 16:33 | |
*** chlong has quit IRC | 16:34 | |
*** fragatina has joined #openstack-ironic | 16:34 | |
*** causten_ has quit IRC | 16:39 | |
jroll | JayF: I'm +2 on that first one, it's necessary | 16:40 |
JayF | jroll: that's what I thought, I am landing it, but wanted to check :) | 16:40 |
JayF | jroll: it's weird to me that d-r is not required to be in g-r for reasons like that, but w/e | 16:40 |
jroll | JayF: yeah, it's a mismatch because the g-r updates won't check that file, but we don't want to hard require d-r | 16:41 |
jroll | pas-ha: did you know this depends-on will make it wait until all branches of the dependency are merged? https://review.openstack.org/#/c/448632/ | 16:41 |
mgould | JayF: dtantsur is on PTO this week | 16:41 |
jroll | this is true | 16:41 |
JayF | jroll: when you roll ironic releases, we might want to push stable releases too fwiw | 16:42 |
pas-ha | jroll: yep, I'm aware, just added that to see gates passing and will remove when requirements patches are merged | 16:42 |
jroll | JayF: indeed | 16:42 |
jroll | pas-ha: ok cool | 16:42 |
*** lucasagomes is now known as lucas-afk | 16:43 | |
*** fragatina has quit IRC | 16:43 | |
JayF | mariojv: I think https://review.openstack.org/#/c/424723/6 is either going to make rescue harder or easier, I'm not 100% sure yet | 16:43 |
openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role. https://review.openstack.org/450884 | 16:45 |
*** mjturek has quit IRC | 16:45 | |
*** jaganathan has joined #openstack-ironic | 16:46 | |
openstackgerrit | Thomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role. https://review.openstack.org/450884 | 16:47 |
*** chlong has joined #openstack-ironic | 16:49 | |
thomas_oneill | okay it's home time again for me, I think I've made some okay progress on this patch today, please review if you have time bifrosters!! :) ^^ | 16:49 |
mgould | thomas_oneill: o/ | 16:50 |
*** jaganathan has quit IRC | 16:50 | |
*** derekh has quit IRC | 16:50 | |
*** Nisha_Agarwal has quit IRC | 16:51 | |
*** trown is now known as trown|lunch | 16:51 | |
*** causten has joined #openstack-ironic | 16:52 | |
vdrok | good night everyone! | 17:01 |
mgould | vdrok: good night! | 17:01 |
*** mgoddard has quit IRC | 17:02 | |
*** fragatina has joined #openstack-ironic | 17:03 | |
*** jaganathan has joined #openstack-ironic | 17:03 | |
*** dsneddon has quit IRC | 17:04 | |
*** crushil has joined #openstack-ironic | 17:05 | |
*** jtaryma has quit IRC | 17:06 | |
*** tuanla_ has quit IRC | 17:07 | |
*** tuan_luong has quit IRC | 17:07 | |
*** mtanino has joined #openstack-ironic | 17:11 | |
*** deadnull has joined #openstack-ironic | 17:12 | |
*** baoli has joined #openstack-ironic | 17:12 | |
*** mtanino has quit IRC | 17:13 | |
*** jtaryma has joined #openstack-ironic | 17:15 | |
*** baoli has quit IRC | 17:18 | |
*** amoralej is now known as amoralej|off | 17:18 | |
*** dsneddon has joined #openstack-ironic | 17:21 | |
*** baoli has joined #openstack-ironic | 17:23 | |
*** mgould is now known as mgould|afk | 17:27 | |
*** jaganathan has quit IRC | 17:29 | |
*** amotoki has quit IRC | 17:31 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing converting BaseDriver.*_interfaces to tuples https://review.openstack.org/450856 | 17:36 |
*** mgoddard has joined #openstack-ironic | 17:46 | |
*** fragatina has quit IRC | 17:47 | |
*** baoli has quit IRC | 17:47 | |
*** fragatina has joined #openstack-ironic | 17:48 | |
*** baoli has joined #openstack-ironic | 17:49 | |
*** baoli has quit IRC | 17:50 | |
*** fragatin_ has joined #openstack-ironic | 17:54 | |
*** fragatina has quit IRC | 17:57 | |
*** baoli has joined #openstack-ironic | 17:59 | |
*** causten has quit IRC | 18:00 | |
*** athomas has quit IRC | 18:02 | |
*** trown|lunch is now known as trown | 18:03 | |
*** tesseract has quit IRC | 18:06 | |
*** ElCoyote_ has quit IRC | 18:07 | |
*** causten has joined #openstack-ironic | 18:07 | |
*** jkilpatr has quit IRC | 18:09 | |
*** jkilpatr has joined #openstack-ironic | 18:09 | |
*** ElCoyote_ has joined #openstack-ironic | 18:11 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 18:32 |
*** deadnull_ has joined #openstack-ironic | 18:48 | |
*** deadnull has quit IRC | 18:49 | |
*** choyj has joined #openstack-ironic | 18:50 | |
*** vills has joined #openstack-ironic | 18:53 | |
openstackgerrit | Debayan Ray proposed openstack/sushy master: Add System Processor/Memory info https://review.openstack.org/446510 | 18:55 |
*** jtaryma has quit IRC | 18:56 | |
*** e0ne has joined #openstack-ironic | 18:58 | |
openstackgerrit | Merged openstack/ironic stable/ocata: Pin version of proliantutils https://review.openstack.org/449504 | 18:59 |
*** ralonsoh has quit IRC | 19:00 | |
mariojv | TheJulia: hey, we have 2x+2 on the first in the series of rolling upgrade patches \o/ https://review.openstack.org/#/c/407491 | 19:01 |
mariojv | TheJulia: feel free to remove -2 whenever you feel that's procedurally ready | 19:01 |
openstackgerrit | Merged openstack/ironic master: Document PXE with Spanning Tree in troubleshooting FAQ https://review.openstack.org/434208 | 19:02 |
mariojv | maybe we ought to get +2s on all the other code in that patch series first though? dunno if it matters too much | 19:03 |
TheJulia | getting +2's on everything is going to drive people crazy | 19:04 |
TheJulia | and we will never get anywhere, I just pulled the -2, I'll review the patch a little later today, I've got an appointment I need to head out for now | 19:05 |
mariojv | haha fair, thx | 19:05 |
choyj | Hi @TheJulia We're trying out the ironic UI and have copied the _2200 py file to the enabled directory. We're not seeing any new panel coming up. What's the panel called? Baremetal? | 19:07 |
TheJulia | choyj: I believe it is baremetal, which is the service type in open stack for ironic | 19:10 |
openstackgerrit | Debayan Ray proposed openstack/sushy master: Add System Processor/Memory info https://review.openstack.org/446510 | 19:10 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: [doc] Update examples in devstack section https://review.openstack.org/450936 | 19:10 |
choyj | ok, thanks! | 19:11 |
*** bfournie has joined #openstack-ironic | 19:15 | |
*** baoli has quit IRC | 19:15 | |
*** bfournie has quit IRC | 19:15 | |
*** bfournie has joined #openstack-ironic | 19:16 | |
*** bfournie has quit IRC | 19:16 | |
*** mgoddard has quit IRC | 19:21 | |
soliosg | choyj: ironic-ui is located in the Admin pane > System > Ironic Bare Metal Provisioning | 19:24 |
soliosg | choyj: you need to restart the horizon server after copying _2200_ironic.py to the enabled directory | 19:25 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] cleanup upgrade settings https://review.openstack.org/450940 | 19:25 |
choyj | Thanks @soliosg, What I was missing was the actual ironic service running in my environment. Which probably disabled the panel due to the missing service. | 19:31 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 19:32 |
*** racedo has quit IRC | 19:37 | |
mariojv | this gate failure (on the patch that tries to fix the dbsync problem) is really weird: https://review.openstack.org/#/c/450733/ | 19:41 |
mariojv | deploy is failing at the stage that copies the configdrive | 19:42 |
mariojv | lots of rpc timeouts at the end of the ironic logs, too, but i'm betting that's just during service shutdown after the tests fail or something | 19:43 |
*** baoli has joined #openstack-ironic | 19:43 | |
jroll | mariojv: that's the one on the whiteboard | 19:45 |
jroll | you know what, I'm gonna fix that now | 19:46 |
jroll | I forgot that was on me and I was waiting for the dbsync thing before releasing everything :x | 19:46 |
mariojv | oh my bad, didn't notice that there were 2 known failures | 19:46 |
jroll | yeah, this one is intermittent | 19:46 |
*** e0ne has quit IRC | 19:46 | |
jroll | mariojv: this fixes, release folks are going to look when CI is done on it https://review.openstack.org/#/c/450949/1 | 19:48 |
JayF | vsaienk0: you know where to kill that standalone job? | 19:49 |
JayF | vsaienk0: that ran against the docs patch? | 19:49 |
JayF | vsaienk0: I can show you if you don't | 19:49 |
*** e0ne has joined #openstack-ironic | 19:49 | |
*** fragatin_ has quit IRC | 19:54 | |
*** fragatina has joined #openstack-ironic | 19:55 | |
openstackgerrit | Merged openstack/ironic master: [doc] Update examples in devstack section https://review.openstack.org/450936 | 19:55 |
jroll | JayF: you asked about stable releases, did you have anything in particular in mind, or just all the things | 19:56 |
JayF | jroll: just a general recollectino of landing a lot of stable stuff | 19:56 |
jroll | ya | 19:56 |
jroll | so many branches :/ | 19:56 |
jroll | suspect we need to backport the oslo.db thing | 19:56 |
JayF | .o(what oslo db thing) | 19:57 |
JayF | I've heard it refenced but seen no patches/bugs | 19:57 |
jroll | the one breaking our gate right now | 19:57 |
JayF | probably because I was gone? | 19:57 |
JayF | oh | 19:57 |
jroll | https://review.openstack.org/#/c/450733/ | 19:57 |
JayF | yeah; I fetched it from the whiteboard once I got clued in | 19:59 |
mariojv | nice, thanks jroll | 20:00 |
jroll | maybe it doesn't need a backport, woo | 20:00 |
*** jtaryma has joined #openstack-ironic | 20:03 | |
*** fragatina has quit IRC | 20:05 | |
*** fragatina has joined #openstack-ironic | 20:06 | |
*** racedo has joined #openstack-ironic | 20:14 | |
*** jkilpatr has quit IRC | 20:16 | |
jroll | vsaienk0: any reason for the depends-on here? they shouldn't be related https://review.openstack.org/#/c/450737/ | 20:18 |
*** jkilpatr has joined #openstack-ironic | 20:18 | |
jroll | I assume just to see the jobs pass | 20:18 |
*** racedo has quit IRC | 20:19 | |
jroll | ^ that could use a +W if someone is around | 20:19 |
*** Goneri has quit IRC | 20:20 | |
*** soulman has joined #openstack-ironic | 20:21 | |
*** Goneri has joined #openstack-ironic | 20:22 | |
JayF | jroll: if it's my working hours, directly ping me for stuff like that :) | 20:22 |
JayF | +2A | 20:22 |
jroll | JayF: yeah, it just isn't a rush due to that depends-on, so I figured why bother :) | 20:22 |
jroll | probably will need to re-approve after the ironic patch merges | 20:23 |
*** ccamacho has quit IRC | 20:25 | |
*** e0ne has quit IRC | 20:26 | |
*** e0ne has joined #openstack-ironic | 20:30 | |
*** mat128 has joined #openstack-ironic | 20:31 | |
*** mat128 has quit IRC | 20:35 | |
*** crushil has quit IRC | 20:38 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 20:38 |
*** e0ne has quit IRC | 20:47 | |
*** trown is now known as trown|outtypewww | 20:50 | |
*** dprince has quit IRC | 20:53 | |
*** crushil has joined #openstack-ironic | 20:53 | |
*** jcoufal has quit IRC | 20:55 | |
*** sambetts is now known as sambetts|afk | 20:55 | |
*** baoli has quit IRC | 20:57 | |
*** e0ne has joined #openstack-ironic | 20:58 | |
*** jtaryma has quit IRC | 21:00 | |
*** e0ne has quit IRC | 21:01 | |
*** e0ne has joined #openstack-ironic | 21:02 | |
*** e0ne has quit IRC | 21:02 | |
*** e0ne has joined #openstack-ironic | 21:02 | |
*** e0ne has quit IRC | 21:03 | |
*** e0ne has joined #openstack-ironic | 21:04 | |
*** e0ne has quit IRC | 21:04 | |
*** e0ne has joined #openstack-ironic | 21:04 | |
*** e0ne has quit IRC | 21:05 | |
*** e0ne has joined #openstack-ironic | 21:05 | |
*** e0ne has quit IRC | 21:05 | |
*** e0ne has joined #openstack-ironic | 21:06 | |
*** e0ne has quit IRC | 21:06 | |
*** Goneri has quit IRC | 21:07 | |
*** ianychoi has quit IRC | 21:07 | |
*** baoli has joined #openstack-ironic | 21:08 | |
*** ianychoi has joined #openstack-ironic | 21:08 | |
*** jtaryma has joined #openstack-ironic | 21:12 | |
*** jrcloud has joined #openstack-ironic | 21:18 | |
*** zackf has quit IRC | 21:20 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements https://review.openstack.org/450995 | 21:26 |
*** Goneri has joined #openstack-ironic | 21:27 | |
*** jjung has quit IRC | 21:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 21:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/451009 | 21:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/451010 | 21:29 |
*** vinsh has joined #openstack-ironic | 21:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/451012 | 21:29 |
*** sayantan_ has quit IRC | 21:36 | |
*** hshiina has quit IRC | 21:39 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 21:42 |
openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 21:47 |
openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 21:47 |
jroll | noooooo I was commenting : | 21:48 |
jroll | :| | 21:48 |
jroll | joanna: just dropped a few more comments on PS 55 | 21:49 |
jlvillal | jroll, You would think she would be concentrating in her golang class ;) | 21:50 |
jroll | jlvillal: probably ahead of the class already :P | 21:50 |
jlvillal | jroll, Probably very true! | 21:51 |
*** bcornec has joined #openstack-ironic | 21:53 | |
jtaryma | jroll: thanks, I commented on the very same minute :) | 22:02 |
jroll | yep :) | 22:02 |
*** jtaryma is now known as joanna_mac | 22:03 | |
joanna_mac | jroll: I already asked about this async thing, it uses call instead of cast, so they're sync. | 22:04 |
jroll | joanna_mac: awesome | 22:05 |
joanna_mac | jlvillal: that's flattering, I just sometimes change my scope when the talk is about sth I already know | 22:05 |
joanna_mac | but I am not ahead! :) | 22:05 |
jroll | joanna_mac: now #openstack-cinder tells me terminate_volume and detach are async :/ | 22:13 |
jroll | https://drive.google.com/file/d/0B1Kp6K43HLHydU5XSWNnZEhqVDA/view?pref=2&pli=1 | 22:13 |
*** joanna_mac has quit IRC | 22:14 | |
*** cdearborn has quit IRC | 22:15 | |
*** baoli has quit IRC | 22:16 | |
*** joanna_mac has joined #openstack-ironic | 22:21 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 22:21 |
joanna_mac | jroll: I heard that you got a different answer | 22:21 |
*** baha has quit IRC | 22:21 | |
*** choyj has quit IRC | 22:22 | |
*** choyj has joined #openstack-ironic | 22:25 | |
JayF | joanna_mac: repost incoming | 22:27 |
JayF | 22:13:17 <jroll> joanna_mac: now #openstack-cinder tells me terminate_volume and detach are async :/ | 22:27 |
JayF | 22:13:20 <jroll> https://drive.google.com/file/d/0B1Kp6K43HLHydU5XSWNnZEhqVDA/view?pref=2&pli=1 | 22:27 |
joanna_mac | the thing is that in the code it doesn't look so. | 22:28 |
joanna_mac | JayF: thanks :) | 22:28 |
*** rpioso has quit IRC | 22:29 | |
*** jaypipes has quit IRC | 22:39 | |
*** wrouesnel has joined #openstack-ironic | 22:49 | |
*** Goneri has quit IRC | 22:52 | |
openstackgerrit | Julian Edwards proposed openstack/ironic master: Update install guide to explain Nova integration https://review.openstack.org/450492 | 22:57 |
joanna_mac | JayF, jroll: yep, scottda meant that call from nova is async | 23:00 |
joanna_mac | not the detach functions itself | 23:00 |
*** bcornec has quit IRC | 23:03 | |
*** soulman has quit IRC | 23:05 | |
*** causten has quit IRC | 23:06 | |
*** causten has joined #openstack-ironic | 23:07 | |
*** Goneri has joined #openstack-ironic | 23:07 | |
*** causten has quit IRC | 23:12 | |
*** hshiina has joined #openstack-ironic | 23:32 | |
*** Fdaisuke has joined #openstack-ironic | 23:40 | |
openstackgerrit | Merged openstack/ironic master: Don't pass sqlite_db in db_options.set_defaults() https://review.openstack.org/450733 | 23:46 |
TheJulia | \o/ | 23:46 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!