Uploaded image for project: 'Insights Experiences'
  1. Insights Experiences
  2. HMS-9551

be: nil pointer dereference when creating an upload repo

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • None
    • Content
    • None
    • insights-content
    • 3
    • None

      GlitchTip errors here: https://glitchtip.devshift.net/insights/issues/3955587

      This has happened several times when trying to create an upload repo in our UI (Repositories page) while using Playwright, happened to rhn-support-tkosciel while testing IB's integration with us. Might be related to how fast does Playwright create the repo, uploads a file and submits it.

      From the GlitchTip stack trace, the error seems to be originating from `AddContentToRepo` function used in the `add_uploads` task.
      Specifically, it looks like the `GetRpmRepositoryByName` can return nil result and error, while the caller only checks for the error being nil.

      A nil check should be added for the result being nil and it should be investigated why it actually gets to that state, where it's trying to search for a non-existing rpm repo, maybe we are missing some task await somewhere.

       

      Its likely that the upload dialog box isn't waiting for the repository to have been created properly.  It should probably show some sort of progress indicator before allowing the user to upload an rpm

              Unassigned Unassigned
              dvagner@redhat.com Dominik Vagner
              None
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: