Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stream fixes #99

Closed
wants to merge 35 commits into from
Closed

Conversation

stevengriffin
Copy link

Status

READY/IN DEVELOPMENT/HOLD

Description

A few sentences describing the overall goals of the pull request's commits.

Related PRs

List related PRs against other branches:

branch PR
other_pr_production link
other_pr_master link

TO-DOs

  • Tests
  • Documentation

Deploy Notes

Notes regarding deployment the contained body of work. These should note any
db migrations, etc.

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

git pull --prune
git checkout <feature_branch>
make test

Impacted Areas in Application

List general components of the application that this PR will affect:

groboclown and others added 30 commits September 8, 2020 16:59
…ty-list checks.

The Red Hat security updates are deprecated, and the old source location now contains just an empty directory tree.  This caused Dagda to attempt to push empty lists into the MongoDB, which generated eliasgranderubio#85.  This fix both adds protections on the MongoDB write by not writing empty lists, and by changing the download URL to one that works.  Note that this is a temporary fix, because the Red Hat update indicates that this is still a deprecated source.  The v2 should be used instead.
…ython libraries. Added logic for zip (already had logic for tar). The zip logis does not yet have a corresponding api-server unit test.
…s requests to use archived version as the site was restructured, causing bug tracking info to no longer be correctly fetched.
@stevengriffin
Copy link
Author

Mistaken PR that was meant for fork, please ignore/delete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants