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

[ENG-6094] Poetry won't install deps properly in when venv is inside project root #10697

Merged

Conversation

opaduchak
Copy link
Contributor

@opaduchak opaduchak commented Aug 8, 2024

Purpose

Fix devenv setup for people who use virtualenv inside project root in conjuction with docker-compose

Changes

Changed mount for docker-compose requirements service

QA Notes

Please make verification statements inspired by your code and what your code touches.

  • Verify
  • Verify

What are the areas of risk?

Any concerns/considerations/questions that development raised?

Documentation

Side Effects

Ticket

ENG-6094

@brianjgeiger brianjgeiger merged commit 99d8b70 into CenterForOpenScience:develop Aug 8, 2024
6 checks passed
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 12, 2024
 into fix-source-tags

* 'develop' of https://github.com/CenterForOpenScience/osf.io:
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 13, 2024
 into add-more-spam-metrics

* 'develop' of https://github.com/CenterForOpenScience/osf.io: (173 commits)
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
  update dataverse dep revision to get changes
  Update CHANGELOG, bump version
  [CR][ENG-5681] Great Big Python Upgrade (CenterForOpenScience#10648)
  Revert "[ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)" (CenterForOpenScience#10666)
  Check Registration READ perms on the Registration - Do not record download metrics for renders
  Fix signature
  Allow DOI metadata updates to be queued
  [ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)
  Update CHANGELOG, bump version
  [ENG-5030] Preprints Phase 2 - BE (CenterForOpenScience#10617)
  Update CHANGELOG, bump version
  Ensure Assumed-HAM users do not get autobanned
  [ENG-5762] Get GV set up in osf docker configs (CenterForOpenScience#10643)
  ...
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 13, 2024
 into clean-up-preprint-permissions

* 'develop' of https://github.com/CenterForOpenScience/osf.io:
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 21, 2024
 into remove-quickfiles-code

* 'develop' of https://github.com/CenterForOpenScience/osf.io:
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 21, 2024
 into remove-quickfiles-code

* 'develop' of https://github.com/CenterForOpenScience/osf.io:
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 22, 2024
 into institutional-dashboard

* 'develop' of https://github.com/CenterForOpenScience/osf.io: (41 commits)
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
  update dataverse dep revision to get changes
  Update CHANGELOG, bump version
  [CR][ENG-5681] Great Big Python Upgrade (CenterForOpenScience#10648)
  Revert "[ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)" (CenterForOpenScience#10666)
  Check Registration READ perms on the Registration - Do not record download metrics for renders
  Fix signature
  Allow DOI metadata updates to be queued
  [ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)
  Update CHANGELOG, bump version
  [ENG-5030] Preprints Phase 2 - BE (CenterForOpenScience#10617)
  Update CHANGELOG, bump version
  Ensure Assumed-HAM users do not get autobanned
  [ENG-5762] Get GV set up in osf docker configs (CenterForOpenScience#10643)
  ...

# Conflicts:
#	api/users/serializers.py
#	conftest.py
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 22, 2024
 into institutional-dashboard

* 'develop' of https://github.com/CenterForOpenScience/osf.io: (41 commits)
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
  update dataverse dep revision to get changes
  Update CHANGELOG, bump version
  [CR][ENG-5681] Great Big Python Upgrade (CenterForOpenScience#10648)
  Revert "[ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)" (CenterForOpenScience#10666)
  Check Registration READ perms on the Registration - Do not record download metrics for renders
  Fix signature
  Allow DOI metadata updates to be queued
  [ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)
  Update CHANGELOG, bump version
  [ENG-5030] Preprints Phase 2 - BE (CenterForOpenScience#10617)
  Update CHANGELOG, bump version
  Ensure Assumed-HAM users do not get autobanned
  [ENG-5762] Get GV set up in osf docker configs (CenterForOpenScience#10643)
  ...

# Conflicts:
#	api/users/serializers.py
#	conftest.py
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 22, 2024
 into institutional-dashboard

* 'develop' of https://github.com/CenterForOpenScience/osf.io: (41 commits)
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
  update dataverse dep revision to get changes
  Update CHANGELOG, bump version
  [CR][ENG-5681] Great Big Python Upgrade (CenterForOpenScience#10648)
  Revert "[ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)" (CenterForOpenScience#10666)
  Check Registration READ perms on the Registration - Do not record download metrics for renders
  Fix signature
  Allow DOI metadata updates to be queued
  [ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)
  Update CHANGELOG, bump version
  [ENG-5030] Preprints Phase 2 - BE (CenterForOpenScience#10617)
  Update CHANGELOG, bump version
  Ensure Assumed-HAM users do not get autobanned
  [ENG-5762] Get GV set up in osf docker configs (CenterForOpenScience#10643)
  ...

# Conflicts:
#	api/users/serializers.py
#	conftest.py
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 22, 2024
 into institutional-dashboard

* 'develop' of https://github.com/CenterForOpenScience/osf.io: (41 commits)
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
  update dataverse dep revision to get changes
  Update CHANGELOG, bump version
  [CR][ENG-5681] Great Big Python Upgrade (CenterForOpenScience#10648)
  Revert "[ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)" (CenterForOpenScience#10666)
  Check Registration READ perms on the Registration - Do not record download metrics for renders
  Fix signature
  Allow DOI metadata updates to be queued
  [ENG-3685] Add permissions for withdrawn registration files (CenterForOpenScience#10650)
  Update CHANGELOG, bump version
  [ENG-5030] Preprints Phase 2 - BE (CenterForOpenScience#10617)
  Update CHANGELOG, bump version
  Ensure Assumed-HAM users do not get autobanned
  [ENG-5762] Get GV set up in osf docker configs (CenterForOpenScience#10643)
  ...

# Conflicts:
#	api/users/serializers.py
#	conftest.py
Johnetordoff pushed a commit to Johnetordoff/osf.io that referenced this pull request Aug 23, 2024
 into remove-quickfiles-code

* 'develop' of https://github.com/CenterForOpenScience/osf.io:
  Hotfixes for: double-encoded urls + dep upgrades
  fixed docker-compose requirements for local setups with venv inside project root (CenterForOpenScience#10697)
  use correct version of module for metrics
  update Dockerfile for recent lints
  remove deprecated version stanza from docker-composes
  update reqs to pull from github commit shas
  correct body location for api.ia POST requests
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.

2 participants