Monday, 2017-01-16

« Sunday, 2017-01-15 Index Tuesday, 2017-01-17 »
*** salv-orl_ has quit IRC00:27
*** dkehn has quit IRC00:31
*** dkehn has joined #openstack-oslo00:31
*** ansmith_ has joined #openstack-oslo00:46
*** dimtruck is now known as zz_dimtruck00:50
*** zz_dimtruck is now known as dimtruck00:50
*** ansmith_ has quit IRC00:52
*** dimtruck is now known as zz_dimtruck01:00
*** tovin07 has joined #openstack-oslo01:27
*** salv-orlando has joined #openstack-oslo01:28
*** salv-orlando has quit IRC01:35
*** amotoki has joined #openstack-oslo01:42
*** gcb has joined #openstack-oslo01:54
*** nkrinner_afk has quit IRC02:12
*** nkrinner_afk has joined #openstack-oslo02:12
*** stevemar has quit IRC02:21
*** zz_dimtruck is now known as dimtruck02:21
*** stevemar has joined #openstack-oslo02:21
*** links has joined #openstack-oslo02:25
*** salv-orlando has joined #openstack-oslo02:31
*** salv-orlando has quit IRC02:36
*** links has quit IRC02:38
*** links has joined #openstack-oslo02:46
*** mriedem has quit IRC03:15
*** salv-orlando has joined #openstack-oslo03:32
*** salv-orlando has quit IRC03:36
*** ihrachys has joined #openstack-oslo03:50
*** ihrachys has quit IRC03:53
*** ihrachys has joined #openstack-oslo03:54
*** ihrachys has quit IRC03:55
openstackgerritCao Xuan Hoang proposed openstack/mox3: Remove vim header from source files  https://review.openstack.org/42049204:07
openstackgerritMerged openstack/oslo.reports: Updated from global requirements  https://review.openstack.org/41610604:16
*** salv-orlando has joined #openstack-oslo04:33
*** salv-orlando has quit IRC04:38
openstackgerritMerged openstack/oslo.context: Updated from global requirements  https://review.openstack.org/41957304:42
*** dimtruck is now known as zz_dimtruck04:49
*** amotoki_ has joined #openstack-oslo05:09
*** amotoki has quit IRC05:12
*** salv-orlando has joined #openstack-oslo05:34
*** salv-orlando has quit IRC05:38
*** Dinesh_Bhor has joined #openstack-oslo06:06
*** salv-orlando has joined #openstack-oslo06:35
*** salv-orlando has quit IRC06:39
*** lpetrut has joined #openstack-oslo06:42
*** greghaynes has quit IRC06:52
*** mnaser has quit IRC06:53
*** mordred has quit IRC06:53
*** greghaynes has joined #openstack-oslo06:58
*** e0ne has joined #openstack-oslo07:00
*** mordred has joined #openstack-oslo07:02
*** greghaynes has quit IRC07:04
*** yamamoto has quit IRC07:04
*** mnaser has joined #openstack-oslo07:10
*** e0ne has quit IRC07:14
*** greghaynes has joined #openstack-oslo07:15
*** tesseract has joined #openstack-oslo07:21
*** hogepodge_ has joined #openstack-oslo07:24
*** nkrinner_afk is now known as nkrinner07:24
*** e0ne has joined #openstack-oslo07:33
*** pcaruana has joined #openstack-oslo07:34
*** salv-orlando has joined #openstack-oslo07:36
*** hogepodge_ has quit IRC07:37
*** salv-orlando has quit IRC07:40
*** rasca has joined #openstack-oslo07:44
*** gcb has quit IRC07:44
*** gcb has joined #openstack-oslo07:46
openstackgerritMerged openstack/oslo.vmware: Fix setting the SOAP headers for remote calls.  https://review.openstack.org/41892107:54
*** eezhova has joined #openstack-oslo07:59
*** amotoki has joined #openstack-oslo08:07
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Use more stable configuration in voting job  https://review.openstack.org/42055608:11
*** amotoki_ has quit IRC08:11
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Use more stable configuration in voting job  https://review.openstack.org/42055608:13
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Fix py35 gate  https://review.openstack.org/42003408:14
*** e0ne has quit IRC08:14
*** shardy has joined #openstack-oslo08:24
*** salv-orlando has joined #openstack-oslo08:37
*** salv-orlando has quit IRC08:41
*** eezhova has quit IRC08:47
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-oslo09:00
*** e0ne has joined #openstack-oslo09:08
*** eezhova has joined #openstack-oslo09:12
*** e0ne has quit IRC09:25
*** e0ne has joined #openstack-oslo09:26
*** yassine has joined #openstack-oslo09:32
*** yassine is now known as Guest701109:32
*** salv-orlando has joined #openstack-oslo09:37
*** salv-orlando has quit IRC09:42
*** lucas-afk is now known as lucasagomes09:47
*** sambetts|afk is now known as sambetts09:53
*** e0ne has quit IRC09:55
*** e0ne has joined #openstack-oslo09:57
*** amotoki has quit IRC10:01
*** amotoki has joined #openstack-oslo10:08
*** dtardivel has joined #openstack-oslo10:10
*** amotoki has quit IRC10:24
*** Guest7011 has quit IRC10:37
*** salv-orlando has joined #openstack-oslo10:38
*** salv-orlando has quit IRC10:43
*** Guest7011 has joined #openstack-oslo10:45
*** cdent has joined #openstack-oslo10:55
*** lpetrut has quit IRC11:03
*** lpetrut has joined #openstack-oslo11:26
*** lpetrut has quit IRC11:30
*** amotoki has joined #openstack-oslo11:37
*** salv-orlando has joined #openstack-oslo11:39
*** salv-orlando has quit IRC11:43
*** lpetrut has joined #openstack-oslo11:59
*** rtheis has joined #openstack-oslo12:07
*** pblaho has joined #openstack-oslo12:10
*** mhickey has joined #openstack-oslo12:17
*** sdague has joined #openstack-oslo12:22
*** nicolasbock has joined #openstack-oslo12:34
*** khushbu_ has joined #openstack-oslo12:37
*** salv-orlando has joined #openstack-oslo12:40
*** salv-orlando has quit IRC12:44
*** lucasagomes is now known as lucas-hungry12:45
*** salv-orlando has joined #openstack-oslo12:46
*** gcb has quit IRC13:01
*** Guest7011 has quit IRC13:13
*** trown|outtypewww is now known as trown13:14
*** khushbu_ has quit IRC13:16
*** gordc has joined #openstack-oslo13:17
*** Guest7011 has joined #openstack-oslo13:20
*** ozamiatin has joined #openstack-oslo13:22
*** ansmith has joined #openstack-oslo13:36
*** jdandrea has joined #openstack-oslo13:39
*** salv-orlando has quit IRC13:46
*** ansmith_ has joined #openstack-oslo13:49
thervedims, We can chat here a bit if you want?13:51
dimsrpodolyaka : dhellmann : seen this before? http://paste.openstack.org/show/595050/13:51
dimstherve :) ++13:51
therveThe issue seems to be that Message.translate is not compatible with str.translate13:51
thervestr.translate takes an escape table13:51
therveMessage.translate takes a locale13:51
therve(I always disliked that Message is-a string, but whatever)13:53
rpodolyakadims: nope :(13:54
*** ihrachys has joined #openstack-oslo13:54
*** lucas-hungry is now known as lucasagomes13:55
*** kgiusti has joined #openstack-oslo14:01
thervedims, https://bugs.launchpad.net/oslo.i18n/+bug/165683114:07
openstackLaunchpad bug 1656831 in oslo.i18n "Message.translate is not compatible with str.translate" [Undecided,New]14:07
dimstherve : wondering if this would be enough http://paste.openstack.org/show/595053/14:08
thervedims, that's a bandaid :/14:08
dimsyep14:08
therveActually no that's just wrong :)14:09
thervedims, We don't get passed a list of locales, but a table of chars14:09
therve(kids at home: that's why you don't inherit from basic types)14:11
dimsk. we should probably wait for dhellmann too :)14:11
*** jeckersb_gone is now known as jeckersb14:17
*** pradk has joined #openstack-oslo14:18
openstackgerritThomas Herve proposed openstack/oslo.i18n: Rename Message.translate method  https://review.openstack.org/42073614:18
therveThat's one solution to look at, at least14:19
*** mkoderer has joined #openstack-oslo14:20
dimsthanks therve14:20
*** Guest7011 has quit IRC14:28
*** links has quit IRC14:30
*** jeckersb is now known as jeckersb_gone14:35
*** mriedem has joined #openstack-oslo14:38
*** jeckersb_gone is now known as jeckersb14:41
*** Guest7011 has joined #openstack-oslo14:42
dhellmanndims, rpodolyaka, therve : why is something passing a Message to SQLAlchemy?14:43
thervedhellmann, We store error messages I believe14:43
dhellmanntherve : ?!14:44
dhellmannwell, whatever is doing that should call .translate() on the message first14:44
dhellmannor not, if we want to store the untranslated error, which may be slightly more correct14:44
dhellmannand fwiw, I don't like that Message subclasses str either, but I was outnumbered at the time14:45
therve:)14:45
*** salv-orlando has joined #openstack-oslo14:47
*** gcb has joined #openstack-oslo14:50
*** salv-orlando has quit IRC14:51
*** e0ne has quit IRC15:00
*** mtanino has joined #openstack-oslo15:01
thervedhellmann, So what's your idea in the end?15:04
*** e0ne has joined #openstack-oslo15:14
*** zz_dimtruck is now known as dimtruck15:14
* dims waits eagerly for the answer as well :)15:25
*** beekneemech is now known as bnemec15:26
*** cdent_ has joined #openstack-oslo15:31
*** cdent has quit IRC15:31
*** cdent_ is now known as cdent15:31
*** nkrinner is now known as nkrinner_afk15:35
*** cdent has quit IRC15:36
*** cdent_ has joined #openstack-oslo15:37
*** jdandrea has quit IRC15:39
*** cdent_ has quit IRC15:42
*** jdandrea has joined #openstack-oslo15:45
*** salv-orlando has joined #openstack-oslo15:48
*** cdent has joined #openstack-oslo15:50
*** rtheis has quit IRC15:51
*** lpetrut has quit IRC15:52
*** salv-orlando has quit IRC15:52
*** ozamiatin_ has joined #openstack-oslo15:55
*** ozamiatin has quit IRC15:55
*** amotoki has quit IRC15:57
gcbseems harlowja is not here15:59
*** cdent has quit IRC16:00
gcbI would like to help chair the weekly meeting today :)16:00
*** mhickey has quit IRC16:00
*** cdent has joined #openstack-oslo16:08
*** jdandrea has quit IRC16:23
*** salv-orlando has joined #openstack-oslo16:30
*** jdandrea has joined #openstack-oslo16:36
*** eezhova has quit IRC16:37
gcbkgiusti,  nice guys have approved https://review.openstack.org/#/c/417629/16:39
kgiustigcb: yes!  Now go get some sleep. :)16:39
gcbkgiusti, yeah, have a nice day16:40
kgiustigcb: ttyl16:40
*** gcb has quit IRC16:42
*** e0ne has quit IRC16:45
*** amotoki has joined #openstack-oslo16:57
*** eezhova has joined #openstack-oslo17:12
dimsdhellmann : ok with this? https://review.openstack.org/#/c/420736/17:14
*** jdandrea has quit IRC17:28
*** tesseract has quit IRC17:29
*** johnsom has quit IRC17:30
*** johnsom has joined #openstack-oslo17:30
*** lucasagomes is now known as lucas-afk17:43
*** rasca has quit IRC17:44
openstackgerritMerged openstack/oslo.messaging: [AMQP 1.0] Fix SSL client authentication  https://review.openstack.org/41762917:45
*** trown is now known as trown|lunch17:49
*** e0ne has joined #openstack-oslo17:58
*** spzala has joined #openstack-oslo17:59
*** sambetts is now known as sambetts|afk18:00
*** jdandrea has joined #openstack-oslo18:02
*** e0ne has quit IRC18:19
openstackgerritgordon chung proposed openstack/oslo.messaging: support kombu4  https://review.openstack.org/40929418:22
*** eezhova has quit IRC18:23
*** shardy has quit IRC18:24
dhellmanndims : oh, sob. an API change during the freeze week?18:25
dims_message.py and _translate.py are private right?18:26
dimsi am yet to test the change. so don't know yet if we need to rush it18:27
dhellmanndims : I left a comment on the bug https://bugs.launchpad.net/oslo.i18n/+bug/165683118:28
openstackLaunchpad bug 1656831 in oslo.i18n "Message.translate is not compatible with str.translate" [Undecided,New]18:28
dhellmanndims : I'd hate to break a bunch of stuff if we can make an application call translate() on the string before passing it to sqlalchemy18:29
dimstrue. that one spot may not be enough though18:32
dhellmanndims: well, that's true, but I can't seem to get you or therve to actually point to the application code that's breaking :-)18:33
dimsoops. sorry one sec18:33
dimssorry we were talking on #heat before coming here, so all the back chatter was there - http://logs.openstack.org/43/420443/1/check/gate-heat-dsvm-functional-orig-mysql-lbaasv2-py35-ubuntu-xenial-nv/d858544/logs/screen-h-eng.txt.gz18:34
*** salv-orlando has quit IRC18:35
*** salv-orlando has joined #openstack-oslo18:36
dhellmanndims : is "test_resource.py" test code?18:36
dimsno, it's the thing that checks for status completion etc.18:39
dhellmannok18:39
dhellmannwhat messages are being given to it?18:39
dhellmannwhere is it getting translatable strings?18:39
dhellmanndims,therve : I'm looking at this as an application data-type issue, for now. heat has a thing it thinks is a string, but that isn't actually. It happens to be a Message, but it could maybe have been a dict or int or something. Where did it come from? Why does it assume a string?18:42
dhellmannI do agree that the translate() method is poorly named, but I don't want to just treat the symptom in oslo.i18n, let's fix the bug that uncovered that issue, too.18:43
dhellmanndims, therve : I have to step out for a bit, brb.18:44
dimsack dhellmann thanks18:44
*** lpetrut has joined #openstack-oslo18:44
*** e0ne has joined #openstack-oslo18:47
*** eezhova has joined #openstack-oslo18:54
*** pblaho has quit IRC19:07
*** trown|lunch is now known as trown19:13
openstackgerritMerged openstack/oslo.messaging: [zmq] Fix py35 gate  https://review.openstack.org/42003419:53
*** jerrygb has joined #openstack-oslo19:57
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Dynamic connections failover  https://review.openstack.org/41946620:09
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Local proxy deployment for minimum connections number  https://review.openstack.org/34655320:11
*** ozamiatin_ has quit IRC20:13
*** spzala has quit IRC20:14
*** dtardivel has quit IRC20:14
*** salv-orlando has quit IRC20:23
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.reports: Updated from global requirements  https://review.openstack.org/42092520:23
*** lpetrut has quit IRC20:27
*** Syed__ has joined #openstack-oslo20:29
*** rtheis has joined #openstack-oslo20:34
openstackgerritJohn Perkins proposed openstack/oslo.config: oslo validator  https://review.openstack.org/38455920:35
*** mhickey has joined #openstack-oslo20:44
*** Guest7011 has quit IRC20:46
*** rtheis has quit IRC20:48
*** spzala has joined #openstack-oslo20:49
*** dkehn has quit IRC20:58
*** dkehn has joined #openstack-oslo20:58
*** jamielennox|away is now known as jamielennox20:59
kevinbentonzzzeek: yo, running into issue with pagination helper21:01
zzzeekkevinbenton: what, the oslo.db one?21:02
kevinbentonzzzeek: yeah21:02
kevinbentonzzzeek: sort_key has a bool column and pass in marker21:02
kevinbentonzzzeek:     ArgumentError: Only '=', '!=', 'is_()', 'isnot()' operators can be used with None/True/False21:02
zzzeekha21:02
zzzeeki think this was a patch i warned them about21:02
zzzeekhmm or perhaps not.21:03
zzzeekok21:03
zzzeekwell21:03
zzzeekboolean is TRUE / FALSE on some databases.  like postgresql.21:03
zzzeekI'd CAST to an integer here21:03
kevinbentonzzzeek: you mean in the helper or should we do it in neutron before we call?21:03
zzzeekkevinbenton: if you can do it in neutron before the call right now, that would be best21:03
zzzeekkevinbenton: ive not worked w/ the pagination thing.    looked a little complicated21:04
kevinbentonzzzeek: oh, i don't know how i would go about making that conversion in neutron21:04
kevinbentonzzzeek: i pass in a string corresponding to a column21:04
zzzeekkevinbenton: well, oh.   welp21:04
kevinbentonzzzeek: i don't think i can do anything to help21:04
zzzeekyeah see i dont write helpers like that :)  strings == bah21:04
zzzeekkevinbenton: oh.  here.   add a @hybrid_property to your model.  it returns cast(cls.my_bool_col, Integer)21:06
*** salv-orlando has joined #openstack-oslo21:09
kevinbentonzzzeek: that can work, but i would need to file patches for vendor stuff in other repos21:09
zzzeekkevinbenton: why. did something change ?21:09
kevinbentonzzzeek: no, this is just a bug that has been lurking21:09
kevinbentonzzzeek: and i came across it21:09
zzzeekkevinbenton: IMO the way paginate works here is entirely wrong and should be leaving it up to the model what it is sorting on at a coarse-grained level21:10
*** dimtruck is now known as zz_dimtruck21:10
zzzeekkevinbenton: becasue when you have a functino take "list of strings that are attributes", you take all the control away from the model21:10
zzzeekit basically says, "turn off all SQLAlchemy functionality, convert into a string format, then unconvert back into a SQLAlchemy format"21:10
zzzeekkevinbenton: ideally each model that uses this should have a single attribute: MyModel.sort_key that is a hybrid or whatever that forms whatever SQL expression is needed for sorting21:11
zzzeekkevinbenton: or paginate accepts a full blown SQL expression21:11
zzzeeknot a list of string attribute names21:11
kevinbentonzzzeek: right. well we do allow users to provide which column they want results to be sorted on21:12
kevinbentonzzzeek: so it needs to be dynamic in that regard21:12
zzzeekkevinbenton: well, postgresql is the only DB that really has a real true/false and it allows them to be compared via > / <.  so if you'd like, file upstream sqlalchemy issue and maybe that can come out in 1.1.621:14
zzzeekkevinbenton: for oslo.db, it would need to inspect the type of each attrbiute, and if boolean then cast() to integer21:16
zzzeekkevinbenton: so, ive given you the solution at every level :).   pick which one(s) you want21:16
kevinbentonzzzeek: the oslo db one :)21:16
kevinbentonzzzeek: i'll work on it21:16
zzzeekkevinbenton: id rather fix SQLA upstream b.c. i dont really know why it throws that error.  it should just let it go through and let the DB complain if need be21:17
kevinbentonzzzeek: well this is something i would like to back-port if possible21:17
kevinbentonzzzeek: so we could do both, no?21:18
zzzeekkevinbenton: sure21:18
*** cdent has quit IRC21:22
*** jdandrea has quit IRC21:27
*** eezhova has quit IRC21:29
*** kgiusti has quit IRC21:31
*** jdandrea has joined #openstack-oslo21:31
*** zz_dimtruck is now known as dimtruck21:32
*** ansmith_ has quit IRC21:42
*** ansmith has quit IRC21:42
*** eezhova has joined #openstack-oslo21:46
*** salv-orlando has quit IRC21:47
*** eezhova has quit IRC21:56
*** trown is now known as trown|outtypewww21:56
*** mhickey has quit IRC21:59
*** jdandrea has quit IRC22:08
*** ihrachys has quit IRC22:10
openstackgerritKevin Benton proposed openstack/oslo.db: Coerce booleans to integer values in paginate_query  https://review.openstack.org/42097422:11
kevinbentonzzzeek: https://review.openstack.org/42097422:11
*** salv-orlando has joined #openstack-oslo22:12
kevinbentonzzzeek: that function smells though the more i look at it. forcing > or < only really makes sense for unique values...22:13
*** spzala has quit IRC22:13
openstackgerritJohn Perkins proposed openstack/oslo.config: oslo validator  https://review.openstack.org/38455922:14
*** yamamoto has joined #openstack-oslo22:15
*** dimtruck is now known as zz_dimtruck22:17
*** zz_dimtruck is now known as dimtruck22:18
*** salv-orl_ has joined #openstack-oslo22:24
*** salv-orlando has quit IRC22:26
*** dimtruck is now known as zz_dimtruck22:28
*** zz_dimtruck is now known as dimtruck22:29
*** jerrygb has quit IRC22:34
openstackgerritMerged openstack/oslo.messaging: [zmq] Dynamic connections failover  https://review.openstack.org/41946622:48
*** mriedem has quit IRC22:51
*** spzala has joined #openstack-oslo23:14
*** spzala has quit IRC23:18
*** jerrygb has joined #openstack-oslo23:22
*** mriedem has joined #openstack-oslo23:23
*** jerrygb has quit IRC23:26
*** jerrygb has joined #openstack-oslo23:34
*** dimtruck is now known as zz_dimtruck23:35
*** spzala has joined #openstack-oslo23:36
*** salv-orl_ has quit IRC23:50
« Sunday, 2017-01-15 Index Tuesday, 2017-01-17 »

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