I was trying to get my local LAVA instance to work with our lab's data base dump. So I had the DB, but not all the media files. LAVA almost works when you do this, but I hit two small issues that prevented me from looking at bundle streams and it their test runs.
I realize this is an edge case, but its handy for when you want to test with a real database, but don't want all the media files. I'm not sure this is the exact way we want to fix this, so rather than do a merge proposal I thought I'd get your thoughts on the patch first.
There were two main issues in the code:
bundle_detail wants to get the "document format" for the media file associated with the bundle. Since the file didn't exist we hit an exception. I added a small check to the get_document_format to handle this by returning "n/a".
bundle_detail and test_run_detail wanted to display the size of media files. This calls a new function content_size which will catch an error if the file doesn't exist and simply return 0.
I'm not sure if this patch is doing to good a job of hiding the fact an error does exist. Maybe we should put a conditional in the template file to display an error when the file doesn't exist also?
-andy
W dniu 21.04.2012 01:10, Andy Doan pisze:
I was trying to get my local LAVA instance to work with our lab's data base dump. So I had the DB, but not all the media files. LAVA almost works when you do this, but I hit two small issues that prevented me from looking at bundle streams and it their test runs.
I realize this is an edge case, but its handy for when you want to test with a real database, but don't want all the media files. I'm not sure this is the exact way we want to fix this, so rather than do a merge proposal I thought I'd get your thoughts on the patch first.
There were two main issues in the code:
bundle_detail wants to get the "document format" for the media file associated with the bundle. Since the file didn't exist we hit an exception. I added a small check to the get_document_format to handle this by returning "n/a".
bundle_detail and test_run_detail wanted to display the size of media files. This calls a new function content_size which will catch an error if the file doesn't exist and simply return 0.
I'm not sure if this patch is doing to good a job of hiding the fact an error does exist. Maybe we should put a conditional in the template file to display an error when the file doesn't exist also?
Are we going to review patches on the mailing list? I don't think our current tools are up to the task.
As for your patches, I like the intent, though the motivation is somewhat non-production. There is something that you may want to use that allows you to work with just the database dump and pull the media files from production on demand. There's a django app for that.
Document format should be in the database, a proper fix would pull it out. Current code will also fail when the document is bonkers and just does not parse as JSON.
File size can also fail in the template so we should double check that we don't call any file size filters in any of the html files. (it looks like {something|filesize})
Thanks ZK
On 04/20/2012 06:53 PM, Zygmunt Krynicki wrote:
W dniu 21.04.2012 01:10, Andy Doan pisze:
I was trying to get my local LAVA instance to work with our lab's data base dump. So I had the DB, but not all the media files. LAVA almost works when you do this, but I hit two small issues that prevented me from looking at bundle streams and it their test runs.
I realize this is an edge case, but its handy for when you want to test with a real database, but don't want all the media files. I'm not sure this is the exact way we want to fix this, so rather than do a merge proposal I thought I'd get your thoughts on the patch first.
There were two main issues in the code:
bundle_detail wants to get the "document format" for the media file associated with the bundle. Since the file didn't exist we hit an exception. I added a small check to the get_document_format to handle this by returning "n/a".
bundle_detail and test_run_detail wanted to display the size of media files. This calls a new function content_size which will catch an error if the file doesn't exist and simply return 0.
I'm not sure if this patch is doing to good a job of hiding the fact an error does exist. Maybe we should put a conditional in the template file to display an error when the file doesn't exist also?
Are we going to review patches on the mailing list? I don't think our current tools are up to the task.
I don't think so. I was trying to convey in the subject of the email that this was an RFC and I didn't think the code itself was worthy of a merge proposal.
As for your patches, I like the intent, though the motivation is somewhat non-production. There is something that you may want to use that allows you to work with just the database dump and pull the media files from production on demand. There's a django app for that.
You are correct - its completely intended for non-production. In theory I like the idea of this app you are referring to. I'd like more details on it, but note since the production isn't reachable, we still need something to fill the gap.
Document format should be in the database, a proper fix would pull it out. Current code will also fail when the document is bonkers and just does not parse as JSON.
k, seems like that might be a follow-on patch?
File size can also fail in the template so we should double check that we don't call any file size filters in any of the html files. (it looks like {something|filesize})
yep - I should have grep'd through the code to check all the other places it went wrong. I was just concerned at the time with what was breaking me.
so - now that I have some feedback, I can try to put together a proper patch set and make a merge proposal
Thanks ZK
W dniu 23.04.2012 05:36, Andy Doan pisze:
On 04/20/2012 06:53 PM, Zygmunt Krynicki wrote:
W dniu 21.04.2012 01:10, Andy Doan pisze:
I was trying to get my local LAVA instance to work with our lab's data base dump. So I had the DB, but not all the media files. LAVA almost works when you do this, but I hit two small issues that prevented me from looking at bundle streams and it their test runs.
I realize this is an edge case, but its handy for when you want to test with a real database, but don't want all the media files. I'm not sure this is the exact way we want to fix this, so rather than do a merge proposal I thought I'd get your thoughts on the patch first.
There were two main issues in the code:
bundle_detail wants to get the "document format" for the media file associated with the bundle. Since the file didn't exist we hit an exception. I added a small check to the get_document_format to handle this by returning "n/a".
bundle_detail and test_run_detail wanted to display the size of media files. This calls a new function content_size which will catch an error if the file doesn't exist and simply return 0.
I'm not sure if this patch is doing to good a job of hiding the fact an error does exist. Maybe we should put a conditional in the template file to display an error when the file doesn't exist also?
Are we going to review patches on the mailing list? I don't think our current tools are up to the task.
I don't think so. I was trying to convey in the subject of the email that this was an RFC and I didn't think the code itself was worthy of a merge proposal.
As for your patches, I like the intent, though the motivation is somewhat non-production. There is something that you may want to use that allows you to work with just the database dump and pull the media files from production on demand. There's a django app for that.
You are correct - its completely intended for non-production. In theory I like the idea of this app you are referring to. I'd like more details on it, but note since the production isn't reachable, we still need something to fill the gap.
http://pypi.python.org/pypi/django-localdevstorage/0.2
I have not used it yet but it sounds very useful
Document format should be in the database, a proper fix would pull it out. Current code will also fail when the document is bonkers and just does not parse as JSON.
k, seems like that might be a follow-on patch?
File size can also fail in the template so we should double check that we don't call any file size filters in any of the html files. (it looks like {something|filesize})
yep - I should have grep'd through the code to check all the other places it went wrong. I was just concerned at the time with what was breaking me.
so - now that I have some feedback, I can try to put together a proper patch set and make a merge proposal
Thanks, that sounds good!
Best regards ZK
linaro-validation@lists.linaro.org
-
Andy Doan -
Zygmunt Krynicki