Tuesday, 2017-03-28

*** crushil has quit IRC00:03
*** gcb has quit IRC00:18
*** oomichi has quit IRC00:19
*** oomichi has joined #openstack-ironic00:23
*** mtanino_ has joined #openstack-ironic00:26
*** mtanino has quit IRC00:27
*** fragatin_ has quit IRC00:32
*** fragatina has joined #openstack-ironic00:32
*** sayantan_ has quit IRC00:42
openstackgerritJulian Edwards proposed openstack/ironic master: Update install guide to explain Nova integration  https://review.openstack.org/45049200:51
*** tovin07 has joined #openstack-ironic00:52
*** jjung has quit IRC00:59
*** amotoki has joined #openstack-ironic01:08
*** amotoki has quit IRC01:14
*** tuanluong has joined #openstack-ironic01:20
*** sasha2 has joined #openstack-ironic01:22
*** MarginHu has joined #openstack-ironic01:27
*** tovin07 has quit IRC01:28
*** tovin07 has joined #openstack-ironic01:29
*** SerenaFeng has joined #openstack-ironic01:31
*** sasha2 has quit IRC01:34
*** SerenaFeng has quit IRC01:36
*** SerenaFeng has joined #openstack-ironic01:40
*** phuongnh has joined #openstack-ironic01:46
*** SerenaFe_ has joined #openstack-ironic01:47
*** SerenaFeng has quit IRC01:49
*** causten has joined #openstack-ironic01:50
*** causten has quit IRC01:50
*** causten has joined #openstack-ironic01:50
*** zhurong has joined #openstack-ironic01:53
openstackgerritAnnie Lezil proposed openstack/ironic-inspector master: Remove translation of log messages Part-2  https://review.openstack.org/44822101:53
openstackgerritklyang proposed openstack/ironic master: Fix some api field lists in docstrings  https://review.openstack.org/45050301:59
*** rloo has quit IRC02:07
*** gcb has joined #openstack-ironic02:07
*** crushil has joined #openstack-ironic02:07
*** mtanino_ has quit IRC02:12
*** oomichi has quit IRC02:28
*** oomichi has joined #openstack-ironic02:32
*** jaganathan has joined #openstack-ironic02:33
*** oomichi has quit IRC02:39
*** oomichi has joined #openstack-ironic02:42
openstackgerritDao Cong Tien proposed openstack/ironic master: Make terminal timeout value configurable  https://review.openstack.org/44913402:43
*** fragatina has quit IRC02:44
*** litao has joined #openstack-ironic02:52
*** MarginHu has quit IRC02:53
*** UForgotten_ has quit IRC03:01
*** UForgotten_ has joined #openstack-ironic03:01
*** UForgotten_ is now known as UForgotten03:01
*** dimak has quit IRC03:20
*** dimak has joined #openstack-ironic03:21
*** g3ek has quit IRC03:27
*** SerenaFe_ has quit IRC03:30
*** links has joined #openstack-ironic03:36
*** g3ek has joined #openstack-ironic03:36
*** crushil has quit IRC03:39
*** jkilpatr has quit IRC03:50
*** joanna has joined #openstack-ironic03:51
*** aparnav has joined #openstack-ironic03:52
*** tovin07 has quit IRC03:53
*** zhurong has quit IRC03:57
*** sasha2 has joined #openstack-ironic04:01
*** aparna has joined #openstack-ironic04:05
*** nmathew has joined #openstack-ironic04:06
*** aparnav has quit IRC04:08
*** vmud213 has joined #openstack-ironic04:12
*** fragatina has joined #openstack-ironic04:13
*** aparna has quit IRC04:17
*** fragatina has quit IRC04:17
*** jaganathan has quit IRC04:23
*** SerenaFeng has joined #openstack-ironic04:32
*** jaganathan has joined #openstack-ironic04:35
*** fragatina has joined #openstack-ironic04:40
*** fragatina has quit IRC04:42
*** fragatina has joined #openstack-ironic04:42
*** SerenaFeng has quit IRC04:55
*** zhurong has joined #openstack-ironic04:55
*** aparnav has joined #openstack-ironic05:03
*** makowals_ has joined #openstack-ironic05:04
*** aNuposic has quit IRC05:05
*** sasha2 has quit IRC05:07
*** makowals has quit IRC05:07
*** makowals has joined #openstack-ironic05:09
*** makowals_ has quit IRC05:11
*** aparnav has quit IRC05:17
*** aparnav has joined #openstack-ironic05:20
*** vmud213 has quit IRC05:27
*** vmud213 has joined #openstack-ironic05:28
openstackgerritklyang proposed openstack/ironic master: Add Error Codes to ironic api references  https://review.openstack.org/44949505:32
*** SerenaFeng has joined #openstack-ironic05:34
*** vishwanathj has quit IRC05:35
*** rcernin has joined #openstack-ironic05:38
*** vikrant has joined #openstack-ironic05:39
*** garyk has joined #openstack-ironic05:54
*** ChubYann has quit IRC05:54
*** aparnav has quit IRC06:09
*** rbartal has joined #openstack-ironic06:20
*** mjura has joined #openstack-ironic06:20
*** yogi has joined #openstack-ironic06:21
*** niks has joined #openstack-ironic06:27
niksFailed to set boot device to PXE: Driver fake_pxe does not support management (disabled or not implemented).06:31
niksany idea06:31
vsaienk0vsaienk006:39
vsaienk0Morning Ironic'ers!06:40
tuanluongMorning Ironic'ers!, vsaienk0 niks06:44
openstackgerrithnyang proposed openstack/ironic master: Add Error Codes to ironic api references  https://review.openstack.org/44949506:44
niksMorning06:44
niksany idea on : Failed to set boot device to PXE: Driver fake_pxe does not support management (disabled or not implemented).06:44
niksi am trying to configure rdo10 using director with vms06:45
niksand using fake_pxe06:45
niksas the driver06:45
*** SerenaFeng has quit IRC06:47
nikshttp://paste.openstack.org/show/604423/06:47
nikstuanluong: any idea06:47
openstackgerritVasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic  https://review.openstack.org/44071906:48
*** oomichi has quit IRC06:48
*** sambetts|afk has quit IRC06:48
tuanluongniks, do you deloy baremental on hardware or VM06:49
*** oomichi has joined #openstack-ironic06:52
*** tovin07 has joined #openstack-ironic06:52
nikstuanluong:  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 compute06:52
*** sambetts_ has joined #openstack-ironic06:52
nikstuanluong:  it is just for POC06:52
niks i do not have enough hardware now06:53
tuanluongniks, I suggest you should try https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack06:54
tuanluongniks, you should try agent_ipmitool instead of fake_pxe because it often use for testing06:56
*** mtreinish has quit IRC06:57
*** mtreinish has joined #openstack-ironic06:58
*** hshiina has joined #openstack-ironic06:58
*** amotoki has joined #openstack-ironic07:13
*** pcaruana has joined #openstack-ironic07:13
*** tesseract has joined #openstack-ironic07:18
*** amotoki has quit IRC07:18
*** aarefiev_afk is now known as aarefiev07:24
aarefievmorning all!07:24
*** jaosorior has joined #openstack-ironic07:29
*** rcernin has quit IRC07:29
*** SerenaFeng has joined #openstack-ironic07:31
*** jaganathan is now known as jaganathan_lunch07:34
*** makowals has quit IRC07:37
*** jaganathan_lunch is now known as jaganathan07:44
*** amarao has joined #openstack-ironic07:45
*** rcernin has joined #openstack-ironic07:47
*** SerenaFeng has quit IRC07:49
*** makowals has joined #openstack-ironic07:49
*** amoralej|off is now known as amoralej07:53
*** racedo has joined #openstack-ironic07:55
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
openstackgerritMerged openstack/bifrost master: Trivial spacing fix.  https://review.openstack.org/45033808:01
*** SerenaFeng has joined #openstack-ironic08:03
*** rcernin has quit IRC08:06
*** rcernin has joined #openstack-ironic08:07
*** rcernin has quit IRC08:10
*** rcernin has joined #openstack-ironic08:10
*** amotoki has joined #openstack-ironic08:14
*** ccamacho has joined #openstack-ironic08:14
*** ralonsoh has joined #openstack-ironic08:16
*** fdegir has joined #openstack-ironic08:16
openstackgerritVasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity  https://review.openstack.org/42512308:16
*** amotoki has quit IRC08:19
*** mgoddard has joined #openstack-ironic08:20
*** derekh has joined #openstack-ironic08:34
*** e0ne has joined #openstack-ironic08:35
*** lucas-afk is now known as lucasagomes08:35
lucasagomesmorning all08:35
fdegirhi08:36
fdegirwe recently started seeing issues with ironic-dbsync08:36
fdegirhere is the log to most recent attempt08:37
fdegirhttps://build.opnfv.org/ci/job/xci-provision-virtual-xenial-daily-ocata/3/console08:37
fdegiranyone is aware of any issues?08:37
*** aparnav has joined #openstack-ironic08:40
*** rcernin has quit IRC08:46
*** rcernin has joined #openstack-ironic08:46
*** rcernin_ has joined #openstack-ironic08:48
*** g3ek has quit IRC08:48
*** aparnav has quit IRC08:49
*** aparnav has joined #openstack-ironic08:49
*** g3ek has joined #openstack-ironic08:51
*** aparna has joined #openstack-ironic08:52
*** aparnav has quit IRC08:52
*** tovin07 has quit IRC08:55
thomas_oneillfdegir: yes I'm getting the same thing actually08:58
thomas_oneillfdegir: I don't suppose you've tried cleaning up and then re-installing using the install.yaml playbook?09:01
thomas_oneillI'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 properly09:01
fdegirthomas_oneill: did that but didn't work09:01
thomas_oneillthe 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
fdegirand it seems it hits both master and ocata09:02
thomas_oneilldang09:02
*** aparna has quit IRC09:03
thomas_oneillfdegir: database completely destroyed and re-installed, and ironic.conf completely removed and re-generated?09:12
*** amotoki has joined #openstack-ironic09:15
fdegirthomas_oneill: yes, we use bifrost which does ironic stuff09:15
fdegirthomas_oneill: and we wipe the database before we run bifrost09:16
thomas_oneillyeah I'm using bifrost too...09:16
*** Serlex has joined #openstack-ironic09:16
fdegirthomas_oneill: this is what we do for cleanup09:17
fdegirthomas_oneill: https://gerrit.opnfv.org/gerrit/gitweb?p=releng.git;a=blob;f=prototypes/bifrost/scripts/destroy-env.sh;hb=HEAD#l3509:17
*** jaganathan has quit IRC09:18
*** amotoki has quit IRC09:19
*** yatinkarel has quit IRC09:20
openstackgerritNisha Agarwal proposed openstack/ironic master: Fixes iLO drivers third party CI  https://review.openstack.org/45067009:20
*** milan has joined #openstack-ironic09:25
milanmorning Ironic! :)09:25
thomas_oneillfdegir: thanks, so you drop the database using the drop command... wonder if the database could be destroyed more destructively09:27
fdegirthomas_oneill: I'm onto something09:28
fdegirthomas_oneill: will come back once I try this09:28
thomas_oneillok :D09:29
*** jaganathan has joined #openstack-ironic09:30
*** yatinkarel has joined #openstack-ironic09:33
*** fragatina has quit IRC09:37
fdegirthomas_oneill: most likely a false hope :/09:38
*** niks has quit IRC09:42
openstackgerritFatih Degirmenci proposed openstack/bifrost master: Set database pool_size in ironic.conf and inspector.conf templates  https://review.openstack.org/45068109:48
fdegirthomas_oneill: ^09:48
thomas_oneilloh nice, ty09:48
vdrokgood morning ironic and all!09:50
*** jaganathan has quit IRC09:50
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Trivial: don't create unused temporary directory  https://review.openstack.org/45068309:53
milanmorning vdrok! :)09:54
vdrokhey milan :)09:54
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Trivial: don't create unused temporary directory  https://review.openstack.org/45068309:54
fdegirthomas_oneill: our verification jobs are passing that point where it fails09:58
lucasagomesvdrok, milan morning!09:59
fdegirthomas_oneill: so the patch will hopefully be merged soon09:59
lucasagomesfolks, 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
openstackgerritFatih Degirmenci proposed openstack/bifrost stable/ocata: Set database pool_size in ironic.conf and inspector.conf templates  https://review.openstack.org/45068610:00
lucasagomesjroll, dtantsur ^10:00
milanmorning lucasagomes! :)10:01
* milan runs lunch :P10:01
*** hshiina has quit IRC10:02
*** yatinkarel has quit IRC10:02
*** mkrai has quit IRC10:04
*** retr0h has quit IRC10:04
*** csmart has quit IRC10:04
*** snecklifter has joined #openstack-ironic10:05
*** jaganathan has joined #openstack-ironic10:08
*** phuongnh has quit IRC10:09
*** tuanluong has quit IRC10:09
*** athomas has quit IRC10:10
*** athomas has joined #openstack-ironic10:11
openstackgerritAparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers  https://review.openstack.org/42257210:11
*** tuanluong has joined #openstack-ironic10:13
*** MattMan has quit IRC10:14
vdrokmorning lucasagomes , oh , cool10:14
*** MattMan has joined #openstack-ironic10:14
*** mjura has quit IRC10:15
lucasagomesyeah, 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
lucasagomeswhich sucks10:16
*** amotoki has joined #openstack-ironic10:16
*** athomas has quit IRC10:16
*** yatinkarel has joined #openstack-ironic10:16
*** SerenaFeng has quit IRC10:17
*** SerenaFeng has joined #openstack-ironic10:20
*** amotoki has quit IRC10:20
*** zhurong has quit IRC10:20
*** mkrai has joined #openstack-ironic10:21
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Deduplicate method in OSC functional tests  https://review.openstack.org/45070210:21
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240210:24
vdroklucasagomes: 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-ironic10:28
lucasagomesvdrok, oh lemme check10:28
lucasagomesvdrok, that's cool, does the g-r check it's dependencies and all ?10:30
vdroklucasagomes: dunno, it appears that fuel-agent does not have a requirements job10:31
lucasagomesand I assume they can be installed separately right ?10:31
lucasagomesoh10:31
lucasagomesI see10:31
vdrokyup, if g-r cannot understand it, better to have a separate project :) also asked yuriyz , it seems to bring troubles with versioning in pbr10:35
*** aparnav has joined #openstack-ironic10:35
*** cdearborn has joined #openstack-ironic10:36
openstackgerritChris Smart proposed openstack/ironic master: WIP: test buildroot IPA image  https://review.openstack.org/44576310:38
*** aparnav has quit IRC10:39
*** mgould|afk is now known as mgould10:39
mgouldhi everyone!10:39
*** aparnav has joined #openstack-ironic10:39
vdrokmorning mgould10:40
*** aparnav has quit IRC10:40
*** aparna has joined #openstack-ironic10:40
mgouldmorning vdrok lucasagomes milan and everyone else10:41
mgouldlucasagomes: nooooooo not more repos...10:41
*** fragatina has joined #openstack-ironic10:41
mgouldhttps://danluu.com/monorepo/10:41
*** athomas has joined #openstack-ironic10:41
mgouldOTOH, we're kinda locked into the "zillions of repos" (anti)pattern now10:42
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/45071110:42
*** appu103 has joined #openstack-ironic10:43
*** aparna has quit IRC10:43
*** aparna has joined #openstack-ironic10:49
*** appu103 has quit IRC10:49
lucasagomesmgould, heh yeah I was trying to avoid it but... How to deal with the dependencies on a single repo ?10:53
lucasagomesalso installation, for example in devstack what I'm doing now is to clone the sushy repo and then run the simulator10:54
lucasagomesbut that's a bit dangerous cause it will always run from master10:54
*** vills has joined #openstack-ironic10:55
*** alexpilotti has quit IRC10:55
* mgould nods10:57
*** jkilpatr has joined #openstack-ironic11:00
*** jjung has joined #openstack-ironic11:01
*** sambetts_ is now known as sambetts11:04
sambettsMorning all11:05
TheJuliaGood morning11:05
mgouldmorning sambetts TheJulia11:05
lucasagomesTheJulia, sambetts morning o/11:06
sambettsHey TheJulia, mgould11:06
*** litao has quit IRC11:09
*** oomichi has quit IRC11:09
*** athomas has quit IRC11:09
openstackgerritJaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector  https://review.openstack.org/42472911:10
*** oomichi has joined #openstack-ironic11:13
*** fragatina has quit IRC11:16
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240211:16
*** aparna has quit IRC11:17
*** amotoki has joined #openstack-ironic11:17
TheJuliathomas_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-ironic11:17
*** athomas has joined #openstack-ironic11:17
vsaienk0sambetts: do you think the sentence should be rephrased? https://review.openstack.org/#/c/415003/12/specs/approved/dynamic-portgroups.rst@1711:20
*** amotoki has quit IRC11:21
thomas_oneillTheJulia: yeah I saw, thanksf or the feedbcak11:25
*** zhurong has quit IRC11:25
TheJuliavsaienk0: By chance are you working bug 16768809 ?11:25
thomas_oneilloops 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 lol11:25
*** aparna has quit IRC11:26
*** appu103 has joined #openstack-ironic11:26
vsaienk0TheJulia: yeah, I'm and vdrok looking onto it https://bugs.launchpad.net/ironic/+bug/167680911:26
openstackLaunchpad bug 1676809 in Ironic "ironic-dbsync failed on CI" [Undecided,Confirmed]11:26
TheJuliavsaienk0: okay, I just confirmed it since it is being seen on bifrost's OPNFV ci as well11:27
vdrokmorning sambetts TheJulia11:27
TheJuliamarking 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-hungry11:28
*** openstackgerrit has quit IRC11:33
*** nmathew has quit IRC11:33
*** mat128 has joined #openstack-ironic11:35
*** aparna has joined #openstack-ironic11:39
*** appu103 has quit IRC11:39
*** rbartal_ has joined #openstack-ironic11:39
*** racedo has quit IRC11:42
*** appu103 has joined #openstack-ironic11:42
*** aparna has quit IRC11:42
*** openstackgerrit has joined #openstack-ironic11:42
openstackgerritVasyl Saienko proposed openstack/ironic master: Don't pass sqlite_db in db_options.set_defaults()  https://review.openstack.org/45073311:42
vsaienk0TheJulia: dtantsur vdrok and all other ironic cores patch to unblock ci https://review.openstack.org/45073311:43
TheJuliavsaienk0: Awesome, that looks... fun :)11:44
garykmgould: maybe you can help me for a sec. I am getting "SSH connect failed: Invalid private key"11:44
*** yogi has quit IRC11:45
*** yogi has joined #openstack-ironic11:45
TheJuliaI guess CI will tell us for sure, but it looks good to me :)11:45
garyki am trying to use pxe_ssh. maybe using wrong information here11:46
*** racedo has joined #openstack-ironic11:47
*** aparna has joined #openstack-ironic11:54
*** appu103 has quit IRC11:54
openstackgerritVasyl Saienko proposed openstack/ironic-inspector master: Don't pass sqlite_db in db_options.set_defaults()  https://review.openstack.org/45073711:55
*** jcoufal has joined #openstack-ironic11:55
*** bfournie has quit IRC11:55
milanaarefiev, hey, wanted to have another philosophical discussion w/ you ;) w/r the multiple filtering backends11:56
*** jaypipes has joined #openstack-ironic11:56
aarefievmilan: hey11:56
milanaarefiev, 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
milanaarefiev, so maybe we should amend the spec w/ [<mac>, <mac>, ...]11:57
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add names to portgroup api tempest tests  https://review.openstack.org/45073911:58
milanaarefiev, 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 remove11:58
milanaarefiev, wdyt?11:58
aarefievmilan: yeah, sounds good to me11:59
milanaarefiev, also a tricky thing w/ IB11:59
milanwe need to accumulate the MAC-IB mapping from all the nodes in HA env11:59
milaninto that table11:59
milan*table == node_id, mac mapping11:59
aarefievmilan: why you don't get macs from node_info object12:00
milanaarefiev, hmmmm12:00
* milan looks12:01
*** SerenaFe_ has joined #openstack-ironic12:03
*** SerenaFe_ has quit IRC12:03
milanaarefiev, 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#L37412:04
milanaarefiev, or you mean the introspection data?12:05
*** SerenaFeng has quit IRC12:05
aarefievmilan: no, I mean our cache12:05
milanaarefiev, but we don't store the mac in the DB, do we?12:06
aarefievmilan: I don't know :)12:06
aarefievyeah, you are right12:07
milanaarefiev, right, we create them and delegate to ironic12:07
milanonce we drop a node cache item, we poll ironic to fetch12:08
milanthe ports again12:08
* milan wonders whether it's sane12:08
*** SerenaFeng has joined #openstack-ironic12:08
milanaarefiev, 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
aarefievmilan: maybe it make sense if new port was added12:09
aarefievmilan: removing node on introspection ?12:09
* milan looks whether we don't have this as a bug12:10
milanaarefiev, after12:10
aarefievmilan: if you remove node, inspector will drop it from cache12:10
aarefiev?12:10
aarefievw/o update filters12:11
milanaarefiev, yup but will it drop the mac addresses from iptables?12:11
*** vills has quit IRC12:11
milanmaybe we've got an iptables rules leak ;)12:11
*** tuanluong has quit IRC12:11
aarefievmilan: hmm, sounds like we should store macs in db12:11
milanaarefiev, I think we might have a bug even right now ;)12:12
aarefievok :)12:12
*** chlong has joined #openstack-ironic12:17
milanaarefiev, 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#L18712:17
* milan needs to think this through 12:17
*** appu103 has joined #openstack-ironic12:18
*** aparna has quit IRC12:18
*** vmud213 has quit IRC12:20
aarefievmilan: yeah, could be optimized12:21
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Fix disk size check output handling  https://review.openstack.org/45075412:21
*** appu103 has quit IRC12:22
*** hshiina has joined #openstack-ironic12:22
*** appu103 has joined #openstack-ironic12:22
milanaarefiev, or if we want to stick to the current implementation, we have to remove the remove_node_ids/macs from the driver spec12:22
milanaarefiev, 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
aarefievmilan: I guess filter's behavior will be driver specific on remove12:24
*** dprince has joined #openstack-ironic12:24
milanaarefiev, right. I've anyways not specified that the whitelist&blacklist logic was additive12:25
milanas we have the remove_node_ids there12:25
milan:-/12:25
*** gcb has quit IRC12:25
*** jaganathan has quit IRC12:26
*** gcb has joined #openstack-ironic12:26
*** Goneri has joined #openstack-ironic12:27
aarefievmilan: so if node was removed and we cache macs we call filter.remove(nodes)12:27
*** bfournie has joined #openstack-ironic12:27
aarefievand drop nodes from cache after all12:27
aarefievright?12:27
milanaarefiev, correct12:27
milanaarefiev, or if we don't introduce the remove logic, we have to have an absolute blacklist/whitelist parameter12:28
*** appu103 has quit IRC12:28
*** lucas-hungry is now known as lucasagomes12:28
*** appu103 has joined #openstack-ironic12:28
*** SerenaFeng has quit IRC12:28
milanso filter.blacklist == node_list after filter.blacklist_nodes(node_list)12:29
milanditto w/ whitelist12:29
milanI guess both ways will have drawbacks12:29
milansecond way (absolute lists) is in addition always working with all nodes currently tracked by Ironic, which may be bad in bigger deployments12:30
aarefievmilan: what do you mean by `working with all nodes`?12:31
milanaarefiev, btw we also agreed to keep the node cache in inspector in sync w/ ironic w/o purging items12:32
aarefievyeah12:32
milanaarefiev, ironic.node_list() or ironic.port_list()12:32
milanis called all the time12:32
milanand then all the items are always passed on to the filter12:32
openstackgerritShivanand Tendulker proposed openstack/ironic master: Call clean_up_instance() during node teardown for Agent deploy  https://review.openstack.org/45014612:33
*** jjung has quit IRC12:33
aarefievmilan: we could pass only nodes to be updated12:33
milanaarefiev, that was my original idea and it requires the remove_node_ids method to be consistent12:34
aarefievmilan: so the problem is how iptables driver works now12:35
milanaarefiev, correct but also you can't get mac address from ironic once the node was removed12:35
aarefievmilan: if we don't cache it12:36
milanaarefiev, exactly so we either cache or always rebuild as we do currently12:36
aarefievwhat if some driver requires macs for clean pu12:36
milanaarefiev, I guess almost all the nodes will work with MACs12:36
* milan can't imagine other criteria to blacklist on as we're talking about DHCP in the end12:37
aarefievmilan: we could hide rebuild chain in iptables impl for now12:38
milanaarefiev, so keep the remove_node_ids in the spec?12:39
aarefievmilan: spec is only words :)12:40
*** appu103 has quit IRC12:40
*** appu103 has joined #openstack-ironic12:40
milanaarefiev, unless someone (ehhhm dtantsur ) -2 it based on a spec :P12:40
*** nmathew has joined #openstack-ironic12:40
*** nmathew has quit IRC12:41
aarefievmilan: we always could sync spec with code12:41
*** trown|outtypewww is now known as trown12:41
*** csmart has joined #openstack-ironic12:41
*** retr0h has joined #openstack-ironic12:41
aarefievwe could miss some details on design stage12:42
milanaarefiev, 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) :D12:42
milanaarefiev, right, so my brain dump so far:12:43
milan* all the drivers will use MAC12:43
milan* we have issues mapping MAC to node ids12:43
milan* we'd better use the remove_node_ids semantics (additive lists)12:43
aarefievmilan: you will pass result node list, why it will calculate twice12:44
milan== let's cache MACs in inspector and change the interface to use MACs instead of node_ids?12:44
milanaarefiev, yeah, if it's calculated by inspector rather than by the driver then once12:45
aarefievmilan: I'm also wondering if some driver requires some other id than macs12:45
aarefievport uuid by neutron12:45
milanaarefiev, right :-/ port uuids make sense12:46
milanso a cache would have to contain that too?12:46
milanit's getting hairy I guess12:46
milannode_id x port_id x MAC x optional IB id?12:47
jrolllucasagomes: \o/ for sushy-tools, +1 there12:47
aarefievmilan: I guess it would be some new port object12:48
lucasagomesjroll, o/ cheers!12:48
milanaarefiev, hmmm, yeah12:48
aarefievmilan: ok, I guess we should go step by step12:48
*** jjung has joined #openstack-ironic12:49
aarefievmilan: lets start from macs12:49
jrollmorning btw12:49
milanmorning jroll! :)12:49
aarefievmorning jroll12:50
milanaarefiev, also, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L729 worries me a bit12:50
milanwe keep the MACs in attributes until the introspection finishes12:51
milanhttps://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/node_cache.py#L29012:52
milanbut the purpose is node lookup so maybe it's OK to duplicate the data in a mac cache table12:54
aarefievmilan: it's temporary data, so i'd not worry a lot12:55
*** sbezverk has quit IRC12:55
*** appu103 has quit IRC12:55
*** appu103 has joined #openstack-ironic12:55
*** mat128 has quit IRC12:57
milanaarefiev, lemme maybe update the POC w/ a mac cache, we can update the spec later?12:58
*** mjura has joined #openstack-ironic12:58
aarefievmilan: we can update spec once code would be ready12:58
milanalright12:59
milanaarefiev, thx for the discussion :)12:59
aarefievnp :)12:59
*** appu103 has quit IRC12:59
*** appu103 has joined #openstack-ironic12:59
* milan .raise(CaffeineLevelsDepleted('productive discussion'))13:00
*** rbudden has joined #openstack-ironic13:03
*** aparna has joined #openstack-ironic13:04
*** appu103 has quit IRC13:04
*** sasha2 has joined #openstack-ironic13:05
*** gcb has quit IRC13:07
*** amoralej is now known as amoralej|lunch13:08
*** garyk has quit IRC13:12
*** baoli has joined #openstack-ironic13:13
*** baoli has quit IRC13:14
*** baoli has joined #openstack-ironic13:14
*** rbartal__ has joined #openstack-ironic13:16
*** yogi has quit IRC13:17
jrollTheJulia: 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 history13:18
*** amotoki has joined #openstack-ironic13:18
jrollTheJulia: I'm looking at https://docs.openstack.org/releasenotes/bifrost/unreleased.html fwiw13:18
TheJuliawhich would be master branch I guess13:19
jrollyeah, I assume so, though bugs in reno have proved me wrong in the past13:19
TheJuliajroll: I was just meaning release a cut of stable/ocata since we have a few nasty bug fixes in there13:20
jrollTheJulia: oh! gotcha13:20
TheJuliamaster branch can continue rolling for now :)13:20
jrollsounds good13:21
* TheJulia may or may not be plotting a major feature13:21
*** jkilpatr has quit IRC13:22
jrollwell this is odd. harumph.13:22
*** jkilpatr has joined #openstack-ironic13:22
jrollhttps://github.com/openstack/ironic-lib/blob/master/tox.ini#L813:22
jrollhttps://github.com/openstack/ironic-lib/blob/stable/ocata/tox.ini#L813:22
* jroll fixes13:22
*** links has quit IRC13:22
*** amotoki has quit IRC13:23
TheJuliajroll: You may want to review https://review.openstack.org/#/c/450733/13:23
TheJuliafunky13:23
openstackgerritJim Rollenhagen proposed openstack/ironic-lib master: Revert "Update UPPER_CONSTRAINTS_FILE for stable/ocata"  https://review.openstack.org/45077713:24
openstackgerritJim Rollenhagen proposed openstack/ironic-lib stable/ocata: Update UPPER_CONSTRAINTS_FILE for stable/ocata  https://review.openstack.org/45077813:24
*** jvgrant_ has quit IRC13:24
jroll^ could I have reviews on those please? :)13:24
*** jvgrant_ has joined #openstack-ironic13:24
jrollJayF: ^ one is stable13:24
jrollTheJulia: +A, will wait for that one for ironic release13:25
TheJuliajroll: awesome, since otherwise everything is broken without it at the moment ;)13:27
jrollTheJulia: just db migrations, right?13:27
jroll"just"13:27
openstackgerritVasyl Saienko proposed openstack/ironic-inspector master: Don't pass sqlite_db in db_options.set_defaults()  https://review.openstack.org/45073713:29
TheJuliaI think creations as well, but I didn't dig too deeply into it since vsaienk0 was looking at it13:29
jrollsure13:29
*** rloo has joined #openstack-ironic13:31
*** crushil has joined #openstack-ironic13:31
*** alexpilotti has joined #openstack-ironic13:31
*** amotoki has joined #openstack-ironic13:34
jrollTheJulia: ironic-ui has network_interface and manual cleaning support, shall we release it as well?13:36
TheJuliaThat does sound like a very good idea :)13:37
jrollalright, will do13:38
* jroll has things lined up but waiting on some patches13:39
TheJuliastory of our lives? :)13:39
jrollheh13:39
*** causten has quit IRC13:40
*** sbezverk has joined #openstack-ironic13:40
openstackgerritMerged openstack/ironic-lib master: Revert "Update UPPER_CONSTRAINTS_FILE for stable/ocata"  https://review.openstack.org/45077713:42
jrolloh, that oslo.db bug killed our entire gate, fun!13:43
openstackgerritMiles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration  https://review.openstack.org/44708513:43
jrollfolks, please don't recheck or approve things until this merges: https://review.openstack.org/#/c/450733/13:43
jlvillaljroll, Is that in the whiteboard?13:45
* jroll looks13:45
jrolljlvillal: the bug was, but status was still 'sporadic'13:46
jlvillaljroll, Thanks13:46
* jroll fixed13:46
rloomorning all, jlvillal, jroll, TheJulia13:46
jlvillalGood morning :)13:46
jrollmorning rloo13:47
mariojvmorning everyone o/13:47
rloojroll: wrt the releases waiting on patches -- do you need any reviews, or just waiting to merge?13:47
jrollrloo: all patches are +W13:47
rloojlvillal: you working EDT hours starting today? :)13:47
rloojroll: cool.13:48
jlvillalrloo, I have to take my wife to the airport today. I figure that will take 2-4 hours.13:48
jlvillalrloo, So I started work early :)13:48
rloojlvillal: :)13:48
mgouldmorning rloo mariojv13:49
*** hoangcx has joined #openstack-ironic13:53
mgouldI'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 deleted13:53
*** hamzy has quit IRC13:54
mgouldmore output: http://paste.openstack.org/show/604484/13:55
*** krtaylor has quit IRC13:56
jlvillalmgould, I'd be more worried about the unable to bind messages.13:58
*** rbartal__ has quit IRC13:58
*** krtaylor has joined #openstack-ironic14:00
*** ElCoyote_ has joined #openstack-ironic14:02
TheJuliamgould: 447085 needs a minor tweak to the release note.  It fails validation presently :(14:02
*** amoralej|lunch is now known as amoralej14:03
*** aparna has quit IRC14:03
*** aparnav has joined #openstack-ironic14:04
mgouldjlvillal: according to netstat, there's nothing listening on those ports :-(14:04
jlvillalmgould, I don't know. Seems like Apache is saying there is something. :(14:05
mgouldTheJulia: thanks14:05
jrollapache loves to restart slowly14:05
jrollI think devstack even does a stop; sleep X; start14:05
jrollon ubuntu anyway14:05
* jlvillal is sure mgould only uses Ubuntu :P14:06
* mgould drags desks around to debug whatever's stopping him from sconnecting to his other devstack VM14:06
mgould[A[B14:06
mgouldjlvillal: this VM's centos14:06
*** _milan_ has joined #openstack-ironic14:07
jlvillalmgould, 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 home14:08
mgouldspeaking of which: I've handed in my notice at Red Hat, so I won't be around after April 12th14:08
jlvillalmgould, Inconcievable!14:08
*** milan has quit IRC14:08
jlvillalmgould, Oh :( Does that mean no more Ironic for you???14:09
TheJuliamgould: Sad to hear, but good luck on your next venture!14:09
* jlvillal sad..14:09
mgouldTheJulia, jlvillal: thanks!14:09
* TheJulia sad too14:09
mariojvmgould: :( gl though14:09
mgouldmariojv: thanks!14:10
jrollmgould: sad for us, congrats to you :)14:10
jlvillalmgould, Willing to share yet what you will be doing next? :)14:10
* mgould is going to do machine-learning at a local security startup14:10
mgouldhttp://zonefox.com14:10
jlvillalCool :)14:10
jrollnice14:11
mgouldI'll miss you all, but remote work was doing my head in14:11
vdrokmorning jroll rloo and jlvillal  !14:11
openstackgerritThomas O'Neill proposed openstack/bifrost master: Abstraction of distribution variables and cleanup role.  https://review.openstack.org/45034714:11
jlvillalmgould, Oh you are changing jobs because you don't want to work remote? I can kind of understand.14:11
rloohi mgould, vdrok!14:12
vdrokoh, mgould , good luck with your new project! :)14:12
jroll\o vdrok14:12
mgouldhi rloo14:12
mgouldjroll, vdrok: thanks!14:12
openstackgerritMario Villaplana proposed openstack/python-ironicclient master: Log warning when API version unspecified  https://review.openstack.org/44215314:12
*** jaosorior has quit IRC14:12
TheJuliamgould: very cool!  Have fun with the new job.14:12
jlvillalmgould, 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
rloooh mgould, congrats!!! remote work isn't for everyone; glad you realized what works for you :)14:13
*** tuan_luong has joined #openstack-ironic14:13
*** tuanla_ has joined #openstack-ironic14:14
jlvillalHi vdrok :)14:14
thomas_oneillTheJulia: hey you mentioned about knowing operators who have (already?) "split the roles apart" - are they on IRC/gerrit?14:14
mgouldrloo, TheJulia: thanks!14:15
vdrokjroll: 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 still14:15
vdrok*new versions of objects14:15
jrollvdrok: 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
vdrokyup14:16
TheJuliathomas_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" thing14:16
TheJuliathomas_oneill: not broken into multiple git repos14:16
vdrokalso, the changes we need to the job appear to be not so straight-forward, as discussed with vsaienk014:16
mgouldyay, I can get into my other virthost!14:17
mgouldturns out our subnet changed this morning before I got in14:17
mgould*obviously* :-/14:17
*** rbartal has quit IRC14:17
jrollvdrok: :( let me know if there's any way I can help14:17
*** rbartal_ has quit IRC14:17
vsaienk0vdrok: jroll so easiest way is to seems not restart ir-api on primary node after upgrade, it should work... but need to check14:18
*** causten has joined #openstack-ironic14:18
jrollvsaienk0: what does moving the api to the subnode require? changing service catalog/configs, anything else?14:19
vdrokjroll: we'll need to skip enrolling the nodes, and do that during subnode setup instead14:20
jrollaha, I see14:20
vsaienk0jroll: yeah, service catalog/configs, and we should enroll ironic nodes when subnode is ready14:20
* jroll needs to read more about how multinode stuff is ordered and such14:22
* jroll wonders if we could do a separate "install and run" phase and "configure resources" phase14:23
jrollthat could be longer term though :)14:23
*** Goneri has quit IRC14:24
openstackgerritMiles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration  https://review.openstack.org/44708514:26
*** hamzy has joined #openstack-ironic14:27
* mgould plays the game called "can I get all my in-progress patches merged before I leave?"14:27
*** aparnav has quit IRC14:27
TheJuliaheh14:30
*** Goneri has joined #openstack-ironic14:30
mgouldif 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-ironic14:33
TheJuliacinerama: ^^^14:34
vsaienk0mgould: we allow more characters in logical names here https://review.openstack.org/#/c/447085/8/ironic_ui/api/ironic_rest_api.py@23014:35
TheJuliafun :(14:37
TheJuliaThat is going to be a number of lines changed14:37
*** crushil has quit IRC14:45
*** vikrant has quit IRC14:48
NobodyCamgood morning Ironic'ers14:52
vdrokgood morning NobodyCam :)14:55
NobodyCamMorning vdrok :)14:55
*** jtaryma has joined #openstack-ironic14:58
openstackgerritVasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test  https://review.openstack.org/42977015:00
*** hoangcx has quit IRC15:00
*** crushil has joined #openstack-ironic15:00
openstackgerritMichael Turek proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd  https://review.openstack.org/44764215:01
openstackgerritMichael Turek proposed openstack/ironic master: Remove logging translation calls from ironic.common  https://review.openstack.org/44808915:01
openstackgerritThomas O'Neill proposed openstack/bifrost master: Abstraction of distribution variables and cleanup role.  https://review.openstack.org/45034715:01
mjturekwhoops vgadiraj, I rebased your patch... sorry about that!15:02
*** jtaryma has quit IRC15:03
openstackgerritThomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role.  https://review.openstack.org/45034715:03
thomas_oneillhow's the test coverage of the keystone install in bifrost?15:04
*** jjung has quit IRC15:08
*** dyasny has joined #openstack-ironic15:08
*** mjura has quit IRC15:08
thomas_oneillhmmm '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.txt15:09
*** rpioso has joined #openstack-ironic15:09
TheJuliathomas_oneill: there is not a specific job that exercises it yet afaik15:10
thomas_oneilloh it's failing at different parts but with this same failure in places. The ci server must be barfing or something, I presume15:10
* TheJulia wonders if someone is clicking the abort button15:10
thomas_oneillit says ABORTED on the 3 tests... where is said abort button? :o15:11
TheJuliain Jenkins console15:11
mgouldvsaienk0: thanks!15:12
TheJuliafor an authenticated and authorized user most likely15:12
* mgould has got his devstack VM to fail in a different way!15:12
mgouldthere were two "Listen 5000" lines in /etc/httpd15:12
mgoulddid we move that setting?15:12
TheJuliathomas_oneill: fwiw, bifrost's jobs on opnfv ci are broken pending a fix in ironic15:13
mgouldanyway, yum remove httpd && rm -rf /etc/httpd made the error go away15:13
thomas_oneilloh yeah I saw about that this morning15:13
TheJuliawheeeee 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_95815:15
TheJuliahttp://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 recheck15:19
*** jjung has joined #openstack-ironic15:19
*** jtaryma has joined #openstack-ironic15:20
jrollindeed, I have a feeling something is just slow there judging by the sleep 5 before the command that returns RTNETLINK answers: Network is unreachable15:20
*** sbezverk has quit IRC15:20
*** _milan_ has quit IRC15:24
*** aarefiev is now known as aarefiev_afk15:24
TheJuliaYeah, I wonder if it could be a little longer or retry once or something. :(15:25
jrollor poll15:26
*** sayantan_ has joined #openstack-ironic15:28
*** baoli has quit IRC15:29
*** rcernin has quit IRC15:31
*** rcernin_ has quit IRC15:31
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update  https://review.openstack.org/45085515:32
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing converting BaseDriver.*_interfaces to tuples  https://review.openstack.org/45085615:32
jlvillaljroll, ^^^ what I mentioned before15:33
jrollcool15:34
jrollthanks!15:34
*** Serlex has quit IRC15:35
jrolljlvillal: do you have a link handy to the patch that fixed that memory bloat bug btw?15:35
jrolldon't go searching, I can do that, just curious :)15:36
*** alexpilotti has quit IRC15:36
* jroll wants to backport that15:36
jlvillaljroll: It is the one I depend on15:36
jlvillaldirectly15:36
jrollah, duh15:36
jrollI thought it had merged but realized I'm waiting for merge to release :D15:36
jlvillaljroll, https://review.openstack.org/#/c/449577/15:36
jrollyep15:36
jrollthanks!15:36
jlvillalYou're welcome15:36
*** alexpilotti has joined #openstack-ironic15:39
* jroll steps out for lunch/errands15:39
*** amarao has quit IRC15:41
*** alexpilotti has quit IRC15:46
*** e0ne has quit IRC15:46
mgouldapparently my pool_size is configured to "ironic.sqlite"15:51
mgouldyeah, I can see how that wouldn't work15:52
*** fragatina has joined #openstack-ironic15:56
*** crushil has quit IRC16:06
vdrokmgould: fix for that is in merge queue16:08
*** bfournie has quit IRC16:12
mgouldvdrok: awesome!16:13
openstackgerritJoanna Taryma proposed openstack/ironic master: Add base cinder common interface  https://review.openstack.org/35562516:13
mgouldvdrok: in ironic?16:14
vdrokmgould: yup, https://review.openstack.org/45073316:15
mgouldexcellent16:15
*** fragatina has quit IRC16:15
*** alexpilotti has joined #openstack-ironic16:15
* mgould rebases his patch on top of it16:16
openstackgerritMiles Gould proposed openstack/ironic master: devstack: install python-dracclient if DRAC enabled  https://review.openstack.org/42532616:16
*** jaganathan has joined #openstack-ironic16:18
mgouldvsaienk0, TheJulia, thanks for reviews16:23
*** Nisha_Agarwal has joined #openstack-ironic16:23
JayFdtantsur: 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 IRC16:24
*** causten has quit IRC16:25
*** pcaruana has quit IRC16:26
*** causten has joined #openstack-ironic16:26
alezilmilan: thanks for reviewing the patch.16:26
*** baha has joined #openstack-ironic16:26
*** aNuposic has joined #openstack-ironic16:28
*** causten_ has joined #openstack-ironic16:29
*** causten has quit IRC16:30
openstackgerritMerged openstack/ironic-lib stable/ocata: Update UPPER_CONSTRAINTS_FILE for stable/ocata  https://review.openstack.org/45077816:31
JayFjroll: dtantsur: Stable patch that should be simple to land: https://review.openstack.org/#/c/448632/16:31
*** jaganathan has quit IRC16:33
*** chlong has quit IRC16:34
*** fragatina has joined #openstack-ironic16:34
*** causten_ has quit IRC16:39
jrollJayF: I'm +2 on that first one, it's necessary16:40
JayFjroll: that's what I thought, I am landing it, but wanted to check :)16:40
JayFjroll: it's weird to me that d-r is not required to be in g-r for reasons like that, but w/e16:40
jrollJayF: yeah, it's a mismatch because the g-r updates won't check that file, but we don't want to hard require d-r16:41
jrollpas-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
mgouldJayF: dtantsur is on PTO this week16:41
jrollthis is true16:41
JayFjroll: when you roll ironic releases, we might want to push stable releases too fwiw16:42
pas-hajroll: yep, I'm aware, just added that to see gates passing and will remove when requirements patches are merged16:42
jrollJayF: indeed16:42
jrollpas-ha: ok cool16:42
*** lucasagomes is now known as lucas-afk16:43
*** fragatina has quit IRC16:43
JayFmariojv: I think https://review.openstack.org/#/c/424723/6 is either going to make rescue harder or easier, I'm not 100% sure yet16:43
openstackgerritThomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role.  https://review.openstack.org/45088416:45
*** mjturek has quit IRC16:45
*** jaganathan has joined #openstack-ironic16:46
openstackgerritThomas O'Neill proposed openstack/bifrost master: Clean-up role and package requirements role.  https://review.openstack.org/45088416:47
*** chlong has joined #openstack-ironic16:49
thomas_oneillokay 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
mgouldthomas_oneill: o/16:50
*** jaganathan has quit IRC16:50
*** derekh has quit IRC16:50
*** Nisha_Agarwal has quit IRC16:51
*** trown is now known as trown|lunch16:51
*** causten has joined #openstack-ironic16:52
vdrokgood night everyone!17:01
mgouldvdrok: good night!17:01
*** mgoddard has quit IRC17:02
*** fragatina has joined #openstack-ironic17:03
*** jaganathan has joined #openstack-ironic17:03
*** dsneddon has quit IRC17:04
*** crushil has joined #openstack-ironic17:05
*** jtaryma has quit IRC17:06
*** tuanla_ has quit IRC17:07
*** tuan_luong has quit IRC17:07
*** mtanino has joined #openstack-ironic17:11
*** deadnull has joined #openstack-ironic17:12
*** baoli has joined #openstack-ironic17:12
*** mtanino has quit IRC17:13
*** jtaryma has joined #openstack-ironic17:15
*** baoli has quit IRC17:18
*** amoralej is now known as amoralej|off17:18
*** dsneddon has joined #openstack-ironic17:21
*** baoli has joined #openstack-ironic17:23
*** mgould is now known as mgould|afk17:27
*** jaganathan has quit IRC17:29
*** amotoki has quit IRC17:31
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing converting BaseDriver.*_interfaces to tuples  https://review.openstack.org/45085617:36
*** mgoddard has joined #openstack-ironic17:46
*** fragatina has quit IRC17:47
*** baoli has quit IRC17:47
*** fragatina has joined #openstack-ironic17:48
*** baoli has joined #openstack-ironic17:49
*** baoli has quit IRC17:50
*** fragatin_ has joined #openstack-ironic17:54
*** fragatina has quit IRC17:57
*** baoli has joined #openstack-ironic17:59
*** causten has quit IRC18:00
*** athomas has quit IRC18:02
*** trown|lunch is now known as trown18:03
*** tesseract has quit IRC18:06
*** ElCoyote_ has quit IRC18:07
*** causten has joined #openstack-ironic18:07
*** jkilpatr has quit IRC18:09
*** jkilpatr has joined #openstack-ironic18:09
*** ElCoyote_ has joined #openstack-ironic18:11
openstackgerritRuby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration  https://review.openstack.org/40855618:32
*** deadnull_ has joined #openstack-ironic18:48
*** deadnull has quit IRC18:49
*** choyj has joined #openstack-ironic18:50
*** vills has joined #openstack-ironic18:53
openstackgerritDebayan Ray proposed openstack/sushy master: Add System Processor/Memory info  https://review.openstack.org/44651018:55
*** jtaryma has quit IRC18:56
*** e0ne has joined #openstack-ironic18:58
openstackgerritMerged openstack/ironic stable/ocata: Pin version of proliantutils  https://review.openstack.org/44950418:59
*** ralonsoh has quit IRC19:00
mariojvTheJulia: hey, we have 2x+2 on the first in the series of rolling upgrade patches \o/ https://review.openstack.org/#/c/40749119:01
mariojvTheJulia: feel free to remove -2 whenever you feel that's procedurally ready19:01
openstackgerritMerged openstack/ironic master: Document PXE with Spanning Tree in troubleshooting FAQ  https://review.openstack.org/43420819:02
mariojvmaybe we ought to get +2s on all the other code in that patch series first though? dunno if it matters too much19:03
TheJuliagetting +2's on everything is going to drive people crazy19:04
TheJuliaand 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 now19:05
mariojvhaha fair, thx19:05
choyjHi @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
TheJuliachoyj: I believe it is baremetal, which is the service type in open stack for ironic19:10
openstackgerritDebayan Ray proposed openstack/sushy master: Add System Processor/Memory info  https://review.openstack.org/44651019:10
openstackgerritRuby Loo proposed openstack/ironic master: [doc] Update examples in devstack section  https://review.openstack.org/45093619:10
choyjok, thanks!19:11
*** bfournie has joined #openstack-ironic19:15
*** baoli has quit IRC19:15
*** bfournie has quit IRC19:15
*** bfournie has joined #openstack-ironic19:16
*** bfournie has quit IRC19:16
*** mgoddard has quit IRC19:21
soliosgchoyj: ironic-ui is located in the Admin pane > System > Ironic Bare Metal Provisioning19:24
soliosgchoyj: you need to restart the horizon server after copying _2200_ironic.py to the enabled directory19:25
openstackgerritVasyl Saienko proposed openstack/ironic master: [Devstack] cleanup upgrade settings  https://review.openstack.org/45094019:25
choyjThanks @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
openstackgerritRuby Loo proposed openstack/ironic master: [PoC] Add fake fields in Node object and RPC  https://review.openstack.org/41546219:32
*** racedo has quit IRC19:37
mariojvthis gate failure (on the patch that tries to fix the dbsync problem) is really weird: https://review.openstack.org/#/c/450733/19:41
mariojvdeploy is failing at the stage that copies the configdrive19:42
mariojvlots 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 something19:43
*** baoli has joined #openstack-ironic19:43
jrollmariojv: that's the one on the whiteboard19:45
jrollyou know what, I'm gonna fix that now19:46
jrollI forgot that was on me and I was waiting for the dbsync thing before releasing everything :x19:46
mariojvoh my bad, didn't notice that there were 2 known failures19:46
jrollyeah, this one is intermittent19:46
*** e0ne has quit IRC19:46
jrollmariojv: this fixes, release folks are going to look when CI is done on it https://review.openstack.org/#/c/450949/119:48
JayFvsaienk0: you know where to kill that standalone job?19:49
JayFvsaienk0: that ran against the docs patch?19:49
JayFvsaienk0: I can show you if you don't19:49
*** e0ne has joined #openstack-ironic19:49
*** fragatin_ has quit IRC19:54
*** fragatina has joined #openstack-ironic19:55
openstackgerritMerged openstack/ironic master: [doc] Update examples in devstack section  https://review.openstack.org/45093619:55
jrollJayF: you asked about stable releases, did you have anything in particular in mind, or just all the things19:56
JayFjroll: just a general recollectino of landing a lot of stable stuff19:56
jrollya19:56
jrollso many branches :/19:56
jrollsuspect we need to backport the oslo.db thing19:56
JayF.o(what oslo db thing)19:57
JayFI've heard it refenced but seen no patches/bugs19:57
jrollthe one breaking our gate right now19:57
JayFprobably because I was gone?19:57
JayFoh19:57
jrollhttps://review.openstack.org/#/c/450733/19:57
JayFyeah; I fetched it from the whiteboard once I got clued in19:59
mariojvnice, thanks jroll20:00
jrollmaybe it doesn't need a backport, woo20:00
*** jtaryma has joined #openstack-ironic20:03
*** fragatina has quit IRC20:05
*** fragatina has joined #openstack-ironic20:06
*** racedo has joined #openstack-ironic20:14
*** jkilpatr has quit IRC20:16
jrollvsaienk0: any reason for the depends-on here? they shouldn't be related https://review.openstack.org/#/c/450737/20:18
*** jkilpatr has joined #openstack-ironic20:18
jrollI assume just to see the jobs pass20:18
*** racedo has quit IRC20:19
jroll^ that could use a +W if someone is around20:19
*** Goneri has quit IRC20:20
*** soulman has joined #openstack-ironic20:21
*** Goneri has joined #openstack-ironic20:22
JayFjroll: if it's my working hours, directly ping me for stuff like that :)20:22
JayF+2A20:22
jrollJayF: yeah, it just isn't a rush due to that depends-on, so I figured why bother :)20:22
jrollprobably will need to re-approve after the ironic patch merges20:23
*** ccamacho has quit IRC20:25
*** e0ne has quit IRC20:26
*** e0ne has joined #openstack-ironic20:30
*** mat128 has joined #openstack-ironic20:31
*** mat128 has quit IRC20:35
*** crushil has quit IRC20:38
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups  https://review.openstack.org/45097020:38
*** e0ne has quit IRC20:47
*** trown is now known as trown|outtypewww20:50
*** dprince has quit IRC20:53
*** crushil has joined #openstack-ironic20:53
*** jcoufal has quit IRC20:55
*** sambetts is now known as sambetts|afk20:55
*** baoli has quit IRC20:57
*** e0ne has joined #openstack-ironic20:58
*** jtaryma has quit IRC21:00
*** e0ne has quit IRC21:01
*** e0ne has joined #openstack-ironic21:02
*** e0ne has quit IRC21:02
*** e0ne has joined #openstack-ironic21:02
*** e0ne has quit IRC21:03
*** e0ne has joined #openstack-ironic21:04
*** e0ne has quit IRC21:04
*** e0ne has joined #openstack-ironic21:04
*** e0ne has quit IRC21:05
*** e0ne has joined #openstack-ironic21:05
*** e0ne has quit IRC21:05
*** e0ne has joined #openstack-ironic21:06
*** e0ne has quit IRC21:06
*** Goneri has quit IRC21:07
*** ianychoi has quit IRC21:07
*** baoli has joined #openstack-ironic21:08
*** ianychoi has joined #openstack-ironic21:08
*** jtaryma has joined #openstack-ironic21:12
*** jrcloud has joined #openstack-ironic21:18
*** zackf has quit IRC21:20
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements  https://review.openstack.org/45099521:26
*** Goneri has joined #openstack-ironic21:27
*** jjung has quit IRC21:27
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/45100821:29
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/45100921:29
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/45101021:29
*** vinsh has joined #openstack-ironic21:29
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/45101221:29
*** sayantan_ has quit IRC21:36
*** hshiina has quit IRC21:39
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups  https://review.openstack.org/45097021:42
openstackgerritAnup Navare proposed openstack/ironic master: Config drive support for Ceph  https://review.openstack.org/40763921:47
openstackgerritJoanna Taryma proposed openstack/ironic master: Add base cinder common interface  https://review.openstack.org/35562521:47
jrollnoooooo I was commenting :21:48
jroll:|21:48
jrolljoanna: just dropped a few more comments on PS 5521:49
jlvillaljroll, You would think she would be concentrating in her golang class ;)21:50
jrolljlvillal: probably ahead of the class already :P21:50
jlvillaljroll, Probably very true!21:51
*** bcornec has joined #openstack-ironic21:53
jtarymajroll: thanks, I commented on the very same minute :)22:02
jrollyep :)22:02
*** jtaryma is now known as joanna_mac22:03
joanna_macjroll: I already asked about this async thing, it uses call instead of cast, so they're sync.22:04
jrolljoanna_mac: awesome22:05
joanna_macjlvillal: that's flattering, I just sometimes change my scope when the talk is about sth I already know22:05
joanna_macbut I am not ahead! :)22:05
jrolljoanna_mac: now #openstack-cinder tells me terminate_volume and detach are async :/22:13
jrollhttps://drive.google.com/file/d/0B1Kp6K43HLHydU5XSWNnZEhqVDA/view?pref=2&pli=122:13
*** joanna_mac has quit IRC22:14
*** cdearborn has quit IRC22:15
*** baoli has quit IRC22:16
*** joanna_mac has joined #openstack-ironic22:21
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups  https://review.openstack.org/45097022:21
joanna_macjroll: I heard that you got a different answer22:21
*** baha has quit IRC22:21
*** choyj has quit IRC22:22
*** choyj has joined #openstack-ironic22:25
JayFjoanna_mac: repost incoming22:27
JayF22:13:17 <jroll> joanna_mac: now #openstack-cinder tells me terminate_volume and detach are async :/22:27
JayF22:13:20 <jroll> https://drive.google.com/file/d/0B1Kp6K43HLHydU5XSWNnZEhqVDA/view?pref=2&pli=122:27
joanna_macthe thing is that in the code it doesn't look so.22:28
joanna_macJayF: thanks :)22:28
*** rpioso has quit IRC22:29
*** jaypipes has quit IRC22:39
*** wrouesnel has joined #openstack-ironic22:49
*** Goneri has quit IRC22:52
openstackgerritJulian Edwards proposed openstack/ironic master: Update install guide to explain Nova integration  https://review.openstack.org/45049222:57
joanna_macJayF, jroll: yep, scottda meant that call from nova is async23:00
joanna_macnot the detach functions itself23:00
*** bcornec has quit IRC23:03
*** soulman has quit IRC23:05
*** causten has quit IRC23:06
*** causten has joined #openstack-ironic23:07
*** Goneri has joined #openstack-ironic23:07
*** causten has quit IRC23:12
*** hshiina has joined #openstack-ironic23:32
*** Fdaisuke has joined #openstack-ironic23:40
openstackgerritMerged openstack/ironic master: Don't pass sqlite_db in db_options.set_defaults()  https://review.openstack.org/45073323:46
TheJulia\o/23:46

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!