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

Unpin labgrid's dependencies #1136

Merged
merged 3 commits into from
Apr 27, 2023

Conversation

Bastian-Krause
Copy link
Member

@Bastian-Krause Bastian-Krause commented Apr 18, 2023

Description
With #1079, crossbar is no longer installed in the same virtualenv as labgrid and its dependencies. We can now loosen labgrid's dependencies.

Use the known-good previously pinned versions as minimum versions.

We cannot unpin crossbar's version because of continuing issues: dependencies being incompatible with Python >= 3.11, PyPI packages depending on non-PyPI packages etc.

Based on #1079.

Checklist

  • PR has been tested (CI only)
  • Man pages have been regenerated

Closes #1153

@Bastian-Krause Bastian-Krause added this to the v23.1 milestone Apr 18, 2023
@Bastian-Krause Bastian-Krause added the dependencies Pull requests that update a dependency file label Apr 18, 2023
@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.4 ⚠️

Comparison is base (7e4dea3) 63.2% compared to head (e881ef6) 62.8%.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #1136     +/-   ##
========================================
- Coverage    63.2%   62.8%   -0.4%     
========================================
  Files         156     156             
  Lines       11571   11571             
========================================
- Hits         7315    7271     -44     
- Misses       4256    4300     +44     

see 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

jluebbe
jluebbe previously approved these changes Apr 26, 2023
@Bastian-Krause
Copy link
Member Author

I'll rebase and undraft this once #1152 is merged.

pyproject.toml Outdated Show resolved Hide resolved
Now that crossbar is no longer installed in the same virtualenv as
labgrid and its dependencies is, we can loosen labgrid's dependencies.

Use the known-good previously pinned versions as minimum versions. For
pytest, relax the minimum version to the latest major version jump.

While at it, let sphinx_rtd_theme specify its dependencies sphinx and
docutils by itself.

Signed-off-by: Bastian Krause <[email protected]>
@Bastian-Krause Bastian-Krause force-pushed the bst/unpin-dependencies branch from d4aae18 to e881ef6 Compare April 27, 2023 07:45
@Emantor Emantor merged commit a8e4831 into labgrid-project:master Apr 27, 2023
@Bastian-Krause Bastian-Krause deleted the bst/unpin-dependencies branch April 28, 2023 10:22
Bastian-Krause added a commit to Bastian-Krause/labgrid that referenced this pull request Jun 22, 2023
Now that labgrid-project#1079 and labgrid-project#1136 are merged, the coordinator no longer supports
ticket authentication. Drop the corresponding authenticator class, do
not advertise ticket auth anymore.

Exporters and clients might still talk to older coordinator versions, so
leave ticket authentication for these components enabled.

Signed-off-by: Bastian Krause <[email protected]>
Bastian-Krause added a commit to Bastian-Krause/labgrid that referenced this pull request Jun 22, 2023
Now that labgrid-project#1079 and labgrid-project#1136 are merged, the coordinator no longer supports
ticket authentication. Drop the corresponding authenticator class, do
not advertise ticket auth anymore.

Exporters and clients might still talk to older coordinator versions, so
leave ticket authentication for these components enabled.

Signed-off-by: Bastian Krause <[email protected]>
userid0x0 pushed a commit to userid0x0/labgrid that referenced this pull request Sep 8, 2023
Now that labgrid-project#1079 and labgrid-project#1136 are merged, the coordinator no longer supports
ticket authentication. Drop the corresponding authenticator class, do
not advertise ticket auth anymore.

Exporters and clients might still talk to older coordinator versions, so
leave ticket authentication for these components enabled.

Signed-off-by: Bastian Krause <[email protected]>
userid0x0 pushed a commit to userid0x0/labgrid that referenced this pull request Sep 8, 2023
Now that labgrid-project#1079 and labgrid-project#1136 are merged, the coordinator no longer supports
ticket authentication. Drop the corresponding authenticator class, do
not advertise ticket auth anymore.

Exporters and clients might still talk to older coordinator versions, so
leave ticket authentication for these components enabled.

Signed-off-by: Bastian Krause <[email protected]>
rpoisel pushed a commit to honeytreelabs/labgrid that referenced this pull request Oct 24, 2023
Now that labgrid-project#1079 and labgrid-project#1136 are merged, the coordinator no longer supports
ticket authentication. Drop the corresponding authenticator class, do
not advertise ticket auth anymore.

Exporters and clients might still talk to older coordinator versions, so
leave ticket authentication for these components enabled.

Signed-off-by: Bastian Krause <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants