pedroalvarez | SotK: well, created new user, went to preferences page, tried to uncheck one of them, and got this: http://paste.baserock.org/rimusezamu | 00:37 |
---|---|---|
pedroalvarez | dropped all its preferences from user_preferences, and now works: http://paste.baserock.org/teyomomaha | 00:42 |
pedroalvarez | :S | 00:42 |
SotK | o.O | 00:43 |
SotK | how completely bizarre | 00:44 |
pedroalvarez | right, new user, and just opening the prefs page: http://paste.baserock.org/tuzoruxiva | 00:47 |
pedroalvarez | madness | 00:47 |
SotK | hmm | 00:47 |
SotK | I wonder why the update function is called so many times | 00:47 |
SotK | I suspect it tries to set defaults for all preferences once per preference | 00:48 |
SotK | but I don't understand why that would only happen for a new user | 00:49 |
* SotK will investigate more tomorrow, too sleepy now :) | 00:50 | |
pedroalvarez | yeah! night! | 00:51 |
*** cody-somerville has quit IRC | 01:29 | |
*** cody-somerville has joined #storyboard | 01:36 | |
*** mrmartin has joined #storyboard | 03:53 | |
*** mrmartin has quit IRC | 05:57 | |
*** thrace has quit IRC | 06:01 | |
*** thrace has joined #storyboard | 06:04 | |
*** mrmartin has joined #storyboard | 07:56 | |
SotK | good morning! | 10:38 |
Zara | morning, storyboard! | 10:38 |
Zara | hm, at the time you two were discussing preferences, I was watching peep show... I'd feel bad, but it was nearly 1am. | 10:39 |
SotK | :) | 10:40 |
SotK | in your notifications roadmap/spec thing, does it make sense to add a requirement for a page on which a user can see all of the things they're subscribed to? | 10:49 |
Zara | yeah, or at least some place (eg: maybe it wouldn't need to be a page but some sort of popup box) | 10:51 |
Zara | I think I'd implied that but maybe jumped over it, and it needs to be explicit, anyway | 10:52 |
Zara | so please add anything that's missing; I think it's at the point where it needs fleshing out from a few people. | 10:52 |
Zara | there are big gaps (especially on the 'how does storyboard currently track a user's subscriptions?') side; I put a lot in about how things were represented in the UX but there's not much detail about the inner workings. | 10:53 |
* SotK will try to flesh out that bit today then | 10:53 | |
Zara | thanks, in return I will try to review all the things! | 10:54 |
SotK | OK, I think I may have discovered (part) of the preferences problem... | 10:56 |
SotK | in Preference.save() there is a comment: "If preferences are defaults, they won't have a $save() method." | 10:58 |
SotK | however, it appears that for some reason, even the default preferences have a $save() method | 10:58 |
SotK | this is why there are 16 POST requests made that all pass a dict of all the preferences when the preferences page is accessed without any preferences set | 10:59 |
SotK | (Preference.save() is called once per unset preference) | 11:00 |
SotK | (but with the expectation that the unset preference won't be saved, as it is just using the default) | 11:00 |
* SotK digs deeper | 11:04 | |
Zara | hehe | 11:05 |
pedroalvarez | I noted the following yesterday: With no preferences in the db, you open the preferences page, madness happen | 11:07 |
pedroalvarez | And, with no preferences in the db, preferences page already open, change preference. Everything ok | 11:07 |
SotK | hmm | 11:09 |
SotK | I suspect its definitely to do with the spam when setting defaults then, since that doesn't happen if the page is already open | 11:09 |
SotK | I dunno why this is only causing an issue for you and the production instance though | 11:09 |
Zara | hm, weird, I got an error when updating my db, but it seems to have worked anyway | 11:16 |
SotK | what error was it? | 11:17 |
Zara | storyboard.common.exception.DBDuplicateEntry: Database object already exists. | 11:18 |
Zara | ERROR: InvocationError: '/home/ubuntu/storyboard/.tox/venv/bin/storyboard-db-manage --config-file ./etc/storyboard.conf upgrade head' | 11:18 |
Zara | ___________________________________ summary ____________________________________ | 11:18 |
Zara | ERROR: venv: commands failed | 11:18 |
Zara | full thing: http://paste.baserock.org/rulekeriza | 11:18 |
Zara | bit weird, but I can now add worklists where I couldn't before because of errors about there being no table for permissions... (testing the permissions patch now) | 11:19 |
Zara | so seems to work, anyway | 11:20 |
SotK | pretty weird | 11:20 |
Zara | hm, creating a board seems to be a bit broken | 11:34 |
Zara | (as in, seems to be locked and I can't edit it) | 11:35 |
Zara | worklists seem okay | 11:35 |
Zara | I got errors about it being unable to convert to a worklist | 11:35 |
SotK | :( | 11:35 |
* SotK will have a look in a moment | 11:35 | |
SotK | default preferences have a $save() method, because they are automatically resolved on every single page by the root state | 11:36 |
Zara | (it might just be that that patch depends on another one that it's not listed as depending on , not sure.) | 11:36 |
SotK | so the check for if a preference is default or not is totally broken | 11:37 |
SotK | \o/ | 11:37 |
Zara | okay, so errors were just because I was clicking 'save' repeatedly too fast. but board not being editable is still an issue, and text still goes off the cards. | 11:45 |
* SotK wonders if pedroalvarez has time to test a fix for the preferences issue today | 11:46 | |
SotK | Zara: what do you mean by "text still goes off the cards"? | 11:46 |
Zara | http://imgur.com/ReMbhNu | 11:47 |
Zara | that's a newly-created board | 11:48 |
SotK | I see :( | 11:48 |
* SotK will do something with overflow | 11:48 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Don't save default preference values https://review.openstack.org/252350 | 11:53 |
SotK | pedroalvarez: I think https://review.openstack.org/252350 should solve the duplicate preferences issue, any chance you'll have time to test it please (since I still don't seem to be able to reproduce it)? | 11:55 |
*** mrmartin has quit IRC | 12:05 | |
pedroalvarez | sure! | 12:05 |
Zara | ah, was about to say I can't easily test that one, cool | 12:05 |
pedroalvarez | feel free to test it too, but I wonder if you had also the issue that it solves | 12:06 |
Zara | yeah, I didn't, that's why I can't easily test it | 12:06 |
Zara | I can check if it breaks other things but can't easily see if it works | 12:07 |
pedroalvarez | dammit, I need to read more carefully | 12:07 |
Zara | heh, no problem | 12:07 |
pedroalvarez | you could try to use my way to reproduce it :) | 12:07 |
Zara | I like having a tidy database! :P | 12:07 |
pedroalvarez | open dashboard, delete rows from user_preferences table, open preferences page, kaboom | 12:07 |
Zara | I've got a lot of review, tbh, but I'll come back to it later if we need more people on it. for now I'm enjoying the fact that nothing has gone kaboom yet. | 12:08 |
pedroalvarez | k, no worries :) I'm happy to test that and give some feedback | 12:11 |
Zara | :) | 12:12 |
pedroalvarez | SotK: tbh, I expected a fix in storyboard-api, given that it looks like a bad use of the API breaks the database | 12:13 |
pedroalvarez | but of course, I have no idea about the codebase | 12:13 |
Zara | (btw, just went back to master of storyboard, and the non-editable board is editable again, so convinced it's the permissions patch that's to blame. The text still goes off the cards though.) | 12:13 |
Zara | that was for my issue from earlier, anyway. | 12:14 |
pedroalvarez | SotK: with https://review.openstack.org/252350 nothing is created in the database | 12:18 |
pedroalvarez | (when opening the preferences page) | 12:18 |
pedroalvarez | which is what the patch says it does | 12:18 |
pedroalvarez | Modifying values after that generates the defaults and works good | 12:18 |
openstackgerrit | Merged openstack-infra/storyboard-webclient: Add a dashboard page for boards and worklists https://review.openstack.org/231973 | 12:27 |
persia | This is probably a FAQ, but how long from merge until production deployment at s.o.o? | 12:34 |
SotK | depends on how busy zuul is | 12:35 |
SotK | normally about an hour or two on average I think | 12:36 |
Zara | yeah, it seems to vary from around an hour to a couple of days, but I've never checked thoroughly. | 12:36 |
persia | Is it zuul's "post" queue? | 12:36 |
SotK | yep | 12:37 |
openstackgerrit | Merged openstack-infra/storyboard-webclient: Fix checkboxes on preferences page https://review.openstack.org/252002 | 12:37 |
pedroalvarez | I've seen it taking a bit longer than what the post queue said | 12:37 |
persia | Excellent. So I can just watch http://status.openstack.org/zuul/ and see the jobs land. | 12:37 |
pedroalvarez | that is, the job in that queue finishes, and still the fix is not in production | 12:37 |
SotK | yeah, I think there is a bit more to be done after the post queue, but I'm not entirely sure how the whole puppet thing works yet | 12:38 |
Zara | I know that sometimes tarballs don't get generated \o/ | 12:38 |
SotK | but the post job creates the tarball which is then deployed by puppet | 12:38 |
pedroalvarez | I tried to find out in the project-config repo, and didn't found any hint | 12:39 |
pedroalvarez | Zara: oh, thats bad, but I don't worry because my deployments depend on git :D | 12:39 |
persia | pedroalvarez: Your deployment subscribes to the gerrit event feed, and runs master? | 12:40 |
pedroalvarez | well, I believe it's pulling from github, (maybe I should use gerrit instead) and installs that in the case of the API, and generates the files and put them in place in the case of the webclient | 12:43 |
pedroalvarez | and yes, master | 12:43 |
SotK | pedroalvarez: there is probably a fix that could be done in the api too, but I have no idea what causes the problem as it doesn't seem to happen for me | 12:43 |
persia | If not gerrit, at least one of the git.o.o mirrors is more likely to receive useful support from the project if it isn't current. | 12:44 |
SotK | and that patch causes the intended behaviour to happen for preferences anyway I think :) | 12:44 |
pedroalvarez | SotK: yes it does | 12:47 |
pedroalvarez | persia: suggestion taken: https://github.com/palvarez89/ansible-role-storyboard/commit/1f7151bf365d6691d764a5e153221b1b864fbe39 | 12:48 |
persia | Unless I'm reading that incorrectly, that polls. Polling should be against git.openstack.org, rather than review.openstack.org. If you want to consume review.openstack.org, you should do so via an event stream. | 12:49 |
pedroalvarez | that would be if I want it to run it whenever an event happens | 12:52 |
pedroalvarez | I don't think I want that yet | 12:52 |
persia | Makes sense, so yeah, you want to use git.openstack.org, rather than review. | 12:52 |
persia | (because the load on review is rather high, so only things that need realtime should not be using a mirror) | 12:53 |
pedroalvarez | makes sense | 12:55 |
* SotK realises the problems with the board permissions patch | 13:24 | |
Zara | ooh? | 13:25 |
SotK | it is uneditable because the old UI doesn't understand the new API's permissions stuff | 13:26 |
SotK | oops | 13:26 |
Zara | hehe, I wondered if it was that but didn't really know how to check | 13:27 |
SotK | also, I can't recreate the breakage in the migration you had :/ | 13:27 |
Zara | (so I was lazy and moved onto looking at other patches for a bit.) | 13:27 |
Zara | :/ well, it didn't seem to affect anything | 13:27 |
Zara | but it was weird. | 13:27 |
SotK | can you paste the output of `select * from worklists;` in your db? | 13:29 |
Zara | sure, though it makes me look like a really weird person: http://paste.baserock.org/deruxoyena | 13:30 |
Zara | so going from the other paste, seems something went wrong at id:12, which I note has an updated_at where the others don't | 13:31 |
Zara | oh wait ignore me | 13:31 |
Zara | that's the line | 13:31 |
Zara | gah | 13:31 |
pedroalvarez | Hello satna father christmas :) | 13:32 |
Zara | \o/ | 13:32 |
pedroalvarez | I'm happy you found your dog :) | 13:32 |
Zara | (12 and 13 have the same name, but I'd've thought that'd error at 13 if that were the issue) | 13:32 |
Zara | haha, it's just a song that was stuck in my head | 13:33 |
* persia mumbles about http://paste.openstack.org/ | 13:33 | |
Zara | I don't even have a dog | 13:33 |
Zara | told you it'd make me look weird =D | 13:33 |
* Zara bookmarks paste.openstack.org | 13:33 | |
Zara | I had baserock's one bookmarked so it was the first thing to come up when I put 'paste' in the address bar | 13:33 |
SotK | Zara: can you paste the result of `select * from permissions;` too please? :) | 13:34 |
* persia has a preference for using context-specific pastebins | 13:34 | |
Zara | http://paste.openstack.org/show/480630/ | 13:35 |
Zara | SotK^ | 13:36 |
* SotK wonders where on earth `| 1 | 2015-12-02 11:12:22 | NULL | edit_worklist_12 | edit_worklist |` came from | 13:38 | |
SotK | hmm actually, did you create that worklist before running the migration? | 13:40 |
SotK | s/create/attempt to create/ | 13:41 |
Zara | I'm not sure | 13:44 |
Zara | the timing suggests it | 13:44 |
SotK | indeed | 13:44 |
Zara | that might've been the thing that reminded me to upgrade | 13:44 |
SotK | and that would explain the partially created permissions | 13:44 |
SotK | since it creates the permission (edit_worklist_12), then tries to reference it in the worklist_permissions table that won't have existed yet | 13:44 |
Zara | yeah, I remember partially creating one before it gave me an error because it didn't understand permissions, which reminded me to upgrade, so that fits | 13:45 |
SotK | meaning that the other permission wouldn't have been created (which also seems to be the case) | 13:45 |
SotK | aha! mystery solved then :) | 13:45 |
Zara | yup, that's the earliest one today so guessing it's that one. | 13:45 |
SotK | does this need a fix, given that it should be impossible to have such a situation in production since the migration will run automatically, before anyone tries to create things? | 13:46 |
Zara | is it guaranteed to always work automatically? | 13:47 |
Zara | (also, what does that mean for poor old worklist 12 at this point?) | 13:47 |
SotK | Zara: if you migrate down to 049, and then do `delete from permissions;` on your db, and then re-run the upgrade migration, it should be fixed | 13:49 |
*** mrmartin has joined #storyboard | 13:49 | |
* SotK doesn't know what happens if a migration breaks in production | 13:49 | |
SotK | I'd imagine a very big mess | 13:49 |
Zara | okay. (yeah, I see poor little worklist 12 is currently 'not found'.) | 13:50 |
Zara | guessing they'd have to revert | 13:50 |
SotK | but then, I don't really want to wipe the permissions table in the migration, although it should already be empty | 13:51 |
SotK | (and definitely is on storyboard.o.o) | 13:52 |
Zara | how would I migrate down to 049 and delete it from permissions? my mysql knowledge is pretty much 'I know how to login and show tables') | 13:52 |
SotK | I've also run the migration on storyboard.sotk.co.uk against a dump of the production db from yesterday without any problems | 13:53 |
SotK | Zara: `tox -e venv "storyboard-db-manage --config-file ./etc/storyboard.conf downgrade 049"` | 13:53 |
Zara | hahaha | 13:54 |
SotK | then connect to your database and run `delete from permissions;` | 13:54 |
Zara | the downgrade command is too sensible; I was expecting something less guessable, like the git commands | 13:55 |
SotK | :D | 13:55 |
Zara | shocked and appalled. | 13:55 |
Zara | ooooohhhh: ERROR 1451 (23000): Cannot delete or update a parent row: a foreign key constraint fails (`storyboard`.`user_permissions`, CONSTRAINT `user_permissions_ibfk_1` FOREIGN KEY (`permission_id`) REFERENCES `permissions` (`id`)) | 13:57 |
SotK | is that when running `delete from permissions;`? | 13:58 |
SotK | if so, do `delete from user_permissions;` first :) | 13:58 |
Zara | hahaha | 13:59 |
Zara | `delete from user_permissions;' worked, but `delete from permissions;` still didn't | 14:00 |
Zara | but this time board_permissions are the constraint | 14:00 |
Zara | so guessing htose also have to go | 14:00 |
SotK | hmmm | 14:00 |
SotK | did the downgrade work? | 14:00 |
Zara | it said it did | 14:01 |
Zara | ubuntu@zara-storyboard:~/storyboard$ tox -e venv "storyboard-db-manage --config-file ./etc/storyboard.conf downgrade 049" | 14:01 |
Zara | venv develop-inst-nodeps: /home/ubuntu/storyboard | 14:01 |
Zara | venv runtests: commands[0] | storyboard-db-manage --config-file ./etc/storyboard.conf downgrade 049 | 14:01 |
Zara | ______________________________________________________________________ summary _______________________________________________________________________ | 14:01 |
Zara | venv: commands succeeded | 14:01 |
Zara | congratulations :) | 14:01 |
Zara | it smiled at me, and everything | 14:01 |
SotK | hmm, that should have deleted the board_permissions table | 14:01 |
SotK | what does `select * from alembic_version;` say? | 14:02 |
Zara | +-------------+ | 14:05 |
Zara | | version_num | | 14:05 |
Zara | +-------------+ | 14:05 |
Zara | | 049 | | 14:05 |
Zara | +-------------+ | 14:05 |
Zara | 1 row in set (0.00 sec) | 14:05 |
Zara | o.O | 14:05 |
Zara | it seems right | 14:05 |
SotK | buh | 14:05 |
SotK | I literally ran it about 10 minutes ago myself xD | 14:06 |
SotK | the `worklist_permissions` and `board_permissions` tables definitely still exist? | 14:07 |
Zara | yup, 70 rows and 16 rows respectively | 14:08 |
SotK | oh, it breaks for me now, but with an error | 14:09 |
Zara | =D | 14:09 |
SotK | that's pretty weird | 14:09 |
SotK | do `drop table worklist_permissions;`, `drop table board_permissions;`, `delete from permissions;`, and I think it should be left in a state that the upgrade migration can handle | 14:10 |
Zara | okay | 14:10 |
Zara | all done | 14:11 |
Zara | time to upgrade again... | 14:11 |
SotK | ugh, my db is in a state after the thing that broke | 14:12 |
Zara | :( the upgrade is working for me this time, at least | 14:13 |
SotK | good :) | 14:14 |
Zara | and I can edit worklist 12 without the world exploding | 14:15 |
SotK | \o/ | 14:15 |
*** mrmartin has quit IRC | 14:16 | |
* SotK tests his fix to the downgrade issue he had | 14:16 | |
SotK | still broken, oops | 14:17 |
* SotK wonders how on earth this worked for Zara | 14:17 | |
SotK | fsvo worked | 14:17 |
Zara | I don't know! :D | 14:17 |
Zara | https://www.youtube.com/watch?v=iz6iENNnf08 | 14:17 |
Zara | I think I switched back to master for the downgrade, then went back to the complex permissions branch for the upgrade | 14:19 |
Zara | I don't see why that'd make any difference to the downgrade, though. | 14:19 |
Zara | it was just where I was at the time, so might as well mention it. | 14:20 |
SotK | aha, that would explain it I think | 14:20 |
SotK | since there would be no downgrade from 050 to 049 in master | 14:20 |
Zara | really? how come? | 14:20 |
Zara | ahhhh | 14:20 |
Zara | right | 14:20 |
SotK | seems dumb that it just sets the version rather than erroring though | 14:20 |
Zara | yeah | 14:21 |
Zara | since that has helped to fix some dodgy understanding of it in my mind, which I'm now undoing | 14:21 |
Zara | oh well, I guess the fix is to fail to do the downgrade, then delete everything and 'upgrade' again | 14:22 |
Zara | and thus I have no idea if downgrading would work | 14:23 |
SotK | downgrading is entirely broken, I must have never actually tested it :/ | 14:23 |
SotK | oh, storyboard meeting in 20 minutes in #openstack-meeting! | 14:40 |
Zara | ack | 14:41 |
SotK | so it looks like the migration is secretly not working properly :/ | 14:48 |
SotK | Zara: please can you paste the result of `select * from worklist_permissions;`? | 14:49 |
Zara | now I've upgraded again? | 14:49 |
SotK | yeah :) | 14:49 |
SotK | I want to see if it is broken for you too :P | 14:50 |
Zara | http://paste.openstack.org/show/480643/ | 14:50 |
SotK | yep, looks broken | 14:50 |
Zara | hm, seem to be duplicated | 14:50 |
SotK | there should be two per worklist | 14:50 |
Zara | oh, so just 39 breaks the pattern? | 14:51 |
Zara | 38, even | 14:51 |
SotK | yep | 14:51 |
SotK | for some reason the very last permission doesn't get added properly | 14:51 |
Zara | \o/ | 14:51 |
SotK | agenda for the meeting is here, feel free to add to it: https://wiki.openstack.org/wiki/Meetings/StoryBoard | 14:54 |
Zara | thanks | 14:54 |
Zara | heh, that makes me feel bad; updated spec thing a bit this week but mainly been catching up on review/discovering how databases can do strange things | 14:55 |
* SotK fixes it with session.flush() | 14:57 | |
SotK | meeting o'clock | 15:00 |
*** ChanServ has quit IRC | 15:34 | |
*** thrace has quit IRC | 15:34 | |
*** b3rnard0 has quit IRC | 15:34 | |
*** davidlenwell has quit IRC | 15:34 | |
*** cody-somerville has quit IRC | 15:34 | |
*** paulsherwood has quit IRC | 15:35 | |
*** lifeless has quit IRC | 15:35 | |
*** SotK has quit IRC | 15:35 | |
*** cody-somerville- has quit IRC | 15:35 | |
*** dabukalam has quit IRC | 15:35 | |
*** pleia2 has quit IRC | 15:35 | |
*** persia has quit IRC | 15:35 | |
*** ttx has quit IRC | 15:35 | |
*** betherly has quit IRC | 15:35 | |
*** jjardon has quit IRC | 15:35 | |
*** coolsvap has quit IRC | 15:35 | |
*** krotscheck has quit IRC | 15:35 | |
*** NikitaKonovalov has quit IRC | 15:35 | |
*** Zara has quit IRC | 15:35 | |
*** Zara_ has quit IRC | 15:35 | |
*** gothicmindfood has quit IRC | 15:35 | |
*** tlbr has quit IRC | 15:35 | |
*** pedroalvarez has quit IRC | 15:35 | |
*** rainya has quit IRC | 15:35 | |
*** wendar has quit IRC | 15:35 | |
*** pleia2 has joined #storyboard | 15:37 | |
*** thrace has joined #storyboard | 15:39 | |
*** b3rnard0 has joined #storyboard | 15:39 | |
*** davidlenwell has joined #storyboard | 15:39 | |
*** ttx has joined #storyboard | 15:39 | |
*** pedroalvarez has joined #storyboard | 15:40 | |
*** persia has joined #storyboard | 15:42 | |
*** betherly has joined #storyboard | 15:42 | |
*** jjardon has joined #storyboard | 15:42 | |
*** SotK has joined #storyboard | 15:43 | |
*** rainya has joined #storyboard | 15:43 | |
*** wendar has joined #storyboard | 15:43 | |
*** NikitaKonovalov has joined #storyboard | 15:53 | |
*** cody-somerville has joined #storyboard | 15:59 | |
*** paulsherwood has joined #storyboard | 15:59 | |
*** lifeless has joined #storyboard | 15:59 | |
*** cody-somerville- has joined #storyboard | 15:59 | |
*** dabukalam has joined #storyboard | 15:59 | |
*** cody-somerville has quit IRC | 16:09 | |
*** paulsherwood has quit IRC | 16:09 | |
*** lifeless has quit IRC | 16:09 | |
*** cody-somerville- has quit IRC | 16:09 | |
*** dabukalam has quit IRC | 16:09 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Implement complex permissions for worklists and boards https://review.openstack.org/235472 | 16:09 |
*** tlbr has joined #storyboard | 16:34 | |
*** gothicmindfood has joined #storyboard | 16:34 | |
*** Zara_ has joined #storyboard | 16:34 | |
*** Zara has joined #storyboard | 16:34 | |
*** krotscheck has joined #storyboard | 16:34 | |
*** coolsvap has joined #storyboard | 16:34 | |
*** cody-somerville has joined #storyboard | 16:34 | |
*** paulsherwood has joined #storyboard | 16:34 | |
*** lifeless has joined #storyboard | 16:34 | |
*** cody-somerville- has joined #storyboard | 16:34 | |
*** dabukalam has joined #storyboard | 16:34 | |
SotK | Zara: are you alive? | 16:44 |
*** cody-somerville has quit IRC | 16:45 | |
*** paulsherwood has quit IRC | 16:45 | |
*** lifeless has quit IRC | 16:45 | |
*** cody-somerville- has quit IRC | 16:45 | |
*** dabukalam has quit IRC | 16:45 | |
*** coolsvap has quit IRC | 16:45 | |
*** krotscheck has quit IRC | 16:45 | |
*** Zara has quit IRC | 16:45 | |
*** Zara_ has quit IRC | 16:45 | |
*** gothicmindfood has quit IRC | 16:45 | |
*** tlbr has quit IRC | 16:45 | |
*** dabukalam has joined #storyboard | 16:46 | |
*** cody-somerville- has joined #storyboard | 16:46 | |
*** lifeless has joined #storyboard | 16:46 | |
*** paulsherwood has joined #storyboard | 16:46 | |
*** cody-somerville has joined #storyboard | 16:46 | |
*** coolsvap has joined #storyboard | 16:46 | |
*** krotscheck has joined #storyboard | 16:46 | |
*** Zara has joined #storyboard | 16:46 | |
*** Zara_ has joined #storyboard | 16:46 | |
*** gothicmindfood has joined #storyboard | 16:46 | |
*** tlbr has joined #storyboard | 16:46 | |
Zara | hey there | 16:48 |
SotK | \o/ | 16:49 |
Zara | what did I miss? | 16:49 |
SotK | 15:44:32 -!- Irssi: Join to #storyboard was synced in 51 secs | 16:50 |
SotK | 16:09:42 < openstackgerrit> Adam Coldrick proposed openstack-infra/storyboard: Implement complex permissions for worklists and boards | 16:50 |
SotK | https://review.openstack.org/235472 | 16:50 |
SotK | 16:44:20 < SotK> Zara: are you alive? | 16:50 |
Zara | 16:42 < Zara> my sources tell me that SotK is now in the same channel, even though nothing has come up on my screen | 16:50 |
Zara | 16:42 < Zara> apparently you're on a swedish server | 16:50 |
Zara | 16:43 < Zara> hm, though actually, I get him with a /whois but /who is still showing nothing | 16:50 |
Zara | 16:43 < Zara> mysterious. | 16:50 |
Zara | 16:44 < Zara> he can see me with /who but not my messages | 16:50 |
persia | netsplits are fun :) | 16:52 |
Zara | I had a party while we were in different channels | 16:52 |
Zara | nobody else came to the party and it was sad. | 16:52 |
Zara | well, on different servers that weren't talking to each other in the same channel | 16:53 |
SotK | so this "uneditable boards" thing is a bit of a mess | 16:53 |
SotK | since the API sends an array now, and you need to pass a flag to angular's $resource if that is the case | 16:54 |
Zara | :D | 16:55 |
SotK | I wonder what happens if you set that flag and don't pass an array... | 16:56 |
Zara | oh, I'm sure it probably makes every past and future patch work perfectly | 16:56 |
Zara | it's a programming secret; don't tell anyone. | 16:57 |
Zara | PS: sorry for ruining your life by finding these bugs; I have a spare openstack marketing expo rubber duck if you want one. | 16:58 |
SotK | that's OK, its better to find these things now that straight after we merge things :) | 16:58 |
SotK | ah, the same thing happens but with the words "array" and "object" swapped | 17:00 |
SotK | In that case I don't have a clue how we can smoothly transition | 17:01 |
*** ChanServ has joined #storyboard | 17:04 | |
*** wolfe.freenode.net sets mode: +o ChanServ | 17:04 | |
* SotK suggests merging them both in quick succession, given the alternative is hardcoding permission levels even more than they already are | 17:07 | |
* SotK gives up and goes home to fly a pretend spaceship to the pretend moon instead | 17:08 | |
*** b3rnard0 is now known as b3rnard0_away | 17:45 | |
Zara | sleep can only help. no idea where today has gone. | 17:47 |
Zara | started sketching out roadmap at the bottom, a bit. I probably can't procrastinate much longer from formatting the wiki page properly instead of cheating with <pre> | 17:48 |
pedroalvarez | heheh it needs some formating :) | 17:54 |
pedroalvarez | why are you using <pre> tags OOI? | 17:54 |
*** b3rnard0_away is now known as b3rnard0 | 18:23 | |
SotK | put it in a story so you can just use markdown :) | 18:25 |
*** jjardon has quit IRC | 19:59 | |
*** jjardon has joined #storyboard | 20:03 | |
pedroalvarez | Ahh, makes sense :) | 22:13 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!