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

Ignore labels when provisioning #2917

Merged
merged 2 commits into from
Feb 8, 2023
Merged

Conversation

masenf
Copy link
Collaborator

@masenf masenf commented Feb 8, 2023

Thanks for contribution

Ignore labels when tox will provision a runtime environment (.tox) so that environment configurations which depend on provisioned plugins or specific tox versions are not accessed in the outer tox process where the configuration would be invalid.

Extend existing test_provision_plugin_runner to also run using labels, which catches this regression.

Fix #2916

  • ran the linter to address style issues (tox -e fix)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

Run the provisioning environment twice, once with the explicit
environment and once passing a label (-m)
Do not attempt to set any test environments as active based on
user-supplied labels when tox will provision a new environment and
re-execute itself.

If any labeled environments depend on a future tox version or external
plugin, their configuration will be invalid and thus cannot be set
active.

fix tox-dev#2916
@masenf masenf force-pushed the provision-w-label branch from c9aa074 to 386dad4 Compare February 8, 2023 01:25
@gaborbernat gaborbernat merged commit c01bcf4 into tox-dev:main Feb 8, 2023
@masenf masenf deleted the provision-w-label branch February 8, 2023 02:20
descope bot referenced this pull request in descope/django-descope Feb 22, 2023
This PR contains the following updates:

| Package | Type | Update | Change | Pending |
|---|---|---|---|---|
| [tox](https://github.com/tox-dev/tox)
([changelog](https://tox.wiki/en/latest/changelog.html)) | dev | patch |
`4.4.4` -> `4.4.5` | `4.4.6` |

---

### Release Notes

<details>
<summary>tox-dev/tox</summary>

### [`v4.4.5`](https://github.com/tox-dev/tox/releases/tag/4.4.5)

[Compare Source](https://github.com/tox-dev/tox/compare/4.4.4...4.4.5)

#### What's Changed

- Bump deps and tools by
[@&#8203;gaborbernat](https://github.com/gaborbernat) in
[https://github.com/tox-dev/tox/pull/2913](https://github.com/tox-dev/tox/pull/2913)
- Bump deps and tools by
[@&#8203;gaborbernat](https://github.com/gaborbernat) in
[https://github.com/tox-dev/tox/pull/2914](https://github.com/tox-dev/tox/pull/2914)
- Ignore labels when provisioning by
[@&#8203;masenf](https://github.com/masenf) in
[https://github.com/tox-dev/tox/pull/2917](https://github.com/tox-dev/tox/pull/2917)

**Full Changelog**: tox-dev/tox@4.4.4...4.4.5

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC4xMDEuMCIsInVwZGF0ZWRJblZlciI6IjM0LjEwMS4wIn0=-->

Co-authored-by: descope[bot] <descope[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Labeled environments cannot use runners from provisioned plugins
2 participants