Monday, 2014-08-18

*** betsy has joined #openstack-dns00:06
*** betsy has quit IRC00:30
*** shakayumi is now known as shakamunyi00:59
*** betsy has joined #openstack-dns01:09
*** betsy has quit IRC01:16
*** nosnos has joined #openstack-dns01:38
*** darshan104 has quit IRC01:43
*** shakayumi has joined #openstack-dns01:46
*** shakamunyi has quit IRC01:49
*** vinod1 has joined #openstack-dns02:25
*** vinod1 has quit IRC03:11
*** vinod1 has joined #openstack-dns03:54
*** vinod1 has quit IRC03:55
*** Krast has joined #openstack-dns03:56
*** nkinder has quit IRC04:34
*** nkinder has joined #openstack-dns04:35
*** nosnos has quit IRC05:59
*** nosnos has joined #openstack-dns06:01
*** darshan104 has joined #openstack-dns06:19
*** tryggvil has joined #openstack-dns09:29
*** mwagner_lap has joined #openstack-dns09:58
*** RaginBajin has joined #openstack-dns11:31
*** mwagner_lap has quit IRC12:28
*** darshan104 has quit IRC12:44
*** darshan104 has joined #openstack-dns12:53
*** darshan104 has quit IRC12:57
*** richm has joined #openstack-dns13:04
*** nkinder has quit IRC13:10
*** darshan104 has joined #openstack-dns13:13
*** darshan104 has quit IRC13:17
*** eankutse has joined #openstack-dns13:17
*** darshan104 has joined #openstack-dns13:17
*** darshan104 has quit IRC13:22
*** darshan104 has joined #openstack-dns13:22
*** darshan104 has quit IRC13:27
*** vinod1 has joined #openstack-dns13:36
*** vinod1 has quit IRC13:38
*** darshan104 has joined #openstack-dns13:38
*** darshan104 has quit IRC13:43
*** darshan104 has joined #openstack-dns13:45
*** betsy has joined #openstack-dns13:46
*** darshan104 has quit IRC13:50
*** mwagner_lap has joined #openstack-dns13:52
*** darshan104 has joined #openstack-dns13:52
*** nkinder has joined #openstack-dns13:56
*** darshan104 has quit IRC13:57
*** Krast_ has joined #openstack-dns14:04
*** Krast has quit IRC14:11
*** nosnos has quit IRC14:16
*** mwagner_lap has quit IRC14:23
*** mwagner_lap has joined #openstack-dns14:24
*** timsim has joined #openstack-dns14:29
*** timsim has joined #openstack-dns14:30
*** vinod1 has joined #openstack-dns14:33
*** mwagner_lap has quit IRC14:34
Kiallbetsy: back in the office?14:34
betsyHeya.14:35
*** mwagner_lap has joined #openstack-dns14:35
betsyYep. I’m back14:35
betsyWhat’s up?14:37
*** mwagner_lap has quit IRC14:41
*** mwagner_lap has joined #openstack-dns14:41
*** paul_glass has joined #openstack-dns14:46
*** paul_glass has quit IRC14:47
*** paul_glass has joined #openstack-dns14:47
*** mwagner_lap has quit IRC14:49
*** mwagner_lap has joined #openstack-dns14:50
*** rustlebee is now known as russellb14:52
*** darshan104 has joined #openstack-dns14:53
*** darshan104 has quit IRC14:57
*** paul_glass has quit IRC15:02
*** paul_glass has joined #openstack-dns15:19
*** vinod1 has quit IRC15:24
Kiallbetsy: sorry, had a call15:25
betsynp15:25
Kiallre the expose NS/SOA stuff, I took a stab and writing a DB migration to pre-generate them all, but didn't have time to update your patch to match ;) .. Wanted to make sure you didn't re-implement!15:26
betsyOh, I started working on it over the weekend15:26
Kiallhah - I was told you were on holidays! I didn't expect you to be back until today15:26
betsyI got back on Friday15:26
betsyDoesn’t seem too hard, but if you’ve already done it, go ahead and check it in15:27
Kiallhttp://paste.openstack.org/show/u1xb5ChCVjmezdbLlfsx/ was where I got to (with lots of room for improvement ;))15:27
betsyI was just getting ready to test mine15:27
Kiallfeel free to ignore mine :)15:29
betsyhmm. Mine’s not too different. I just did everything in the one loop thru the domains - create the SOA recordset, the SOA record, the NS recordset and the NS record15:30
betsyOh, you did the same thing15:30
betsyAt first glance I thought you were doing the ns records separately15:30
betsyWhy do you have managed set to True for the NS recordsets and records?15:31
KiallWell, we need to ensure users can't edit the, for example, SOA record or inbuilt NS records15:32
Kiallin-theory - managed=True is meant to do that, but I went looking for where that's implemented, and I think that code got refactored away at some point :/15:32
betsyI also didn’t check to see if there were existing NS recordsets. I thought about it, but was thinking there wouldn’t be any15:34
betsyIs that not correct?15:34
KiallYes, there can be an existing NS RRset15:34
betsyOh, good to know. But not existing SOAs?15:35
*** vinod1 has joined #openstack-dns15:35
KiallNo, SOA will be rejected as an invalid rtype, NS is allowed though - e.g. for delegating a subdomain to another nameserver15:35
Kialland we never blocked creating NS records for the domain itself, so it seems we have some customers doing that :)15:36
betsyOh, true. Duh. I’m adding that now.15:36
betsyAs far as the missing unique constraint - I didn’t run into that in testing, but had to add it to get it to pass tox15:37
betsyI guess that’s running sqlite15:37
KiallYea, SQLite drops indexes when you ALTER TABLE, so they have to be re-added (but only for SQLite..)15:37
KiallIt's *really* annoying.. I always forget ;)15:38
betsyYeah. I saw your comment and added that.15:38
betsyI couldn’t figure out why it wouldn’t pass tox on that, because it was fine for me.15:38
KiallYea, it's silently dropped inside of SQLite, so there's not even any logs etc.. It just goes *poof*15:38
betsyOh, I was also wondering about the created_ad and updated_at15:39
betsyWhy do those need to be explicitly added?15:40
betsyJust to make them match the zone info because they should have been created with the zone?15:41
betsyAnd I guess you didn’t add the updated_date, but the example I was looking at — 035 — did15:41
Kiallwell, everything needs a created_at - that should never be NULL.. and I figured the zone creation date was a reasonable choice15:42
betsyI assumed that was created by the db when it was inserted15:43
Kiallupdated_at .. I didn't really think about it, but I suppose NOW() would be a reasonable choice?15:43
betsyok. I’ll add that15:43
KiallIt's possible it is, I didn't think of that ;)15:43
betsyOk. I’ll play with it and see15:43
*** rmoe has quit IRC15:48
*** rmoe has joined #openstack-dns15:57
*** zain1 has joined #openstack-dns15:59
*** zain has quit IRC15:59
vinod1kiall: I had a question on your sqlalchemy core change - https://review.openstack.org/#/c/109833/16:00
vinod1What is the reasoning for creating this file - https://review.openstack.org/#/c/109833/3/designate/sqlalchemy/utils.py?16:00
Kiallheya16:05
KiallSo, oslo.db has a method that is 99% the same, but it's build for ORM - where the "SELECT .. WHERE" function is called filter_by rather than "where"16:06
Kiallso, I had to copy+edit.. At some point we should get oslo.db updated to support both, but even if we did today, it'll be a while before we can require that version of oslo.db as the minimun!16:07
Kiallminimum*16:07
vinod1Thanks Kiall for the clarification16:08
*** zain has joined #openstack-dns16:11
*** paul_glass has quit IRC16:12
*** zain1 has quit IRC16:12
*** paul_glass has joined #openstack-dns16:13
*** mwagner_lap has quit IRC16:23
*** paul_glass has quit IRC16:35
*** zain has quit IRC16:37
*** zain1 has joined #openstack-dns16:37
*** crc32 has joined #openstack-dns16:55
*** paul_glass has joined #openstack-dns17:21
*** zain2 has joined #openstack-dns17:21
*** zain1 has quit IRC17:25
*** vinod1 has quit IRC17:32
*** vinod1 has joined #openstack-dns17:49
*** darshan104 has joined #openstack-dns17:50
*** rossk has joined #openstack-dns17:52
*** darshan104 has quit IRC17:54
*** darshan104 has joined #openstack-dns18:15
vinod1kiall: With the requirements.txt file, I now see an error - No distributions matching the version for oslo.config>=1.4.0.0a318:30
Kiallvinod1: you could be on too old a version of pip?18:30
Kiallhttps://pypi.python.org/pypi/oslo.config <-- 1.4.0.0a3 exists, but it's an "wheel" rather than tarball, which is a new-ish format18:31
vinod1Thanks Kiall - that worked18:32
KiallThose wheels have caused untold amounts of pain for people ;)18:33
*** vinod1 has quit IRC18:37
*** jaiswaro has joined #openstack-dns19:33
rjrjranyone get a chance to look at this: https://review.openstack.org/#/c/114397/19:39
*** jaiswaro has quit IRC19:49
*** shakayumi has quit IRC19:53
*** nkinder has quit IRC19:54
*** simonmcc__ has joined #openstack-dns19:57
*** zain2 has quit IRC19:58
*** betsy has quit IRC19:58
*** simonmcc_ has quit IRC19:58
*** simonmcc__ is now known as simonmcc_19:59
*** tryggvil has quit IRC19:59
*** betsy has joined #openstack-dns20:01
*** zain has joined #openstack-dns20:02
*** vinod1 has joined #openstack-dns20:33
*** paul_glass has quit IRC20:46
*** RaginBajin has quit IRC20:48
*** nkinder has joined #openstack-dns21:10
*** EricGonczer_ has joined #openstack-dns21:11
*** EricGonczer_ has quit IRC21:23
*** eankutse has quit IRC21:30
*** timsim has quit IRC21:41
*** crc32 has quit IRC21:46
*** vinod1 has quit IRC22:17
openstackgerritA change was merged to openstack/designate: Migrate to SQLAlchemy Core  https://review.openstack.org/10983322:32
*** EricGonczer_ has joined #openstack-dns22:56
*** eankutse has joined #openstack-dns23:10
*** eankutse has quit IRC23:14
*** darshan104 has quit IRC23:20
*** crc32 has joined #openstack-dns23:21
*** vinod1 has joined #openstack-dns23:23
*** vinod1 has quit IRC23:23
*** darshan104 has joined #openstack-dns23:24
*** vinod1 has joined #openstack-dns23:26
*** vinod1 has quit IRC23:26
*** EricGonczer_ has quit IRC23:41
*** darshan104 has quit IRC23:41
*** darshan104 has joined #openstack-dns23:43

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