*** robertmyers has joined #openstack-trove | 00:39 | |
andreykurilin | amrith: wow. Most jobs had already succeeded. | 00:57 |
---|---|---|
andreykurilin | We did it! | 00:57 |
*** robertmyers has quit IRC | 02:39 | |
openstackgerrit | Merged openstack/trove: unwedge the gate https://review.openstack.org/426535 | 03:04 |
*** rcernin has joined #openstack-trove | 03:11 | |
*** rcernin has quit IRC | 03:13 | |
*** rcernin has joined #openstack-trove | 03:14 | |
*** miqui has quit IRC | 03:19 | |
*** rcernin has quit IRC | 03:25 | |
*** links has joined #openstack-trove | 03:56 | |
*** links has quit IRC | 03:57 | |
*** gouthamr has quit IRC | 04:09 | |
*** chhavi has joined #openstack-trove | 04:15 | |
openstackgerrit | Merged openstack/trove: Add i18n translation to extensions 3/3 https://review.openstack.org/406271 | 06:19 |
*** rcernin has joined #openstack-trove | 07:11 | |
*** rcernin has quit IRC | 07:12 | |
*** rcernin has joined #openstack-trove | 07:12 | |
*** tesseract has joined #openstack-trove | 07:50 | |
*** pcaruana has joined #openstack-trove | 08:15 | |
openstackgerrit | Riccardo Pittau proposed openstack/trove: Adding mongodb support to xenial https://review.openstack.org/403790 | 08:30 |
rpittau | hi all o/ | 09:11 |
rpittau | seeing an exception when running root-disable in newton | 09:13 |
rpittau | BadRequest: Username root is not valid: 'root' is not a valid user name. (HTTP400) | 09:13 |
rpittau | root-enable works fine | 09:14 |
*** jschwarz is now known as jschwarz|lunch | 09:27 | |
*** rcernin has quit IRC | 09:31 | |
*** dasTor_ has quit IRC | 09:31 | |
*** dasTor has joined #openstack-trove | 09:32 | |
*** rcernin has joined #openstack-trove | 09:33 | |
*** tosky has joined #openstack-trove | 09:38 | |
*** chhavi has quit IRC | 09:45 | |
*** dalees has quit IRC | 10:23 | |
*** dalees has joined #openstack-trove | 10:28 | |
*** chhavi has joined #openstack-trove | 10:29 | |
*** dalees has quit IRC | 11:19 | |
*** dalees has joined #openstack-trove | 11:20 | |
*** dalees has quit IRC | 11:31 | |
*** dalees has joined #openstack-trove | 11:32 | |
amrith | andreykurilin, yes, we did it (something) | 11:53 |
amrith | have to still figure out why I had to whack one method to just nullify it | 11:53 |
amrith | this is a problem for upgrade | 11:53 |
amrith | andreykurilin, thanks much for your help | 11:53 |
amrith | dougshelley66, you will want to take a look at this change very closely as it will cause your hair to curl | 11:53 |
*** dalees has quit IRC | 12:15 | |
*** dalees has joined #openstack-trove | 12:16 | |
amrith | hi rpittau | 12:21 |
rpittau | hi amrith | 12:21 |
amrith | not sure what that is. what datastore? | 12:21 |
rpittau | mysql | 12:21 |
amrith | in newton? | 12:22 |
rpittau | it seems a bug, the root user should have ignore_users set to [] | 12:22 |
rpittau | correct | 12:22 |
rpittau | I have client version 2.7.0 and server 6.0.0 | 12:23 |
rpittau | server = guestagent | 12:23 |
amrith | no | 12:27 |
amrith | no matter what user, there is a set of ignore_users | 12:28 |
amrith | for example, the internal users. | 12:28 |
amrith | you should NOT be able to muck with those through trove | 12:28 |
amrith | that's my understanding of how ignore_users works | 12:28 |
rpittau | yes but I found that in mysql the root user is a special user | 12:28 |
amrith | you should ONLY be able to manipulate root user through root-enable/disable commands | 12:28 |
rpittau | correct | 12:29 |
amrith | yes, that's why it should be in ignore_users. | 12:29 |
rpittau | yes | 12:29 |
rpittau | but when I'm using root-disable | 12:29 |
rpittau | I get that excepion | 12:29 |
rpittau | exception* | 12:29 |
amrith | ok, i understand. I am merely saying that the statement "the root user should have ignore_users set to []" is false. | 12:29 |
rpittau | oh ok, because it's how it's set in mysql, at least my understanding of how it's implemented as special mysql user | 12:30 |
amrith | root is a db user | 12:30 |
amrith | a db user has no ignore_users list | 12:30 |
amrith | an ignore_users list is a server wide configuration | 12:30 |
amrith | hopefully per datastore | 12:30 |
rpittau | yes | 12:31 |
rpittau | hold on, let me find that in the code | 12:31 |
amrith | so I don't understand the statement "the root user should have ignore_users set to []" | 12:31 |
rpittau | I'm sorry, it should set the ignore_users to [] | 12:33 |
rpittau | if you look at the RootUser class for mysql | 12:34 |
rpittau | Overrides _ignore_users from the MySQLUser class. | 12:35 |
rpittau | under trove/guestagent/db/models.py | 12:36 |
rpittau | line 1022 | 12:36 |
amrith | one second | 12:39 |
rpittau | sure :) | 12:39 |
amrith | ok, am back | 12:49 |
amrith | hmm, what's /db/? | 12:50 |
amrith | is this specific to newton? | 12:50 |
rpittau | yes | 12:50 |
amrith | or are you on master? | 12:50 |
rpittau | I'm on newton | 12:50 |
amrith | one second, let me switch branches | 12:51 |
amrith | weird | 12:52 |
amrith | ok, so where are you getting this exception? | 12:53 |
rpittau | it's in trove/guestagent/db/models.py | 12:54 |
rpittau | line 953 | 12:54 |
rpittau | the _is_valid_user_name function is checking the _ignore_users value | 12:54 |
rpittau | I guess the ignore_users value is set to [] only during root-enable ? | 12:56 |
amrith | should also set it in root-disable | 13:26 |
amrith | I think | 13:26 |
amrith | not in general | 13:26 |
amrith | but in root-disable | 13:26 |
amrith | maybe | 13:27 |
amrith | this is all different in ocata | 13:27 |
amrith | (master) | 13:27 |
rpittau | yeah, I saw it | 13:27 |
rpittau | I believe the disable command sent by the client is a delete | 13:27 |
rpittau | that's probably why the root user is still ignored | 13:27 |
amrith | but since you are talking about newton, fixing this will be hard | 13:35 |
rpittau | I understand | 13:35 |
amrith | because the only fixes allowed in newton (stable) will be backports. | 13:35 |
rpittau | I will give it a try with ocata | 13:36 |
amrith | so it would be great if you can find a fix you can backport into newton from master | 13:36 |
amrith | but I don't see one immediately | 13:36 |
rpittau | me neither, the structure is very different | 13:36 |
*** jschwarz|lunch is now known as jschwarz | 14:01 | |
*** gcb has joined #openstack-trove | 14:13 | |
*** gouthamr has joined #openstack-trove | 14:19 | |
*** robertmyers has joined #openstack-trove | 14:29 | |
*** rcernin has quit IRC | 14:48 | |
*** rcernin has joined #openstack-trove | 15:00 | |
*** pmackinn has joined #openstack-trove | 15:28 | |
*** rajinir has joined #openstack-trove | 15:33 | |
*** chhavi has quit IRC | 16:08 | |
*** rcernin has quit IRC | 16:24 | |
*** johnma has joined #openstack-trove | 16:26 | |
*** chhavi has joined #openstack-trove | 16:40 | |
*** tesseract has quit IRC | 16:58 | |
*** johnma has quit IRC | 17:05 | |
*** johnma has joined #openstack-trove | 17:06 | |
*** gcb has quit IRC | 17:07 | |
*** trevormc has joined #openstack-trove | 17:09 | |
*** trevormc has quit IRC | 17:30 | |
*** harlowja has joined #openstack-trove | 18:02 | |
*** trevormc has joined #openstack-trove | 18:16 | |
*** chhavi has quit IRC | 18:28 | |
*** tqtran has joined #openstack-trove | 18:29 | |
*** tosky has quit IRC | 18:44 | |
openstackgerrit | Merged openstack/trove: Add i18n translation to others 3/3 https://review.openstack.org/406327 | 20:31 |
*** gouthamr has quit IRC | 21:07 | |
*** gouthamr has joined #openstack-trove | 21:35 | |
openstackgerrit | Trevor McCasland proposed openstack/trove: TrivialFix: Use os_cache instead of no_cache https://review.openstack.org/426938 | 21:38 |
openstackgerrit | Trevor McCasland proposed openstack/trove: TrivialFix: Use os_cache instead of no_cache https://review.openstack.org/426938 | 21:46 |
*** robertmyers has quit IRC | 22:45 | |
openstackgerrit | Trevor McCasland proposed openstack/trove: Fix Deprecation: context property tenant https://review.openstack.org/426960 | 22:51 |
openstackgerrit | Trevor McCasland proposed openstack/trove: Fix Deprecation: context property tenant https://review.openstack.org/426960 | 23:30 |
openstackgerrit | Merged openstack/trove: Add i18n tranlastion to guestagent datastore 2/5 https://review.openstack.org/406283 | 23:37 |
openstackgerrit | Trevor McCasland proposed openstack/trove: Fix Deprecation: context property tenant https://review.openstack.org/426960 | 23:41 |
*** gouthamr has quit IRC | 23:49 | |
trevormc | I have an odd suspicion that the gate cannot bring up an instance for the same reason I can't. I don't know what's causing it but whenever I do operations as the demo user I can bring up an instance, otherwise I get timeouts. | 23:52 |
trevormc | This job failed because of a timeout and this line made me think it has to do with being an admin because module create works just not as admin. http://logs.openstack.org/80/406280/4/gate/gate-trove-scenario-dsvm-mysql-single-ubuntu-xenial/a28aa8e/console.html#_2017-01-30_22_23_45_621403 | 23:52 |
trevormc | s/create/delete | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!