| *** jtomasek has quit IRC | 00:04 | |
| *** dcritch has joined #tripleo | 00:04 | |
| *** rlandy has quit IRC | 00:11 | |
| *** tbonds has joined #tripleo | 00:11 | |
| *** bswartz has quit IRC | 00:12 | |
| *** chlong has joined #tripleo | 00:32 | |
| *** Goneri has quit IRC | 00:35 | |
| *** limao has joined #tripleo | 00:39 | |
| *** jjung has quit IRC | 00:43 | |
| *** karimb has quit IRC | 00:44 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-common master: Open log in utf-8 to prevent UnicodeEncodeError https://review.openstack.org/434520 | 00:47 |
|---|---|---|
| *** pabelanger has left #tripleo | 00:48 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci master: Exclude /var/log/journal from devstack-logs https://review.openstack.org/434467 | 00:52 |
| *** chlong has quit IRC | 01:08 | |
| openstackgerrit | Merged openstack/puppet-tripleo master: tuning: manage keystone resources only at step3 https://review.openstack.org/433954 | 01:11 |
| *** ansmith has quit IRC | 01:15 | |
| *** dsneddon has joined #tripleo | 01:17 | |
| *** funzo_ has quit IRC | 01:21 | |
| *** thrash is now known as thrash|g0ne | 01:37 | |
| *** snecklifter has quit IRC | 02:06 | |
| *** cdearborn has joined #tripleo | 02:06 | |
| *** snecklifter has joined #tripleo | 02:06 | |
| *** dsneddon_ has joined #tripleo | 02:07 | |
| *** dsneddon has quit IRC | 02:07 | |
| *** [1]cdearborn has joined #tripleo | 02:10 | |
| diana_clarke | How do you know if the tripleo quickstart script exited successfully? Following this: https://github.com/openstack/tripleo-quickstart | 02:12 |
| diana_clarke | After 78 minutes, The "PLAY RECAP" doesn't show errors, nor does the scroll prior, but it still doesn't match the docs, so I'm not convinced it worked. | 02:14 |
| *** fzdarsky__ has joined #tripleo | 02:14 | |
| diana_clarke | I'm assuming I should have an overcloud after a successful quickstart run. Is that correct? | 02:15 |
| *** yamahata has quit IRC | 02:16 | |
| diana_clarke | For example, the docs [1] say I should be able to source the overcloudrc from the undercloud, but I don't see an overcouldrc. | 02:16 |
| diana_clarke | [1] https://docs.openstack.org/developer/tripleo-quickstart/accessing-undercloud.html | 02:17 |
| diana_clarke | Nor do I have anything overcloud related in ssh.config.ansible, based on this: https://docs.openstack.org/developer/tripleo-quickstart/accessing-overcloud.html | 02:18 |
| *** fzdarsky_ has quit IRC | 02:18 | |
| *** funzo has joined #tripleo | 02:22 | |
| *** yamahata has joined #tripleo | 02:24 | |
| *** cdearborn has quit IRC | 02:26 | |
| *** funzo has quit IRC | 02:26 | |
| *** Guest81465 has joined #tripleo | 02:31 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: Remove test-environments/multinode_major_upgrade.yaml https://review.openstack.org/426375 | 02:31 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: CI fix broken references to multinode-core.yaml https://review.openstack.org/434443 | 02:31 |
| *** xuao has joined #tripleo | 02:33 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Align multinode and multinode_upgrade CI services https://review.openstack.org/434421 | 02:33 |
| diana_clarke | The last tasks I see are: "validate the overcloud", and "Execute tempest against the overcloud", yet I don't seem to have an overcloud. | 02:35 |
| diana_clarke | Does anyone have a dump of what a successful pass looks like? | 02:36 |
| *** tzumainn has quit IRC | 02:39 | |
| *** yamahata has quit IRC | 02:57 | |
| *** akuznetsov has joined #tripleo | 03:02 | |
| *** akuznetsov has quit IRC | 03:04 | |
| *** ramishra has joined #tripleo | 03:04 | |
| openstackgerrit | Xu Ao proposed openstack/diskimage-builder master: DOC:Delete a obsolete note in the doc of python-brickclient https://review.openstack.org/434600 | 03:12 |
| *** panda|zZ has quit IRC | 03:18 | |
| *** dougbtv_ has joined #tripleo | 03:19 | |
| *** funzo has joined #tripleo | 03:21 | |
| *** funzo has quit IRC | 03:26 | |
| *** saneax-_-|AFK is now known as saneax | 03:29 | |
| *** [1]cdearborn has quit IRC | 03:47 | |
| *** yamahata has joined #tripleo | 03:52 | |
| *** sudipto has joined #tripleo | 03:57 | |
| *** sudipto_ has joined #tripleo | 03:57 | |
| *** diana_clarke has quit IRC | 04:00 | |
| *** cmyster has quit IRC | 04:01 | |
| *** links has joined #tripleo | 04:02 | |
| *** psahoo has joined #tripleo | 04:02 | |
| *** nmathew has joined #tripleo | 04:07 | |
| nmathew | gm | 04:08 |
| *** cmyster has joined #tripleo | 04:08 | |
| *** zhihui has quit IRC | 04:08 | |
| *** Guest81465 has left #tripleo | 04:19 | |
| *** limao has quit IRC | 04:22 | |
| *** limao has joined #tripleo | 04:23 | |
| *** ipsecguy_ has quit IRC | 04:25 | |
| *** ipsecguy has joined #tripleo | 04:26 | |
| *** funzo has joined #tripleo | 04:27 | |
| *** limao has quit IRC | 04:27 | |
| *** sasha2 has quit IRC | 04:30 | |
| *** funzo has quit IRC | 04:32 | |
| *** dougbtv_ has quit IRC | 04:32 | |
| openstackgerrit | Numan Siddique proposed openstack/tripleo-heat-templates master: ovn: Add iptable rules for OVN DB server ports https://review.openstack.org/434643 | 04:33 |
| *** mdnadeem has joined #tripleo | 04:35 | |
| *** jerrygb has quit IRC | 04:41 | |
| *** sudipto has quit IRC | 04:44 | |
| *** sudipto_ has quit IRC | 04:44 | |
| *** fultonj has quit IRC | 04:53 | |
| *** tbonds has quit IRC | 04:58 | |
| *** tbonds has joined #tripleo | 04:59 | |
| *** limao has joined #tripleo | 05:00 | |
| *** sudipto has joined #tripleo | 05:07 | |
| *** sudipto_ has joined #tripleo | 05:07 | |
| *** sdake has joined #tripleo | 05:08 | |
| *** tbonds has quit IRC | 05:09 | |
| *** vpickard has quit IRC | 05:09 | |
| *** saneax is now known as saneax-_-|AFK | 05:12 | |
| *** sdake_ has joined #tripleo | 05:13 | |
| *** prateek has joined #tripleo | 05:13 | |
| *** sdake has quit IRC | 05:14 | |
| *** vpickard has joined #tripleo | 05:18 | |
| *** saneax-_-|AFK is now known as saneax | 05:22 | |
| *** Lokesh_Jain_ has quit IRC | 05:23 | |
| *** funzo has joined #tripleo | 05:28 | |
| *** jaosorior has joined #tripleo | 05:31 | |
| *** funzo has quit IRC | 05:32 | |
| *** tbonds has joined #tripleo | 05:34 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job https://review.openstack.org/426863 | 05:34 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job https://review.openstack.org/426863 | 05:34 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Deploy novajoin in the TLS-everywhere job https://review.openstack.org/427481 | 05:43 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 05:43 |
| *** twi99y has joined #tripleo | 05:45 | |
| *** chem` has joined #tripleo | 05:46 | |
| *** twi99y_ has quit IRC | 05:47 | |
| *** chem has quit IRC | 05:48 | |
| *** vpickard_afk has joined #tripleo | 05:49 | |
| *** vpickard has quit IRC | 05:49 | |
| *** tbonds has quit IRC | 05:50 | |
| *** cmyster has quit IRC | 05:50 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Fix overcloud-scale failure msg and regression https://review.openstack.org/425352 | 05:52 |
| *** sdake_ has quit IRC | 05:54 | |
| *** tbonds has joined #tripleo | 06:03 | |
| *** tbonds has quit IRC | 06:08 | |
| *** tbonds has joined #tripleo | 06:09 | |
| *** saneax is now known as saneax-_-|AFK | 06:11 | |
| *** apetrich has quit IRC | 06:12 | |
| *** gatuus has quit IRC | 06:18 | |
| *** dsneddon_ has quit IRC | 06:19 | |
| *** funzo has joined #tripleo | 06:28 | |
| *** dsneddon has joined #tripleo | 06:32 | |
| *** funzo has quit IRC | 06:33 | |
| *** jaganathan has joined #tripleo | 06:34 | |
| *** agurenko has joined #tripleo | 06:43 | |
| *** udesale has joined #tripleo | 06:43 | |
| *** ealcaniz has joined #tripleo | 06:45 | |
| *** dsneddon has quit IRC | 06:53 | |
| *** saneax-_-|AFK is now known as saneax | 06:54 | |
| *** abregman has joined #tripleo | 06:55 | |
| *** apetrich has joined #tripleo | 06:56 | |
| *** ealcaniz has quit IRC | 06:57 | |
| *** hjensas has quit IRC | 06:57 | |
| *** muskan_ has joined #tripleo | 06:59 | |
| *** ealcaniz has joined #tripleo | 07:01 | |
| *** rcernin has joined #tripleo | 07:03 | |
| *** iranzo has joined #tripleo | 07:03 | |
| *** iranzo has joined #tripleo | 07:03 | |
| *** leanderthal|afk is now known as leanderthal | 07:05 | |
| *** paramite has joined #tripleo | 07:09 | |
| *** cmyster has joined #tripleo | 07:09 | |
| *** jtomasek has joined #tripleo | 07:12 | |
| *** tesseract has joined #tripleo | 07:13 | |
| *** tremble has joined #tripleo | 07:13 | |
| *** tremble has joined #tripleo | 07:13 | |
| *** limao has quit IRC | 07:17 | |
| *** limao has joined #tripleo | 07:17 | |
| *** ekuris has quit IRC | 07:21 | |
| *** xuao has quit IRC | 07:24 | |
| *** cmyster has quit IRC | 07:26 | |
| *** funzo has joined #tripleo | 07:29 | |
| *** jprovazn has joined #tripleo | 07:31 | |
| *** cmyster has joined #tripleo | 07:32 | |
| *** funzo has quit IRC | 07:35 | |
| *** dsariel has joined #tripleo | 07:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-ui master: Imported Translations from Zanata https://review.openstack.org/434699 | 07:39 |
| bandini | morning | 07:41 |
| openstackgerrit | Michele Baldessari proposed openstack-infra/tripleo-ci master: Collect /var/lib/pacemaker/cib in the tar files from nodes https://review.openstack.org/425151 | 07:41 |
| *** karimb has joined #tripleo | 07:41 | |
| matbu | bandini: o/ | 07:46 |
| bandini | yo matbu | 07:46 |
| marios | o/ peeps | 07:46 |
| bandini | heya marios | 07:46 |
| chem` | hi people :) | 07:47 |
| *** chem` is now known as chem | 07:47 | |
| matbu | wow the dream team :D | 07:47 |
| bandini | chem o/ | 07:47 |
| bandini | eheh | 07:47 |
| marios | matbu: we're doomed! | 07:47 |
| *** hjensas has joined #tripleo | 07:48 | |
| matbu | hehe | 07:49 |
| *** amoralej|off is now known as amoralej | 07:50 | |
| *** flepied has quit IRC | 07:50 | |
| *** dsneddon has joined #tripleo | 07:51 | |
| *** mcornea has joined #tripleo | 07:58 | |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK https://review.openstack.org/430277 | 08:00 |
| *** alop has joined #tripleo | 08:01 | |
| *** shardy has joined #tripleo | 08:03 | |
| shardy | morning all | 08:06 |
| marios | o/ | 08:06 |
| shardy | marios: Hey when you get a moment can you please check https://review.openstack.org/#/c/434468/ and https://review.openstack.org/#/c/434250/ | 08:06 |
| shardy | marios: those two add the upgrade -> converge steps to CI | 08:07 |
| *** jlinkes has joined #tripleo | 08:07 | |
| shardy | seems to be working OK, but the next step is to add nova | 08:07 |
| marios | shardy: yeah will do (I was thinking about this yesterday , i.e. we'll need a converge env file) | 08:07 |
| shardy | marios: yup, I added one, we can perhaps symlink the old named ones, or just update the docs? | 08:07 |
| shardy | This patch https://review.openstack.org/#/c/434348/ adds nova support but isn't yet passing, I'll work on getting that green today | 08:08 |
| shardy | I think we can go ahead with the update + converge changes anyway tho | 08:08 |
| marios | shardy: i think we'll need docs generally for the O upgrades (I'll try get something started, been meaning too for few days now but _other_stuff_) | 08:08 |
| shardy | marios: Yeah, agreed - I just wasn't sure how much we'd try to maintain the same filenames, vs just update docs and remove all the old stuff | 08:09 |
| *** yprokule has joined #tripleo | 08:09 | |
| shardy | the latter would be cleaner I suppose, but perhaps slightly less convenient for operators | 08:09 |
| marios | shardy: yeah i think it is significant enough of a change that we can/should change the docs | 08:09 |
| jaosorior | shardy: is there somewhere in CI where I can see the values that were given to the overcloud stack? | 08:09 |
| shardy | marios: Ok, cool, at least then we can purge all the old pieces which should be less confusing | 08:10 |
| marios | shardy: right, we should have a cleanup of extraconfig/tasks and environments/ | 08:10 |
| shardy | jaosorior: you mean the env files, or the parameters? | 08:10 |
| jaosorior | shardy: the parameters | 08:10 |
| *** aufi has joined #tripleo | 08:10 | |
| shardy | jaosorior: there's a stack show in the postci log IIRC which should show them | 08:10 |
| shardy | we should probably add an "openstack stack environment show" to that output too | 08:11 |
| shardy | as that will show everything for all stacks | 08:11 |
| jaosorior | shardy: cause I'm setting the DnsServers, but I don't find that in the postci logs | 08:11 |
| shardy | jaosorior: yeah if it's only passed to nested stacks via parameter_defaults it may not be captured, we need to add openstack stack environment show overcloud somewhere I think | 08:12 |
| shardy | that will dump all parameter_defaults and resource_registry entries which would be helpful I think | 08:12 |
| marios | shardy: did you see https://review.openstack.org/#/c/434367/ - after discussion yesterday with jarda it appears it would be favourable to allow operator to drive the ceph upgrade. at this point i think we can just use roles_data.yaml to determine that... i think we're good with https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/major_upgrade_steps.j2.yaml#L124 | 08:12 |
| jaosorior | shardy: any idea where the stack show is done? | 08:13 |
| marios | shardy: so all that remains is to fixup the tripleo_upgrade_node.sh to also deal with ceph like at https://review.openstack.org/#/c/434367/1/extraconfig/tasks/tripleo_upgrade_node.sh (well, this just happened last night, so we'll first need to sanity check with the ceph guys, gfidente already helping out there, and then test it) | 08:13 |
| marios | shardy: so we could keep both i mean | 08:13 |
| shardy | jaosorior: actually you can probably see the values in the heat API logs (/logs/var/log/heat/heat-api.txt.gz) | 08:13 |
| marios | shardy: the operator will decide whether ceph will be auto upgraded with controller steps, or manually like computes, based on settig the roles_data flag | 08:14 |
| *** pcaruana has joined #tripleo | 08:14 | |
| shardy | https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/get_host_info.sh#L38 | 08:14 |
| shardy | jaosorior: ^^ there I think | 08:14 |
| shardy | you could add openstack stack environment show overcloud there | 08:15 |
| matbu | shardy: i wonder, i add a review to add pacemaker in CI | 08:15 |
| jaosorior | :qa | 08:15 |
| jaosorior | shardy: funky, I don't see that value in the heat api logs | 08:15 |
| shardy | marios: ack, sounds good, note that ceph isn't tested in CI yet, and I hit an issue with it locally, so we'll need to get that covered and/or locally tested | 08:16 |
| shardy | it worked for me when we merged, but yesterday it hung upgrading OSD for some reason | 08:16 |
| shardy | matbu: yeah I was thinking we could add that after we get nova working | 08:16 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 08:16 |
| marios | shardy: ack | 08:16 |
| shardy | matbu: feel free to push a patch like https://review.openstack.org/#/c/434348/ though if you want to start looking at it | 08:17 |
| shardy | we can work on them in parallel I think | 08:17 |
| matbu | shardy: yep sure | 08:17 |
| *** gatuus_ has joined #tripleo | 08:19 | |
| shardy | matbu: note in https://review.openstack.org/#/c/434443/2/ci/environments/multinode_major_upgrade.yaml | 08:19 |
| *** gatuus has joined #tripleo | 08:19 | |
| shardy | matbu: I removed the Services::Core service as it was not needed, but when we re-enable pacemaker I think we will need it for the newton overcloud deploy, as it includes the core pacemaker p-t profile | 08:19 |
| matbu | shardy: ack; the review: https://review.openstack.org/#/c/429630 | 08:19 |
| shardy | then we'll need to switch it to OS::Heat::None during the upgrade | 08:19 |
| matbu | shardy: i have made also one for tripleo-ci repo, need to be rebase and rework a bit | 08:20 |
| *** pcaruana has quit IRC | 08:20 | |
| *** rasca has joined #tripleo | 08:20 | |
| shardy | matbu: I think since EmilienM is working on https://review.openstack.org/#/c/426375/ we should avoid changing the env in tripleo-ci | 08:21 |
| shardy | probably we should be rebasing on that and just changing things in t-h-t | 08:21 |
| matbu | ha yep right | 08:22 |
| *** pcaruana has joined #tripleo | 08:22 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/mitaka: Ensure working compute node during the whole migration. https://review.openstack.org/428093 | 08:22 |
| shardy | hopefully we can get that passing and landed today to avoid the duplication | 08:22 |
| *** d0ugal has joined #tripleo | 08:23 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates master: CI fix broken references to multinode-core.yaml https://review.openstack.org/434443 | 08:23 |
| *** flepied has joined #tripleo | 08:24 | |
| *** dsneddon has quit IRC | 08:24 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 08:27 |
| openstackgerrit | Thomas Herve proposed openstack/instack-undercloud master: Use Swift as a Zaqar backend. https://review.openstack.org/434727 | 08:27 |
| *** cylopez has joined #tripleo | 08:27 | |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras master: Implement overcloud sanity checks role https://review.openstack.org/429716 | 08:30 |
| *** funzo has joined #tripleo | 08:31 | |
| *** zoli|gone is now known as zoli | 08:32 | |
| *** zoli is now known as zoli|wfh | 08:32 | |
| *** jpena|away is now known as jpena | 08:35 | |
| *** funzo has quit IRC | 08:36 | |
| *** nyechiel has joined #tripleo | 08:41 | |
| *** owalsh-afk has quit IRC | 08:44 | |
| *** jpena is now known as jpena|away | 08:48 | |
| *** jpich has joined #tripleo | 08:51 | |
| *** akrivoka has joined #tripleo | 08:52 | |
| *** ccamacho has joined #tripleo | 08:52 | |
| *** fragatin_ has joined #tripleo | 08:53 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add manual ceph-osd upgrade if operator prefers this https://review.openstack.org/434367 | 08:53 |
| *** jpena|away is now known as jpena | 08:53 | |
| *** zoli|wfh is now known as zoliXXL | 08:54 | |
| *** yprokule_ has joined #tripleo | 08:56 | |
| *** fragatina has quit IRC | 08:57 | |
| *** yprokule has quit IRC | 08:57 | |
| *** yprokule_ is now known as yprokule | 08:57 | |
| *** fragatin_ has quit IRC | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #tripleo | 09:00 | |
| *** b00tcat has quit IRC | 09:01 | |
| marios | shardy: easy one when you have 2 mins please https://review.openstack.org/#/c/434305/ | 09:04 |
| marios | shardy: (it cant merge as it is a sub-review from https://review.openstack.org/#/c/433641/ based on mcornea's comments there from testing) | 09:06 |
| marios | shardy: but would be good to get it ready ;) | 09:06 |
| *** yamahata has quit IRC | 09:06 | |
| shardy | marios: ack, looking | 09:08 |
| *** b00tcat has joined #tripleo | 09:08 | |
| shardy | marios: looks good - it'd be nice if during pike we could move away from the script looking at hiera, and instead have it run an ansible playbook based on the service template upgrade_tasks | 09:10 |
| shardy | but for a first pass this looks fine, thanks! | 09:10 |
| *** abregman has quit IRC | 09:10 | |
| marios | shardy: yeah i think thats the next iteration, i.e. for those roles where we have disable_upgrade_deployment to true, run the actual ansible playbook from the service upgrade_tasks (they are all there anyway, e.g. nova-compute has them already) | 09:11 |
| marios | shardy: thanks | 09:12 |
| shardy | marios: yeah, we just need to dump the playbook on each node and run it from the script I think | 09:12 |
| shardy | https://review.openstack.org/#/c/433641/5/extraconfig/tasks/tripleo_upgrade_node.sh | 09:13 |
| *** pmannidi has quit IRC | 09:13 | |
| shardy | we could do the same with the puppt apply and drive that via ansible I guess, then ultimately folks could drive the entire workflow from outside the nodes in future | 09:13 |
| *** derekh has joined #tripleo | 09:15 | |
| *** abregman has joined #tripleo | 09:16 | |
| shardy | marios, chem: question on https://review.openstack.org/#/c/433641 | 09:16 |
| shardy | the order looks backwards compared to the heat orchestrated workflow? | 09:17 |
| *** bogdando has joined #tripleo | 09:18 | |
| marios | shardy: no it should be fine... the run_puppet is not invoked there | 09:18 |
| marios | shardy: i mean it is just declared... | 09:18 |
| shardy | marios: maybe I need more coffee, but I'm still missing something | 09:20 |
| shardy | we have a group: script SoftwareConfig | 09:20 |
| shardy | which contains a script | 09:20 |
| marios | shardy: added comment | 09:20 |
| shardy | that joints together a bunch of scripts | 09:20 |
| *** yamahata has joined #tripleo | 09:20 | |
| marios | shardy: https://review.openstack.org/#/c/433641/5/extraconfig/tasks/run_puppet.sh just declares the function | 09:21 |
| shardy | each one isn't encapsulated by cat > << EOF blocks | 09:21 |
| shardy | marios: Ah, Ok so that's one part of it | 09:21 |
| marios | shardy: (but doesn't execute it) | 09:21 |
| shardy | we join them together, then it's actually run from tripleo_upgrade_node, I see | 09:21 |
| shardy | but how are we eriting, vs running, this script? | 09:22 |
| shardy | writing I mean | 09:22 |
| marios | shardy: and then we just use that function declaration in the heredoc yeah its weird | 09:22 |
| marios | shardy: it is a bash heredoc which dumps /root/tripleo_upgrade_node.sh | 09:22 |
| marios | https://review.openstack.org/#/c/433641/5/extraconfig/tasks/tripleo_upgrade_node.sh@11 | 09:22 |
| *** dtantsur|afk is now known as dtantsur | 09:23 | |
| marios | shardy: and then the operator will do 'upgrade-non-controller.sh --node compute0' which expects that file to exist | 09:23 |
| *** bogdando has quit IRC | 09:25 | |
| shardy | marios: Ah, Ok , it was the declare -f I was missing | 09:25 |
| shardy | that's weird ;) | 09:25 |
| shardy | thanks for clarifying | 09:25 |
| marios | shardy: yeah its not nice :( but it allows us to declare the upgrade file and pass in params (e.g. we were passing in the nova pin previously here from the templates and writing into the upgrade script for the operator to invoke later) | 09:25 |
| * shardy would have joined the script together by appending cat blocks, but whatever | 09:25 | |
| shardy | marios: Ok, cool - I was just confused, perhaps another coffee is a good idea, thanks! :) | 09:26 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Deploy novajoin in the TLS-everywhere job https://review.openstack.org/427481 | 09:26 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 09:26 |
| marios | shardy: result looks like this fyi/fwiw from yesterday http://paste.openstack.org/raw/598999/ | 09:27 |
| shardy | marios: ack, thanks, looks good, +2 | 09:29 |
| *** panda has joined #tripleo | 09:29 | |
| *** yamahata has quit IRC | 09:30 | |
| *** milan has joined #tripleo | 09:30 | |
| *** yprokule_ has joined #tripleo | 09:31 | |
| shardy | marios: hey did you see my comment yesterday on https://review.openstack.org/#/c/424715/ ? | 09:32 |
| *** nyechiel has quit IRC | 09:32 | |
| shardy | I think the CI failures are real, and we're trying to run upgradeinit every deploy, not only on upgrade | 09:32 |
| *** openstackgerrit has quit IRC | 09:32 | |
| shardy | but we got away with it before because the default was basically a no-op | 09:33 |
| *** yprokule has quit IRC | 09:33 | |
| *** yprokule_ is now known as yprokule | 09:33 | |
| marios | shardy: yes i did and will revisit next :D thanks | 09:33 |
| shardy | marios: Ok cool, let me know if I can help | 09:33 |
| marios | shardy: ok, we should make it 'enabled' via the env file probably simplest way to go for now i think | 09:34 |
| shardy | marios: Yeah I was thinking the easiest way is to add another UpgradeInitCommandCommon parameter or something | 09:34 |
| shardy | then we set it in the upgrade-composable-steps env, and unset in the converge? | 09:34 |
| *** bugzy has joined #tripleo | 09:34 | |
| shardy | maybe some other way, but the heat states only tell us create vs update, not when we're upgrading | 09:35 |
| *** lucas-afk is now known as lucasagomes | 09:35 | |
| marios | shardy: ack will do thanks | 09:35 |
| *** nyechiel has joined #tripleo | 09:36 | |
| *** openstackgerrit has joined #tripleo | 09:36 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add major-upgrade-converge environment. https://review.openstack.org/434468 | 09:36 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/newton: Allow the override of pacemaker::corosync::settle_tries https://review.openstack.org/426114 | 09:37 |
| openstackgerrit | Merged openstack/tripleo-ui master: Imported Translations from Zanata https://review.openstack.org/434699 | 09:38 |
| openstackgerrit | Merged openstack/tripleo-docs master: Add yum clean all when updating undercloud https://review.openstack.org/434447 | 09:42 |
| openstackgerrit | Thierry Carrez proposed openstack/tripleo-ui master: Fix sdist tarball name in setup.cfg https://review.openstack.org/434778 | 09:51 |
| *** saibarspeis has joined #tripleo | 09:51 | |
| *** Arpit has joined #tripleo | 09:55 | |
| Arpit | Hii | 09:55 |
| jpich | honza: Morning! You might have some thoughts on https://review.openstack.org/#/c/434778/ (now creating different issues for the release job) | 09:56 |
| Arpit | I want install Openstack on tripleo | 09:56 |
| Arpit | can anyone help me?? | 09:56 |
| sshnaidm|off | jaosorior, hi | 09:57 |
| *** athomas has joined #tripleo | 09:57 | |
| *** sshnaidm|off is now known as sshnaidm | 09:57 | |
| shardy | Arpit: hi! Here are our docs which describe how to get started | 09:59 |
| shardy | https://docs.openstack.org/developer/tripleo-docs/ | 09:59 |
| sshnaidm | jaosorior, wrt dev env problem, I've updated etherpad, see lines 50-54. Seems like it was in the scripts, but I didn't add it to etherpad.. https://etherpad.openstack.org/p/tripleo-ci-devenvs | 09:59 |
| shardy | Arpit: also there is an ansible based tool called tripleo-quickstart which automates some steps for you: | 09:59 |
| shardy | https://github.com/openstack/tripleo-quickstart/ | 09:59 |
| Arpit | Thanks Shardy | 09:59 |
| jaosorior | sshnaidm: duuude, thanks! | 09:59 |
| Arpit | fine shardy | 10:00 |
| Arpit | its great | 10:00 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere overcloud deploy arguments https://review.openstack.org/422310 | 10:00 |
| *** salmankhan has joined #tripleo | 10:01 | |
| jaosorior | sshnaidm: how acquainted are you with quickstart? | 10:02 |
| *** udesale has quit IRC | 10:03 | |
| *** ckyriakidou has joined #tripleo | 10:05 | |
| sshnaidm | jaosorior, more or less, why? | 10:05 |
| *** gfidente has joined #tripleo | 10:06 | |
| *** gfidente has quit IRC | 10:06 | |
| *** gfidente has joined #tripleo | 10:06 | |
| jaosorior | sshnaidm: I'm trying to replicate some parts of what I'm doing in CI for the TLS-everywhere job in quickstart. And for this, I need to update the cloud-init package in the overcloud images. Any idea where to do that? | 10:06 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add Newton to Ocata UpgradeInitCommonCommand https://review.openstack.org/434795 | 10:06 |
| *** flepied has quit IRC | 10:07 | |
| sshnaidm | jaosorior, we have "modify-image" role that can upload things and run scripts on the image | 10:07 |
| jaosorior | I guess that could work | 10:07 |
| jaosorior | sshnaidm: how do I use that? | 10:07 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add Newton to Ocata UpgradeInitCommonCommand https://review.openstack.org/424715 | 10:08 |
| sshnaidm | jaosorior, for example: https://review.openstack.org/#/c/426869/8/roles/install-built-repo/tasks/inject_repo.yml | 10:09 |
| sshnaidm | jaosorior, or https://review.openstack.org/#/c/433287/1/roles/repo-setup/tasks/inject_repos_into_image.yml | 10:09 |
| sshnaidm | jaosorior, or a few examples here: https://github.com/openstack/tripleo-quickstart-extras/blob/61ef59ff2cb64bede6da55347d9c2fa6f0870492/roles/modify-image/README.md | 10:10 |
| *** sdake has joined #tripleo | 10:10 | |
| jaosorior | sshnaidm: any recommendation on where to add this modification? | 10:10 |
| jaosorior | the overcloud-prep-image role doesn't really seem to modify the images :/ | 10:10 |
| jaosorior | sshnaidm: also, should I just upload the rpm into the quickstart-extras repo? | 10:11 |
| jaosorior | panda: ^^ | 10:11 |
| panda | jaosorior: what about here roles/libvirt/setup/undercloud/templates/update_image.sh.j2 ? | 10:14 |
| marios | gfidente: hey, thanks for you comments on https://review.openstack.org/#/c/434367/ I updated it. we need to get this tested and landed soonish :) grateful for any time you can spare ;) | 10:15 |
| gfidente | marios ack on it | 10:15 |
| marios | gfidente: i left one question about the startup | 10:15 |
| marios | its OK everybody, the faidentee has arrived. \o/ | 10:16 |
| marios | thanks man | 10:16 |
| openstackgerrit | Merged openstack/tripleo-ui master: Update author field in package.json https://review.openstack.org/434335 | 10:16 |
| jaosorior | panda: will that take effect on the overcloud-full image? | 10:16 |
| gfidente | marios the more the spring gets closer | 10:16 |
| gfidente | the more I miss you | 10:16 |
| panda | jaosorior: yes | 10:17 |
| marios | miss you too the faidentee so much | 10:17 |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh https://review.openstack.org/431029 | 10:17 |
| jaosorior | panda: I could use that, is there a reason why that's recommended over using the modify-image role? | 10:18 |
| panda | jaosorior: https://github.com/openstack/tripleo-quickstart/blob/master/roles/libvirt/setup/undercloud/tasks/main.yml#L51 | 10:18 |
| *** limao has quit IRC | 10:18 | |
| jaosorior | panda: oh, but that only works in devmode then | 10:19 |
| panda | jaosorior: yes | 10:19 |
| panda | jaosorior: fi you want to do it generally, yes, you'll have to add a new task using the genric modify-image role,, creating a script that installs the cloud-itni, and put those task probably under https://github.com/openstack/tripleo-quickstart/blob/master/roles/libvirt/setup/undercloud/tasks/main.yml#L51 | 10:21 |
| jaosorior | panda: that would be fine for me | 10:22 |
| *** Arpit has quit IRC | 10:22 | |
| panda | jaosorior: the problem is that except for devmode, overcloud-full image is already in the undercloud image | 10:22 |
| *** suuuper has joined #tripleo | 10:22 | |
| jaosorior | panda: but, modify-image would anyway run virt-customize on those images | 10:23 |
| *** flepied has joined #tripleo | 10:24 | |
| panda | jaosorior: yes, but it doesn't modify an image inside another image | 10:25 |
| jaosorior | panda: what do you mean? | 10:25 |
| *** udesale has joined #tripleo | 10:26 | |
| panda | when not in devmode, the undercloud image already contains the overcloud image. so to modify the overcloud image inside the undercloud image you'd have to extrace the overcloud image from the undercloud image, modify it and inject it again | 10:26 |
| *** panda is now known as panda|brb | 10:27 | |
| *** sdake has quit IRC | 10:28 | |
| jaosorior | panda|brb: but, wouldn't this be done at runtime? if I do this in the right role, it would happen when the undercloud is running and before the overcloud images are uploaded to glance | 10:28 |
| *** funzo has joined #tripleo | 10:33 | |
| *** tosky has joined #tripleo | 10:33 | |
| *** psahoo has quit IRC | 10:34 | |
| *** psahoo has joined #tripleo | 10:34 | |
| *** funzo has quit IRC | 10:37 | |
| *** salmankhan has quit IRC | 10:37 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere overcloud deploy arguments https://review.openstack.org/422310 | 10:38 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add "install" option to modify-image role https://review.openstack.org/434803 | 10:38 |
| *** panda|brb is now known as panda | 10:38 | |
| chem | marios: commented on https://review.openstack.org/#/c/434367/2 | 10:38 |
| chem | marios: if I'm not misleading quoting is a killer in those "deliver a script" upgrade | 10:39 |
| chem | marios: maybe we should think of another way. The $(declare -f ...) is nice but maybe we can do better, will give it some though. In the meantime, let's quote :) | 10:39 |
| marios | chem: ack thanks will have a look in a bit | 10:40 |
| gfidente | chem marios no no wait | 10:41 |
| gfidente | no quotes | 10:41 |
| gfidente | I tested those steps a million times | 10:41 |
| marios | gfidente: yeah we could use 'ENDOFCAT' too | 10:41 |
| gfidente | put 'ENDOFCAT' in quotes instead | 10:41 |
| gfidente | IMHO it's risky to escape/quote all the steps | 10:41 |
| marios | gfidente: but then we have to remove all the \ where we do have them | 10:41 |
| gfidente | yeah | 10:41 |
| jaosorior | sshnaidm: by the way, could you check this out https://review.openstack.org/#/c/427481/ ? | 10:42 |
| panda | jaosorior: then I think the only other place suited for this is the overcloud-prep-image role, | 10:42 |
| marios | gfidente: like what do we do about https://review.openstack.org/#/c/434367/2/extraconfig/tasks/tripleo_upgrade_node.sh@93 | 10:42 |
| jaosorior | panda: that works for me | 10:42 |
| marios | chem: gfidente where we do want it to evaluate $ROLE but not \$step, i mean when it is writing it | 10:42 |
| jaosorior | panda: thanks dude | 10:42 |
| lhinds | jaosorior: is MongoDB under TLS everywhere? | 10:44 |
| *** owalsh has joined #tripleo | 10:44 | |
| gfidente | marios ok but seems to be a single instance where we need it evaluated outside the script? | 10:45 |
| jaosorior | lhinds: not yet. | 10:45 |
| lhinds | jaosorior: ok, but you have it on your plan? | 10:45 |
| chem | gfidente: marios if we put the 'EOF' then we have to use another trick to pass the ROLE variable. | 10:46 |
| gfidente | chem yeah but it's a single instance issuer | 10:46 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add Newton to Ocata UpgradeInitCommonCommand https://review.openstack.org/424715 | 10:46 |
| jaosorior | lhinds: well, it was in the initial plan. Stuff is really fuzzy at the moment due to the containers bits. So it might loose priority :/ | 10:47 |
| *** gatuus_ has quit IRC | 10:47 | |
| *** gatuus has quit IRC | 10:47 | |
| chem | gfidente: yep, but how ? we should then concatene string somehow ? | 10:47 |
| *** salmankhan has joined #tripleo | 10:47 | |
| lhinds | jaosorior: yep, makes sense. i was going to look into implementing auth there | 10:47 |
| chem | marios: gfidente maybe the best here would be to put the ceph handling function is another file and use the $(declare -f ...) trick | 10:48 |
| marios | chem: yeah that could be another way to go too | 10:48 |
| chem | gfidente: and there are all those $(declare -f ...) as well wich are evaluated at ... compile time (!=run time) | 10:48 |
| lhinds | jaosorior: wait? is mongodb likely to be migrated from with the containers effort? | 10:49 |
| marios | shardy: updated https://review.openstack.org/#/c/424715/ when you get a sec thanks | 10:49 |
| gfidente | so what do you mean "compile" time | 10:49 |
| gfidente | it's a shell script | 10:49 |
| jaosorior | lhinds: well, it would be migrated to run over a container... maybe? I'm not sure to be honest | 10:49 |
| chem | gfidente: heat "compile time" vs user "run time" | 10:49 |
| lhinds | jaosorior: I know zaqar is using it, and ceilometer on controller | 10:49 |
| *** hewbrocca_afk is now known as hewbrocca | 10:50 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add Newton to Ocata UpgradeInitCommonCommand https://review.openstack.org/424715 | 10:50 |
| chem | gfidente: does it make sense ? I just want to say those scripts are parsed twiced | 10:50 |
| gfidente | yeah sec | 10:50 |
| gfidente | I think | 10:50 |
| gfidente | there was a nicer way to do that now | 10:50 |
| chem | s/twiced/twice/ | 10:50 |
| gfidente | so I think jistr left an idea about that too in another yaml | 10:52 |
| gfidente | we can use map_merge to replace in the shell script the placeholders | 10:52 |
| *** ckyriakidou has left #tripleo | 10:52 | |
| gfidente | instead of feeding them as variables | 10:52 |
| gfidente | but this means changing the templates were we trigger the .sh | 10:52 |
| chem | gfidente: hum... i see, then you put 'eof' and you're done with quoting | 10:54 |
| gfidente | and escaping yes | 10:54 |
| chem | gfidente: but then the $(declare -f ) won't work anymore and we rely on it a lot. | 10:55 |
| gfidente | it will work, you replace it with a value in the template itself | 10:55 |
| jistr | gfidente, chem: i'm not 100% sure i understand what we're trying to solve now, so excuse me if i don't :) but | 10:55 |
| gfidente | you don't need to evaluate it in the script | 10:55 |
| marios | chem: gfidente sec posting an update using the declare -f for now | 10:55 |
| jistr | what i remember suggesting was splitting the script writing, "the compile time", into two parts | 10:56 |
| jistr | one which echoes the function and variable definitions, that would use the double-quote mode | 10:56 |
| jistr | and the rest which is passed in as-is, using single-quote mode | 10:57 |
| *** salmankhan1 has joined #tripleo | 10:57 | |
| jistr | that way you will worry about escaping only in a small part of the script | 10:57 |
| gfidente | jistr yeah that's also an option | 10:57 |
| marios | jistr: i think we ended up doing that somewhere in fact (using >1 heredoc) | 10:57 |
| jistr | right yea | 10:57 |
| *** salmankhan has quit IRC | 10:58 | |
| gfidente | I was going to replace the placeholders straight in heat | 10:58 |
| marios | chem: jistr gfidente for now i'm going to use the pacemaker_common_functions since we are using those already here anyway | 10:58 |
| jistr | echo "vars functions" > script | 10:58 |
| jistr | echo 'actual logic' >> script | 10:58 |
| marios | gimme few faidentee SRSLY! | 10:58 |
| gfidente | your backslash on the much loved keyboard will break | 10:59 |
| gfidente | and you will be sad forever | 10:59 |
| marios | :( | 11:00 |
| *** salmankhan has joined #tripleo | 11:00 | |
| chem | jistr: marios gfidente https://xkcd.com/1638/ | 11:01 |
| jistr | :D | 11:01 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add manual ceph-osd upgrade if operator prefers this https://review.openstack.org/434367 | 11:01 |
| chem | hehe backslash ftw ! | 11:01 |
| marios | haha chem | 11:01 |
| *** dtantsur is now known as dtantsur|bbl | 11:01 | |
| *** salmankhan1 has quit IRC | 11:01 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: OVN bridge mappings for tripleo https://review.openstack.org/402293 | 11:02 |
| mcornea | marios: hey, I'm seeing an error, I think it's related to the ceph patch: /var/lib/heat-config/heat-config-script/98323c5f-b402-466f-b551-e319b0d85aa8: line 385: OSD_IDS: unbound variable | 11:02 |
| chem | mcornea: yep, marios is on it :) | 11:03 |
| marios | mcornea: thanks yeah it wasn't being written correctly | 11:03 |
| chem | mcornea: we are in the backslash hell | 11:03 |
| mcornea | great guys, thank yoou | 11:03 |
| *** udesale has quit IRC | 11:03 | |
| gfidente | but | 11:06 |
| gfidente | if we default to the ansible tasks | 11:06 |
| gfidente | why is mcornea hitting that? | 11:07 |
| marios | gfidente: i guess he is testing it against a ceph node | 11:07 |
| gfidente | mcornea ^^ are you testing the manual procedure for the OSDs? | 11:07 |
| mcornea | gfidente: marios no, it showed up during major-upgrade-composable-steps.yaml | 11:08 |
| mcornea | gfidente: marios http://paste.openstack.org/show/599199/ | 11:09 |
| chem | marios: what we would need is a heat "file" resource that create a file on the specified servers | 11:09 |
| marios | mcornea: gfidente hmm maybe because we have -eu in https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/major_upgrade_steps.j2.yaml#L61 | 11:09 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 11:10 |
| gfidente | marios mcornea sorry guys, I don't see any reference to OSD_IDS in master because we removed the legacy scripts | 11:10 |
| gfidente | and we haven't landed the shells for those doing it manually | 11:11 |
| marios | gfidente: i think mcornea had v2/3 of https://review.openstack.org/#/c/434367/ applied | 11:11 |
| mcornea | gfidente: I had https://review.openstack.org/#/c/434367/ applied to my tht | 11:11 |
| gfidente | mcornea marios ah okay, sorry, didn't get that | 11:11 |
| marios | gfidente: so when he was trying to run it, on the part where we deliver the script, it failed (we deliver the script at //github.com/openstack/tripleo-heat-templates/blob/master/puppet/major_upgrade_steps.j2.yaml#L61 ) | 11:11 |
| * marios deathstare faidentee | 11:11 | |
| gfidente | mcornea ok so back to the original question | 11:12 |
| gfidente | I'd prefer defaulting to the ansible tasks | 11:12 |
| marios | gfidente: you should join today call and tell that to jarda | 11:12 |
| marios | gfidente: someone will have to make a cal on what the default will be | 11:12 |
| gfidente | but I suppose those are already tested as part of the "bigger testing effort around upgrades" ? | 11:12 |
| gfidente | marios ack | 11:13 |
| marios | gfidente: not to a great extent i mean testing | 11:13 |
| marios | gfidente: so either way we should choose one and there is no preference at the moment afaik | 11:13 |
| gfidente | well the ansible tasks are going to be executed whenever people is deploying a ceph osd | 11:13 |
| gfidente | so one reason for me to prefer that is that they get a much bigger exposure | 11:13 |
| gfidente | to testing | 11:13 |
| marios | gfidente: well, unless they specify disable_upgrade_deployment: True | 11:13 |
| marios | gfidente: so i guess, the decision is, what does the default roles_data.yaml say, though ultimately they will have to edit their own | 11:14 |
| gfidente | yeah | 11:14 |
| gfidente | so we'll upgrade the monitors via ansbile tasks anyway | 11:14 |
| marios | gfidente: yeah mons get upgraded,unless they live on a roles where the flags say no | 11:15 |
| *** flepied has quit IRC | 11:15 | |
| marios | gfidente: right now they are on the controller (by default i mean right now) | 11:15 |
| gfidente | shardy magically provided _batch_ upgrade for us | 11:15 |
| marios | gfidente: so yeah, they have the batch_upgrade_tasks .. .the ceph mons will happen before any other controller services in fact | 11:15 |
| mcornea | gfidente: I would feel more comfortable with running the upgrade manually on each node because you are more in control of what's happening on each nodes. The ansible tasks are running behind the scenes so the user doesn't know which of the node is getting updated at a certain point in time. If there's a failure you need to debug, see where the tasks failed and so on. | 11:16 |
| gfidente | right I mean people worked to get there | 11:16 |
| gfidente | mcornea right that is a valid concern | 11:16 |
| *** jaosorior has quit IRC | 11:16 | |
| gfidente | but we have to deal with it being either automated or not be | 11:16 |
| jpich | honza: Thanks for the helpful follow-up!! | 11:16 |
| honza | jpich: np! | 11:16 |
| gfidente | and if we buy the idea for openstack | 11:16 |
| gfidente | I don't see why not for ceph | 11:16 |
| *** skramaja has quit IRC | 11:16 | |
| gfidente | that's why I'm saying shardy had to work on specific requirements to make the upgrade happen on one node at a time | 11:17 |
| gfidente | before the others | 11:17 |
| *** jaganathan has quit IRC | 11:17 | |
| *** jaosorior has joined #tripleo | 11:17 | |
| gfidente | not only that, but upgrading manually the osds gives you more control | 11:17 |
| gfidente | but you'd still get the mons upgraded via ansible | 11:17 |
| *** [1]cdearborn has joined #tripleo | 11:17 | |
| *** karthiks has quit IRC | 11:18 | |
| *** numans has quit IRC | 11:18 | |
| marios | gfidente: plus we're not removing any of the 'auto' ceph osd upgrade stuff | 11:18 |
| marios | gfidente: so it is both | 11:18 |
| gfidente | so I'd actually keep the manual process as an option for those who prefer it | 11:19 |
| gfidente | but default to the ansible version | 11:20 |
| gfidente | so I am actually going to re-enable ceph in the update job :) | 11:22 |
| *** saneax is now known as saneax-_-|AFK | 11:22 | |
| mcornea | gfidente: well, I guess if we do thorough testing (on physical ceph deployments) I'd say we're safe with the ansible tasks. but we still need to test both the approaches | 11:23 |
| gfidente | mcornea ack on that | 11:23 |
| mcornea | gfidente: so maybe we can do most of the testing with the default approach and have a few jobs with doing individual nodes upgrades | 11:24 |
| gfidente | marios shall I add it to updates or upgrades actually? | 11:24 |
| *** sudipto has quit IRC | 11:25 | |
| *** sudipto_ has quit IRC | 11:25 | |
| *** lucasagomes is now known as lucas-bbl | 11:26 | |
| marios | gfidente: upgrades - matbu knows more about that he added the jobs | 11:27 |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci master: Enable Ceph in the updates job https://review.openstack.org/434832 | 11:28 |
| *** flepied has joined #tripleo | 11:31 | |
| gfidente | marios matbu how is it triggered? does it run in ovb or multinode? | 11:31 |
| *** funzo has joined #tripleo | 11:34 | |
| *** adarazs is now known as adarazs_lunch | 11:34 | |
| marios | gfidente: https://review.openstack.org/#/c/404831/ | 11:36 |
| *** nmathew has quit IRC | 11:37 | |
| *** funzo has quit IRC | 11:38 | |
| *** jkilpatr_ has quit IRC | 11:39 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Add removed wait_for script to validate role https://review.openstack.org/434239 | 11:45 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add "install" option to modify-image role https://review.openstack.org/434803 | 11:46 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 11:46 |
| *** saneax-_-|AFK is now known as saneax | 11:46 | |
| *** jaganathan has joined #tripleo | 11:46 | |
| *** skramaja has joined #tripleo | 11:46 | |
| *** karthiks has joined #tripleo | 11:46 | |
| *** owalsh has quit IRC | 11:46 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Make overcloud image upload honor --image-path https://review.openstack.org/407261 | 11:47 |
| *** numans has joined #tripleo | 11:47 | |
| *** jjung has joined #tripleo | 11:47 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Mock time.sleep in unit tests https://review.openstack.org/429909 | 11:48 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add release notes for the HA-by-default change https://review.openstack.org/421226 | 11:48 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Align HCI environment file with list of services from roles_data https://review.openstack.org/434848 | 11:49 |
| mcornea | matbu: hi! could you please rebase https://review.openstack.org/#/c/431398/ ? | 11:52 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add manual ceph-osd upgrade if operator prefers this https://review.openstack.org/434367 | 11:53 |
| *** ansmith has joined #tripleo | 11:55 | |
| *** agurenko has quit IRC | 11:57 | |
| *** jkilpatr has joined #tripleo | 12:00 | |
| *** sdake has joined #tripleo | 12:02 | |
| *** jpena is now known as jpena|lunch | 12:05 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 12:06 |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci master: Enable Ceph in multinode upgrades job (via HCI). https://review.openstack.org/434855 | 12:06 |
| *** athomas has quit IRC | 12:06 | |
| *** rbrady has quit IRC | 12:08 | |
| EmilienM | hello | 12:10 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove deprecated metering_secret https://review.openstack.org/431118 | 12:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove unused nova_url from neutron config https://review.openstack.org/423301 | 12:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Configuring a default ntp server. https://review.openstack.org/420573 | 12:11 |
| openstackgerrit | Merged openstack/python-tripleoclient master: Mock os.path.isfile correctly https://review.openstack.org/433222 | 12:11 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Fix a typo in mysql.pp https://review.openstack.org/416855 | 12:11 |
| EmilienM | shardy: hi - can we land https://review.openstack.org/#/c/434443/ and https://review.openstack.org/#/c/434421/ before I branch? | 12:12 |
| *** panda is now known as panda|lunch | 12:12 | |
| EmilienM | shardy: http://logs.openstack.org/21/434421/2/check/gate-tripleo-heat-templates-pep8-ubuntu-xenial/24735ee/console.html#_2017-02-16_02_36_56_131608 | 12:13 |
| shardy | EmilienM: ack, looking | 12:13 |
| EmilienM | shardy: I think you need to change the heat_template_version on https://review.openstack.org/#/c/434443/2/ci/environments/multinode-core.yaml | 12:13 |
| shardy | EmilienM: yes https://review.openstack.org/#/c/434443/ can land, we just need reviews | 12:14 |
| shardy | EmilienM: yup I did in in ps3 | 12:14 |
| shardy | https://review.openstack.org/#/c/434443/3 passed CI | 12:15 |
| EmilienM | shardy: can you approve it please? | 12:15 |
| shardy | marios: hey can you check ^^ so we can land before we branch? | 12:15 |
| shardy | EmilienM: well it's my patch | 12:15 |
| EmilienM | oh right | 12:15 |
| shardy | so could you review? | 12:15 |
| *** noslzzp has quit IRC | 12:15 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Align multinode and multinode_upgrade CI services https://review.openstack.org/434421 | 12:15 |
| EmilienM | shardy: I rebased https://review.openstack.org/#/c/434421/ | 12:15 |
| EmilienM | to pass CI | 12:15 |
| shardy | EmilienM: thanks, sorry I was waiting for https://review.openstack.org/#/c/434443 to pass | 12:16 |
| EmilienM | shardy: do we need to backport all of these to be backported into newton? | 12:16 |
| *** noslzzp has joined #tripleo | 12:16 | |
| marios | shardy: looking | 12:16 |
| shardy | EmilienM: No, I think only ocata | 12:16 |
| EmilienM | shardy: ok | 12:16 |
| shardy | EmilienM: some further adjustments may be needed when we add pacemaker support to the upgrade job | 12:16 |
| shardy | but for now I think this is OK | 12:16 |
| EmilienM | ok | 12:17 |
| EmilienM | shardy: I think https://review.openstack.org/#/c/434421/3 can be approved too | 12:17 |
| EmilienM | marios: ^ | 12:17 |
| shardy | EmilienM: No I think we need that to pass CI with nova enabled | 12:17 |
| shardy | EmilienM: I may abandon it and instead we can go with your patch: | 12:18 |
| *** ansmith has quit IRC | 12:18 | |
| EmilienM | shardy: it's a bit confusing for me and you seem to know better. can you do it please? | 12:18 |
| shardy | https://review.openstack.org/#/c/434348/ | 12:18 |
| shardy | EmilienM: sure - we both posted similar patches at the same time yesterday | 12:18 |
| EmilienM | shardy: oops, sorry :/ | 12:18 |
| EmilienM | shardy: feel free to take over my patch or abandon it | 12:19 |
| EmilienM | shardy: I also think we need to land https://review.openstack.org/#/c/426375/ asap | 12:19 |
| EmilienM | shardy: people are confused, even folks working on upgrades :/ | 12:19 |
| shardy | https://review.openstack.org/#/c/434348/ abandoned, and I'll take over https://review.openstack.org/#/c/434348 to try to get nova passing CI | 12:19 |
| * shardy is not confused | 12:19 | |
| shardy | ;) | 12:20 |
| EmilienM | shardy: yeah I know :D | 12:20 |
| EmilienM | shardy: thx for the help | 12:20 |
| EmilienM | shardy: can I recheck https://review.openstack.org/#/c/426375/ without Depends-On now? | 12:20 |
| shardy | EmilienM: you are right though, there's a lot of changes in flight (as always!) | 12:20 |
| shardy | EmilienM: I think you need to wait for it to merge, or leave the Depends-On | 12:21 |
| EmilienM | shardy: the depends-on is on the right patch? | 12:23 |
| shardy | EmilienM: yes, removing the multinode_upgrade env from tripleo-ci won't work until https://review.openstack.org/#/c/434443/ lands | 12:24 |
| *** sudipto_ has joined #tripleo | 12:24 | |
| *** sudipto has joined #tripleo | 12:24 | |
| EmilienM | shardy: ok! thanks :-) | 12:24 |
| shardy | np :) | 12:24 |
| * shardy goes for lunch | 12:24 | |
| *** shardy is now known as shardy_lunch | 12:24 | |
| jaosorior | EmilienM: when you get some time can you review this patch for CI? https://review.openstack.org/#/c/427481/ | 12:25 |
| EmilienM | jaosorior: I don't think we want to carry an rpm in the git repo | 12:26 |
| EmilienM | jaosorior: it should be handled by RDO | 12:26 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Ansible role for built packages install https://review.openstack.org/426869 | 12:26 |
| *** athomas has joined #tripleo | 12:26 | |
| jaosorior | EmilienM: so, cloud-init is not part of RDO | 12:26 |
| jaosorior | it's part of base CentOS/RHEL | 12:26 |
| EmilienM | so the dependencies | 12:26 |
| jaosorior | and it gets even more complicated, cause the version that's there is broken, and the newest is broken as well | 12:26 |
| jaosorior | so we needed something in the middle, that actually works | 12:27 |
| EmilienM | jaosorior: then, it should be in cbs | 12:27 |
| jaosorior | (the issue has been reported already) | 12:27 |
| jaosorior | EmilienM: cbs? | 12:27 |
| EmilienM | in https://cbs.centos.org/repos/cloud7-openstack-ocata-testing/ | 12:27 |
| EmilienM | look https://cbs.centos.org/repos/cloud7-openstack-ocata-testing/x86_64/os/Packages/ | 12:28 |
| EmilienM | we have a ton of packages non-openstack | 12:28 |
| EmilienM | jaosorior: ask apevec or number80 on #rdo | 12:28 |
| gfidente | supermarios | 12:30 |
| gfidente | ^_^ | 12:30 |
| gfidente | marios ^^ doest that blink? | 12:31 |
| marios | this is what happens when you make the mistake of saying 'hello' to faidentee | 12:31 |
| marios | gfidente: chem fyi http://paste.openstack.org/raw/599209/ | 12:32 |
| *** limao has joined #tripleo | 12:32 | |
| *** limao has quit IRC | 12:33 | |
| *** limao has joined #tripleo | 12:33 | |
| *** funzo has joined #tripleo | 12:34 | |
| *** adarazs_lunch is now known as adarazs | 12:35 | |
| chem | marios: LGTM | 12:35 |
| *** dbecker has joined #tripleo | 12:35 | |
| jaosorior | EmilienM: is a solution like this acceptable https://review.openstack.org/#/c/430688/9/toci_gate_test.sh ? having to configure an extra repo for just one package seems a bit overkill to me | 12:37 |
| *** sudipto has quit IRC | 12:39 | |
| *** sudipto_ has quit IRC | 12:39 | |
| EmilienM | jaosorior: only if it's temporary and well documented | 12:39 |
| EmilienM | jaosorior: imho, all packages should go in RDO | 12:39 |
| *** funzo has quit IRC | 12:39 | |
| EmilienM | we shouldn't do some things like ssh & cloudinit tricks | 12:39 |
| EmilienM | well, ssh is special because it's related to our way to run CI | 12:39 |
| jaosorior | EmilienM: apevec just said it shouldn't be in RDO | 12:39 |
| EmilienM | though cloudinit affects all people wanted to try out the TLS everywhere thing | 12:39 |
| *** sdake_ has joined #tripleo | 12:40 | |
| EmilienM | jaosorior: in that case, do like ssh with a TODO to remove it when possible. | 12:40 |
| jaosorior | EmilienM: anyway, it's meant to be temporary; hopefully the bug will be fixed soon | 12:40 |
| EmilienM | jaosorior: ok, go ahead | 12:40 |
| *** sdake has quit IRC | 12:42 | |
| *** jerrygb has joined #tripleo | 12:42 | |
| *** jerrygb has quit IRC | 12:42 | |
| *** jerrygb has joined #tripleo | 12:43 | |
| jaosorior | sshnaidm: are you around? | 12:44 |
| sshnaidm | jaosorior, yep | 12:45 |
| jaosorior | sshnaidm: hey dude, you got acccess to 66.187.229.139 ? | 12:45 |
| jaosorior | would like to host a package temporarily as in here https://review.openstack.org/#/c/430688/9/toci_gate_test.sh | 12:45 |
| sshnaidm | jaosorior, yep, np | 12:45 |
| pradk | can i request some reviews on https://review.openstack.org/#/c/433824/ | 12:46 |
| jaosorior | sshnaidm: this rpm https://review.openstack.org/#/c/427481/26/packages/cloud-init-0.7.6-9.el7.x86_64.rpm,unified | 12:46 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui master: Cleanup Nodes selectors https://review.openstack.org/431515 | 12:46 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui master: Update DeploymentPlan page node counts https://review.openstack.org/433449 | 12:46 |
| *** pkovar has joined #tripleo | 12:47 | |
| *** psahoo has quit IRC | 12:48 | |
| sshnaidm | jaosorior, http://66.187.229.139/test/cloud-init-0.7.6-9.el7.x86_64.rpm | 12:49 |
| *** dougbtv has joined #tripleo | 12:49 | |
| sshnaidm | jaosorior, but pay attention not to use http_proxy | 12:49 |
| jaosorior | sshnaidm: will do, thanks man | 12:52 |
| EmilienM | jaosorior: in the code, if you can put URL of BZ or anything that can help anyone to track it, that would be cool | 12:53 |
| *** muskan_ has quit IRC | 12:54 | |
| *** salmankhan has quit IRC | 12:54 | |
| jaosorior | EmilienM: will do | 12:55 |
| *** bfournie has quit IRC | 12:55 | |
| *** chlong has joined #tripleo | 12:55 | |
| *** mdnadeem has quit IRC | 12:56 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Deploy novajoin in the TLS-everywhere job https://review.openstack.org/427481 | 12:56 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 12:56 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Ansible role for built packages install https://review.openstack.org/426869 | 12:57 |
| EmilienM | tripleo RC1 has been released, in a few min we'll have stable/ocata for all tripleo projects except tripleo-ui (need to be released later today) | 12:57 |
| openstackgerrit | OpenStack Release Bot proposed openstack/instack-undercloud stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434871 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/instack-undercloud master: Update reno for stable/ocata https://review.openstack.org/434872 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/instack stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434873 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/puppet-tripleo stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434874 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/puppet-tripleo master: Update reno for stable/ocata https://review.openstack.org/434875 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-common stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434876 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-common master: Update reno for stable/ocata https://review.openstack.org/434877 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-heat-templates stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434878 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-heat-templates master: Update reno for stable/ocata https://review.openstack.org/434879 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-image-elements stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434880 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-image-elements master: Update reno for stable/ocata https://review.openstack.org/434881 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-puppet-elements stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434882 | 12:59 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-puppet-elements master: Update reno for stable/ocata https://review.openstack.org/434883 | 12:59 |
| EmilienM | ok I think people know now \o/ | 13:00 |
| jaosorior | lol | 13:01 |
| jaosorior | nice | 13:01 |
| jpich | woohoo! | 13:01 |
| jpich | Hopefully we resolve the issue with tripleo-ui soon | 13:01 |
| *** lucas-bbl is now known as lucasagomes | 13:01 | |
| EmilienM | jpich: yeah, no pressure, no worries | 13:01 |
| *** jpena|lunch is now known as jpena | 13:04 | |
| marios | matbu: will you rebase https://review.openstack.org/#/c/431398/ we really need this one too ! | 13:06 |
| matbu | marios: yep sorry | 13:07 |
| marios | shardy_lunch: have you seen https://review.openstack.org/#/c/431398/ (otherwise, puppet get applied to the disabled roles too) | 13:07 |
| matbu | marios: i forgot, i was diging the ovs weirdness | 13:07 |
| *** salmankhan has joined #tripleo | 13:07 | |
| marios | matbu: ack np just remembered it | 13:07 |
| matbu | gerrit is slooowwww | 13:08 |
| marios | matbu: i grabbed puppet-pacemaker and puppet-tripleo from deloeran repo manually on my ctrls to see if i can get further on the puppet apply | 13:09 |
| marios | matbu: but i am skeptical (there may be some new required param in puppet-tripleo which my templates aren't passing or something) | 13:09 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 13:09 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK https://review.openstack.org/430277 | 13:10 |
| matbu | marios: you still have issue with puppet ? which ones ? (i got severals those days) | 13:10 |
| *** ccamacho is now known as ccamacho|lunch | 13:10 | |
| marios | matbu: well, several, i mean i hit one, fixed it (patched the files from the reviews o_O) then another etc... so today i just decided to try the upstram package ... | 13:11 |
| marios | matbu: first on was like Error while evaluating a Resource Statement, Class[Pacemaker::Stonith]: has no parameter named 'tries' | 13:12 |
| marios | matbu: then i got Error while evaluating a Resource Statement, Pacemaker::Resource::Ocf[galera]: has no parameter named 'location_rule | 13:12 |
| *** Trpger has joined #tripleo | 13:12 | |
| marios | matbu: then Error while evaluating a Resource Statement, Unknown resource type: 'pacemaker::constraint::order | 13:12 |
| *** shardy_lunch is now known as shardy | 13:12 | |
| openstackgerrit | Gabriele Cerami proposed openstack/tripleo-quickstart-extras master: [WIP] collect logs: use wildcard instead of specific user name for home collection https://review.openstack.org/434375 | 13:12 |
| matbu | marios: erf yep looks like a PITA | 13:13 |
| marios | matbu: afaics the changes that add those are all in puppet-tripleo/puppet-pacemaker (which is why i grabbed those onese specifically) | 13:13 |
| *** lucasagomes is now known as lucas-brb | 13:13 | |
| *** Trpger has quit IRC | 13:13 | |
| shardy | marios: Hey I'm not sure why that's needed see https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/post-upgrade.j2.yaml#L26 | 13:14 |
| shardy | marios: the include should do the same thing AFAICT | 13:14 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Deploy novajoin in the TLS-everywhere job https://review.openstack.org/427481 | 13:14 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job (novajoin) https://review.openstack.org/427482 | 13:14 |
| shardy | marios: Ah | 13:15 |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud-resource-registry-puppet.j2.yaml#L5 | 13:15 |
| shardy | it should be fixed there instead I think, will comment | 13:15 |
| openstackgerrit | Gabriele Cerami proposed openstack/tripleo-quickstart master: main script: add zuul-clone default path for upstream ci https://review.openstack.org/434900 | 13:16 |
| EmilienM | shardy: sorry, branch has been done. From now, backports for THT things | 13:16 |
| EmilienM | shardy: fine for you? | 13:16 |
| *** panda|lunch is now known as panda | 13:16 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh https://review.openstack.org/431029 | 13:16 |
| marios | shardy: yeah i was confused when i first saw that but i realised that on master we aren't using post-upgrade in the resource registry (is the comment i put at https://review.openstack.org/#/c/431398/1/environments/major-upgrade-composable-steps.yaml ) | 13:16 |
| shardy | marios: yeah, the mistake is in the base resource_registry | 13:17 |
| shardy | commented | 13:17 |
| matbu | shardy: yep marios commented abou | 13:17 |
| *** rhallisey has joined #tripleo | 13:17 | |
| shardy | it's only ever used for upgrades, so we can fix it there I think | 13:17 |
| shardy | EmilienM: ack, thanks! | 13:17 |
| matbu | shardy: but i put there because i thought we wanted to keep the all-in-one way | 13:17 |
| marios | shardy: cool thanks | 13:17 |
| shardy | matbu: yeah, I think we can't now because of nova | 13:17 |
| matbu | shardy: yep right | 13:17 |
| shardy | https://review.openstack.org/#/c/434250/ | 13:18 |
| shardy | matbu: when ^^ lands I'll remove the all-in-one env | 13:18 |
| shardy | or at least mark it as experimental and for developer use only | 13:18 |
| matbu | shardy: ack so then it make sense to add to resource-registry | 13:18 |
| shardy | I think it will still work provided the all-in-one role has upgrades enabled tho | 13:18 |
| shardy | matbu: yup, I think so, thanks for spotting it! :) | 13:18 |
| shardy | it was my mistake | 13:18 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: CI fix broken references to multinode-core.yaml https://review.openstack.org/434443 | 13:19 |
| *** dprince has joined #tripleo | 13:19 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: CI fix broken references to multinode-core.yaml https://review.openstack.org/434908 | 13:19 |
| EmilienM | shardy: backport to ocata ^ | 13:19 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: CI fix broken references to multinode-core.yaml https://review.openstack.org/434908 | 13:19 |
| *** jprovazn has quit IRC | 13:19 | |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras master: Add sanity checks to the baremetal overcloud and upgrades playbooks https://review.openstack.org/433914 | 13:19 |
| *** trown|outtypewww is now known as trown | 13:20 | |
| shardy | EmilienM: do you want bugs for all ocata backports yet, or are we OK to just propose the patches, then tag the final release later? | 13:20 |
| EmilienM | shardy: no need now | 13:21 |
| shardy | EmilienM: ack, thanks | 13:21 |
| EmilienM | shardy: if it's ok | 13:21 |
| shardy | EmilienM: yeah I agree, makes things easier until the final release | 13:21 |
| *** jprovazn has joined #tripleo | 13:21 | |
| EmilienM | yeah | 13:21 |
| *** limao has quit IRC | 13:24 | |
| *** diana_clarke has joined #tripleo | 13:25 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh https://review.openstack.org/431029 | 13:26 |
| *** bkopilov_ has quit IRC | 13:27 | |
| *** bkopilov has quit IRC | 13:27 | |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci master: Enable Ceph in multinode upgrades job (via HCI). https://review.openstack.org/434855 | 13:28 |
| *** jayg|g0n3 is now known as jayg | 13:28 | |
| matbu | shardy: im pushing the rebase of the review, so it will drop your comment, but then i'll address it ... idk why gerrit is so slow to me | 13:28 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-heat-templates master: Apply post-upgrade step to not run puppet in post upgrade https://review.openstack.org/431398 | 13:28 |
| *** bfournie has joined #tripleo | 13:29 | |
| *** limao has joined #tripleo | 13:31 | |
| shardy | matbu: ack, no problem, thanks! | 13:31 |
| dprince | flaper87: not even going to reply to your email. You guys picked the wrong side though :( | 13:31 |
| *** thrash|g0ne is now known as thrash | 13:32 | |
| shardy | dprince: FWIW It's not about sides - our current logo doesn't meet the common look/theme requirements defined by the foundation, so we have to reach a compromise | 13:33 |
| flaper87 | dprince: tbh, didn't know there was a side. Those are just opinions and I think it's fine to express them | 13:33 |
| shardy | I like the one we have, but we'll have to also have something foundation approved, I don't think there is any other choice because they want a consistent look for marketing reasons | 13:33 |
| dprince | shardy: that isn't what I heard | 13:34 |
| *** abregman is now known as abregman|mtg | 13:34 | |
| dprince | shardy: They can choose to use things as is I think | 13:34 |
| *** abregman|mtg has quit IRC | 13:34 | |
| shardy | honestly, I like the old logo quite a bit, but it's not worth burning a lot of developer brain cycles on IMO, particularly when we can still use the old logo where we want | 13:34 |
| dprince | shardy: and if they must have an alternate, then fine. But we can keep the existing one | 13:34 |
| hewbrocca | wut no changing the owl | 13:34 |
| flaper87 | TBH, I think we have bigger things to worry about and I'd be happy to let someone else take care of the logo stuff but I just got here so *Shrugs* | 13:34 |
| shardy | " But for official channels like the website, we need a logo to represent TripleO that’s cohesive with the rest of the set. " | 13:34 |
| shardy | dprince: that is what was in the mail EmilienM forwarded at the very start of the thread | 13:35 |
| dprince | shardy: nah, that is just an overstatement | 13:35 |
| dprince | shardy: They can use our logo as is I think. It would be fine for them | 13:35 |
| dprince | shardy: You drank the koolaid man | 13:35 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-heat-templates master: Apply post-upgrade step to not run puppet in post upgrade https://review.openstack.org/431398 | 13:36 |
| shardy | dprince: well I was approached about this months ago when I was PTL, and I said we already had a logo | 13:36 |
| dprince | shardy: that was the right answer ;) | 13:36 |
| hewbrocca | which rocks | 13:36 |
| shardy | dprince: and at no time, ever, was approving it for foundation use offered as an option | 13:36 |
| shardy | so it's not about drinking any koolaid, and yes it does feel more coorporate than open-source | 13:36 |
| shardy | but honestly I don't think arguing is going to help, we'll have to reach a compromise | 13:37 |
| dprince | shardy: yeah, well the tactic now being used (by Monty and co) is that we are bad people if we don't have a common set of logos | 13:37 |
| EmilienM | folks it's a logo | 13:37 |
| diana_clarke | n00b questions incoming.... | 13:37 |
| dprince | shardy: the foundation shouldn't even bother with any of this | 13:37 |
| diana_clarke | How do you know if the tripleo quickstart script exited successfully? I'm following this: https://github.com/openstack/tripleo-quickstart | 13:37 |
| EmilienM | I would love people focusing on the right things, specially in a release time | 13:37 |
| diana_clarke | I'm assuming I should have an overcloud after a successful quickstart run. Is that correct? | 13:37 |
| dprince | EmilienM: I know. So let us have what we want! | 13:38 |
| EmilienM | diana_clarke: welcome here Diana | 13:38 |
| diana_clarke | After 78 minutes, The "PLAY RECAP" doesn't show errors, nor does the scroll prior, but it still doesn't match the docs, so I'm not convinced it worked. | 13:38 |
| diana_clarke | For example, the docs [1] say I should be able to source the overcloudrc from the undercloud, but I don't see an overcloudrc. | 13:38 |
| diana_clarke | [1] https://docs.openstack.org/developer/tripleo-quickstart/accessing-undercloud.html | 13:38 |
| flaper87 | EmilienM: exactly my thoughts, fwiw | 13:38 |
| diana_clarke | Nor do I have anything overcloud related in the resulting ssh.config.ansible which I see refered here in the docs [2]. | 13:38 |
| diana_clarke | [2] based on this: https://docs.openstack.org/developer/tripleo-quickstart/accessing-overcloud.html | 13:38 |
| diana_clarke | EmilienM: Thanks! | 13:39 |
| *** fultonj has joined #tripleo | 13:39 | |
| *** links has quit IRC | 13:39 | |
| gfidente | dprince flaper87 I liked and would push for what ccamacho|lunch said | 13:40 |
| *** rlandy has joined #tripleo | 13:40 | |
| gfidente | I'd propose they iterate over the existing logo, proposing minimal changes to fit within "requirements" (so she says in the email) | 13:40 |
| gfidente | but, I'd also like to see the "requirements" fwiw | 13:40 |
| flaper87 | gfidente: if someone has the time for that, sure. | 13:41 |
| flaper87 | what that person is? dunno | 13:41 |
| *** sudipto_ has joined #tripleo | 13:41 | |
| *** sudipto has joined #tripleo | 13:41 | |
| gfidente | flaper87 well, I assume same people who spent lot of time for the blue one | 13:41 |
| EmilienM | diana_clarke: at the end of the quickstart run, the undercloud should be ready and the overcloud nodes started, ready to be configured | 13:41 |
| gfidente | could have approached things this way, iterating over existing logos | 13:41 |
| EmilienM | diana_clarke: can you maybe paste the output in a pastbin please? So we can look at details | 13:41 |
| shardy | diana_clarke: Hi, welcome! | 13:42 |
| flaper87 | gfidente: well, not really. but again, I didn't work on it so I'd probably go and ask them why that was not the case | 13:42 |
| shardy | trown: Hey re ^^ is there a reference to the quickstart tags, as IIRC it only deploys the undercloud by default? | 13:42 |
| diana_clarke | The last tasks run are: "validate the overcloud", and then "Execute tempest against the overcloud". Does that look like the end of a successful run? | 13:43 |
| EmilienM | diana_clarke: yes, exactly | 13:43 |
| EmilienM | diana_clarke: just curious, on what kind of HW do you run quickstart? | 13:44 |
| shardy | diana_clarke: yes, but if you log onto the undercloud and do . stackrc && openstack stack list you should see the "overcloud" heat stack in CREATE_COMPLETE state | 13:44 |
| shardy | and nova list will show you the running overcloud nodes, etc | 13:44 |
| diana_clarke | EmilienM: But I don't have any overcloud config etc... should I? That is, are these docs current/correct? https://docs.openstack.org/developer/tripleo-quickstart/accessing-overcloud.html https://docs.openstack.org/developer/tripleo-quickstart/accessing-undercloud.html | 13:45 |
| sshnaidm | diana_clarke, you have ssh.config.ansible in your working dir, usually ~/.quickstart, right? | 13:48 |
| diana_clarke | shardy: Most of the time I get a 504 when I do a "heat stack-list" or a "openstack stack list", but when it does work, I just get an empty table. | 13:48 |
| *** sdake_ is now known as sdake | 13:48 | |
| diana_clarke | sshnaidm: Yes, but ssh.config.ansible only has undercloud info. The docs seems to suggest I should also see overcloud stanzas. | 13:49 |
| *** lucas-brb is now known as lucasagomes | 13:49 | |
| *** dbecker has quit IRC | 13:49 | |
| sshnaidm | diana_clarke, can you show the log of the run please? | 13:49 |
| honza | jtomasek: webpack 1 => 2 converstion tool (cli) is almost ready https://github.com/webpack/webpack-cli/pull/40 | 13:50 |
| diana_clarke | sshnaidm: where would you like it? openstack pastebin will truncate it. | 13:50 |
| sshnaidm | diana_clarke, if you have 504 and don't have overcloud nodes in ssh file, most likely the deployment failed | 13:50 |
| honza | jtomasek: but lol, you probably also follow @dabramov on twitter, nevermind :) | 13:51 |
| *** tosky has quit IRC | 13:52 | |
| diana_clarke | sshnaidm: That's what I figured too... (obviously, lol). But it appeared to complete successfully... So you have logs/docs showing what a successful run looks like? | 13:52 |
| sshnaidm | diana_clarke, gist on github? | 13:52 |
| diana_clarke | s/So/Do/ | 13:52 |
| *** dbecker has joined #tripleo | 13:52 | |
| *** ealcaniz has quit IRC | 13:52 | |
| sshnaidm | diana_clarke, for example: http://logs.openstack.org/27/433627/1/experimental-tripleo/gate-tripleo-ci-centos-7-ovb-nonha-oooq-nv/0cd618b/console.html#_2017-02-14_14_53_44_968837 | 13:53 |
| sshnaidm | diana_clarke, look for "TASK [did the deployment pass or fail?]" | 13:53 |
| diana_clarke | sshnaidm: okay, thanks. checking... brb | 13:54 |
| *** bkopilov_ has joined #tripleo | 13:54 | |
| *** bkopilov has joined #tripleo | 13:54 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 13:54 |
| marios | matbu: minor nit https://review.openstack.org/#/c/431398/3/overcloud-resource-registry-puppet.j2.yaml | 13:55 |
| openstackgerrit | Merged openstack/python-tripleoclient master: Fix fencing action parameter name. https://review.openstack.org/433649 | 13:56 |
| matbu | marios: yep thanks, /me updating | 13:57 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Update reno for stable/ocata https://review.openstack.org/434879 | 13:57 |
| openstackgerrit | Merged openstack/tripleo-common master: Update reno for stable/ocata https://review.openstack.org/434877 | 13:57 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Update reno for stable/ocata https://review.openstack.org/434875 | 13:57 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements master: Update reno for stable/ocata https://review.openstack.org/434883 | 13:57 |
| openstackgerrit | Merged openstack/tripleo-image-elements master: Update reno for stable/ocata https://review.openstack.org/434881 | 13:57 |
| *** jmelvin has joined #tripleo | 13:57 | |
| *** ealcaniz has joined #tripleo | 13:58 | |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-heat-templates master: Apply post-upgrade step to not run puppet in post upgrade https://review.openstack.org/431398 | 13:59 |
| *** milan has quit IRC | 13:59 | |
| *** sudipto has quit IRC | 13:59 | |
| *** sudipto_ has quit IRC | 13:59 | |
| *** noslzzp has quit IRC | 13:59 | |
| *** ansmith has joined #tripleo | 13:59 | |
| *** sasha2 has joined #tripleo | 14:00 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: remove duplicate network_environment from undercloud-deploy role https://review.openstack.org/434924 | 14:00 |
| *** noslzzp has joined #tripleo | 14:00 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: Fix quickstart issues https://review.openstack.org/425459 | 14:01 |
| *** gatuus has joined #tripleo | 14:02 | |
| *** gatuus_ has joined #tripleo | 14:02 | |
| *** jprovazn has quit IRC | 14:03 | |
| trown | shardy: ya --tags all will do full deploy ... we are considering dropping tags for something better in pike | 14:03 |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud master: Set project name explicitly to service for panko https://review.openstack.org/434925 | 14:04 |
| *** kberger has quit IRC | 14:06 | |
| *** kberger has joined #tripleo | 14:06 | |
| jaosorior | trown: how would it look like if not with tags? | 14:07 |
| trown | jaosorior: using vars instead of tags | 14:07 |
| trown | jaosorior: so flow control could be done on the config level instead | 14:08 |
| trown | jaosorior: we already use that, it is just some pretty big effort to switch all tags to that interface instead | 14:08 |
| jaosorior | right | 14:08 |
| jaosorior | makes sense | 14:08 |
| diana_clarke | sshnaidm: Seeing that log vs mine really helped. Good tip! My logs don't show those final deploy overcloud steps, but I think I finally know why... | 14:08 |
| *** abregman has joined #tripleo | 14:09 | |
| *** morazi has joined #tripleo | 14:09 | |
| *** lblanchard has joined #tripleo | 14:10 | |
| diana_clarke | sshnaidm: I suspect it's because I followed this command in the quickstart docs: "bash quickstart.sh $VIRTHOST" rather than the one under it with the "--tags all" option. | 14:10 |
| sshnaidm | diana_clarke, so you maybe you installed undercloud only | 14:11 |
| *** cdearborn has joined #tripleo | 14:11 | |
| *** ealcaniz has quit IRC | 14:11 | |
| *** dsavineau has joined #tripleo | 14:11 | |
| sshnaidm | diana_clarke, it's default behavior | 14:12 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/tripleo-docs master: Add some documentation about VirtualBMC and pxe_ipmitool https://review.openstack.org/434286 | 14:12 |
| *** ealcaniz has joined #tripleo | 14:12 | |
| diana_clarke | This isn't true though, right? "This script will output instructions at the end to access the deployed undercloud." | 14:13 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 14:13 |
| shardy | diana_clarke: if it runs successfully, there's a little print showing how to access the undercloud | 14:13 |
| shardy | so sounds like something went wrong in your case | 14:13 |
| diana_clarke | shardy: This is what I see: https://gist.github.com/dianaclarke/2ebbaeb7e0454a5efc501207dd55a33f | 14:13 |
| *** eck`gone is now known as eck` | 14:13 | |
| shardy | https://github.com/openstack/tripleo-quickstart/blob/master/quickstart.sh#L454 | 14:14 |
| shardy | diana_clarke: that message ^^ is what you should see I think | 14:14 |
| *** gatuus_ has quit IRC | 14:15 | |
| *** gatuus has quit IRC | 14:15 | |
| diana_clarke | shardy: Thanks, I wondered if I should be seeing some indication of success. Other than no errors in the plat recap. | 14:15 |
| diana_clarke | *play | 14:15 |
| shardy | diana_clarke: yeah to me that looks like it exited early, but it's not a super great error message | 14:15 |
| jaosorior | diana_clarke: you could still try to log into your undercloud like this: ssh -F ~/.quickstart/ssh.config.ansible undercloud | 14:16 |
| diana_clarke | I don't even see an error message... or is that info dump at the end an error message? | 14:16 |
| diana_clarke | jaosorior: yup, I can get to the undercloud just fine | 14:17 |
| jaosorior | diana_clarke: I actually don't see the error message there either :/ it's pretty confusing why it didn't just print the instructions on success | 14:17 |
| *** [2]cdearborn has joined #tripleo | 14:17 | |
| shardy | https://github.com/openstack/tripleo-quickstart/blob/master/quickstart.sh#L449 | 14:17 |
| shardy | I think it got there, then the conditional to print the message evaluated to false | 14:17 |
| shardy | so as jaosorior said, you can probably log on to the undercloud and go ahead, but we can't currently see why it evaluated false | 14:18 |
| *** owalsh has joined #tripleo | 14:18 | |
| *** michapma_alt has quit IRC | 14:19 | |
| diana_clarke | Thanks, folks! I'll try running the quickstart again, this time with "--tags all". See you in a few hours ;) | 14:19 |
| thrash | diana_clarke: late to the party, but are you using devmode? | 14:19 |
| *** ansmith has quit IRC | 14:20 | |
| *** iranzo has quit IRC | 14:20 | |
| *** michapma_alt has joined #tripleo | 14:20 | |
| *** ccamacho|lunch is now known as ccamacho | 14:20 | |
| diana_clarke | thrash: No, not yet. I'm just trying to get easy mode to work first ;) | 14:20 |
| *** lblanchard has quit IRC | 14:20 | |
| *** lblanchard has joined #tripleo | 14:21 | |
| thrash | diana_clarke: ok. I'm wondering if it didn't spit out the "instructions" because you used the extras as well, and went all the way through an overcloud deployment. | 14:22 |
| *** ansmith has joined #tripleo | 14:23 | |
| *** liverpooler has joined #tripleo | 14:23 | |
| *** iranzo has joined #tripleo | 14:23 | |
| *** iranzo has joined #tripleo | 14:23 | |
| *** funzo has joined #tripleo | 14:24 | |
| ccamacho | gfidente dprince flaper87 I do like a lot the foundation initiative of aligning all projects image (logos) the only thing is that they didn't asked and started from scratch something that we already had (I like what they are doing i.e. with Ironic) Im hopeful about what they said about improving what we currently have **I drafted something briefly yesterday and I hope they will listen to us :) | 14:24 |
| *** tzumainn has joined #tripleo | 14:25 | |
| *** [1]cdearborn has quit IRC | 14:25 | |
| adarazs | matbu, EmilienM, trown, sshnaidm, panda, rlandy, bkero: https://etherpad.openstack.org/p/tripleo-ci-squad-meeting -- CI squad meeting in an hour, add topics if you'd like | 14:26 |
| adarazs | matbu: please join to discuss the update/upgrade part. | 14:27 |
| *** adarazs is now known as adarazs_afk | 14:27 | |
| *** skramaja has quit IRC | 14:28 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Ansible role for built packages install https://review.openstack.org/426869 | 14:28 |
| EmilienM | adarazs_afk: thank you sir | 14:29 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 14:29 |
| *** sudipto_ has joined #tripleo | 14:29 | |
| *** sudipto has joined #tripleo | 14:29 | |
| *** jprovazn has joined #tripleo | 14:29 | |
| *** suuuper has quit IRC | 14:30 | |
| *** pblaho has quit IRC | 14:30 | |
| rlandy | adarazs: matbu and I will be 15 mins late for squad meeting - we have another updates related meeting | 14:31 |
| jtomasek | honza: cool | 14:31 |
| *** Goneri has joined #tripleo | 14:31 | |
| *** cdearborn has quit IRC | 14:32 | |
| *** [2]cdearborn is now known as cdearborn | 14:32 | |
| dprince | ccamacho: I would be more okay with it if they didn't take the attitude that we are "bad" or "not community" if we keep our original too | 14:32 |
| dprince | ccamacho: like, they can go and style one for their website. Sure. But it always could just point back to the original | 14:33 |
| EmilienM | amoralej, jpena: I might need your help | 14:33 |
| EmilienM | sounds like some jobs are failing on stable/ocata | 14:33 |
| EmilienM | https://review.openstack.org/#/c/434878/ | 14:33 |
| amoralej | EmilienM, checking | 14:33 |
| dprince | ccamacho: it is the position that we are being forced to adopt their version as well that I'm not a fan of | 14:33 |
| EmilienM | amoralej: http://logs.openstack.org/78/434878/1/check/gate-tripleo-ci-centos-7-nonha-multinode/5adbd7d/console.html#_2017-02-16_13_12_25_215249 | 14:33 |
| jpena | EmilienM: is that python-tripleoclient failing to install? | 14:34 |
| dprince | ccamacho: and it pains me that they have managed to convince most of the core team this is the only option. When in fact it isn't. We can keep whichever logo we want IMO | 14:34 |
| EmilienM | jpena: no, it sounds like THT | 14:34 |
| amoralej | EmilienM, we are working on it | 14:34 |
| EmilienM | amoralej: excellent! | 14:34 |
| amoralej | we've replaced heat-templates by heat-agents and we messed it up | 14:34 |
| EmilienM | i'm not sure it's related | 14:34 |
| EmilienM | the error is with Error: Package: instack-undercloud-6.0.0-0.20170215205650.a4e4105.el7.centos.noarch (delorean) right? | 14:35 |
| EmilienM | oh | 14:35 |
| EmilienM | Available: openstack-heat-templates-0-0.5.1e6015dgit.el7.noarch (openstack-newton) | 14:35 |
| EmilienM | amoralej: that ^? | 14:35 |
| amoralej | yes | 14:35 |
| EmilienM | ok ok | 14:35 |
| EmilienM | amoralej: let me know if I can help | 14:36 |
| amoralej | EmilienM, https://github.com/redhat-openstack/rdoinfo/commit/607b4cc8100f57cc84202e80fe90da5f10233814 | 14:36 |
| ccamacho | dprince ++ I agree on that, they should not force us about using something not approved by the community, but, Im not against of working to improve what we currently have to make it "fit" their style guidelines. I think the error was to said "Hey guys, this will be your new logo" And we said? What? Which new logo?? Let's wait to see that they will propose :) | 14:37 |
| shardy | matbu, owalsh: Hey any idea about https://paste.fedoraproject.org/paste/U3B7X9ZaPomWC-TQDxNO315M1UNdIGYhyRLivL9gydE= | 14:38 |
| shardy | it seems nova api db_sync breaks as there's a duplicate 021 migration | 14:38 |
| jrist | ok | 14:39 |
| shardy | is there a placeholder we have mistakenly added to packaging or something? | 14:39 |
| *** saneax is now known as saneax-_-|AFK | 14:39 | |
| matbu | shardy: oops paste not found | 14:40 |
| d0ugal | EmilienM: Do you think the -2 can be lifted here? https://review.openstack.org/#/c/425047/ | 14:40 |
| shardy | http://paste.openstack.org/show/599247/ | 14:40 |
| shardy | matbu: try ^^ sorry, it's not line-wrapped | 14:41 |
| dprince | ccamacho: sometimes keeping an original is cool too though (https://www.kernel.org/). Not to say they can't *style* it for their website if they need it too (https://en.wikipedia.org/wiki/Tux). But if you read between their lines what really sets me off, (and why I'm dissappointed in some of the responses from TripleO members) is they've managed to convince you that we are bad or not community if we keep | 14:41 |
| dprince | our original. *That* is what I read between the lines here, and it is a bad way to interact | 14:41 |
| shardy | 021_build_requests_instance_mediumtext.py 021_placeholder.py | 14:41 |
| shardy | it looks like 021_placeholder.py shouldn't be included? | 14:41 |
| EmilienM | ccamacho: yes, it sounds like a good plan | 14:42 |
| owalsh | shardy: looks like 028 was backported to 021 in stable/newton | 14:42 |
| EmilienM | ccamacho: and that's what we agreed as a team, I thought at least | 14:42 |
| shardy | owalsh: Hmm, weird, this should be after the package upgrade to ocata | 14:42 |
| * shardy checks repos | 14:42 | |
| EmilienM | d0ugal: done | 14:43 |
| d0ugal | EmilienM: thanks! | 14:43 |
| EmilienM | d0ugal: -1 though :P | 14:43 |
| d0ugal | EmilienM: It is just an ever so slighly better version of https://github.com/openstack/tripleo-common/blob/master/releasenotes/notes/5.8.0-d1ca2298ba598431.yaml#L34-L35 | 14:44 |
| owalsh | shardy: it's a nova bug, 028 needs to be renamed to 021_build_requests_instance_mediumtext.py in ocata & master | 14:44 |
| EmilienM | d0ugal: ok | 14:45 |
| shardy | owalsh: ack, OK I see - want to to raise a LP bug? | 14:45 |
| shardy | python-nova-15.0.0-0.20170215034806.bdeb05d.el7.centos.noarch is the version I'm testing | 14:45 |
| owalsh | shardy: yes please | 14:45 |
| shardy | owalsh: ack, on it, thanks! | 14:45 |
| *** apetrich has quit IRC | 14:46 | |
| ccamacho | dprince I believe we are a quite strong community and non of us are bad, I mean we are discussing about it, right? They can always use the stylized version of our current logo and if we like it enough well we have also the freedom of using it or not :) (In any case saying "Use this" is not the most polite way of asking for changes) | 14:46 |
| *** ealcaniz has quit IRC | 14:47 | |
| *** apetrich has joined #tripleo | 14:47 | |
| *** tosky has joined #tripleo | 14:47 | |
| *** suuuper has joined #tripleo | 14:48 | |
| dprince | ccamacho: when I read the conversation on IRC it sounds like most people prefered the original. Thats all. Like "we'll miss him", etc. And when I ask... it doesn't seem like people think there was a choice. THe conclusion was we had to adopt their version | 14:48 |
| *** athomas has quit IRC | 14:50 | |
| *** jjung has quit IRC | 14:50 | |
| cschwede_ | can someone please review https://review.openstack.org/#/c/414460/ ? it fixes the up-/downscaling/node replacement bug with Swift. pls ping me if you have questions on that patch | 14:51 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras master: Implement overcloud sanity checks role https://review.openstack.org/429716 | 14:51 |
| dprince | ccamacho: so you see, the power is theirs now. They have convinced you that if you don't use the same version they pick then you are inferior | 14:51 |
| shardy | owalsh: https://bugs.launchpad.net/nova/+bug/1665370 | 14:51 |
| openstack | Launchpad bug 1665370 in OpenStack Compute (nova) "api db_sync fails on upgrade due to duplicate migration files" [Undecided,New] | 14:51 |
| pradk | can we get these two simple changes in asap - https://review.openstack.org/#/c/434920/ and https://review.openstack.org/#/c/434925/ | 14:52 |
| EmilienM | cschwede_: sounds like agood backport candidate, I'll let you do it when it merges | 14:52 |
| pradk | these are blocking CF | 14:52 |
| dprince | ccamacho: believe this is a team thing if you want though. But I don't think the team would've chosen this week/time to recreate our Logo (right in the middle of a release) | 14:52 |
| ccamacho | hey dprince I think there is some confusion there, what I have in my head is: "They will try to adapt our current logo version to their styles guidelines, from there they will have an official version for the foundation and we are free to adopted" | 14:52 |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart-extras master: Consider also newton for bulk introspection https://review.openstack.org/434952 | 14:53 |
| shardy | owalsh: I'm not quite on stable/newton tip with my image but I don't see any recent fixes which are related | 14:54 |
| dprince | ccamacho: right, there is confustion. Ours is the vintange, or offical version if you like. Theres is just a stylized version for their website | 14:54 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras master: Implement overcloud sanity checks role https://review.openstack.org/429716 | 14:54 |
| dprince | ccamacho: saying it that way honors the creativity that went into this | 14:54 |
| EmilienM | cschwede_: +2 - do you think we need a release note on top? | 14:54 |
| EmilienM | pradk: I'll review it shortly. Please make sure it's backported in puppet-panko and released by mwhahaha and backported in instack-undercloud (I'll release rc2 next week) | 14:55 |
| dprince | ccamacho: saying theirs is *official* is just another way of saying we are the ones who aren't in compliance | 14:55 |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart-extras master: Consider also newton for bulk introspection https://review.openstack.org/434952 | 14:55 |
| pradk | EmilienM, sure | 14:56 |
| cschwede_ | EmilienM: yep, i'd like to do a backport once it merges. i can add a reno - is it ok if that's a follow up? | 14:56 |
| EmilienM | pradk: I think you missed to update rspec tests | 14:56 |
| EmilienM | cschwede_: no prob | 14:56 |
| pradk | EmilienM, also https://review.openstack.org/#/c/433824/ | 14:56 |
| pradk | EmilienM, i'll check | 14:56 |
| owalsh | shardy: thanks, I'll bring it up on #openstack-nova | 14:56 |
| ccamacho | dprince +1 or we can say that we are vintage revolutionaries | 14:56 |
| EmilienM | pradk: +2. Same thing please make sure it's backported. | 14:57 |
| cschwede_ | EmilienM: ok, i'll write a reno today and will submit it later | 14:57 |
| dprince | ccamacho: right, and ditto for Ironic. Who have also been "shamed" into this new line of thinking and should consider their old bear a keepsake | 14:57 |
| EmilienM | cschwede_: thx + backport too | 14:57 |
| openstackgerrit | Gabriele Cerami proposed openstack/tripleo-quickstart-extras master: publish: don't fail in case the logs dir is a link to a directory https://review.openstack.org/434955 | 14:58 |
| *** athomas has joined #tripleo | 15:00 | |
| *** hjensas has quit IRC | 15:02 | |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart-extras master: Consider also newton for bulk introspection https://review.openstack.org/434952 | 15:05 |
| *** chlong has quit IRC | 15:05 | |
| jrist | jpich: recheck passed | 15:06 |
| jrist | no need for the my little pony comb | 15:07 |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci master: Enable Ceph in multinode upgrades job (via HCI). https://review.openstack.org/434855 | 15:08 |
| jpich | jrist: You never know when a yak might need some good shavin' | 15:08 |
| *** jkilpatr has quit IRC | 15:09 | |
| *** jkilpatr has joined #tripleo | 15:09 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-common master: Open log in utf-8 to prevent UnicodeEncodeError https://review.openstack.org/434520 | 15:09 |
| jrist | jpich: thanks for taking care of that | 15:09 |
| *** thrash is now known as thrash|biab | 15:12 | |
| jpich | jrist: No problems, jpena in #rdo should get all the credit for resolving this :) | 15:14 |
| jrist | heh | 15:14 |
| jrist | jpena++ | 15:14 |
| jaosorior | EmilienM: today was the CI squad meeting, right? | 15:14 |
| EmilienM | jaosorior: yes | 15:15 |
| EmilienM | in 15 min | 15:15 |
| jaosorior | EmilienM: what channel? | 15:15 |
| EmilienM | jaosorior: on BJ | 15:15 |
| EmilienM | jaosorior: https://bluejeans.com/u/whayutin | 15:15 |
| jaosorior | ko | 15:15 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart-extras master: Allow direct override of the overcloud's DNS servers https://review.openstack.org/434963 | 15:16 |
| trown | jaosorior: like ^ | 15:16 |
| jaosorior | trown: nice! | 15:17 |
| *** suuuper has quit IRC | 15:17 | |
| *** suuuper has joined #tripleo | 15:17 | |
| *** saneax-_-|AFK is now known as saneax | 15:18 | |
| jaosorior | trown: that was exactly what I needed, thanks dude | 15:18 |
| trown | jaosorior: ya, we have lots of those dicts in our configs, anything we need to override should be done like that | 15:18 |
| *** PhilSliderS has joined #tripleo | 15:19 | |
| *** hjensas has joined #tripleo | 15:19 | |
| *** hjensas has quit IRC | 15:19 | |
| *** hjensas has joined #tripleo | 15:19 | |
| *** adarazs_afk is now known as adarazs | 15:21 | |
| *** yprokule has quit IRC | 15:24 | |
| *** saibarspeis has quit IRC | 15:26 | |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Skipping metadata definition tests https://review.openstack.org/434970 | 15:27 |
| EmilienM | all rc1 bugs moved to rc2 | 15:27 |
| EmilienM | for High / Critical bugs, please use rc2 unless you think it's not going to make it on time | 15:28 |
| EmilienM | then use pike-1 and add the ocata-backport-potential tag | 15:28 |
| EmilienM | trown, panda, sshnaidm: meeting time! | 15:30 |
| adarazs | jaosorior: ^ if you'd like to join. :) | 15:30 |
| adarazs | https://redhat.bluejeans.com/4113567798/ | 15:30 |
| *** chem has quit IRC | 15:30 | |
| EmilienM | some stats: tripleo rc1: 62 Fix Released & 7 blueprints Implemented | 15:30 |
| jaosorior | adarazs: I'm in | 15:31 |
| jpich | go us! | 15:31 |
| d0ugal | Do we have CI issues on the Ocata branch? | 15:32 |
| EmilienM | d0ugal: yes | 15:32 |
| *** sudipto has quit IRC | 15:32 | |
| *** sudipto_ has quit IRC | 15:32 | |
| d0ugal | EmilienM: How can I help? | 15:32 |
| EmilienM | d0ugal: amoralej is working on it. Paste me the link to make sure it's the same we have | 15:32 |
| EmilienM | d0ugal: I think it's under control, thanks | 15:33 |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient stable/ocata: Make overcloud image upload honor --image-path https://review.openstack.org/434975 | 15:33 |
| d0ugal | EmilienM: top ~5 patches. https://review.openstack.org/#/q/status:open+project:openstack/mistral,n,z | 15:33 |
| EmilienM | looks like something else: | 15:34 |
| EmilienM | http://logs.openstack.org/25/434825/1/gate/gate-tripleo-ci-centos-7-scenario003-multinode-puppet/92b1e24/console.html#_2017-02-16_13_29_56_054914 | 15:34 |
| EmilienM | amoralej, jpena ^ are you aware about that one? | 15:34 |
| amoralej | yes EmilienM | 15:34 |
| EmilienM | d0ugal: if it's blocking you, we can make the job non voting for mistral | 15:34 |
| EmilienM | amoralej: any eta on the fix? just to take decision if we set the job non voting for mistral in the meantime | 15:34 |
| jpena | EmilienM: yes, give us a few minutes and it should be fixed. I'll ping you when we can recheck | 15:34 |
| d0ugal | EmilienM: it is blocking backports for Mistral - not sure how urgent they are. | 15:35 |
| EmilienM | ok :D | 15:35 |
| *** jaganathan has quit IRC | 15:37 | |
| gfidente | matbu you around? | 15:38 |
| matbu | gfidente: in mtg | 15:39 |
| matbu | gfidente: but yes :) | 15:39 |
| jaosorior | hrybacki: if you have some time can you help me debug what I messed up here https://review.openstack.org/#/c/422310/ ? | 15:40 |
| hrybacki | jaosorior: aye | 15:40 |
| gfidente | matbu somewhere the multinode upgrade job must be saying that it wantes NovaCompute on the controller role | 15:42 |
| gfidente | where is it? | 15:42 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add sanity checks to the baremetal overcloud and upgrades playbooks https://review.openstack.org/433914 | 15:43 |
| gfidente | matbu environments/multinode.yaml ? | 15:43 |
| matbu | gfidente: i guess it's tht/ci/scenario | 15:43 |
| gfidente | ack ci/environments | 15:44 |
| matbu | https://github.com/openstack/tripleo-heat-templates/tree/master/ci/environments | 15:44 |
| matbu | yep sorry | 15:44 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-heat-templates master: Add missing releasenotes for Swift ring management https://review.openstack.org/434977 | 15:44 |
| gfidente | I am adding CephOSD there | 15:44 |
| gfidente | for more fun | 15:44 |
| *** fragatina has joined #tripleo | 15:44 | |
| matbu | gfidente: hehe nice thanks :) | 15:44 |
| gfidente | ty | 15:44 |
| matbu | we love fun | 15:44 |
| gfidente | and breakage! | 15:45 |
| gfidente | add Ceph* for for fun and breakage | 15:45 |
| *** fragatina has quit IRC | 15:45 | |
| matbu | hehe + upgrade , you win | 15:45 |
| gfidente | yah | 15:46 |
| *** fragatina has joined #tripleo | 15:46 | |
| *** saneax is now known as saneax-_-|AFK | 15:46 | |
| hrybacki | jaosorior: simple bug, just need to move `enable_tls_everywhere` into extras-common/defaults -- I added a comment to your review | 15:47 |
| jaosorior | hrybacki: which file? | 15:47 |
| * hrybacki grabs lines | 15:48 | |
| hrybacki | jaosorior: line 31: https://review.openstack.org/#/c/422310/9/roles/overcloud-deploy/defaults/main.yml | 15:48 |
| hrybacki | to | 15:48 |
| hrybacki | https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/extras-common/defaults/main.yml | 15:48 |
| hrybacki | all of the roles inherit from ^^ and ^^ inherits from tripleo-quickstart/roles/common/ | 15:48 |
| *** rbrady has joined #tripleo | 15:49 | |
| *** rbrady has joined #tripleo | 15:49 | |
| *** salmankhan has quit IRC | 15:50 | |
| *** salmankhan has joined #tripleo | 15:50 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Update ci environment for multinode jobs to support Ceph https://review.openstack.org/434981 | 15:51 |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci master: Enable Ceph in multinode upgrades job https://review.openstack.org/434855 | 15:52 |
| *** twi99y has quit IRC | 15:55 | |
| *** twi99y has joined #tripleo | 15:55 | |
| marios | mascotgate | 15:55 |
| gfidente | EmilienM that does not enable Ceph in the multinode by default | 15:56 |
| gfidente | because the roles are disabled | 15:56 |
| gfidente | it allows us to use ceph in the multinode jobs though, when the roles are enabled in the heat registry | 15:57 |
| gfidente | like for the upgrade job | 15:57 |
| *** prateek has quit IRC | 15:59 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 16:00 |
| *** dtantsur|bbl is now known as dtantsur | 16:02 | |
| *** twi99y_ has joined #tripleo | 16:02 | |
| *** fultonj has quit IRC | 16:02 | |
| *** rcernin has quit IRC | 16:03 | |
| *** twi99y has quit IRC | 16:03 | |
| openstackgerrit | Merged openstack/instack-undercloud master: Update reno for stable/ocata https://review.openstack.org/434872 | 16:03 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 16:03 |
| openstackgerrit | Merged openstack/tripleo-ui master: Fix sdist tarball name in setup.cfg https://review.openstack.org/434778 | 16:04 |
| *** twi99y_ has quit IRC | 16:04 | |
| EmilienM | gfidente: in a call, brb | 16:04 |
| *** hewbrocca is now known as hewbrocca_afk | 16:04 | |
| *** twi99y has joined #tripleo | 16:04 | |
| *** thrash|biab is now known as thrash | 16:05 | |
| *** jprovazn is now known as jprovazn_bbl | 16:06 | |
| *** saibarspeis has joined #tripleo | 16:08 | |
| owalsh | shardy: not sure yum would install 2 versions of python-nova. Were they installed using rpm? | 16:10 |
| *** pcaruana has quit IRC | 16:11 | |
| *** derekh has quit IRC | 16:12 | |
| *** tremble has quit IRC | 16:12 | |
| *** nyechiel has quit IRC | 16:12 | |
| *** abregman has quit IRC | 16:13 | |
| shardy | owalsh: yes the newton version was installed via dib into my overcloud-full image, then yum update updated to the ocata version | 16:13 |
| jaosorior | sshnaidm, EmilienM: Can you check this out https://review.openstack.org/#/c/427481/ ? | 16:13 |
| *** chem has joined #tripleo | 16:13 | |
| shardy | owalsh: both were RPM installs, I'm trying to reproduce manually for more info | 16:13 |
| *** trown is now known as trown|outtypewww | 16:13 | |
| owalsh | shardy: I mean rpm command vs yum | 16:14 |
| *** tesseract has quit IRC | 16:14 | |
| *** panda is now known as panda|off | 16:14 | |
| owalsh | shardy: yum shouldn't do this AFAIK | 16:14 |
| shardy | owalsh: Hmm, the update was definitely done via yum | 16:14 |
| gfidente | EmilienM ah remove the -1 ! | 16:14 |
| gfidente | :) | 16:14 |
| shardy | Feb 15 19:02:12 localhost yum[99309]: Updated: 1:python-nova-15.0.0-0.20170215034806.bdeb05d.el7.centos.noarch | 16:15 |
| EmilienM | gfidente: ok I'm back | 16:15 |
| EmilienM | gfidente: ok I see | 16:15 |
| *** derekh has joined #tripleo | 16:15 | |
| EmilienM | gfidente: +2 | 16:15 |
| shardy | owalsh: amoralej has attempted to reproduce outside tripleo with the same delorean hash, and it worked OK | 16:15 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-ui stable/ocata: Update .gitreview for stable/ocata https://review.openstack.org/434989 | 16:15 |
| openstackgerrit | OpenStack Release Bot proposed openstack/tripleo-ui master: Update reno for stable/ocata https://review.openstack.org/434990 | 16:15 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: Fix quickstart issues https://review.openstack.org/425459 | 16:15 |
| EmilienM | gfidente: sorry for confusion | 16:15 |
| gfidente | EmilienM now +2 is too much, we don't know if it breakes | 16:16 |
| shardy | so I'm confused, perhaps something specific to either the upgrade workflow or my environment | 16:16 |
| gfidente | ahahahah | 16:16 |
| gfidente | *breaks | 16:16 |
| EmilienM | jrist, honza, jpich, jtomasek : TripleO UI Ocata RC1 release - good work folks | 16:16 |
| jpich | woohoo \o/ | 16:16 |
| jtomasek | yay! | 16:16 |
| jrist | EmilienM: thanks so much! | 16:17 |
| EmilienM | jrist, honza, jpich, jtomasek : from now, please backport to stable/ocata what you need to be released in RC2 | 16:17 |
| jistr | ccamacho: hi, fwiw i'm also +1 on your new logo, it's nice to keep the original community concept while at the same time adhering to common OpenStack style. In case we couldn't use your new logo directly, perhaps the illustrators could at least have it as a baseline and do whatever tweaks are necessary. Nice work :) | 16:17 |
| jtomasek | **\o/** | 16:17 |
| jrist | EmilienM: ew | 16:17 |
| jrist | EmilienM: but thanks :) | 16:17 |
| jpich | EmilienM: Roger! | 16:17 |
| jrist | jtomasek: is that a flail or cheerleader pom poms? | 16:17 |
| jtomasek | it's double-flail! | 16:17 |
| jrist | :o | 16:17 |
| jpich | hidden under pompoms | 16:17 |
| jtomasek | but pompoms are cool too lol | 16:18 |
| ccamacho | hey jistr thanks! hopefully they will listen to us | 16:18 |
| EmilienM | ccamacho: I think they will, they are really willing to collaborate. And yes, +1 to what jistr said, nice work here | 16:20 |
| marios | jistr: +1 ccamacho i mentioned your examples in my post too, hopefully they take notice of it | 16:20 |
| owalsh | shardy: hmm, so that suggests an issue in the ansible yum module? | 16:21 |
| jaosorior | bnemec: got time for a review https://review.openstack.org/#/c/427481/ ? :D | 16:22 |
| jpena | EmilienM: can you try rechecking the job that failed before? | 16:22 |
| EmilienM | jpena: sure thing | 16:23 |
| *** sdake_ has joined #tripleo | 16:24 | |
| EmilienM | jpena, d0ugal: I tried recheck on a mistral patch: https://review.openstack.org/#/c/434825/ | 16:24 |
| *** aufi has quit IRC | 16:24 | |
| EmilienM | we'll know in 10 min | 16:24 |
| *** leanderthal is now known as leanderthal|afk | 16:26 | |
| *** sdake has quit IRC | 16:26 | |
| *** yamahata has joined #tripleo | 16:26 | |
| shardy | owalsh: not sure atm, currently trying to reproduce directly with yum | 16:27 |
| *** gatuus has joined #tripleo | 16:27 | |
| *** gatuus_ has joined #tripleo | 16:27 | |
| shardy | one possibility is the postinst scriptlet hung after we installed the new package but before we uninstalled the old one, but I don't see any evidence of that in the logs | 16:27 |
| jpena | EmilienM: oh, it was a stable/ocata change, I may still have to fix that | 16:28 |
| jpena | let's wait and see | 16:28 |
| EmilienM | jpena: yeah, it seems like stable/ocata jobs are failing | 16:28 |
| EmilienM | jpena: watching 434825 in zuul, I'll let you know | 16:29 |
| EmilienM | adarazs: thx for chairing the tripleo CI meeting | 16:29 |
| jpena | EmilienM: ack, I'll keep an eye on it | 16:29 |
| adarazs | EmilienM: np :) | 16:29 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Don't output the PublicVip separately to /etc/hosts https://review.openstack.org/434105 | 16:30 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Exclude /etc/alternatives from logs https://review.openstack.org/434994 | 16:31 |
| sshnaidm | trown|outtypewww, adarazs, panda|off, jaosorior can you please review/merge ^^ | 16:31 |
| *** yamahata has quit IRC | 16:31 | |
| jrist | jpich: what's the recheck you just did recheck /opt/stack/new/tripleo-ci/toci_gate_test.sh: No such file or directory | 16:32 |
| EmilienM | sshnaidm: +2 | 16:32 |
| *** jcoufal has joined #tripleo | 16:32 | |
| jpich | jrist: I pick the error I find in the logs when I do a recheck, so it's easier to reference/know what happened | 16:33 |
| jrist | oh | 16:33 |
| *** jlinkes has quit IRC | 16:34 | |
| openstackgerrit | Justin Kilpatrick proposed openstack/tripleo-quickstart-extras master: Add introspection with retries option https://review.openstack.org/403677 | 16:34 |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo master: mariadb: Move generation of systemd drop-in to puppet-tripleo https://review.openstack.org/408250 | 16:34 |
| *** rbrady has quit IRC | 16:35 | |
| EmilienM | honza, jtomasek: can you please review https://review.openstack.org/#/c/434990/ ? | 16:36 |
| jtomasek | EmilienM: done | 16:38 |
| EmilienM | jtomasek: thx sir | 16:38 |
| *** panda|off is now known as panda|bbl | 16:40 | |
| *** limao has quit IRC | 16:41 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common master: Implement check node counts as a custom action https://review.openstack.org/429757 | 16:42 |
| trozet | dciabrin: when you get a chance could you respond to https://review.openstack.org/#/c/408250 ? | 16:42 |
| *** anton has quit IRC | 16:43 | |
| dciabrin | trozet, will do! | 16:44 |
| trozet | dciabrin: ty | 16:44 |
| trozet | dciabrin: i checked stable/newton RDO and puppet-systemd is in, so we shoudl be good to use it | 16:44 |
| *** abehl has joined #tripleo | 16:44 | |
| dciabrin | trozet: ok great.. glad you chased it btw, it went a bit under my radar :) | 16:45 |
| *** anton has joined #tripleo | 16:46 | |
| trozet | dciabrin: another user hit it and pinged me askign where the fix is haha..otherwise i had forgotten as well | 16:46 |
| openstackgerrit | Merged openstack/tripleo-ui master: Update reno for stable/ocata https://review.openstack.org/434990 | 16:46 |
| d0ugal | thrash: would you mind re-reviewing https://review.openstack.org/#/c/425047/ ? | 16:54 |
| thrash | d0ugal: not at all | 16:54 |
| d0ugal | thanks! | 16:54 |
| *** twi99y_ has joined #tripleo | 16:55 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Add virtual_packages support to norpm provider https://review.openstack.org/435011 | 16:55 |
| mwhahaha | EmilienM, bandini -^ | 16:55 |
| pradk | can someone +A this please https://review.openstack.org/#/c/434925/ .. super simple | 16:55 |
| EmilienM | dprince, flaper87: if you feel confortable in merging docker work in THT, you can go ahead from now. | 16:55 |
| dprince | EmilienM: ack, thanks | 16:56 |
| EmilienM | mwhahaha: crazy how a single line will probably save minutes of deployment :) | 16:56 |
| mwhahaha | no that comes next :D | 16:56 |
| mwhahaha | that prevents the stupid restarts | 16:57 |
| EmilienM | mwhahaha: ha | 16:57 |
| *** twi99y has quit IRC | 16:59 | |
| *** chlong has joined #tripleo | 16:59 | |
| *** b00tcat has quit IRC | 17:00 | |
| *** bogdando has joined #tripleo | 17:01 | |
| *** psahoo has joined #tripleo | 17:01 | |
| *** derekh has quit IRC | 17:01 | |
| *** psahoo has quit IRC | 17:02 | |
| *** hewbrocca_afk is now known as hewbrocca | 17:04 | |
| EmilienM | matbu: please make sure https://review.openstack.org/#/c/431398 is backported once it lands (stable/ocata) | 17:04 |
| *** dsneddon_afk is now known as dsneddon | 17:04 | |
| *** derekh has joined #tripleo | 17:05 | |
| jaosorior | shardy: if you have time could you check this out https://review.openstack.org/#/c/427481/ ? | 17:05 |
| EmilienM | trozet: do you want https://review.openstack.org/#/c/433295/ in stable/ocata? | 17:06 |
| trozet | EmilienM: no doesnt need to be | 17:07 |
| EmilienM | trozet: ack | 17:07 |
| trozet | EmilienM: i guess that means ocata branch was pulled ;) | 17:07 |
| trozet | EmilienM: can you remove the -2s on the other patches ? | 17:07 |
| shardy | jaosorior: which job is it that runs this, the fakeha-caserver one? | 17:08 |
| EmilienM | trozet: yes, I will very shortly | 17:08 |
| trozet | EmilienM: cool, thanks | 17:08 |
| EmilienM | gfidente: when merged, please backport https://review.openstack.org/#/c/425747/1 | 17:08 |
| EmilienM | trozet: done, let me know if I missed a patch | 17:09 |
| trozet | EmilienM: looks good, also fpan has some vpp ones | 17:10 |
| trozet | https://review.openstack.org/#/c/427437/ | 17:10 |
| trozet | https://review.openstack.org/#/c/427436/ | 17:10 |
| EmilienM | done | 17:10 |
| *** rbowen has quit IRC | 17:13 | |
| diana_clarke | How long does it take the tripleo quickstart with "--tags all" to run for you guys? It's been 2+ hours for me so far. | 17:13 |
| jaosorior | diana_clarke: it does take a while :/ | 17:14 |
| jaosorior | diana_clarke: took more or less 2 hours for me today | 17:14 |
| diana_clarke | jaosorior: Thanks! I was worried it was just hung. Fingers crossed :) | 17:15 |
| *** rbowen has joined #tripleo | 17:15 | |
| jaosorior | diana_clarke: what's the task that's running at the moment? | 17:15 |
| diana_clarke | jaosorior: "Deploy the overcloud" | 17:15 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-common master: Open log in utf-8 to prevent UnicodeEncodeError https://review.openstack.org/434520 | 17:16 |
| shardy | diana_clarke: you can log onto the undercloud via a different terminal, then source stackrc, and do openstack stack event list --follow overcloud | 17:16 |
| shardy | that'll give you a good idea what's going on with the deploy, as you should see the events changing | 17:17 |
| shardy | diana_clarke: when you drive deployment directly via "openstack overcloud deploy" you see those same events | 17:17 |
| shardy | which is why you may like to do that outside of quickstart when you're happy the environment is working | 17:17 |
| shardy | (that's why we don't default to tags all) | 17:18 |
| EmilienM | marios: is https://review.openstack.org/#/c/433641/5 ready to go? | 17:19 |
| *** rasca has quit IRC | 17:20 | |
| *** jpich has quit IRC | 17:20 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: TEST: DONT RECHECK or REVIEW: periodic jobs https://review.openstack.org/359215 | 17:22 |
| EmilienM | d0ugal, jpena, amoralej: still waiting for 434825 to kick off a multinode job | 17:23 |
| EmilienM | d0ugal: if you want, we can propose the job non voting for now | 17:23 |
| *** bogdando has quit IRC | 17:23 | |
| EmilienM | d0ugal: so you guys can land patches | 17:23 |
| EmilienM | d0ugal: it will be non voting only for mistral project | 17:23 |
| amoralej | EmilienM, repos should be ok now | 17:27 |
| diana_clarke | shardy: Neat. I'll take note and try that next time. Sadly, this one is DOA, so I need to start again ;( | 17:27 |
| d0ugal | EmilienM: I guess it depends how likely it is to be fixed soon | 17:27 |
| d0ugal | EmilienM: Renat will be gone for the next 10 hours or so at least I think | 17:28 |
| *** derekh has quit IRC | 17:28 | |
| EmilienM | d0ugal, amoralej : ok so let's see if job is passing before disabling voting | 17:29 |
| EmilienM | d0ugal: I just wanted to make sure you guys don't hate us too much :D | 17:29 |
| d0ugal | k | 17:29 |
| d0ugal | EmilienM: haha, I don't mind but I am not the one trying to backport stuff :-D | 17:29 |
| shardy | diana_clarke: you don't need to tear down the entire environment | 17:30 |
| shardy | diana_clarke: you can log on to the undercloud and see why the deploy failed (I assume it failed?) | 17:30 |
| shardy | well, you might want to tear it down but if it got to deploying the overcloud it'd be good to see why it failed | 17:31 |
| shardy | diana_clarke: a good command to find out os "openstack stack failures list overcloud" | 17:32 |
| shardy | it'll tell you all the things that failed which may provide some clues | 17:32 |
| diana_clarke | shardy: Ran out of disk space. Do you happen to know how much I'll need? | 17:32 |
| shardy | diana_clarke: my undercloud has a 50G disk, 33G used | 17:33 |
| diana_clarke | shardy: hmmm... I gave it 60G | 17:33 |
| shardy | perhaps someone can give another value as my env isn't configured via quickstart | 17:34 |
| shardy | I'm not sure what additional things could be pulled in, I'd expect the usage to be same/similar tho | 17:34 |
| *** rbowen has quit IRC | 17:35 | |
| diana_clarke | shardy: 40G was what I was told when I asked last week. And it was sufficient for the non all tags runs. I thought I was playing it extra safe by giving it 60G this time. | 17:35 |
| *** rbowen has joined #tripleo | 17:36 | |
| *** chem has quit IRC | 17:36 | |
| *** chem has joined #tripleo | 17:37 | |
| *** cylopez has quit IRC | 17:38 | |
| shardy | diana_clarke: Strange, maybe worth a du -h | sort -h to track down where the space has gone (did it sit there for hours and create huge logs full of errors?) | 17:38 |
| shardy | diana_clarke: my undercloud has been running for over a month and it's only using 33G | 17:38 |
| *** mhenkel_ has joined #tripleo | 17:39 | |
| *** shardy is now known as shardy_afk | 17:40 | |
| *** karthiks has quit IRC | 17:40 | |
| *** owalsh has quit IRC | 17:41 | |
| *** jschluet is now known as jschlueter | 17:45 | |
| *** owalsh has joined #tripleo | 17:48 | |
| *** gfidente is now known as gfidente|afk | 17:49 | |
| *** jpena is now known as jpena|away | 17:52 | |
| *** rhallisey has quit IRC | 17:55 | |
| *** thrash is now known as thrash|g0ne | 17:55 | |
| *** rwsu has joined #tripleo | 17:55 | |
| *** rhallisey has joined #tripleo | 17:55 | |
| *** dtantsur is now known as dtantsur|afk | 17:56 | |
| *** sudipto_ has joined #tripleo | 17:59 | |
| *** sudipto has joined #tripleo | 17:59 | |
| EmilienM | amoralej: http://logs.openstack.org/25/434925/1/gate/gate-tripleo-ci-centos-7-nonha-multinode/2c4894b/console.html#_2017-02-16_17_32_03_147563 | 18:02 |
| EmilienM | ouch | 18:02 |
| *** flepied has quit IRC | 18:02 | |
| amoralej | ooops | 18:03 |
| EmilienM | amoralej: it sounds like it affects all multinode jobs now | 18:03 |
| *** paramite has quit IRC | 18:03 | |
| amoralej | so, reverting https://review.rdoproject.org/r/#/c/5302/ ? | 18:03 |
| *** jprovazn_bbl is now known as jprovazn | 18:03 | |
| *** twi99y has joined #tripleo | 18:04 | |
| *** twi99y_ has quit IRC | 18:05 | |
| *** twi99y_ has joined #tripleo | 18:05 | |
| *** twi99y has quit IRC | 18:05 | |
| EmilienM | therve: you still around? | 18:06 |
| *** leifmadsen_ has joined #tripleo | 18:08 | |
| EmilienM | amoralej: let's revert for now, unless it's blocking anything else, let's revert it | 18:09 |
| EmilienM | amoralej: https://review.rdoproject.org/r/#/c/5317/ | 18:10 |
| *** fultonj has joined #tripleo | 18:10 | |
| therve | EmilienM, Yeah | 18:11 |
| therve | I guess the other package is not ready yet | 18:11 |
| *** leifmadsen has quit IRC | 18:11 | |
| therve | bbl | 18:13 |
| *** lucasagomes is now known as lucas-afk | 18:14 | |
| jtomasek | jrist, honza: I'll ask:) why the .gitreview change? | 18:15 |
| honza | Is this some kind of new rule introduced for this release? | 18:16 |
| jrist | EmilienM: ^ | 18:16 |
| jrist | probably better to ask in #openstack-release | 18:16 |
| jtomasek | I don't recall it happening before | 18:16 |
| jrist | since it was done by the openstack-release bot | 18:16 |
| jrist | :) | 18:16 |
| jrist | honza: do you mind looking at our release job failure for tripleo-ui? | 18:19 |
| jrist | The job to build the npm package for tripleo-ui has failed. | 18:19 |
| jrist | - tripleo-ui-nodejs6-npm-publish-tarball http://logs.openstack.org/45/452b4cb427a86e4b097b9711fb045a211b5e7039/release/tripleo-ui-nodejs6-npm-publish-tarball/6ec2420/ : FAILURE in 35m 53s | 18:19 |
| jrist | - tripleo-ui-npm-upload tripleo-ui-npm-upload : SKIPPED | 18:19 |
| honza | not that again | 18:19 |
| honza | jrist: yes, i'll look | 18:20 |
| honza | jrist: what patch is it on? | 18:20 |
| *** jkilpatr has quit IRC | 18:22 | |
| jrist | honza: I unfortunately don't have that info | 18:22 |
| *** jkilpatr has joined #tripleo | 18:22 | |
| jrist | honza: it's just an email from dhellman | 18:22 |
| honza | jrist: it's an npm timeout issue | 18:23 |
| honza | jrist: am i on that thread? or is it directly to you? | 18:24 |
| jrist | honza: openstack-dev | 18:24 |
| jrist | [openstack-dev] [tripleo] tripleo-ui release job failure | 18:24 |
| jrist | just happened about half hour ago | 18:24 |
| *** athomas has quit IRC | 18:25 | |
| *** pkovar has quit IRC | 18:26 | |
| *** mcornea has quit IRC | 18:28 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment https://review.openstack.org/422310 | 18:30 |
| honza | jrist: yes, sorry, found it and responded | 18:31 |
| honza | jrist: there is almost no info in the logs | 18:31 |
| openstackgerrit | Feng Pan proposed openstack/os-net-config master: Add support for VPP interface https://review.openstack.org/424426 | 18:33 |
| *** jaosorior has quit IRC | 18:34 | |
| EmilienM | honza: you might want to add more debug https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n416 | 18:36 |
| EmilienM | does npm has --debug? | 18:36 |
| *** twi99y has joined #tripleo | 18:37 | |
| jrist | it's a ... dependency | 18:37 |
| EmilienM | it has -d | 18:37 |
| jrist | (debug module) | 18:37 |
| jrist | yeah | 18:37 |
| EmilienM | https://docs.npmjs.com/misc/config | 18:37 |
| EmilienM | of that's npm-config | 18:37 |
| jrist | -v | 18:37 |
| *** jrist has left #tripleo | 18:38 | |
| *** jrist has joined #tripleo | 18:38 | |
| honza | npm install --verbose | 18:38 |
| honza | is what you're after | 18:38 |
| EmilienM | jrist, honza: so please patch project-config | 18:38 |
| honza | will do | 18:38 |
| EmilienM | and let's recheck this job again with the new command | 18:38 |
| EmilienM | thanks | 18:38 |
| *** fzdarsky__ is now known as fzdarsky|afk | 18:40 | |
| *** twi99y_ has quit IRC | 18:40 | |
| bnemec | Wow, ovb jobs returning results before multinode? What madness is this!? | 18:40 |
| slagle | probably 2 more cycles or so and multinode jobs will be hitting the timeout | 18:42 |
| slagle | we used to routinely have jobs finish < 50 minutes. now they're routinely > 60m | 18:42 |
| EmilienM | bnemec: which one exactly? | 18:43 |
| EmilienM | bnemec: that's why we created scenarios, to avoid timeout due to the # of services | 18:43 |
| bnemec | EmilienM: https://review.openstack.org/#/c/434981/ and https://review.openstack.org/#/c/434855/ | 18:44 |
| *** IdentSecure has joined #tripleo | 18:44 | |
| *** IdentSecure has left #tripleo | 18:44 | |
| bnemec | I think the check queue is just quite long today. | 18:45 |
| bnemec | Probably because of the log server issues yesterday. | 18:45 |
| bnemec | Rechecking all the things takes a lot of resources. | 18:45 |
| EmilienM | bnemec, slagle: no, they disabled osic-cloud for networking outage, which means it takes more time to have multinode jobs to be kicked off | 18:45 |
| EmilienM | it has nothing to do with runtime (yet ;)) | 18:45 |
| bnemec | That would do it too. | 18:46 |
| *** tosky has quit IRC | 18:46 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Exclude /etc/alternatives from logs https://review.openstack.org/434994 | 18:46 |
| slagle | EmilienM: that would help explain it. still the inevitable OpenStack resource creep can't be stopped | 18:46 |
| EmilienM | slagle: yeah... | 18:47 |
| honza | EmilienM: jrist: https://review.openstack.org/#/c/435063/ jtomasek | 18:47 |
| *** thrash|g0ne is now known as thrash | 18:47 | |
| EmilienM | honza: +1, even if your commit message is a bit too short to explain the motivation and background of such a patch. | 18:51 |
| honza | EmilienM: i was going to write something like "To increase debugability of the npm scripts" but it seemed dumb and too obvious | 18:51 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 18:52 |
| bnemec | Over the past month we've made changes that probably saved 45 minutes on some jobs, and we're still running slightly slower than a month ago. | 18:53 |
| bnemec | It's crazy how fast you regress performance when you're constantly merging tons of patches. | 18:53 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 18:54 |
| honza | tripleo-ui 3.0.0 on npm https://www.npmjs.com/package/tripleo-ui | 18:55 |
| EmilienM | honza: good news! | 18:56 |
| *** saibarspeis has quit IRC | 18:57 | |
| *** sudipto has quit IRC | 18:59 | |
| *** sudipto_ has quit IRC | 18:59 | |
| bnemec | EmilienM: slagle: Hmm, do we have a dependency problem in the multinode jobs? http://logs.openstack.org/11/435011/1/check/gate-tripleo-ci-centos-7-nonha-multinode/636931c/console.html | 18:59 |
| bnemec | I'm seeing several unrelated patches failing on the missing openstack-heat-templates path. | 19:00 |
| openstackgerrit | Bob Fournier proposed openstack/instack-undercloud master: Install Ironic inspector plugins https://review.openstack.org/434356 | 19:00 |
| EmilienM | bnemec: yeah, that's why we did https://review.rdoproject.org/r/#/c/5317/ | 19:00 |
| *** akrivoka has quit IRC | 19:00 | |
| EmilienM | bnemec: I had to revert therve's patch, it seems we're missing something here | 19:00 |
| openstackgerrit | Feng Pan proposed openstack/puppet-tripleo master: Add VPP service https://review.openstack.org/427436 | 19:01 |
| openstackgerrit | Bob Fournier proposed openstack/instack-undercloud master: Install Ironic inspector plugins https://review.openstack.org/434356 | 19:01 |
| *** jidar has quit IRC | 19:01 | |
| bnemec | EmilienM: , | 19:01 |
| bnemec | EmilienM: Oops | 19:01 |
| bnemec | EmilienM: If we're not using anything from those templates, they probably just need to be removed from the instack call in the bootstrap script. | 19:02 |
| bnemec | (that was an impressive typo - not only did I type a spurious , but I then followed it up by hitting enter instead of backspace) | 19:03 |
| *** twi99y has quit IRC | 19:03 | |
| *** twi99y has joined #tripleo | 19:03 | |
| EmilienM | bnemec: most probably, let me look | 19:04 |
| *** ipsecguy_ has joined #tripleo | 19:07 | |
| *** rbrady has joined #tripleo | 19:09 | |
| *** ipsecguy has quit IRC | 19:09 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/mitaka: Allow the override of pacemaker::corosync::settle_tries https://review.openstack.org/426115 | 19:14 |
| *** gfidente|afk has quit IRC | 19:15 | |
| slagle | bnemec: EmilienM we used to get the heat hooks from openstack-heat-templates, but those are packaged now | 19:15 |
| *** hewbrocca is now known as hewbrocca_afk | 19:16 | |
| *** owalsh is now known as owalsh-afk | 19:16 | |
| *** rbrady has quit IRC | 19:16 | |
| openstackgerrit | Feng Pan proposed openstack/tripleo-heat-templates master: Add OpenDaylightConnectionProtocol parameter to opendaylight-api service https://review.openstack.org/435074 | 19:17 |
| *** salmankhan has quit IRC | 19:19 | |
| *** dprince has quit IRC | 19:19 | |
| openstackgerrit | Feng Pan proposed openstack/tripleo-heat-templates master: Add VPP composable service https://review.openstack.org/427437 | 19:22 |
| *** Goneri has quit IRC | 19:24 | |
| *** jidar has joined #tripleo | 19:28 | |
| *** zoliXXL is now known as zoli|afk | 19:30 | |
| *** suuuper has quit IRC | 19:35 | |
| *** prateek has joined #tripleo | 19:36 | |
| *** fragatina has quit IRC | 19:37 | |
| *** fragatina has joined #tripleo | 19:38 | |
| *** fragatina has quit IRC | 19:40 | |
| *** fragatina has joined #tripleo | 19:40 | |
| openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates master: Enables increasing mariadb open files for noha deployments https://review.openstack.org/435081 | 19:41 |
| *** salmankhan has joined #tripleo | 19:41 | |
| *** jkilpatr has quit IRC | 19:42 | |
| *** jkilpatr has joined #tripleo | 19:42 | |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo master: mariadb: Move generation of systemd drop-in to puppet-tripleo https://review.openstack.org/408250 | 19:43 |
| *** jkilpatr has quit IRC | 19:46 | |
| *** jkilpatr has joined #tripleo | 19:46 | |
| jrist | honza: +1'd | 19:48 |
| *** fragatina has quit IRC | 19:48 | |
| honza | jrist: thanks | 19:48 |
| *** sasha2 has quit IRC | 19:51 | |
| *** sasha2 has joined #tripleo | 19:52 | |
| *** chlong has quit IRC | 19:56 | |
| openstackgerrit | Feng Pan proposed openstack/tripleo-heat-templates master: Add OpenDaylightConnectionProtocol parameter to opendaylight-api service https://review.openstack.org/435074 | 20:07 |
| mwhahaha | bandini, EmilienM: i figured out why we're restarting running all the dbsync stuff again and restarting nova-api in step 5on updates | 20:07 |
| EmilienM | mwhahaha: the virtual package thing? | 20:08 |
| mwhahaha | nope | 20:08 |
| dsneddon | EmilienM, Do you know if there is a reason that the tripleo-docs still recommend using a RHEL 7.1 guest image for disk-image-builder rather than a RHEL 7.2 image? | 20:08 |
| dsneddon | https://docs.openstack.org/developer/tripleo-docs/basic_deployment/basic_deployment_cli.html | 20:08 |
| mwhahaha | dsneddon: probably cause it's out of date, i think you need a 7.3 image actually | 20:09 |
| dsneddon | mwhahaha, Yeah, good point! | 20:09 |
| EmilienM | dsneddon: what alex said | 20:09 |
| EmilienM | mwhahaha: stop spoiling us! what is it? :) | 20:09 |
| sshnaidm | EmilienM, do we still have problem with templates or I can recheck? | 20:10 |
| sshnaidm | bnemec, ^ | 20:10 |
| EmilienM | sshnaidm: recheck should work now | 20:10 |
| EmilienM | I see successful jobs | 20:10 |
| mwhahaha | EmilienM: https://github.com/openstack/puppet-nova/blob/master/manifests/scheduler/filter.pp#L159 we pass [] in so the provider tries to set [] which doesn't actually output anything into the config | 20:10 |
| mwhahaha | EmilienM: so puppet thinks it's writing something and it's not | 20:10 |
| *** sdake_ has quit IRC | 20:10 | |
| *** sdake has joined #tripleo | 20:11 | |
| mwhahaha | EmilienM: so a workaround is to set it to $::os_service_default if [] gets passed in | 20:11 |
| sshnaidm | EmilienM, bnemec can you please review/merge the patch https://review.openstack.org/#/c/425459/ ? It should resolve logging issues we have | 20:11 |
| sshnaidm | I ran recheck there, should pass I hope | 20:11 |
| *** liverpooler has quit IRC | 20:12 | |
| EmilienM | mwhahaha: do we need this kind of logic ? https://github.com/openstack/puppet-nova/blob/master/manifests/scheduler/filter.pp#L165-L170 | 20:14 |
| mwhahaha | EmilienM: yes, i am preparing a patch | 20:14 |
| mwhahaha | https://bugs.launchpad.net/puppet-nova/+bug/1665443 | 20:14 |
| openstack | Launchpad bug 1665443 in puppet-nova "idempotency issue with nova::scheduler::filter::scheduler_available_filters" [High,New] | 20:14 |
| *** prateek has quit IRC | 20:15 | |
| *** Goneri has joined #tripleo | 20:15 | |
| *** salmankhan has quit IRC | 20:15 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-docs master: Add docs for scaling for deployed-server https://review.openstack.org/435094 | 20:16 |
| *** amoralej is now known as amoralej|off | 20:16 | |
| mwhahaha | are multinode jobs fixed yet? | 20:20 |
| EmilienM | mwhahaha: yes | 20:21 |
| EmilienM | mwhahaha: recheck ︵ /(.□. \ | 20:21 |
| mwhahaha | on it :D | 20:22 |
| *** sshnaidm is now known as sshnaidm|off | 20:23 | |
| *** jprovazn has quit IRC | 20:24 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci master: Remove openstack-heat-templates from bootstrap elements path https://review.openstack.org/435102 | 20:24 |
| bnemec | slagle: EmilienM: Okay, hopefully ^ will work then. | 20:24 |
| EmilienM | bnemec: thanks | 20:27 |
| EmilienM | bnemec: indeed, it looks like the perfect fix | 20:30 |
| EmilienM | bnemec: once it merges, we can revert the revert | 20:30 |
| *** panda|bbl is now known as panda | 20:36 | |
| *** salmankhan has joined #tripleo | 20:39 | |
| *** kberger has quit IRC | 20:39 | |
| *** gatuus has quit IRC | 20:45 | |
| *** gatuus_ has quit IRC | 20:45 | |
| EmilienM | bnemec: https://review.rdoproject.org/r/#/c/5337/1 | 20:49 |
| *** salmankhan has quit IRC | 20:55 | |
| *** fultonj has quit IRC | 20:55 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Switch to net-config-multinode https://review.openstack.org/426372 | 20:57 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Switch to net-config-multinode https://review.openstack.org/426372 | 20:57 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml https://review.openstack.org/426375 | 20:58 |
| *** lblanchard has quit IRC | 20:58 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: upgrades: switch 'newton' to $UPGRADE_RELEASE https://review.openstack.org/433209 | 20:59 |
| *** fragatina has joined #tripleo | 21:02 | |
| *** jayg is now known as jayg|g0n3 | 21:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: multinode/upgrade: add nova services to environment https://review.openstack.org/434348 | 21:03 |
| EmilienM | hum, gate-tripleo-ci-centos-7-nonha-multinode SUCCESS in 2h 15m 37s | 21:04 |
| EmilienM | though it's not that long usually it's less than 1h | 21:04 |
| EmilienM | probably due to osic cloud | 21:05 |
| slagle | they just disabled osic cloud today though aiui | 21:05 |
| EmilienM | err no it was on centos-7-2-node-rax-ord-7341598 | 21:05 |
| EmilienM | http://logs.openstack.org/24/433824/4/check/gate-tripleo-ci-centos-7-nonha-multinode/05d3e89/console.html#_2017-02-16_18_31_35_979054 | 21:05 |
| slagle | my earlier comment stands on it's own | 21:05 |
| *** hjensas has quit IRC | 21:05 | |
| EmilienM | slagle: yes :/ | 21:06 |
| slagle | the job takes ~10 minutes longer | 21:06 |
| slagle | on average | 21:06 |
| *** hjensas has joined #tripleo | 21:06 | |
| EmilienM | jumping on infra chan | 21:06 |
| bnemec | Ha, I was just looking at that job thinking "WTF?" | 21:06 |
| therve | EmilienM, zaneb; https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/bootstrap-overcloud-full.sh#L47 is the reason the dep removal broke CI | 21:06 |
| bnemec | It got scheduled to an old 486 box in the basement of the DC. :-) | 21:07 |
| *** sdake has quit IRC | 21:07 | |
| slagle | TripleO is still the best "find bad hardware" tool i've ever seen :) | 21:07 |
| EmilienM | bnemec: lol yeah | 21:07 |
| EmilienM | therve: exactly, bnemec has fix: https://review.openstack.org/435102 | 21:07 |
| bnemec | slagle: Or "find bugs with nested virt" tool. :-) | 21:07 |
| zaneb | therve: presumably it would actually fail to build the disk images even if that bogus path weren't there though? | 21:08 |
| zaneb | therve: since it's using the d-i-b elements instead of installing RPMs? | 21:08 |
| bnemec | zaneb: It's not building images in multinode though. It's just applying elements directly to an already booted environment. | 21:08 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-docs master: Update instructions to point to RHEL 7.3 guest image (not 7.1) https://review.openstack.org/435118 | 21:10 |
| therve | zaneb, Yeah it's not dib, it's instack | 21:11 |
| dsneddon | EmilienM, mwhahaha: tripleo-docs update for RHEL 7.3 guest image: https://review.openstack.org/#/c/435118/ | 21:11 |
| dsneddon | EmilienM, If you want a separate Lauchpad bug, I'll file it and update the commit message. | 21:11 |
| EmilienM | dsneddon: nice | 21:11 |
| EmilienM | dsneddon: no need imho | 21:11 |
| zaneb | therve, bnemec: so my best guess was that the correct solution is to add an element to tripleo-image-elements that installs the heat-agents from their RPMs. is that crazy? | 21:12 |
| bnemec | EmilienM: Do we need a promotion or something before the multinode bug is fixed? Because http://logs.openstack.org/10/406210/2/check/gate-tripleo-ci-centos-7-nonha-multinode/d709a57/console.html just failed recently on it. | 21:12 |
| therve | zaneb, Presumably it's already done, and we don't need that anymore | 21:12 |
| bnemec | zaneb: I think that's already being done. The path was just leftover cruft from when we got them from elements. | 21:12 |
| bnemec | I guess we'll see based on the results on https://review.openstack.org/435102 | 21:13 |
| zaneb | heh, yep, we'll soon find out I guess :) | 21:13 |
| EmilienM | bnemec: mhh I don't think so | 21:14 |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Skipping metadata definition tests https://review.openstack.org/434970 | 21:14 |
| EmilienM | bnemec: let me look in zuul | 21:14 |
| EmilienM | bnemec: I don't think so, I see some jobs deploying the overcloud now | 21:15 |
| therve | zaneb, I guess the remaining question is whether there are things in heat-agents that we want in ocato | 21:15 |
| therve | zaneb, Therefore if we want to sync the repo | 21:16 |
| *** salmankhan has joined #tripleo | 21:16 | |
| *** ccamacho has quit IRC | 21:16 | |
| bnemec | Unfortunately it looks like https://review.openstack.org/435102 is going to fail the multinode job on one of those repo-setup timeouts. :-/ | 21:19 |
| bnemec | But the updates job looks promising, so I think it worked. | 21:19 |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Sort ResourceGroup resource list https://review.openstack.org/435121 | 21:20 |
| *** karimb has quit IRC | 21:21 | |
| EmilienM | bnemec: I think only multinode jobs were affected, isn't? | 21:21 |
| EmilienM | bnemec: looking at http://tripleo.org/cistatus.html | 21:21 |
| bnemec | EmilienM: Yeah, I mean the multinode-updates job. | 21:21 |
| *** karimb has joined #tripleo | 21:21 | |
| EmilienM | oh sorry ok | 21:22 |
| bnemec | It got past the bootstrap step, so apparently we really weren't using anything from that directory. | 21:22 |
| bnemec | I wonder if I should just rebase that patch though. Otherwise it's going to take a long time for the multinode job to eventually time out. | 21:23 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 21:23 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add option to build delorean changes and include the repo https://review.openstack.org/434004 | 21:24 |
| *** iranzo has quit IRC | 21:24 | |
| *** flepied has joined #tripleo | 21:27 | |
| EmilienM | bnemec: I would rebase it | 21:29 |
| bnemec | Bleh, nothing new to rebase on. | 21:30 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci master: Remove openstack-heat-templates from bootstrap elements path https://review.openstack.org/435102 | 21:30 |
| * bnemec removes a comma from the commit message | 21:31 | |
| * bnemec uses too many commas anyway | 21:31 | |
| bnemec | On the plus side, we can cancel ovb jobs sanely now. | 21:32 |
| bnemec | Which we've done 1684 times since that patch merged. | 21:32 |
| bnemec | Apparently we cancel a _lot_ of jobs. | 21:33 |
| *** sdake has joined #tripleo | 21:33 | |
| dsneddon | bnemec, Is that what happens when a patchset gets updated before the tests are finished running? | 21:34 |
| bnemec | dsneddon: Yeah, that's a common one. | 21:34 |
| *** jkilpatr has quit IRC | 21:34 | |
| bnemec | I think there are others too, like a patch merging while ovb jobs are still running. | 21:34 |
| *** ccamacho has joined #tripleo | 21:34 | |
| openstackgerrit | Feng Pan proposed openstack/os-net-config master: Add support for VPP interface https://review.openstack.org/424426 | 21:36 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci master: Use bootstrap subnodes environment in CI https://review.openstack.org/425888 | 21:37 |
| *** karimb has quit IRC | 21:37 | |
| *** pmannidi has joined #tripleo | 21:40 | |
| *** jerrygb has quit IRC | 21:41 | |
| *** salmankhan has quit IRC | 21:42 | |
| *** ansmith has quit IRC | 21:44 | |
| *** eck` is now known as eck`gone | 21:44 | |
| *** salmankhan has joined #tripleo | 21:56 | |
| openstackgerrit | Merged openstack/instack-undercloud master: Set project name explicitly to service for panko https://review.openstack.org/434925 | 22:02 |
| *** leifmadsen_ is now known as leifmadsen | 22:04 | |
| *** chlong has joined #tripleo | 22:05 | |
| *** salmankhan has quit IRC | 22:05 | |
| openstackgerrit | Feng Pan proposed openstack/os-net-config master: Add support for VPP interface https://review.openstack.org/424426 | 22:07 |
| *** gbarros has joined #tripleo | 22:09 | |
| *** jkilpatr has joined #tripleo | 22:10 | |
| *** gbarros has quit IRC | 22:11 | |
| *** chlong has quit IRC | 22:11 | |
| *** gbarros has joined #tripleo | 22:12 | |
| *** gbarros_ has joined #tripleo | 22:13 | |
| *** b00tcat` has joined #tripleo | 22:13 | |
| *** salmankhan has joined #tripleo | 22:13 | |
| *** gbarros has quit IRC | 22:16 | |
| *** zoli|afk is now known as zoli|wfh | 22:19 | |
| *** bfournie has quit IRC | 22:21 | |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud stable/ocata: Set project name explicitly to service for panko https://review.openstack.org/435140 | 22:24 |
| EmilienM | pradk: you're free to +2 +A this patch ^ | 22:24 |
| pradk | cool | 22:25 |
| slagle | EmilienM: so pike is officially open? | 22:33 |
| slagle | now that we've branched? | 22:33 |
| EmilienM | slagle: yes it is | 22:34 |
| EmilienM | slagle: though the focus should be on stabilization and ocata-rc2 imho | 22:34 |
| *** gatuus has joined #tripleo | 22:35 | |
| slagle | for sure | 22:36 |
| *** gatuus_ has joined #tripleo | 22:38 | |
| *** pmannidi has quit IRC | 22:39 | |
| *** flepied has quit IRC | 22:39 | |
| openstackgerrit | Merged openstack/tripleo-quickstart master: provision/remote: add missing become lines https://review.openstack.org/426238 | 22:41 |
| *** rhallisey has quit IRC | 22:42 | |
| EmilienM | bnemec, slagle: sounds like we don't have /var/log anymore? http://logs.openstack.org/78/434878/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-updates/769fd98/logs/overcloud-novacompute-0/var/log/ | 22:45 |
| EmilienM | bnemec, slagle, err /var/log/nova | 22:45 |
| EmilienM | also see http://logs.openstack.org/78/434878/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-updates/769fd98/logs/ | 22:45 |
| EmilienM | overcloud-controller wasn't extracted | 22:45 |
| bnemec | Yeah, looks like something went wrong there. | 22:46 |
| bnemec | /var/log/nova is present in the tar file. | 22:46 |
| bnemec | EmilienM: Yeah, that job timed out during postci. | 22:47 |
| bnemec | /home/jenkins/workspace/gate-tripleo-ci-centos-7-ovb-updates/devstack-gate/functions.sh: line 1134: 17167 Killed timeout -s 9 ${REMAINING_TIME}m bash -c "source $WORKSPACE/devstack-gate/functions.sh && $cmd" | 22:47 |
| EmilienM | oops | 22:48 |
| slagle | BOOM, no logs for you | 22:48 |
| *** morazi has quit IRC | 22:51 | |
| *** jmelvin has quit IRC | 22:53 | |
| *** dbecker has quit IRC | 22:55 | |
| *** tbonds has quit IRC | 22:56 | |
| EmilienM | so I see something very interesting | 22:56 |
| EmilienM | in tripleo ci master, nova.conf [placement] has os_interface set | 22:56 |
| EmilienM | but not in stable/ocata | 22:56 |
| EmilienM | the THT patch and Nova patch that makes it possible are both in stable/ocata, I checked the version of rpms | 22:56 |
| EmilienM | I even see it in the etc/puppet/hieradata/service_configs.json | 22:56 |
| EmilienM | "nova::placement::os_interface": "internal" | 22:57 |
| EmilienM | but somehow, it's not applied on the nova nodes | 22:57 |
| *** ccamacho has quit IRC | 22:58 | |
| *** dcritch has quit IRC | 22:58 | |
| *** twi99y_ has joined #tripleo | 22:58 | |
| EmilienM | oh it's puppet-nova | 22:59 |
| *** saneax-_-|AFK is now known as saneax | 23:00 | |
| EmilienM | we need https://review.openstack.org/#/c/435143/ | 23:00 |
| *** twi99y has quit IRC | 23:02 | |
| *** saneax is now known as saneax-_-|AFK | 23:02 | |
| *** bfournie has joined #tripleo | 23:10 | |
| *** rbrady has joined #tripleo | 23:10 | |
| *** panda is now known as panda|zZ | 23:12 | |
| diana_clarke | 132 minutes later, the tripleo quickstart failed again. This time I see "failed" under: "did the deployment pass or fail?". Where should I look for more info as to why? | 23:19 |
| EmilienM | diana_clarke: can you paste some logs please? | 23:19 |
| openstackgerrit | Merged openstack/tripleo-quickstart master: provide instructions on how to only use quickstart as instack-virt-setup https://review.openstack.org/422260 | 23:19 |
| diana_clarke | Here are the logs: https://gist.github.com/dianaclarke/57a26f36d69111a2f1eef3c57ebd9ce4 | 23:19 |
| diana_clarke | EmilienM: Sorry, was about to :) No rush, BTW. I need to feed the kids, etc, so I probably won't be around much for a few hours. And I'm sure you'd like an evening too. I'll circle back tomorrow. | 23:20 |
| EmilienM | diana_clarke: no worries, let's catch-up later. If I find something I'll email you. Enjoy family time! | 23:21 |
| diana_clarke | PS. I haven't looked much at those logs yet. It might be obvious, but I'm cooking dinner now. | 23:21 |
| *** zoli|wfh is now known as zoli|gone | 23:23 | |
| *** rlandy is now known as rlandy|afk | 23:29 | |
| *** lblanchard has joined #tripleo | 23:31 | |
| *** tbonds has joined #tripleo | 23:41 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml https://review.openstack.org/426375 | 23:51 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml https://review.openstack.org/426375 | 23:52 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: upgrades: switch 'newton' to $UPGRADE_RELEASE https://review.openstack.org/433209 | 23:52 |
| *** sshnaidm|off has quit IRC | 23:56 | |
| *** cdearborn has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!