Re: [openstack-dev] [tripleo] CI promotion blockers
On 2 January 2018 at 16:30, Alex Schultzwrote: > On Tue, Jan 2, 2018 at 9:08 AM, Julie Pichon wrote: >> Hi! >> >> On 27 December 2017 at 16:48, Emilien Macchi wrote: >>> - Keystone removed _member_ role management, so we stopped using it >>> (only Member is enough): https://review.openstack.org/#/c/529849/ >> >> There's been so many issues with the default member role and Horizon >> over the years, that one got my attention. I can see that >> puppet-horizon still expects '_member_' for role management [1]. >> However trying to understand the Keystone patch linked to in the >> commit, it looks like there's total freedom in which role name to use >> so we can't just change the default in puppet-horizon to use 'Member' >> as other consumers may expect and settle on '_member_' in their >> environment. (Right?) >> >> In this case, the proper way to fix this for TripleO deployments may >> be to make the change in instack-undercloud (I presume in [2]) so that >> the default role is explicitly set to 'Member' for us? Does that sound >> like the correct approach to get to a working Horizon? >> > > We probably should at least change _member_ to Member in > puppet-horizon. That fixes both projects for the default case. Oh, I thought there was no longer a default and that TripleO was creating the 'Member' role by itself? Fixing it directly in puppet-horizon sounds ideal in general, if changing the default value isn't expected to cause other issues. Thanks, Julie > > Thanks, > -Alex > >> Julie >> >> [1] >> https://github.com/openstack/puppet-horizon/blob/master/manifests/init.pp#L458 >> [2] >> https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.yaml.template#L622 __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [tripleo] CI promotion blockers
On Tue, Jan 2, 2018 at 9:08 AM, Julie Pichonwrote: > Hi! > > On 27 December 2017 at 16:48, Emilien Macchi wrote: >> - Keystone removed _member_ role management, so we stopped using it >> (only Member is enough): https://review.openstack.org/#/c/529849/ > > There's been so many issues with the default member role and Horizon > over the years, that one got my attention. I can see that > puppet-horizon still expects '_member_' for role management [1]. > However trying to understand the Keystone patch linked to in the > commit, it looks like there's total freedom in which role name to use > so we can't just change the default in puppet-horizon to use 'Member' > as other consumers may expect and settle on '_member_' in their > environment. (Right?) > > In this case, the proper way to fix this for TripleO deployments may > be to make the change in instack-undercloud (I presume in [2]) so that > the default role is explicitly set to 'Member' for us? Does that sound > like the correct approach to get to a working Horizon? > We probably should at least change _member_ to Member in puppet-horizon. That fixes both projects for the default case. Thanks, -Alex > Julie > > [1] > https://github.com/openstack/puppet-horizon/blob/master/manifests/init.pp#L458 > [2] > https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.yaml.template#L622 > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [tripleo] CI promotion blockers
Hi! On 27 December 2017 at 16:48, Emilien Macchiwrote: > - Keystone removed _member_ role management, so we stopped using it > (only Member is enough): https://review.openstack.org/#/c/529849/ There's been so many issues with the default member role and Horizon over the years, that one got my attention. I can see that puppet-horizon still expects '_member_' for role management [1]. However trying to understand the Keystone patch linked to in the commit, it looks like there's total freedom in which role name to use so we can't just change the default in puppet-horizon to use 'Member' as other consumers may expect and settle on '_member_' in their environment. (Right?) In this case, the proper way to fix this for TripleO deployments may be to make the change in instack-undercloud (I presume in [2]) so that the default role is explicitly set to 'Member' for us? Does that sound like the correct approach to get to a working Horizon? Julie [1] https://github.com/openstack/puppet-horizon/blob/master/manifests/init.pp#L458 [2] https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.yaml.template#L622 __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [tripleo] CI promotion blockers
We've got promotion today, thanks Wes & Sagi for your help. On Sun, Dec 31, 2017 at 9:06 AM, Emilien Macchiwrote: > Here's an update on what we did the last days after merging these > blockers mentioned in the previous email: > > - Ignore a failing test in tempest (workaround) > https://review.rdoproject.org/r/#/c/8/ until > https://review.openstack.org/#/c/526647/ is merged. It allowed RDO > repos to be consistent again, so we could have the latest patches in > TripleO, tested by Promotion jobs. > - scenario001 is timeouting a lot, we moved tacker/congress to > scenario007, and also removed MongoDB that was running for nothing. > - tripleo-ci-centos-7-containers-multinode was timeouting a lot, we > removed cinder and some other services already covered by scenarios, > so tripleo-ci-centos-7-containers-multinode is like ovb, testing the > minimum set of services (which is why we created this job). > - fixing an ipv6 issue in puppet-tripleo: > https://review.openstack.org/#/c/530219/ > > All of the above is merged. > Now the remaining blocker is to update the RDO CI layout for promotion jobs: > See https://review.rdoproject.org/r/#/c/9/ and > https://review.rdoproject.org/r/#/c/11120/ > Once it merges and job runs, we should get a promotion. > > Let me know any question, > > On Wed, Dec 27, 2017 at 8:48 AM, Emilien Macchi wrote: >> Just a heads-up about what we've done the last days to make progress >> and hopefully get a promotion this week: >> >> - Disabling voting on scenario001, 002 and 003. They timeout too much, >> we haven't figured out why yet but we'll look at it this week and next >> week. Hopefully we can re-enable voting today or so. >> - Kolla added Sensu support and it broke our container builds. It >> should be fixed by https://review.openstack.org/#/c/529890/ and >> https://review.openstack.org/530232 >> - Keystone removed _member_ role management, so we stopped using it >> (only Member is enough): https://review.openstack.org/#/c/529849/ >> - Fixup MTU configuration for CI envs: >> https://review.openstack.org/#/c/527249 >> - Reduce memory for undercloud image convert: >> https://review.openstack.org/#/c/530137/ >> - Remove policy.json default rules from Heat in THT: >> https://review.openstack.org/#/c/530225 >> >> That's pretty all. Due to the lack of reviewers during the Christmas >> time, we had to land some patches ourselves. If there is any problem >> with one of them, please let us know. We're trying to maintain CI is >> good shape this week and it's a bit of a challenge ;-) >> -- >> Emilien Macchi > > > > -- > Emilien Macchi -- Emilien Macchi __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [tripleo] CI promotion blockers
Here's an update on what we did the last days after merging these blockers mentioned in the previous email: - Ignore a failing test in tempest (workaround) https://review.rdoproject.org/r/#/c/8/ until https://review.openstack.org/#/c/526647/ is merged. It allowed RDO repos to be consistent again, so we could have the latest patches in TripleO, tested by Promotion jobs. - scenario001 is timeouting a lot, we moved tacker/congress to scenario007, and also removed MongoDB that was running for nothing. - tripleo-ci-centos-7-containers-multinode was timeouting a lot, we removed cinder and some other services already covered by scenarios, so tripleo-ci-centos-7-containers-multinode is like ovb, testing the minimum set of services (which is why we created this job). - fixing an ipv6 issue in puppet-tripleo: https://review.openstack.org/#/c/530219/ All of the above is merged. Now the remaining blocker is to update the RDO CI layout for promotion jobs: See https://review.rdoproject.org/r/#/c/9/ and https://review.rdoproject.org/r/#/c/11120/ Once it merges and job runs, we should get a promotion. Let me know any question, On Wed, Dec 27, 2017 at 8:48 AM, Emilien Macchiwrote: > Just a heads-up about what we've done the last days to make progress > and hopefully get a promotion this week: > > - Disabling voting on scenario001, 002 and 003. They timeout too much, > we haven't figured out why yet but we'll look at it this week and next > week. Hopefully we can re-enable voting today or so. > - Kolla added Sensu support and it broke our container builds. It > should be fixed by https://review.openstack.org/#/c/529890/ and > https://review.openstack.org/530232 > - Keystone removed _member_ role management, so we stopped using it > (only Member is enough): https://review.openstack.org/#/c/529849/ > - Fixup MTU configuration for CI envs: https://review.openstack.org/#/c/527249 > - Reduce memory for undercloud image convert: > https://review.openstack.org/#/c/530137/ > - Remove policy.json default rules from Heat in THT: > https://review.openstack.org/#/c/530225 > > That's pretty all. Due to the lack of reviewers during the Christmas > time, we had to land some patches ourselves. If there is any problem > with one of them, please let us know. We're trying to maintain CI is > good shape this week and it's a bit of a challenge ;-) > -- > Emilien Macchi -- Emilien Macchi __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
[openstack-dev] [tripleo] CI promotion blockers
Just a heads-up about what we've done the last days to make progress and hopefully get a promotion this week: - Disabling voting on scenario001, 002 and 003. They timeout too much, we haven't figured out why yet but we'll look at it this week and next week. Hopefully we can re-enable voting today or so. - Kolla added Sensu support and it broke our container builds. It should be fixed by https://review.openstack.org/#/c/529890/ and https://review.openstack.org/530232 - Keystone removed _member_ role management, so we stopped using it (only Member is enough): https://review.openstack.org/#/c/529849/ - Fixup MTU configuration for CI envs: https://review.openstack.org/#/c/527249 - Reduce memory for undercloud image convert: https://review.openstack.org/#/c/530137/ - Remove policy.json default rules from Heat in THT: https://review.openstack.org/#/c/530225 That's pretty all. Due to the lack of reviewers during the Christmas time, we had to land some patches ourselves. If there is any problem with one of them, please let us know. We're trying to maintain CI is good shape this week and it's a bit of a challenge ;-) -- Emilien Macchi __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev