Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2658

Fix App Store search by configuring playbooks

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23

      Description

      Searching generates an error.

      To reproduce the bug:

      • Go to a deployed app store (e.g., https://devappstore5.bioviz.org)
      • Enter search term (e.g., "bioinformatics")
      • Type ENTER to run the search
      • Observe there is an internal server error generated

        Attachments

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-2563 [ 18746 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20 [ 108 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 23 - Dec 4 [ 108, 109 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Description Searching generates an error.

          To reproduce the bug:

          * Go to a deployed app store (e.g., https://devappstore5.biovizi.org)
          * Enter search term (e.g., "bioinformatics")
          * Type ENTER to run the search
          * Observe there is an internal server error generated
          Searching generates an error.

          To reproduce the bug:

          * Go to a deployed app store (e.g., https://devappstore5.bioviz.org)
          * Enter search term (e.g., "bioinformatics")
          * Type ENTER to run the search
          * Observe there is an internal server error generated
          Hide
          ann.loraine Ann Loraine added a comment -

          Noted by Sameer:

          I guess there is a write problem, the error as follows:

          • OSError: The path to your Whoosh index '/home/ubuntu/appstore/appstore/whoosh_index' is not writable for the current user/group.

          our search uses whoosh_index and that directory is not writable

          and

          It works when I change the permissions manually, we need to do that in the playbook.

          Show
          ann.loraine Ann Loraine added a comment - Noted by Sameer: I guess there is a write problem, the error as follows: OSError: The path to your Whoosh index '/home/ubuntu/appstore/appstore/whoosh_index' is not writable for the current user/group. our search uses whoosh_index and that directory is not writable and It works when I change the permissions manually, we need to do that in the playbook.
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          To fix:

          Modify the appstore playbooks (https://bitbucket.org/lorainelab/appstore-playbooksto) include a step that changes the group affiliation of the whoosh_index directory to "www-data".
          During deployment, the cloned appstore files will all have user "ubuntu" and group "ubuntu," which is required for future updating of the clone.
          The apache process (httpd) is owned by user and group www-data.
          The ubuntu user is added to the group "www-data".
          So if the whoosh index directory has group "www-data," then apache/httpd will be able to write to the directory and properly update the index when users submit apps or when the index is first used.

          Note: The test site mentioned above has been properly configured manually. So this is why you will probably not observe the bug in action. Also, you will probably be unlikely to observe the bug on your local because local development deployments happen under the developers user id.

          Show
          ann.loraine Ann Loraine added a comment - - edited To fix: Modify the appstore playbooks ( https://bitbucket.org/lorainelab/appstore-playbooksto ) include a step that changes the group affiliation of the whoosh_index directory to "www-data". During deployment, the cloned appstore files will all have user "ubuntu" and group "ubuntu," which is required for future updating of the clone. The apache process (httpd) is owned by user and group www-data. The ubuntu user is added to the group "www-data". So if the whoosh index directory has group "www-data," then apache/httpd will be able to write to the directory and properly update the index when users submit apps or when the index is first used. Note: The test site mentioned above has been properly configured manually. So this is why you will probably not observe the bug in action. Also, you will probably be unlikely to observe the bug on your local because local development deployments happen under the developers user id.
          ann.loraine Ann Loraine made changes -
          Summary Fix App Store search Fix App Store search by configuring playbooks
          ann.loraine Ann Loraine made changes -
          Story Points 0.25 0.5
          cdias1 Chester Dias (Inactive) made changes -
          Assignee Chester Dias [ cdias1 ]
          cdias1 Chester Dias (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          cdias1 Chester Dias (Inactive) added a comment -

          Looks like that folder is getting generated as part of migration steps in the playbooks or later. Not as a part of deployment. I am changing the whole /home/ubuntu/appstore/appstore/ affiliation to ubuntu:www-data
          that should work too i think

          Show
          cdias1 Chester Dias (Inactive) added a comment - Looks like that folder is getting generated as part of migration steps in the playbooks or later. Not as a part of deployment. I am changing the whole /home/ubuntu/appstore/appstore/ affiliation to ubuntu:www-data that should work too i think
          Show
          cdias1 Chester Dias (Inactive) added a comment - Please Review : https://bitbucket.org/chesterdias/appstore-playbooks/commits/345e31a5b3acf53a4b830d40cb66cdadc85d0530
          cdias1 Chester Dias (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          cdias1 Chester Dias (Inactive) made changes -
          Assignee Chester Dias [ cdias1 ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11 [ 108, 109 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23 [ 108, 109, 110 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ann.loraine Ann Loraine added a comment -

          Please rebase on the latest master branch and resolve any conflicts that emerge.

          Show
          ann.loraine Ann Loraine added a comment - Please rebase on the latest master branch and resolve any conflicts that emerge.
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Chester Dias [ cdias1 ]
          ann.loraine Ann Loraine made changes -
          Assignee Chester Dias [ cdias1 ] Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Fixed

          Show
          ann.loraine Ann Loraine added a comment - Fixed
          ann.loraine Ann Loraine made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          ann.loraine Ann Loraine made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          ann.loraine Ann Loraine made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ann.loraine Ann Loraine made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

            People

            • Assignee:
              ann.loraine Ann Loraine
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: