Thursday, 2019-04-11

*** bobh has joined #openstack-mistral01:22
*** bobh has quit IRC02:31
*** bobh has joined #openstack-mistral02:32
*** bobh has quit IRC02:36
*** bobh has joined #openstack-mistral04:30
*** bobh has quit IRC04:35
rakhmerovmgariepy: hi, I guess so04:38
rakhmerovlooking at your patch...04:39
rakhmerovooh, seems like the patch is not ready for review yet04:40
openstackgerritMerged openstack/mistral stable/rocky: Add release note for I04ba85488b27cb05c3b81ad8c973c3cc3fe56d36  https://review.openstack.org/65148805:04
*** bobh has joined #openstack-mistral05:35
*** bobh has quit IRC05:39
*** jtomasek has quit IRC05:48
*** pgaxatte has joined #openstack-mistral06:25
*** bobh has joined #openstack-mistral07:14
*** bobh has quit IRC07:18
openstackgerritMerged openstack/mistral master: Fix an expression context for all_errors_handled()  https://review.openstack.org/65119508:04
*** jtomasek has joined #openstack-mistral08:06
openstackgerritRenat Akhmerov proposed openstack/mistral stable/stein: Fix an expression context for all_errors_handled()  https://review.openstack.org/65171708:15
openstackgerritjacky06 proposed openstack/mistral master: Drop py35 jobs  https://review.openstack.org/65142608:31
*** bobh has joined #openstack-mistral09:56
*** bobh has quit IRC10:00
vgvolegHi all, please let me know if you agree with this https://blueprints.launchpad.net/mistral/+spec/add-sleep-option-to-echo-action12:01
vgvolegi'm ready to push this changes if you don't mind :)12:02
mgariepyrakhmerov, hello, yep i got some issue with the tests ..12:05
openstackgerritMarc Gariépy (mgariepy) proposed openstack/mistral master: Add reply-to to std.email  https://review.openstack.org/65158212:25
openstackgerritMarc Gariépy (mgariepy) proposed openstack/mistral master: Add reply-to to std.email  https://review.openstack.org/65158212:37
*** d0ugal has quit IRC13:46
*** d0ugal has joined #openstack-mistral14:01
*** gkadam has joined #openstack-mistral14:02
*** gkadam has quit IRC14:03
*** bobh has joined #openstack-mistral14:44
*** bobh has quit IRC14:49
*** bobh has joined #openstack-mistral14:50
mgariepyrakhmerov, it's ready for review now, feel free to update it if you need to I really don't mind.14:51
*** d0ugal has quit IRC15:35
*** pgaxatte has quit IRC15:58
vgvolegGuys, I've used flake8 for mistral and found 3 bad places16:10
vgvolegmistral/tests/unit/engine/test_commands.py:363:1: E305 expected 2 blank lines after class or function definition, found 116:10
*** smrcascao has joined #openstack-mistral16:10
vgvolegmistral/tests/unit/engine/test_commands.py:452:1: E305 expected 2 blank lines after class or function definition, found 116:10
vgvolegmistral/db/sqlalchemy/migration/cli.py:141:1: E305 expected 2 blank lines after class or function definition, found 116:11
vgvolegWhat should I do to fix this in upstream? Should I create a bug? It breaks my CI :D16:12
mgariepyor submit a patch to fix it ?16:14
vgvolegoh, it's a misunderstanding16:24
vgvolegWhen I asked about a bug, I mean could this be called a bug?16:25
vgvolegof course I can submit a patch, I just want to know the "right bureaucratic" way16:26
mgariepyhehe ok ;)16:29
mgariepyno idea then, i only submitted 2 patch for the project.16:29
*** jtomasek has quit IRC17:37
*** bobh has quit IRC17:58
*** bobh has joined #openstack-mistral18:03
*** bobh has quit IRC18:13
*** bobh has joined #openstack-mistral18:16
*** bobh has quit IRC22:10
*** aspiers has quit IRC23:21
*** aspiers has joined #openstack-mistral23:27

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