*** openstackgerrit has joined #openstack-ansible | 00:12 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Only adjust user_variables.yml if pre_config_osa is true https://review.openstack.org/605484 | 00:12 |
---|---|---|
*** ansmith has quit IRC | 00:42 | |
*** cshen has joined #openstack-ansible | 00:46 | |
*** cshen has quit IRC | 00:51 | |
*** Emine has quit IRC | 00:55 | |
*** sawblade6 has joined #openstack-ansible | 01:51 | |
*** mino_ has joined #openstack-ansible | 01:53 | |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_nova master: use import_playbook instead of include https://review.openstack.org/593435 | 01:54 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_glance master: use import_playbook instead of include https://review.openstack.org/593439 | 01:54 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_cinder master: Switch include to include_tasks or import_playbook https://review.openstack.org/585307 | 01:56 |
*** sawblade6 has quit IRC | 01:56 | |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_sahara master: Update messaging notification configuration https://review.openstack.org/605111 | 02:03 |
*** spatel has joined #openstack-ansible | 02:12 | |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_magnum master: Update messaging notification configuration https://review.openstack.org/605112 | 02:15 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_designate master: Update messaging notification configuration https://review.openstack.org/605109 | 02:16 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_trove master: Update messaging notification configuration https://review.openstack.org/605114 | 02:16 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_blazar master: Update messaging notification configuration https://review.openstack.org/605113 | 02:17 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_tacker master: Update messaging notification configuration https://review.openstack.org/605108 | 02:17 |
*** Bhujay has joined #openstack-ansible | 02:19 | |
*** Bhujay has joined #openstack-ansible | 02:20 | |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_tacker master: Remove the unnecessary verbose defined https://review.openstack.org/605117 | 02:31 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_tacker master: Remove the unused tacker_horizon_dashboard_disable defined https://review.openstack.org/605571 | 02:38 |
cloudnull | evenings all | 02:43 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-os_nova master: Stop running map_cell0 all the time https://review.openstack.org/602979 | 02:46 |
*** markvoelker has joined #openstack-ansible | 02:50 | |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client master: Fix apt pinning for ceph https://review.openstack.org/605429 | 02:59 |
*** Bhujay has quit IRC | 02:59 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-os_searchlight master: Add bionic to galaxy_info metadata https://review.openstack.org/588314 | 03:01 |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Completed required variables for README.rst https://review.openstack.org/583981 | 03:05 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_barbican master: Add bugs url link to README https://review.openstack.org/598522 | 03:07 |
openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Add an address of project source code repository. https://review.openstack.org/595507 | 03:08 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_swift master: Add bugs url link to README https://review.openstack.org/598558 | 03:08 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_ceilometer master: Add bugs url link to README https://review.openstack.org/598561 | 03:09 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_tacker master: Add bugs url link to README https://review.openstack.org/598563 | 03:09 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_cloudkitty master: Add bugs url link to README https://review.openstack.org/598564 | 03:10 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_watcher master: Add bugs url link to README https://review.openstack.org/598566 | 03:10 |
*** spatel has quit IRC | 03:11 | |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_panko master: Add bugs url link to README https://review.openstack.org/598567 | 03:11 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_trove master: Add bugs url link to README https://review.openstack.org/598570 | 03:11 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_designate master: Add bugs url link to README https://review.openstack.org/598575 | 03:12 |
openstackgerrit | ShangXiao proposed openstack/openstack-ansible-os_searchlight master: Add bugs url link to README https://review.openstack.org/598576 | 03:12 |
openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Remove duplicate link in README https://review.openstack.org/598485 | 03:13 |
openstackgerrit | Merged openstack/openstack-ansible-os_sahara master: Add the project source code repository in README https://review.openstack.org/584220 | 03:17 |
openstackgerrit | caoyuan proposed openstack/openstack-ansible-os_zaqar master: Completed required variables for README.rst https://review.openstack.org/583976 | 03:19 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Add the project source code repository in README https://review.openstack.org/586796 | 03:19 |
openstackgerrit | Merged openstack/openstack-ansible-os_molteniron master: Add the project source code repository in READM https://review.openstack.org/586792 | 03:20 |
openstackgerrit | Merged openstack/openstack-ansible-os_watcher master: Add the project source code repository in README https://review.openstack.org/586802 | 03:21 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-ui master: add the project source code repository https://review.openstack.org/600597 | 03:21 |
openstackgerrit | Merged openstack/openstack-ansible-pip_install master: Add the project source code repository in README https://review.openstack.org/586804 | 03:21 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: [Trivial Fix] Replace Chinese punctuation with English punctuation https://review.openstack.org/601140 | 03:22 |
openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Add the project source code repository in README https://review.openstack.org/586801 | 03:22 |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: [Trivial Fix] Replace Chinese punctuation with English punctuation https://review.openstack.org/601147 | 03:23 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-agent master: Add the project source code repository in README https://review.openstack.org/586793 | 03:23 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_client master: add the project source code repository https://review.openstack.org/600609 | 03:24 |
openstackgerrit | Merged openstack/openstack-ansible-os_sahara master: add the project source code repository https://review.openstack.org/600615 | 03:25 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-agent master: add the project source code repository https://review.openstack.org/600604 | 03:25 |
openstackgerrit | Merged openstack/openstack-ansible-os_molteniron master: add the project source code repository https://review.openstack.org/600619 | 03:25 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts master: add the project source code repository https://review.openstack.org/600596 | 03:25 |
openstackgerrit | Merged openstack/openstack-ansible-pip_install master: add the project source code repository https://review.openstack.org/600618 | 03:25 |
openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/605394 | 03:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_blazar master: modify the format of release notes link https://review.openstack.org/600361 | 03:26 |
openstackgerrit | Merged openstack/openstack-ansible-repo_build master: Add the project source code repository in README https://review.openstack.org/586806 | 03:26 |
openstackgerrit | Merged openstack/openstack-ansible-repo_build master: add the project source code repository https://review.openstack.org/600607 | 03:26 |
openstackgerrit | Merged openstack/openstack-ansible-repo_server master: add the project source code repository https://review.openstack.org/600605 | 03:27 |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: add the project source code repository https://review.openstack.org/600620 | 03:27 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: rabbitmq_upgrade_check: Do not complain for upgrades on distro packages https://review.openstack.org/605417 | 03:27 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: Add the project source code repository in README https://review.openstack.org/586805 | 03:27 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: add the project source code repository https://review.openstack.org/600601 | 03:27 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Enable the xinetd service for reboot https://review.openstack.org/605500 | 03:33 |
openstackgerrit | Merged openstack/openstack-ansible-os_zaqar master: Trivial: Fix the pep8 warning https://review.openstack.org/589062 | 03:39 |
openstackgerrit | Merged openstack/openstack-ansible-os_zaqar master: add the project source code repository https://review.openstack.org/600595 | 03:39 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/605395 | 03:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: [Trivial Fix] Replace Chinese punctuation with English punctuation https://review.openstack.org/601146 | 03:40 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: tests-ansible-env-prep: Fix plugin, tests and ops testing on Vagrant https://review.openstack.org/592527 | 03:42 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Drop un-used packages from role https://review.openstack.org/600670 | 03:45 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Add source code information to readme https://review.openstack.org/599934 | 03:45 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Add the project source code repository in README https://review.openstack.org/586790 | 03:45 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: add the project source code repository https://review.openstack.org/600611 | 03:45 |
*** sawblade6 has joined #openstack-ansible | 03:48 | |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/605393 | 03:49 |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Add the project source code repository in README https://review.openstack.org/584225 | 03:49 |
*** sawblade6 has quit IRC | 03:53 | |
*** dxiri has joined #openstack-ansible | 03:53 | |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_server master: Add the project source code repository in README https://review.openstack.org/584221 | 03:54 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_server master: add the project source code repository https://review.openstack.org/600598 | 03:54 |
*** dxiri has quit IRC | 03:57 | |
*** udesale has joined #openstack-ansible | 04:18 | |
openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Bring gather vars in line with other OSA roles https://review.openstack.org/586818 | 04:19 |
openstackgerrit | Merged openstack/openstack-ansible-os_nova master: add the project source code repository https://review.openstack.org/600602 | 04:19 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Add additional API tests to confirm functionality https://review.openstack.org/605194 | 04:38 |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Add source code repository notes link to README https://review.openstack.org/583082 | 04:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: add the project source code repository https://review.openstack.org/600617 | 04:40 |
*** shyamb has joined #openstack-ansible | 04:48 | |
*** sawblade6 has joined #openstack-ansible | 04:49 | |
*** dave-mccowan has joined #openstack-ansible | 04:49 | |
*** cshen has joined #openstack-ansible | 04:54 | |
*** sawblade6 has quit IRC | 04:55 | |
*** cshen has quit IRC | 04:58 | |
*** macza has joined #openstack-ansible | 05:03 | |
*** macza has quit IRC | 05:08 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add changes to the sysconfig defaults file https://review.openstack.org/605586 | 05:32 |
*** Bhujay has joined #openstack-ansible | 05:35 | |
*** sawblade6 has joined #openstack-ansible | 05:49 | |
openstackgerrit | Merged openstack/openstack-ansible master: Remove keystone_admin-back load balancer manipulation https://review.openstack.org/605494 | 05:53 |
openstackgerrit | Merged openstack/openstack-ansible stable/ocata: Use loop_control for haproxy keystone back-end enablement https://review.openstack.org/605519 | 05:53 |
openstackgerrit | Merged openstack/openstack-ansible stable/queens: Use loop_control for haproxy keystone back-end enablement https://review.openstack.org/605517 | 05:53 |
*** sawblade6 has quit IRC | 05:54 | |
*** DanyC has joined #openstack-ansible | 05:55 | |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: zuul.d: Update jobs for package installations https://review.openstack.org/604988 | 05:59 |
*** shyamb has quit IRC | 06:11 | |
*** shyamb has joined #openstack-ansible | 06:19 | |
*** shyamb has quit IRC | 06:25 | |
*** cshen has joined #openstack-ansible | 06:29 | |
*** DanyC has quit IRC | 06:32 | |
*** DanyC has joined #openstack-ansible | 06:32 | |
*** pcaruana has joined #openstack-ansible | 06:33 | |
*** jamesdenton has quit IRC | 06:35 | |
*** shyamb has joined #openstack-ansible | 06:36 | |
*** DanyC has quit IRC | 06:52 | |
*** sawblade6 has joined #openstack-ansible | 06:53 | |
*** sawblade6 has quit IRC | 06:57 | |
*** sawblade6 has joined #openstack-ansible | 06:58 | |
openstackgerrit | Ionuț Bîru proposed openstack/openstack-ansible-os_ceilometer master: Remove deprecated param from ceilometer-upgrade https://review.openstack.org/605601 | 06:59 |
*** sawblade6 has quit IRC | 07:02 | |
ioni | i failed at mentioning the bug number in commit message :( | 07:06 |
jrosser | ioni: if you put a link to the upstream review url instead of the commit-id in the commit message then folks can also navigate to it outside gerrit | 07:09 |
*** shyamb has quit IRC | 07:11 | |
*** sawblade6 has joined #openstack-ansible | 07:16 | |
*** chkumar|off is now known as chandankumar | 07:18 | |
*** sawblade6 has quit IRC | 07:20 | |
*** shardy has joined #openstack-ansible | 07:21 | |
*** mma has joined #openstack-ansible | 07:22 | |
*** sawblade6 has joined #openstack-ansible | 07:26 | |
ioni | i'm running rocky! | 07:29 |
ioni | :D | 07:29 |
*** cshen has quit IRC | 07:32 | |
*** sawblade6 has quit IRC | 07:36 | |
*** gkadam has joined #openstack-ansible | 07:36 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova stable/rocky: Compute: Enable vGPU in Nova if exist on the host https://review.openstack.org/603928 | 07:36 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova stable/rocky: Revert region name option to os_region_name for Cinder https://review.openstack.org/596577 | 07:38 |
*** mwe1 has left #openstack-ansible | 07:43 | |
*** tosky has joined #openstack-ansible | 07:45 | |
*** shyamb has joined #openstack-ansible | 07:52 | |
*** sawblade6 has joined #openstack-ansible | 07:56 | |
*** cshen has joined #openstack-ansible | 07:59 | |
*** sawblade6 has quit IRC | 08:01 | |
*** cshen has quit IRC | 08:03 | |
*** lkoranda has quit IRC | 08:12 | |
*** DanyC has joined #openstack-ansible | 08:15 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: AppArmor: Handle required profiles for neutron services https://review.openstack.org/604695 | 08:21 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/604286 | 08:21 |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/rocky: Adding [service_catalog] in ironic.conf https://review.openstack.org/605311 | 08:33 |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/queens: Adding [service_catalog] in ironic.conf https://review.openstack.org/605313 | 08:33 |
*** sep has quit IRC | 08:34 | |
*** sawblade6 has joined #openstack-ansible | 08:49 | |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Trivial: Fix the pep8 warning https://review.openstack.org/589051 | 08:49 |
evrardjp | odyssey4me: I will issue a release today then, our final rc :) | 08:49 |
*** gkadam is now known as gkadam-afk | 08:52 | |
*** sawblade6 has quit IRC | 08:54 | |
*** gkadam-afk is now known as gkadam | 08:59 | |
ioni | evrardjp, can you review https://review.openstack.org/#/c/605601/ ? | 09:03 |
evrardjp | ioni: nice | 09:05 |
*** electrofelix has joined #openstack-ansible | 09:05 | |
ioni | evrardjp, do i need to modify the commit message? | 09:06 |
evrardjp | yours? | 09:07 |
evrardjp | yours is improvable, but compared to ceilometer's one it's already nicer :D | 09:08 |
noonedeadpunk | logan- are you here? | 09:08 |
evrardjp | noonedeadpunk: logan- is not europe based, so probably later in the day | 09:08 |
evrardjp | ioni: it would be nice if you could follow's jrosser opinion too, but I find it good enough myself. If you change it, I will revote (if it's done quick!) | 09:09 |
noonedeadpunk | ioni: yeah, good commit, I was looking for it, but had no time for publishing change | 09:09 |
ioni | evrardjp, well, i'm not very familiar with how to resend the change into the same review | 09:10 |
ioni | last time I failed and sent a new review | 09:10 |
noonedeadpunk | evrardjp: got it. I've just wanted to tolk a bit about his commit https://review.openstack.org/#/c/504094/3 (which I've commented a bit...) | 09:10 |
evrardjp | ioni: git add . ; git commit --amend ; git review | 09:10 |
evrardjp | ioni: keep the commit-id intact and it should be fine | 09:11 |
evrardjp | ioni: you can also edit in gerrit webinterface | 09:11 |
ioni | ok, what's the syntax to link to the bug report? :D | 09:12 |
ioni | # id didn't work | 09:12 |
noonedeadpunk | I think it's not very polite from my side to comment core's patch, but I just want to help as I experience some issues with current version | 09:13 |
odyssey4me | nicolasbock: could you propose backports for https://review.openstack.org/605500 to rocky, queens, pike, ocata when you have a moment | 09:13 |
noonedeadpunk | ioni> https://docs.openstack.org/infra/manual/developers.html#development-workflow | 09:13 |
odyssey4me | evrardjp: https://review.openstack.org/605504 is release critical, without that a multi-node build is non-functional | 09:14 |
*** shyamb has quit IRC | 09:16 | |
openstackgerrit | Ionuț Bîru proposed openstack/openstack-ansible-os_ceilometer master: Remove skip-metering-database deprecated param from ceilometer-upgrade https://review.openstack.org/605601 | 09:17 |
ioni | ok, i've updated the message | 09:18 |
ioni | please vote | 09:18 |
ioni | should i choose cherry-pick in stable/queens? | 09:19 |
*** Bhujay has quit IRC | 09:19 | |
noonedeadpunk | ioni: probably rocky at least, but only after commit being merged | 09:20 |
ioni | ups, i ment rocky | 09:20 |
ioni | :D | 09:20 |
ioni | ok | 09:20 |
*** shyamb has joined #openstack-ansible | 09:24 | |
*** Emine has joined #openstack-ansible | 09:24 | |
ioni | i still have one issue with ceilometer: https://bugs.launchpad.net/openstack-ansible/+bug/1794688 | 09:24 |
openstack | Launchpad bug 1794688 in openstack-ansible "Rocky upgrade nova_oslomsg_notify_host_group' is undefined" [Undecided,New] | 09:24 |
ioni | but couldn't figure it out how to fix it | 09:25 |
noonedeadpunk | ioni: I've placed variables in user_variables_ceilometer.yml | 09:25 |
*** Bhujay has joined #openstack-ansible | 09:25 | |
ioni | noonedeadpunk, because it's not read from os_nova ? | 09:26 |
noonedeadpunk | it looks like this http://paste.openstack.org/show/730993/ | 09:26 |
ioni | noonedeadpunk, hmm | 09:27 |
noonedeadpunk | actually I didn't dig too deep, as it's smth that is easier to put into vars file | 09:27 |
ioni | noonedeadpunk, you added them all because there are problems with all? | 09:28 |
noonedeadpunk | ioni: with most:) | 09:29 |
ioni | noonedeadpunk, still don't get it :D | 09:31 |
ioni | noonedeadpunk, i've grepped ansible roles for nova_rabbitmq_telemetry_host_group and is not defined or used | 09:32 |
ioni | for rocky | 09:32 |
*** spatel has joined #openstack-ansible | 09:33 | |
noonedeadpunk | ioni: actually it's in use only for ceilometer when you enable nova_ceilometer_enabled | 09:33 |
noonedeadpunk | and used in ceilometer.conf.j2 | 09:34 |
noonedeadpunk | at last in queens (I still don't have rocky for some reason) | 09:34 |
*** spatel has quit IRC | 09:37 | |
*** sawblade6 has joined #openstack-ansible | 09:49 | |
noonedeadpunk | * s/last/least/g | 09:50 |
*** sawblade6 has quit IRC | 09:54 | |
*** shyamb has quit IRC | 09:57 | |
*** shyamb has joined #openstack-ansible | 09:57 | |
noonedeadpunk | ioni: yep, you're right, for rocky it is now nova_oslomsg_notify_vhost | 10:03 |
ioni | noonedeadpunk, yep | 10:03 |
noonedeadpunk | but still you probably should try specifying this variable into user_variables | 10:06 |
ioni | noonedeadpunk, i did that and it works | 10:09 |
ioni | kinda strange since I do not define anywhere to enable nova ceilometer | 10:09 |
ioni | like you have | 10:09 |
*** hamzaachi has joined #openstack-ansible | 10:09 | |
ioni | but still want to configure | 10:09 |
noonedeadpunk | ioni: probably nova_ceilometer_enabled is set to enabled by default on rocky in case of ceilometer being installed | 10:10 |
jrosser | noonedeadpunk: is https://review.openstack.org/#/c/504094/3 trying to cover all situations, including where the ceph mon is an OSA container - the OSA connection plugin would be required there rather than SSH? | 10:10 |
noonedeadpunk | jrosser: I thought yes, but I didn't test ceph osd as a container created by OSA. But it should work both with generic SSH and connection plugin. I wrote about SSH probably, as I'm having external ceph deployment and for me it was a case | 10:16 |
noonedeadpunk | *ceph mon as a container | 10:17 |
*** shardy is now known as shardy_mtg | 10:18 | |
noonedeadpunk | jrosser you mean that delegate_to doesn't use connection plugin for it's connection? | 10:18 |
*** hamzaachi has quit IRC | 10:23 | |
*** hamzaachi has joined #openstack-ansible | 10:23 | |
noonedeadpunk | jrosser: yes, delegate_to utilize connection plugin, so it should suit every type of connection | 10:25 |
*** sep has joined #openstack-ansible | 10:25 | |
*** hamzaachi has quit IRC | 10:37 | |
*** hamzaachi has joined #openstack-ansible | 10:44 | |
*** udesale has quit IRC | 10:54 | |
openstackgerrit | Merged openstack/openstack-ansible-os_heat master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/605389 | 10:56 |
*** shyamb has quit IRC | 10:56 | |
*** shyamb has joined #openstack-ansible | 10:58 | |
openstackgerrit | Merged openstack/openstack-ansible-os_designate master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/605391 | 11:06 |
*** spatel has joined #openstack-ansible | 11:10 | |
*** macza has joined #openstack-ansible | 11:11 | |
*** hamzaachi has quit IRC | 11:13 | |
*** macza has quit IRC | 11:15 | |
*** spatel has quit IRC | 11:16 | |
*** hamzaachi has joined #openstack-ansible | 11:16 | |
openstackgerrit | Kaio Kassiano Moura Oliveira proposed openstack/openstack-ansible-os_monasca master: Add support for openSUSE https://review.openstack.org/588584 | 11:17 |
*** hamzaachi has quit IRC | 11:22 | |
*** joabdearaujo has joined #openstack-ansible | 11:26 | |
logan- | jrosser noonedeadpunk: that patch came out of some work I was doing a while back to test osa without openssh-server installed in the containers. since i believe we mask ssh server in the containers now, i'm not sure why our ceph gates are green if someone hasn't merged something like that already. might want to check if ceph_client has something similar now. | 11:35 |
* logan- has to go afk for a while | 11:35 | |
*** Tahvok has left #openstack-ansible | 11:42 | |
*** tosky__ has joined #openstack-ansible | 11:44 | |
*** tosky has quit IRC | 11:44 | |
*** tosky has joined #openstack-ansible | 11:45 | |
*** olivierb has joined #openstack-ansible | 11:47 | |
*** pcaruana has quit IRC | 11:50 | |
*** Bhujay has quit IRC | 11:51 | |
*** Bhujay has joined #openstack-ansible | 11:52 | |
*** Bhujay has quit IRC | 11:53 | |
*** Bhujay has joined #openstack-ansible | 11:53 | |
*** fghaas has joined #openstack-ansible | 11:54 | |
*** Bhujay has quit IRC | 11:54 | |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Remove role used for upgrade tests https://review.openstack.org/603903 | 11:55 |
*** olivierb has quit IRC | 11:58 | |
*** mino_ has quit IRC | 11:58 | |
*** olivierb has joined #openstack-ansible | 11:59 | |
fghaas | Hi everyone, do we have a built-in ability to drop a random file (or symlink) into a given location (a) on a host or container, (b) in a venv? Case in point: (a) for Cinder, some backends require configuration files to be dropped in a specific location on the cinder-volume host. The Huawei OceanStor backend being one example. (b) For Heat, some vendors support custom resource types via pip-installable plugins which when instal | 12:01 |
fghaas | led with heat_pip_packages obviously end up in the venv, and must then be symlinked from /usr/lib/heat. Example in that case would be F5. | 12:01 |
noonedeadpunk | logan- I truly believe that we need this change. As currently only this part makes problems if someone uses non-standart ssh port on all hosts | 12:05 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: AppArmor: Handle required profiles for neutron services https://review.openstack.org/604695 | 12:07 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/604286 | 12:07 |
noonedeadpunk | logan- I was trying to achieve smth common, but as I was unable to set wait_for_connection target hosts (with delegation_to localhost), like it's possible with wait_for, I defer this idea and just tried to add port defenition into ssh command. But the idea of using wait_for with delegation to ceph_mons is good enough, as it should connect to the ceph mons via ssh and test connection to the container which should futher interact | 12:10 |
noonedeadpunk | with ceph | 12:10 |
*** spatel has joined #openstack-ansible | 12:10 | |
*** spatel has quit IRC | 12:11 | |
*** shyamb has quit IRC | 12:13 | |
odyssey4me | fghaas: we've got extra/optional plugins implemented for neutron which establishes a pattern for extra drivers - and the cinder role could do with some of those - that pattern works well for pip installable things... then in nova we have some prior art for symlinking things installed via packages into the venv as well | 12:14 |
*** shyamb has joined #openstack-ansible | 12:14 | |
*** shardy_mtg is now known as shardy | 12:14 | |
odyssey4me | fghaas: if you could provides a gist or something with the procedure you want to have automated, then I'd be happy to implement it via ansible for you - it may be useful to register a bug, then reference the gist in it | 12:14 |
*** tosky__ has quit IRC | 12:15 | |
*** jamesdenton has joined #openstack-ansible | 12:16 | |
fghaas | odyssey4me: here's some documentation from F5: https://clouddocs.f5.com/products/openstack/heat-plugins/newton/ | 12:16 |
jamesdenton | mornin' | 12:16 |
fghaas | Like I mentioned, I can get the package installed with heat_pip_packages. It's the symlink into /usr/lib/heat that currently can't happen in an automated fashion. | 12:18 |
fghaas | (and this symlink has to be replaced each time the venv is updated, of course) | 12:18 |
odyssey4me | fghaas: it's a bit strange that it wants that symlink - surely that directory should be in the venv too? | 12:19 |
odyssey4me | ie /openstack/venvs/heat.../usr/lib/heat | 12:20 |
fghaas | Hmm. That I haven't tried. But goof point. | 12:20 |
fghaas | *good* point of course :) | 12:20 |
odyssey4me | I can work up a patch really quickly - do you have a way to test it? | 12:20 |
fghaas | For Cinder, there's this old (Mitaka) documentation that apparently hasn't been carried over into later versions: https://docs.openstack.org/mitaka/config-reference/block-storage/drivers/huawei-storage-driver.html#volume-driver-configuration | 12:20 |
fghaas | Right now my test env is busy doing a full queens install, but I can certainly do that tomorrow or Monday (note, there's absolutely no urgency on this on my part, so don't drop anything else for this please) | 12:21 |
odyssey4me | ok, let me finish up what I'm currently doing and I'll work up some patches for these, they're simple enough - the only thing is that I'm obviously not able to test them | 12:22 |
mgariepy | morning everyone | 12:22 |
fghaas | Unrelated, but as it happened it bit me with that F5 thing: we seem to have no way to use a git+https egg with *_pip_packages. If one does, the requirements.txt file ends up with that link being in there *twice*, i.e. garbled, and then the wheel build breaks. | 12:26 |
fghaas | (I was only able to test that in pike though, so I didn't file a bug because I couldn't tell whether it had been fixed in rocky) | 12:26 |
jrosser | noonedeadpunk: wouldnt you resolve non standard ssh ports through your .ssh/config on the deploy host - trouble is i'm not actually sure what issue you are running into | 12:30 |
nicolasbock | > [nicolasbock](https://matrix.to/#/@nicolasbock:matrix.org): could you propose backports for https://review.openstack.org/605500 to rocky, queens, pike, ocata when you have a moment | 12:30 |
nicolasbock | Yes odyssey4me | 12:30 |
odyssey4me | fghaas: you'd have to put the package name in the *_pip_packages list, and put the git_https://<repo>@<SHA/tag/branch>#egg=<package name> in the variable repo_build_upper_constraints_overrides | 12:30 |
odyssey4me | that should have been git+https | 12:30 |
noonedeadpunk | jrosser: yeah, I've resolved it already with ssh config, but I think this ssh command is something that should to be replaced | 12:31 |
odyssey4me | fghaas: note that repo_build_upper_constraints_overrides is a list variable | 12:31 |
fghaas | odyssey4me: oh cool, thanks for that | 12:31 |
noonedeadpunk | jrosser: and setting ssh config is a bit not the "ansible way" as for me... | 12:32 |
odyssey4me | fghaas: hopefully it'll be easier to simplify all that stuff in stein - I'm doing a bunch of rework to make the python build process easier to understand and use | 12:32 |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-ceph_client stable/rocky: Fix apt pinning for ceph https://review.openstack.org/605669 | 12:35 |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-ceph_client stable/queens: Fix apt pinning for ceph https://review.openstack.org/605670 | 12:36 |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-ceph_client stable/pike: Fix apt pinning for ceph https://review.openstack.org/605671 | 12:36 |
mgariepy | do we still maintain ocata ? | 12:36 |
noonedeadpunk | mgariepy: I'm not sure about OSA, but it's not supposed to be maintained generally... | 12:39 |
*** pcaruana has joined #openstack-ansible | 12:39 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Update repo_build test-vars data https://review.openstack.org/605672 | 12:40 |
mgariepy | the patch conflict anyway. and i don't think someone would want to install ocata now. and older install should still works. | 12:40 |
odyssey4me | mgariepy: a little, if you know a fix and are prepared to do it - go for it, otherwise whoever wants it can see your fix in later branches and fix it | 12:41 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/rocky: Update repo_build test-vars data https://review.openstack.org/605675 | 12:44 |
*** ansmith has joined #openstack-ansible | 12:44 | |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-ceph_client stable/ocata: Fix apt pinning for ceph https://review.openstack.org/605677 | 12:46 |
mgariepy | ok, i did a little fix then :) haha | 12:46 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/queens: Update repo_build test-vars data https://review.openstack.org/605679 | 12:47 |
*** shyam89 has joined #openstack-ansible | 12:50 | |
*** shyamb has quit IRC | 12:52 | |
openstackgerrit | Kaio Kassiano Moura Oliveira proposed openstack/openstack-ansible-os_monasca master: Add support for openSUSE https://review.openstack.org/588584 | 12:52 |
*** shyam89 has quit IRC | 12:55 | |
*** kstev has joined #openstack-ansible | 12:57 | |
*** hamzaachi has joined #openstack-ansible | 13:02 | |
ebbex | odyssey4me: If I want to bump the 'spicehtml5_git_install_branch' for stable/pike to fix an issue with screens being upside down. Would it be preferrable take it from one of the commits to master that actually updated it an cherry-pick it down? | 13:04 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/pike: Update repo_build test-vars data https://review.openstack.org/605691 | 13:05 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/ocata: Update repo_build test-vars data https://review.openstack.org/605692 | 13:06 |
odyssey4me | ebbex: no, you can do it directly - however, please ensure that the later stable branches also include the same fix already so that we don't have a regression when upgrading | 13:08 |
odyssey4me | so start the fix in the most recent branch with the same bug | 13:08 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/queens: Remove galera_client meta-dependency https://review.openstack.org/603772 | 13:10 |
openstackgerrit | Arx Cruz proposed openstack/openstack-ansible-tests master: Add stackviz directory to the list of find_files https://review.openstack.org/604292 | 13:12 |
*** scroll2 has joined #openstack-ansible | 13:22 | |
openstackgerrit | Erik Berg proposed openstack/openstack-ansible stable/queens: Bump SHA to fix flipping screens https://review.openstack.org/605700 | 13:24 |
*** spatel has joined #openstack-ansible | 13:27 | |
spatel | jamesdenton: morning! | 13:27 |
spatel | I got my SR-IOV result and want to share with you.. its very interesting..! | 13:28 |
spatel | i have meeting in 5min after that i will show you my result. | 13:28 |
ioni | spatel, looking forward | 13:36 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Remove broken uptime tests during upgrade https://review.openstack.org/605151 | 13:37 |
jamesdenton | spatel cool, lemme know | 13:41 |
*** spatel has quit IRC | 13:50 | |
openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_heat stable/rocky: Setup oslo.messaging extra packages for optional drivers https://review.openstack.org/605727 | 13:58 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul: Add template for jobs using distribution packages https://review.openstack.org/605734 | 14:04 |
*** Leo_m has joined #openstack-ansible | 14:09 | |
*** skiedude has joined #openstack-ansible | 14:11 | |
*** spatel has joined #openstack-ansible | 14:17 | |
spatel | jamesdenton: ioni sorry i was in meeting | 14:17 |
spatel | here is the CPU load graph non-SRIOV vs SRIOV https://ibb.co/mQ0XHU | 14:17 |
jamesdenton | no worries | 14:17 |
spatel | drastic difference | 14:17 |
jamesdenton | indeed | 14:17 |
ioni | spatel, nice | 14:17 |
odyssey4me | wow, that's very cool spatel | 14:17 |
spatel | my application running very smooth zero packet loss at high load | 14:17 |
ioni | spatel, latency is better ? | 14:18 |
spatel | very good | 14:19 |
spatel | let me share more graph | 14:19 |
spatel | hold on | 14:19 |
ioni | spatel, so this graph is showing the load of the compute host | 14:20 |
ioni | compute host that only has your client vm or other vms? | 14:20 |
spatel | This is load on VM instance | 14:20 |
ioni | ok | 14:21 |
mnaser | spotz: SRIOV means you cant use any filters right | 14:21 |
spatel | i will hold on collecting all info... | 14:21 |
*** skiedude has quit IRC | 14:24 | |
spatel | This is VM instance graph and its very strange, its interrupt ratio https://ibb.co/cFKdP9 | 14:26 |
spatel | ioni: this is compute node load but i don't have historic data because this is new compute node... but i can check historic compute node https://ibb.co/cw7nHU | 14:28 |
ioni | spatel, it's not quite relevant | 14:29 |
ioni | it doesn't show the drop | 14:29 |
spatel | This is smokeping data for that VM https://ibb.co/e7kDrp | 14:30 |
cloudnull | mornings | 14:31 |
spatel | morning! | 14:32 |
spatel | I am going with all SR-IOV, converting my all compute node in SR-IOV ( i have to make lots of network changes :( ) | 14:32 |
spatel | I am going to test DPDK soon and compare that result too | 14:32 |
cloudnull | very cool! | 14:33 |
spatel | If jamesdenton around i have question related Interrupt rate? | 14:34 |
jamesdenton | i'm around, but not sure i can help you with that | 14:35 |
jamesdenton | i'll do my best | 14:35 |
spatel | just curious why its high, does SR-IOV using more interrupts.. | 14:35 |
*** fghaas has quit IRC | 14:36 | |
spatel | but all and all I think i am great with SR-IOV :) | 14:36 |
jamesdenton | You're seeing an increase in the VM? | 14:36 |
spatel | yes inside VM interrupt rate is high | 14:36 |
jamesdenton | i suspect it's just transferred from the hypervisor to the VM itself in this case | 14:41 |
jamesdenton | but that isn't really my wheelhouse | 14:41 |
*** dxiri has joined #openstack-ansible | 14:44 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add changes to the sysconfig defaults file https://review.openstack.org/605586 | 14:44 |
*** evrardjp has quit IRC | 14:45 | |
*** fghaas has joined #openstack-ansible | 14:45 | |
*** evrardjp has joined #openstack-ansible | 14:47 | |
*** evrardjp has quit IRC | 14:58 | |
hwoarang | cloudnull: hello. i was looking to nspawn_host for leap 15 but i noticed you already started working on it on https://review.openstack.org/#/c/592177/ | 14:59 |
hwoarang | do you plan to pick it up again so should i look at it myself? | 14:59 |
cloudnull | I was starting to add it, if you have a handle on what needs to be done feel free to do make it go otherwise Id be happy to pick it up again . | 15:00 |
*** hamzy_ has quit IRC | 15:00 | |
hwoarang | no clue what needs to be done yet :) i will have a look soon-ish | 15:00 |
*** shardy is now known as shardy_afk | 15:00 | |
*** skiedude has joined #openstack-ansible | 15:02 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Remove skip-metering-database deprecated param from ceilometer-upgrade https://review.openstack.org/605601 | 15:02 |
*** gillesMo has joined #openstack-ansible | 15:03 | |
spatel | jamesdenton: thanks i will dig more and collect more data to see any pattern i can find | 15:03 |
openstackgerrit | Ionuț Bîru proposed openstack/openstack-ansible-os_ceilometer stable/rocky: Remove skip-metering-database deprecated param from ceilometer-upgrade https://review.openstack.org/605768 | 15:04 |
openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cinder master: Added volume usage collection, when cinder_ceilometer_enabled https://review.openstack.org/605769 | 15:05 |
*** evrardjp has joined #openstack-ansible | 15:07 | |
*** shardy_afk is now known as shardy | 15:07 | |
openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cinder master: Added volume usage collection, when cinder_ceilometer_enabled https://review.openstack.org/605769 | 15:08 |
mgariepy | spatel, for your SR-IOV config, did you need to move the pci_passthrough_whitelist under [pci] in nova.conf ? | 15:19 |
*** kstev has quit IRC | 15:20 | |
*** jbadiapa has quit IRC | 15:20 | |
openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-galera_server stable/rocky: Enable the xinetd service for reboot https://review.openstack.org/605776 | 15:20 |
openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-galera_server stable/queens: Enable the xinetd service for reboot https://review.openstack.org/605777 | 15:21 |
openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-galera_server stable/pike: Enable the xinetd service for reboot https://review.openstack.org/605778 | 15:21 |
openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-galera_server stable/ocata: Enable the xinetd service for reboot https://review.openstack.org/605779 | 15:21 |
spatel | mgariepy: let me check i just did all config via OSA | 15:22 |
*** dave-mccowan has quit IRC | 15:23 | |
*** jralbert has joined #openstack-ansible | 15:24 | |
jralbert | Is there a facility in OSA to extend systemd limits for services OSA installs? Specifically we encountered issues with LimitFILENO on cinder-volume service, and I ended up updating its systemd unit file by hand to extend the limits; it would be nice to be able to do this via configuration. | 15:26 |
*** hamzy_ has joined #openstack-ansible | 15:26 | |
*** shardy has quit IRC | 15:26 | |
*** jbadiapa has joined #openstack-ansible | 15:27 | |
cloudnull | jralbert yes you can extend the systemd service units for the openstack services. | 15:27 |
* cloudnull gets the override | 15:27 | |
cloudnull | https://github.com/openstack/openstack-ansible-os_cinder/blob/master/defaults/main.yml#L311-L345 | 15:28 |
cloudnull | specifically https://github.com/openstack/openstack-ansible-os_cinder/blob/master/defaults/main.yml#L337 | 15:28 |
cloudnull | all of the services have an init override option which is a dictionary | 15:28 |
jralbert | And that just drives the same config override system as other conf files | 15:29 |
jralbert | ? | 15:29 |
jralbert | that is, dict key maps to ini key? | 15:29 |
*** hamzy_ has quit IRC | 15:30 | |
openstackgerrit | Panagiotis Karalis proposed openstack/openstack-ansible-os_tacker master: [WIP]Introduce the Barbican in XCI for SFC-HA scenario https://review.openstack.org/605784 | 15:30 |
cloudnull | so you should be able to add something like so: http://paste.openstack.org/show/731026/ | 15:31 |
cloudnull | yes it allows you to treat any INI file like a dict with each section being a top level key in this case [Service] | 15:32 |
cloudnull | jralbert sorry http://paste.openstack.org/show/731028/ | 15:33 |
spatel | mgariepy: here is my config snippet of /etc/nova/nova.conf | 15:33 |
spatel | http://paste.openstack.org/show/731027/ | 15:33 |
spatel | in [DEFAULT] section | 15:34 |
jralbert | cloudnull: Glorious, thanks for the assist | 15:34 |
mgariepy | https://docs.openstack.org/nova/pike/admin/pci-passthrough.html#enable-pci-passthrough-compute | 15:35 |
cloudnull | ++ all of the services we deploy should have a similar variable enabling tunables in the service units. | 15:35 |
cloudnull | jralbert happy to help | 15:35 |
mgariepy | also from https://docs.openstack.org/nova/queens/configuration/sample-config.html # Deprecated group/name - [DEFAULT]/pci_passthrough_whitelist | 15:37 |
openstackgerrit | Merged openstack/openstack-ansible-os_nova stable/rocky: Compute: Enable vGPU in Nova if exist on the host https://review.openstack.org/603928 | 15:40 |
spatel | mgariepy: is there any issue with that if that is not in [pci] section or its a bug? | 15:46 |
*** shardy has joined #openstack-ansible | 15:46 | |
*** chandankumar is now known as chkumar|off | 15:46 | |
*** macza has joined #openstack-ansible | 15:47 | |
mgariepy | for now it works i guess :D | 15:48 |
mgariepy | but might be dropped at some point | 15:48 |
mgariepy | i'll patch it up, the config in osa won't change. only the template. | 15:48 |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-os_nova master: Update the pci config for nova. https://review.openstack.org/605789 | 15:49 |
*** gyee has joined #openstack-ansible | 15:53 | |
*** rpittau has quit IRC | 15:54 | |
*** gyee has quit IRC | 15:54 | |
*** cshen has joined #openstack-ansible | 15:56 | |
*** gyee has joined #openstack-ansible | 15:57 | |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible master: Add openstackclient bash completion https://review.openstack.org/605795 | 16:00 |
*** fghaas has quit IRC | 16:04 | |
*** mma has quit IRC | 16:06 | |
*** cshen has quit IRC | 16:11 | |
mgariepy | any cores willing to push those ? https://review.openstack.org/#/q/owner:gariepy.marc%2540gmail.com+status:open+project:openstack/openstack-ansible-ceph_client | 16:12 |
*** jralbert has quit IRC | 16:13 | |
*** hamzaachi has quit IRC | 16:20 | |
openstackgerrit | Logan V proposed openstack/openstack-ansible-ceph_client master: Remove the dependency on SSH for monitors https://review.openstack.org/504094 | 16:28 |
*** cshen has joined #openstack-ansible | 16:31 | |
noonedeadpunk | logan- but with wait_for_connection and delegate_to we're connectiong to ceph mon, and then from it we're trying to reach cinder/galera/gnocchi container, which defenitelly shouldn't be reachable from ceph mon (if ceph mon is not a container deployed by osa) | 16:35 |
logan- | ? | 16:36 |
noonedeadpunk | logan-: I'm about https://review.openstack.org/#/c/504094/ | 16:36 |
*** DanyC has quit IRC | 16:36 | |
logan- | yeah, I know, but delegate_to simply changes the target host of the task, it does not attempt to create the connection from any other host than the ansible deployment host | 16:37 |
logan- | unless the ansible connection calls for that | 16:37 |
logan- | (ie if you had a jump host defined in your ssh config or something) | 16:37 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: switch documentation job to new PTI https://review.openstack.org/594681 | 16:38 |
*** gillesMo has quit IRC | 16:38 | |
noonedeadpunk | delegate_to is great here, but I'm not sure, that wait_for_connection from external ceph monitor should work properly, as wait_for_connection establishes connection to host, for which task has been runned. So we're connecting to monitor, and then from it returning back to container. Or I'm missing smth? | 16:40 |
logan- | as long as you can ssh to the external ceph monitor (ie run ansible tasks on it normally as you do now with ceph_client), then the wait_for_connection will succeed as well | 16:42 |
logan- | it is just a module to confirm that ansible can successfully run tasks on an inventory host | 16:42 |
*** cshen has quit IRC | 16:43 | |
noonedeadpunk | logan-: yep, I know, I mean that delegate_to establishes SSH connection from deployment host to monitor, and wait_for_connection is supposed to open connection from monitor to cinder container mgmt ip | 16:44 |
*** cshen has joined #openstack-ansible | 16:44 | |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client stable/rocky: Fix apt pinning for ceph https://review.openstack.org/605669 | 16:44 |
noonedeadpunk | probably you're right, and there won't be a problem in most cases, as cinder have to have connection to monitors, but for example in my case it connects not via mgmt ip, but over storage ip | 16:46 |
logan- | there is no connection to cinder since the task is delegated. instead the connection is performed to the ceph monitors | 16:48 |
noonedeadpunk | logan-: hm, than my understanding of delegate_to is fundamentaly wrong. Went reading ansible docs... | 16:49 |
noonedeadpunk | sorry for confusing then | 16:49 |
noonedeadpunk | my appologizes | 16:49 |
*** cshen has quit IRC | 16:49 | |
logan- | no prob, if you have the opportunity to test the patch out it would be great to hear how it works for you. if you run into any problems some output with -vvvvv enabled on the playbook run should be helpful | 16:51 |
jrosser | noonedeadpunk: in your user_variables, you've specified you external ceph_mons? have you put the storage network ip there or whichever your "bare metal ssh" network is? | 16:51 |
noonedeadpunk | jrosser: yep, everything is in place. But also I've added 1 mon, which is unreachable from deployment host for testing. I'll try to run test one more time now, probably I've made smth wrong first time | 16:53 |
jrosser | what i am asking is which IP you have used - if you've defined ceph_mons as the storage net ip then it would explain why you see ansible trying to connect on the storage network | 16:54 |
logan- | well here is one annoyance | 16:56 |
logan- | http://paste.openstack.org/raw/731035/ | 16:56 |
logan- | wait_for_connection does not set the task state correctly when the connection fails :/ | 16:56 |
noonedeadpunk | jrosser: we have a bit tricky conf. Our management net is 172.20.0.9, storage net is placed on infiniband with net 10.157.208.2. deployment host doesn't have access to infiniband. And ceph mon is reached by our internal datacenter private net 10.10.10.145 | 17:00 |
jrosser | right - sd in your config what is ceph_mons? | 17:01 |
jrosser | *so | 17:01 |
noonedeadpunk | cinder/glance/gnocchi containers have direct access to 172.20.0.0/24 and 10.157.208.0/24. ceph_mons is 10.10.10.145 us user_variables | 17:01 |
jrosser | and you see ansible trying to connect to 10.157.208.<> things? i'm just trying to understand becasue I have a very similar situation, which works | 17:03 |
olivierb | Hello, any reason why some pip tasks are now failing in queens branch for some containers (note that I am behind a proxy in restricted network access mode) TASK [pip_install : Install PIP] [Errno 111] Connection refused | 17:04 |
noonedeadpunk | logan-: as for me wait_for_connection is missing host param, which would suit perfectly and present for wait_for... | 17:04 |
olivierb | going to one of this container and typing wget http://INTERNAL_VIP:8181/os-releases/17.1.2/centos-7.5-x86_64/ also fails with same error | 17:05 |
olivierb | AFAIK it was working fine middle of last week | 17:05 |
noonedeadpunk | jrosser: please give me some time to test it one more time from scratch. I'm on meeting now and can't verify all details.... | 17:05 |
jrosser | noonedeadpunk: sure no worries :) | 17:06 |
olivierb | one weird thing I noticed in that within the container ip route | grep 172.29.236 gives 172.29.236.0/22 dev eth1 proto kernel scope link src 172.29.237.176 | 17:08 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Ensure VM's are shut down before doing image save https://review.openstack.org/605814 | 17:09 |
olivierb | but on the hosts br-mgmt is on eth0.10 | 17:10 |
kaiokmo | evrardjp: could you please take a look at https://review.openstack.org/#/c/588584/? | 17:11 |
olivierb | but may be this irrelevant as this is the way containers interfaces are created on br-mgmt | 17:12 |
jrosser | olivierb: there is a diagram here which may help https://docs.openstack.org/project-deploy-guide/openstack-ansible/draft/app-networking.html | 17:13 |
*** fghaas has joined #openstack-ansible | 17:15 | |
*** gkadam has quit IRC | 17:20 | |
olivierb | jrosser thx for the info, digging a little bit deeper, I found that IP connectivity is OK (ping seems to work on all 172.29.236.X assigned addresses) however HTTP requests are refused | 17:20 |
*** dxiri has quit IRC | 17:20 | |
olivierb | on port 8181 which seems OK on the infra nodes | 17:21 |
*** Emine has quit IRC | 17:21 | |
*** dxiri has joined #openstack-ansible | 17:21 | |
noonedeadpunk | jrosser logan- sorry for misguide you, as in sandbox with the same config everything works correctly... So probably I have some customizations on main deployment host and I will figure this out | 17:23 |
noonedeadpunk | I hope it will be merged into master soon) | 17:24 |
jrosser | noonedeadpunk: no worries! Hope you find what’s up | 17:27 |
goldenfri | I am running OSA queens and I don't have an option to edit a flavor, is that a bug, an option I haven't set or am I doing it wrong? :) | 17:29 |
odyssey4me | goldenfri: a flavor is created/modified by using the openstack client - OSA doesn't do it for you | 17:31 |
goldenfri | yea I know, I'm talking about in horizon | 17:31 |
goldenfri | I can't only update metadata or delete flavors | 17:31 |
odyssey4me | goldenfri: I don't know if horizon has ever provided an option to create flavors. | 17:32 |
odyssey4me | Horizon is primarily a user tool, not an admin tool. | 17:32 |
*** kstev has joined #openstack-ansible | 17:32 | |
goldenfri | I can create them just fine in horizon, but in a mitaka stack I have I can edit the basic parameters, but that doesn't seem to be an option in queens | 17:32 |
kaiokmo | odyssey4me: it does | 17:32 |
odyssey4me | Ah, things have changed then. :p | 17:33 |
* odyssey4me is feeling old | 17:33 | |
odyssey4me | If horizon can do it, then horizon deployed by OSA can do it. It's might be a matter of changing a setting, or making sure a policy is right, or something. | 17:34 |
*** Emine has joined #openstack-ansible | 17:34 | |
noonedeadpunk | goldenfri: you're right, you can create image with horizon, but you basically can't modify it. I believe that in mitaka glance api v1 was used, and in queens it's glance v2. Also horizon changed since mitaka... | 17:35 |
*** cshen has joined #openstack-ansible | 17:35 | |
goldenfri | aww boo ok, that is annoying | 17:35 |
goldenfri | thanks | 17:36 |
noonedeadpunk | you may ask about this feature in #openstack-horizon folks | 17:36 |
kaiokmo | goldenfri: please note that you can create flavors in horizon, but not edit them | 17:38 |
kaiokmo | you can however update flavor's metadata | 17:38 |
noonedeadpunk | odyssey4me: for my surprise, guys from my ops department use only horizon for all management and by far they had no problems with missing options, and I'm really puzzled by this fact | 17:38 |
*** nsmeds_ has joined #openstack-ansible | 17:39 | |
odyssey4me | noonedeadpunk: haha, just wait until they discover the openstack CLI :p | 17:39 |
nsmeds_ | hey there - can someone confirm that openstack-ansible supports installing Ironic in addition to regular Nova services? the note at top of https://docs.openstack.org/openstack-ansible-os_ironic/latest/configure-ironic.html is worrisome | 17:40 |
mnaser | nsmeds_: thanks to prometheanfire he did some really good work to overhaul | 17:41 |
odyssey4me | nsmeds_: oh man, that doc entry is really old and should be removed - it was added in mitaka, and ironic's been suppored and used decently since newton | 17:41 |
mnaser | but we dont have extensive ironic CI testing | 17:41 |
mnaser | nsmeds_: ever pushed a patch to gerrit before? :p | 17:41 |
nsmeds_ | thanks guys - appreciate it | 17:41 |
odyssey4me | yeah, unfortunately trying to test a thing that deploys are metal nodes in a cloud is hard | 17:41 |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible master: Add openstackclient bash completion https://review.openstack.org/605795 | 17:41 |
odyssey4me | *bare metal | 17:43 |
*** kstev has quit IRC | 17:43 | |
nsmeds_ | makes sense | 17:43 |
noonedeadpunk | I think most problems with such testing is to emulate ipmi.... | 17:43 |
nsmeds_ | :thumbsup: | 17:43 |
mnaser | there's some stuff around it | 17:44 |
mnaser | maybe we can learn some lessons from ironic | 17:44 |
*** jamesdenton has quit IRC | 17:44 | |
*** kstev has joined #openstack-ansible | 17:44 | |
odyssey4me | ok folks, I'm out for the night - cores, I'd appreciate it if you could babysit https://review.openstack.org/#/q/I0d7e4804bdc92ffe7a679060a686e684c01fcd1b until they're all merged, those are release blockers | 17:46 |
prometheanfire | nsmeds_: ya, docs need updating :| | 17:51 |
*** kstev has quit IRC | 17:52 | |
mnaser | odyssey4me: will try to keep an eye, thank you and have a good night | 17:57 |
mnaser | i will not be able to run next weeks meeting as i will be at openstack days nordic | 17:59 |
mnaser | would someone mind running next week meeting please? | 17:59 |
*** cshen has quit IRC | 18:05 | |
prometheanfire | if someone has a minute, https://review.openstack.org/422908 (os_ironic) | 18:10 |
prometheanfire | https://docs.openstack.org/ironic/latest/configuration/sample-config.html is refrence for the config update | 18:10 |
*** dxiri has quit IRC | 18:11 | |
*** olivierb has quit IRC | 18:15 | |
*** olivierb has joined #openstack-ansible | 18:16 | |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic master: remove oneview support from ironic https://review.openstack.org/605829 | 18:24 |
*** jamesdenton has joined #openstack-ansible | 18:31 | |
*** skiedude has quit IRC | 18:35 | |
*** olivierb has quit IRC | 18:43 | |
*** olivierb has joined #openstack-ansible | 18:43 | |
*** hamzaachi has joined #openstack-ansible | 18:47 | |
openstackgerrit | Andy Smith proposed openstack/openstack-ansible master: Add qdrouterd role for oslo.messaging backend deployment https://review.openstack.org/570261 | 18:53 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Refactor Filebeat configuration file https://review.openstack.org/605837 | 18:55 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Add Instance ID to logs https://review.openstack.org/605838 | 18:57 |
kaiokmo | mnaser: openstack-ansible-role-jobs refer to the tasks listed here, right? https://github.com/openstack/openstack-ansible-tests/blob/master/zuul.d/project-templates.yaml#L3-L22 | 18:59 |
*** DanyC has joined #openstack-ansible | 18:59 | |
kaiokmo | if a role doesn't support a distro, it will fail for the respective check/job. | 19:00 |
mnaser | thats correct kaiokmo | 19:00 |
mnaser | however that patch introduces/fixed support for that | 19:01 |
mnaser | so we can add it in the same patch :) | 19:01 |
mnaser | cloudnull: thanks for putting down the foundation for super cool basis for elk | 19:01 |
mnaser | its been great and i'm happy to be pushing stuff back up | 19:01 |
kaiokmo | in case of centos-7, do I need to override the template behavior in the role zuul.d/project.yaml and set it to non-voting? | 19:03 |
kaiokmo | because I have another patch for centos-7 support | 19:03 |
*** idlemind has joined #openstack-ansible | 19:03 | |
kaiokmo | and the opensuse patch will obviously fail on functional-centos-7 | 19:04 |
cloudnull | mnaser thanks for using it! Its a team effort :) | 19:10 |
mnaser | cloudnull: we have now every deployment shipping all data to one centralized cluster | 19:11 |
mnaser | with the two patches above, i'll be able to add something to append an extra set of tags so we can send tags to filter by environment | 19:11 |
mnaser | because now we can programatically just merge it to the tags | 19:11 |
cloudnull | excellent ! | 19:12 |
*** dxiri has joined #openstack-ansible | 19:14 | |
*** dcdamien has quit IRC | 19:18 | |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cinder master: Fix Cinder backup to use full paths https://review.openstack.org/605846 | 19:22 |
*** cshen has joined #openstack-ansible | 19:22 | |
guilhermesp | it's been very helpful this elk cluster 🤘🏼 | 19:23 |
*** dcdamien has joined #openstack-ansible | 19:28 | |
*** fghaas has left #openstack-ansible | 19:30 | |
*** hamzaachi has quit IRC | 19:31 | |
*** cshen has quit IRC | 19:34 | |
*** hamzy has joined #openstack-ansible | 19:38 | |
*** hamzaachi has joined #openstack-ansible | 19:40 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Retrieve ilo drivers via webserver https://review.openstack.org/422908 | 19:43 |
*** hamzy has quit IRC | 19:44 | |
nsmeds_ | here's a question for y'all related to nspawn container creation, ansible, and ssh (i'm currently trying to dig through ansible code to solve this, however I'll throw it here as well) | 19:44 |
nsmeds_ | how does Ansible connect to each container to configure it, if my SSH key and user isn't created in the container? | 19:45 |
nsmeds_ | all hints appreciated ^^ | 19:45 |
nsmeds_ | I've found the cache-prep-script (ansible chroot's into the cache container image and runs bash script) - however no user is created there and SSHD service is disabled | 19:46 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Fix blank microversion https://review.openstack.org/605855 | 19:48 |
nsmeds_ | ah - `root` user configured with `/root/.ssh/authorized_keys` | 19:50 |
*** DanyC has quit IRC | 19:52 | |
*** dxiri has quit IRC | 19:53 | |
*** dxiri has joined #openstack-ansible | 19:53 | |
*** pcaruana has quit IRC | 19:54 | |
*** kstev has joined #openstack-ansible | 19:56 | |
*** kstev has quit IRC | 19:56 | |
*** nsmeds_ has quit IRC | 19:58 | |
*** cshen has joined #openstack-ansible | 20:05 | |
openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Fixes too low security group rules quota https://review.openstack.org/600826 | 20:09 |
openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Break out config files for each octavia service https://review.openstack.org/603228 | 20:09 |
*** hamzy has joined #openstack-ansible | 20:09 | |
*** nsmeds_ has joined #openstack-ansible | 20:13 | |
mnaser | nsmeds_: we have a custom connection script | 20:13 |
mnaser | that actually runs commands in containers | 20:13 |
*** hamzy has quit IRC | 20:20 | |
*** hamzy has joined #openstack-ansible | 20:20 | |
*** skiedude has joined #openstack-ansible | 20:24 | |
*** hamzy has quit IRC | 20:25 | |
skiedude | I've run into an issue following https://docs.openstack.org/openstack-ansible/queens/admin/scale-environment.html#add-a-compute-host -- when it tries to install hte qemu packages it fails because the highest version of seavgabios-bin isn't available. And upon closer inspection there are quite a few yum repos missing on this new compute host | 20:26 |
skiedude | I've been poking through the playbooks it has you run, but I'm not sure where it puts down the yum repos, and why it didn't put down all the CentOS-* | 20:26 |
*** Emine has quit IRC | 20:27 | |
*** Emine has joined #openstack-ansible | 20:28 | |
*** hamzy has joined #openstack-ansible | 20:28 | |
skiedude | looks like its specifically Centos-Base.repo that is missing | 20:29 |
*** cshen has quit IRC | 20:29 | |
*** macza has quit IRC | 20:31 | |
*** macza has joined #openstack-ansible | 20:32 | |
mnaser | cloudnull: jrosser are your auditd dashboards working? for some reason it just ships a message here | 20:32 |
skiedude | sigh disregard, its teh bloody kickstart for the server that dorked it | 20:32 |
*** hamzy has quit IRC | 20:34 | |
*** hamzy has joined #openstack-ansible | 20:34 | |
openstackgerrit | Arx Cruz proposed openstack/openstack-ansible-tests master: Add stackviz directory to the list of find_files https://review.openstack.org/604292 | 20:39 |
jrosser | mnaser: btw i had tons more luck with those dashboards in firefox - chrome gets very unhappy with some of them | 20:40 |
*** hamzy has quit IRC | 20:43 | |
*** ansmith has quit IRC | 20:48 | |
jrosser | mnaser: i think you are right.... auditbeat dashboard working, filebeat auditd dashboard not working | 20:55 |
mnaser | yeah so | 20:55 |
mnaser | apparently | 20:55 |
mnaser | when you ship stuff to logstash | 20:55 |
mnaser | it doesnt actaully parse it | 20:55 |
mnaser | module uses the ingest stuff so it would only work if you shipped straight to elasticsearch | 20:55 |
mnaser | im writing a small filter.. | 20:55 |
guilhermesp | jrosser: you mean the opentack custom dashboards? | 20:56 |
openstackgerrit | Arx Cruz proposed openstack/openstack-ansible-tests master: Add stackviz directory to the list of find_files https://review.openstack.org/604292 | 20:59 |
jrosser | guilhermesp: i have trouble with the openstack log dashboard now, after an "incident" with my index patterns meant i had to delete / recreate them all | 20:59 |
jrosser | guilhermesp: look at this http://paste.openstack.org/show/730730/ see how my recreated "*" index pattern is formed differently now i've deleted/recreated it with kibana 6.4.1 UI | 21:01 |
*** DanyC has joined #openstack-ansible | 21:02 | |
*** hamzaachi has quit IRC | 21:13 | |
*** spatel has quit IRC | 21:13 | |
*** jamesdenton has quit IRC | 21:15 | |
*** spatel has joined #openstack-ansible | 21:16 | |
guilhermesp | slightly different jrosser | 21:19 |
jrosser | yes and unfortunatley for the openstack log dashboard it means that each visualisation says "Could not locate that index-pattern (id: *), [click here to re-create it](#/management/kibana/index)" | 21:20 |
guilhermesp | that's odd, the purpose of * is to aggregate all beats index-pattern | 21:21 |
guilhermesp | lemme paste for one of my visualizations | 21:21 |
guilhermesp | in my 6.4.0 kibana | 21:22 |
jrosser | the "*" is still there, it's just in the title field rather than the id field of the index pattern | 21:22 |
jrosser | but thats really poor as theres now a difference in structure between the index patterns created automatically by the different beats at the cli, and those you might create through the UI | 21:23 |
*** spatel has quit IRC | 21:26 | |
guilhermesp | http://paste.openstack.org/show/731048/ so then I was debugging the "not found" indexes I just changed the index from "index": "filebeat-*", to "index": "*" | 21:26 |
*** spatel has joined #openstack-ansible | 21:27 | |
guilhermesp | so none of these visualizations working for you righ? https://imgur.com/a/tyO67Kn | 21:29 |
*** nsmeds_ has quit IRC | 21:30 | |
jrosser | they used to - then i had to delete/recreate my index patterns | 21:31 |
jrosser | and now they don't work. i think if i recreate the "*" index pattern using the API rather than the UI I will be able to get it back again | 21:32 |
*** spatel has quit IRC | 21:35 | |
guilhermesp | Cool, let us know if works jrosser | 21:36 |
*** ansmith has joined #openstack-ansible | 21:40 | |
*** DanyC has quit IRC | 21:50 | |
*** vnogin has joined #openstack-ansible | 21:54 | |
*** skiedude has quit IRC | 21:59 | |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cinder master: Fix Cinder backup to use full class path https://review.openstack.org/605846 | 22:01 |
*** Leo_m has quit IRC | 22:01 | |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Fix auditd logs using Filebeat https://review.openstack.org/605869 | 22:04 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cinder master: Fix Cinder backup to use full class path https://review.openstack.org/605846 | 22:05 |
mnaser | jrosser: cloudnull ^ the fixup | 22:06 |
*** spatel has joined #openstack-ansible | 22:07 | |
openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/ocata: Enable the xinetd service for reboot https://review.openstack.org/605779 | 22:07 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/queens: Enable the xinetd service for reboot https://review.openstack.org/605777 | 22:14 |
*** cshen has joined #openstack-ansible | 22:25 | |
spatel | cloudnull: ^^ | 22:25 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/rocky: Enable the xinetd service for reboot https://review.openstack.org/605776 | 22:25 |
spatel | there? | 22:25 |
*** spatel has quit IRC | 22:28 | |
*** vnogin has quit IRC | 22:29 | |
*** cshen has quit IRC | 22:32 | |
*** KeithMnemonic has quit IRC | 22:33 | |
*** KeithMnemonic has joined #openstack-ansible | 22:33 | |
*** macza has quit IRC | 22:38 | |
*** macza has joined #openstack-ansible | 22:38 | |
*** macza has quit IRC | 22:51 | |
*** macza_ has joined #openstack-ansible | 22:51 | |
*** tosky has quit IRC | 22:53 | |
*** olivierb has quit IRC | 23:08 | |
*** olivierb has joined #openstack-ansible | 23:11 | |
*** spatel has joined #openstack-ansible | 23:17 | |
*** macza_ has quit IRC | 23:18 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Ensure VM's are shut down before doing image save https://review.openstack.org/605814 | 23:19 |
*** spatel has quit IRC | 23:21 | |
*** olivierb has quit IRC | 23:29 | |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/rocky: Clean up role tests https://review.openstack.org/603857 | 23:38 |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client stable/queens: Fix apt pinning for ceph https://review.openstack.org/605670 | 23:51 |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client stable/pike: Fix apt pinning for ceph https://review.openstack.org/605671 | 23:51 |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client stable/ocata: Fix apt pinning for ceph https://review.openstack.org/605677 | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!