Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Update django-waffle to use clintonb's fix for django 1.8+ #706

Merged

Conversation

pomegranited
Copy link
Contributor

Uses @clintonb 's django-waffle==0.21.0 to fix errors when using waffle_switch and waffle_flag management commands.

JIRA tickets: OSPR

Discussions: cf jazzband/django-waffle#244, jsocol/django-waffle@v0.12.0

Partner information: 3rd party hosted

Deployment targets: insights.edx.org, insights.stage.edx.org

Merge deadline: open-releae/ginkgo.rc1, if possible cc @nedbat

Testing instructions:

  1. Deploy Insights on an analytics devstack using the current master.
  2. Try listing the waffle flags:
    sudo su insights
    ./manage.py waffle_switch --list --settings=analytics_dashboard.settings.production
    
    Note AttributeError: type object 'BaseCommand' has no attribute 'option_list'
  3. Checkout the branch from this PR: jill/update-django-waffle
  4. Install the pip requirements to upgrade django-waffle:
    make requirements.py
    
  5. Re-run the waffle_switch command from step 2, and note success.

Reviewers

  • OpenCraft reviewer TBD
  • edX reviewers TBD - @edx/analytics CC @thallada

@openedx-webhooks
Copy link

Thanks for the pull request, @pomegranited! It looks like you're a member of a company that does contract work for edX. If you're doing this work as part of a paid contract with edX, you should talk to edX about who will review this pull request. If this work is not part of a paid contract with edX, then you should ensure that there is an OSPR issue to track this work in JIRA, so that we don't lose track of your pull request.

Create an OSPR issue for this pull request.

@codecov-io
Copy link

codecov-io commented Jul 16, 2017

Codecov Report

Merging #706 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #706   +/-   ##
=======================================
  Coverage   94.85%   94.85%           
=======================================
  Files          73       73           
  Lines        5775     5775           
  Branches      490      490           
=======================================
  Hits         5478     5478           
  Misses        219      219           
  Partials       78       78

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f20a60...7702ffd. Read the comment docs.

@openedx-webhooks
Copy link

Thanks for the pull request, @pomegranited! I've created OSPR-1826 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.

Feel free to add as much of the following information to the ticket:

  • supporting documentation
  • edx-code email threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

If you like, you can add yourself to the AUTHORS file for this repo, though that isn't required. Please see the CONTRIBUTING file for more information.

@openedx-webhooks openedx-webhooks added needs triage open-source-contribution PR author is not from Axim or 2U labels Jul 16, 2017
Copy link
Contributor

@clintonb clintonb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but someone who actually manages/deploys this application should also sign off.

Copy link
Contributor

@thallada thallada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @pomegranited ! This looks good to me too 👍

@thallada
Copy link
Contributor

I'm going to merge this and then make a PR to cherry-pick this change into the ginkgo release branch.

@thallada thallada merged commit cfdbb39 into openedx-unsupported:master Jul 17, 2017
@pomegranited
Copy link
Contributor Author

pomegranited commented Jul 17, 2017

Thank you @thallada and @clintonb !

@pomegranited pomegranited deleted the jill/update-django-waffle branch July 17, 2017 15:34
nedbat added a commit that referenced this pull request Jul 31, 2017
Ginkgo: Update django-waffle to use clintonb's fix for django 1.8+ (#706)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs triage open-source-contribution PR author is not from Axim or 2U
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants