| *** pma78 has quit IRC | 02:09 | |
| *** pma78 has joined #puppet-openstack | 02:10 | |
| *** mschuppert has quit IRC | 02:40 | |
| *** irclogbot_0 has quit IRC | 02:56 | |
| *** irclogbot_3 has joined #puppet-openstack | 03:16 | |
| *** ykarel|away has joined #puppet-openstack | 04:05 | |
| *** ykarel|away has quit IRC | 04:30 | |
| *** ykarel has joined #puppet-openstack | 04:45 | |
| *** openstackgerrit has joined #puppet-openstack | 05:59 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Add CentOS8 support and jobs https://review.opendev.org/698142 | 05:59 |
|---|---|---|
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Test CentOS8 jobs with temporary repos https://review.opendev.org/699731 | 06:01 |
| *** rcernin_ has joined #puppet-openstack | 06:09 | |
| *** rcernin has quit IRC | 06:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/699914 | 06:13 |
| *** pgaxatte has joined #puppet-openstack | 07:02 | |
| *** rcernin_ has quit IRC | 07:09 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration stable/train: Add CentOS8 support and jobs https://review.opendev.org/699204 | 07:30 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration stable/train: [DNM] Test CentOS8 jobs with temporary repos https://review.opendev.org/699923 | 07:35 |
| *** irclogbot_3 has quit IRC | 07:44 | |
| *** irclogbot_3 has joined #puppet-openstack | 07:46 | |
| *** pkopec has joined #puppet-openstack | 07:46 | |
| *** ykarel is now known as ykarel|lunch | 07:48 | |
| *** mschuppert has joined #puppet-openstack | 07:50 | |
| *** tosky has joined #puppet-openstack | 08:14 | |
| *** tkajinam has quit IRC | 08:15 | |
| *** jpena|off is now known as jpena | 08:38 | |
| *** ykarel|lunch is now known as ykarel | 08:50 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration stable/train: Add CentOS8 support and jobs https://review.opendev.org/699204 | 08:55 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration stable/train: [DNM] Test CentOS8 jobs with temporary repos https://review.opendev.org/699923 | 09:01 |
| *** ykarel is now known as ykarel|afk | 09:16 | |
| *** rpittau|afk is now known as rpittau | 09:18 | |
| *** rcernin_ has joined #puppet-openstack | 09:21 | |
| *** rcernin_ has quit IRC | 09:29 | |
| *** derekh has joined #puppet-openstack | 09:41 | |
| *** ykarel|afk is now known as ykarel | 09:42 | |
| *** gfidente|afk is now known as gfidente | 09:51 | |
| *** gfidente has quit IRC | 10:03 | |
| *** tosky_ has joined #puppet-openstack | 10:07 | |
| *** tosky is now known as Guest97727 | 10:07 | |
| *** tosky_ is now known as tosky | 10:07 | |
| *** Guest97727 has quit IRC | 10:08 | |
| *** gfidente has joined #puppet-openstack | 10:34 | |
| ykarel | tobias-urdin, hi | 10:50 |
| ykarel | tobias-urdin, can u review centos8 patches https://review.opendev.org/#/q/topic:rdo-centos8+(status:open+OR+status:merged)+project:openstack/puppet-openstack-integration | 10:50 |
| tobias-urdin | ykarel: sure, a little skeptical about the custom Puppetfile since we won't get the new module version proposed by the OpenStack proposal bot | 11:00 |
| tobias-urdin | we are also piling up a lot of fixme/todo's in p-o-i, maybe a later issue but we have a lot of them even now | 11:00 |
| ykarel | tobias-urdin, re. TODO/FIXME most of them are due to external dependencies, we will be clearing those as soon as we get fixes | 11:03 |
| ykarel | tobias-urdin, and re. Puppetfile i understand your concern | 11:03 |
| ykarel | we need to work upon that part | 11:04 |
| tobias-urdin | i wonder if we just get that new minor mariadb version in rdo, if we can bump puppetlabs-mysql and skip that whole part | 11:04 |
| tobias-urdin | talked to amoralejo about that some week ago | 11:04 |
| ykarel | tobias-urdin, hmm i saw some discussion around it but it didn't progressed much | 11:05 |
| ykarel | i will check again | 11:05 |
| ykarel | tobias-urdin, with the proposed patches we are not enabling the jobs for now | 11:06 |
| ykarel | once these patches will be merged, we will be setting up or promotion jobs | 11:06 |
| ykarel | once those get's stable, we will start running those jobs as non-voting | 11:06 |
| ykarel | in upstream check | 11:06 |
| ykarel | and once everything stabilizes we will move them voting and remove centos7 jobs | 11:07 |
| ykarel | from master | 11:07 |
| tobias-urdin | ykarel: im fine with all the changes if we really need them right now, added comments for all my concerns on the change 698142 | 11:10 |
| ykarel | tobias-urdin, THanks, /me check if we can address them right now, or take as a followup | 11:11 |
| tobias-urdin | my main concern is the Puppetfile which we maybe can close by just upping the mariadb minor version and uncap puppetlabs-mysql https://review.opendev.org/#/c/635728/ | 11:12 |
| tobias-urdin | all others, is as you say, issues with dependencies that we can fix along the way | 11:13 |
| ykarel | tobias-urdin, puppet-rabbitmq also needs update in Train one due to newer rabbitmq in centos8, i forgot to mention in commit message | 11:15 |
| ykarel | but /me checking all your comments and will post back on review once i have those answeres | 11:16 |
| tobias-urdin | looks like it's set to v10.0.0 in both files for puppet-rabbitmq | 11:18 |
| ykarel | in master ^^, in train different | 11:21 |
| *** jpena is now known as jpena|lunch | 11:58 | |
| *** ykarel is now known as ykarel|afk | 12:36 | |
| *** ykarel|afk has quit IRC | 12:41 | |
| *** gfidente has quit IRC | 12:45 | |
| *** gfidente has joined #puppet-openstack | 12:54 | |
| *** jpena|lunch is now known as jpena | 13:11 | |
| *** skramaja has quit IRC | 13:26 | |
| *** ykarel|afk has joined #puppet-openstack | 14:15 | |
| *** ykarel|afk is now known as ykarel | 14:18 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Add CentOS8 support and jobs https://review.opendev.org/698142 | 14:21 |
| *** tosky_ has joined #puppet-openstack | 14:23 | |
| *** tosky is now known as Guest27890 | 14:23 | |
| *** tosky_ is now known as tosky | 14:23 | |
| *** Guest27890 has quit IRC | 14:26 | |
| *** morazi has joined #puppet-openstack | 14:44 | |
| ykarel | EmilienM, can u review https://review.opendev.org/#/q/topic:rdo-centos8+(status:open+OR+status:merged)+project:openstack/puppet-openstack-integration | 14:46 |
| ykarel | tobias-urdin, can u also check reposnses from amoralej there | 14:46 |
| EmilienM | maybe? :P | 14:46 |
| ykarel | EmilienM, do u know reason of WARNING: class included by absolute name (::$class) on line 18? | 14:48 |
| ykarel | i am seeing this in stable/train patch | 14:48 |
| EmilienM | yes | 14:48 |
| ykarel | and i see there are some recent patches for such fixes | 14:48 |
| ykarel | is it already fixed? | 14:48 |
| EmilienM | one sec, /me pull link | 14:48 |
| EmilienM | ykarel: check https://review.opendev.org/#/c/697930/ | 14:48 |
| EmilienM | tobias-urdin did all the work for all modules | 14:49 |
| ykarel | EmilienM, ack, /me facing it in stable/train poi patch https://review.opendev.org/#/c/699204/ | 14:49 |
| ykarel | but doesn't look related to my patch | 14:50 |
| *** derekh has quit IRC | 14:54 | |
| *** theintern_ has joined #puppet-openstack | 15:03 | |
| *** jpena is now known as jpena|off | 15:04 | |
| tobias-urdin | EmilienM: ykarel i assume this patch fails to disable it https://review.opendev.org/#/c/697853/ | 15:08 |
| tobias-urdin | my patch restricted to version 1.x while this enables 2.x but disables that check | 15:08 |
| tobias-urdin | we can only run 2.x in master since it's fixed there now | 15:08 |
| tobias-urdin | need to pass disable_relative_classname_inclusion | 15:10 |
| tobias-urdin | here https://github.com/openstack/puppet-openstack-integration/blob/master/Rakefile#L6 | 15:10 |
| tobias-urdin | is my guess | 15:10 |
| *** jpena|off is now known as jpena | 15:13 | |
| ykarel | tobias-urdin, hmm possible, /me not aware of this part though | 15:20 |
| *** pkopec_ has joined #puppet-openstack | 15:48 | |
| *** pgaxatte has quit IRC | 15:50 | |
| *** pkopec_ has quit IRC | 15:57 | |
| *** rpittau is now known as rpittau|afk | 16:02 | |
| *** gfidente has quit IRC | 16:12 | |
| *** ykarel is now known as ykarel|away | 16:32 | |
| EmilienM | ykarel|away: I briefly reviewed https://review.opendev.org/698142 and I'm not happy with the Puppetfile proposal | 16:46 |
| EmilienM | I'll review more later | 16:46 |
| EmilienM | I have a lot in my backlog this week | 16:46 |
| ykarel|away | EmilienM, ack Thanks, if other things are fine i can try other uproaches | 16:51 |
| ykarel|away | for PUppetfile | 16:51 |
| ykarel|away | please comment if u have some suggestions around it, but NOTE some thing are temporary until we can get rid of Centos7 | 16:52 |
| EmilienM | k | 16:53 |
| *** pkopec has quit IRC | 16:58 | |
| *** ykarel|away has quit IRC | 17:14 | |
| *** jpena is now known as jpena|off | 17:44 | |
| openstackgerrit | Merged openstack/puppet-neutron stable/queens: CiscoNexus: Update and Remove deprecated vars https://review.opendev.org/699418 | 17:55 |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/699914 | 17:56 |
| *** morazi has quit IRC | 19:11 | |
| *** irclogbot_3 has quit IRC | 20:06 | |
| *** irclogbot_0 has joined #puppet-openstack | 20:10 | |
| *** theintern_ has quit IRC | 20:46 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack_spec_helper stable/queens: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700073 | 21:37 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack_spec_helper stable/rocky: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700074 | 21:37 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack_spec_helper stable/stein: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700075 | 21:38 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack_spec_helper stable/train: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700076 | 21:49 |
| *** tkajinam has joined #puppet-openstack | 23:04 | |
| openstackgerrit | Jake Yip proposed openstack/puppet-ovn master: Add Debian support for initialising DB https://review.opendev.org/700084 | 23:15 |
| openstackgerrit | Jake Yip proposed openstack/puppet-ovn master: Add clustering support for ovsdb https://review.opendev.org/700085 | 23:15 |
| *** tkajinam has quit IRC | 23:17 | |
| *** tkajinam has joined #puppet-openstack | 23:17 | |
| *** tosky has quit IRC | 23:22 | |
| *** bnemec has quit IRC | 23:24 | |
| *** bnemec has joined #puppet-openstack | 23:24 | |
| *** rcernin has joined #puppet-openstack | 23:39 | |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper stable/train: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700076 | 23:52 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper stable/stein: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700075 | 23:52 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper stable/rocky: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700074 | 23:52 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper stable/queens: Revert "Disable relative_classname_inclusion" https://review.opendev.org/700073 | 23:52 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-nova master: DNM: Test relateive name inclusion https://review.opendev.org/700087 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!