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.

Sign up for GitHub

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

Jump to bottom

Unable to install avocado-framework-plugin-varianter-yaml-to-mux:102 cleanly #5713

Closed
AYUSHJAIN951 opened this issue Jun 27, 2023 · 4 comments · Fixed by #5778
Closed

Unable to install avocado-framework-plugin-varianter-yaml-to-mux:102 cleanly #5713

AYUSHJAIN951 opened this issue Jun 27, 2023 · 4 comments · Fixed by #5778
Assignees
@richtja
Labels
bug
Milestone
#103 (LTS release...

Comments

@AYUSHJAIN951
Copy link
Contributor

While upgrading 101.0 ->102.0, avocado-framework-plugin-varianter-yaml-to-mux seems to be having version issues

 git clone https://github.com/open-power-host-os/tests.git
 cd tests
 sed -i "s/work', ''/work', 'git+https:\/\/github.com\/avocado-framework\/avocado.git'/g" config/wrapper/env.conf
 head -15 config/wrapper/env.conf
 ./avocado-setup.py --bootstrap --install-deps

Ideally it should be installing avocado and avocado-framework-plugin-varianter-yaml-to-mux with 102.0 release

But currently it seems pip is not able to fetch and install avocado-framework-plugin-varianter-yaml-to-mux:102.0 and instead installing 101.0 .

root@AMDtesthost_Ubuntu:~/tests# avocado -v
Traceback (most recent call last):
  File "/usr/local/bin/avocado", line 5, in <module>
    from avocado.core.main import main
  File "/usr/local/lib/python3.10/dist-packages/avocado/__init__.py", line 35, in <module>
    initialize_plugins()
  File "/usr/local/lib/python3.10/dist-packages/avocado/core/__init__.py", line 290, in initialize_plugins
    InitDispatcher().map_method("initialize")
  File "/usr/local/lib/python3.10/dist-packages/avocado/core/dispatcher.py", line 207, in __init__
    super().__init__("avocado.plugins.init")
  File "/usr/local/lib/python3.10/dist-packages/avocado/core/enabled_extension_manager.py", line 25, in __init__
    super().__init__(namespace, invoke_kwds)
  File "/usr/local/lib/python3.10/dist-packages/avocado/core/extension_manager.py", line 94, in __init__
    plugin = ep.load()
  File "/usr/local/lib/python3.10/dist-packages/pkg_resources/__init__.py", line 2517, in load
    self.require(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/pkg_resources/__init__.py", line 2540, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/usr/local/lib/python3.10/dist-packages/pkg_resources/__init__.py", line 829, in resolve
    dist = self._resolve_dist(
  File "/usr/local/lib/python3.10/dist-packages/pkg_resources/__init__.py", line 875, in _resolve_dist
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.VersionConflict: (avocado-framework 102.0 (/usr/local/lib/python3.10/dist-packages), Requirement.parse('avocado-framework==101.0'))

2023-06-27 06:27:13 DEBUG   : [helper.py:49] Running pip3 install -U avocado-framework-plugin-varianter-yaml-to-mux
2023-06-27 06:27:14 DEBUG   : [helper.py:55] Collecting avocado-framework-plugin-varianter-yaml-to-mux
  Downloading avocado_framework_plugin_varianter_yaml_to_mux-101.0-py3-none-any.whl (10 kB)
Collecting avocado-framework==101.0 (from avocado-framework-plugin-varianter-yaml-to-mux)
  Downloading avocado_framework-101.0-py3-none-any.whl (430 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 430.7/430.7 kB 3.0 MB/s eta 0:00:00
Requirement already satisfied: PyYAML>=4.2b2 in /usr/lib/python3/dist-packages (from avocado-framework-plugin-varianter-yaml-to-mux) (5.4.1)
Requirement already satisfied: setuptools in /usr/local/lib/python3.10/dist-packages (from avocado-framework==101.0->avocado-framework-plugin-varianter-yaml-to-mux) (68.0.0)
Installing collected packages: avocado-framework, avocado-framework-plugin-varianter-yaml-to-mux
  Attempting uninstall: avocado-framework
    Found existing installation: avocado-framework 102.0
    Uninstalling avocado-framework-102.0:
      Successfully uninstalled avocado-framework-102.0
Successfully installed avocado-framework-101.0 avocado-framework-plugin-varianter-yaml-to-mux-101.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
2023-06-27 06:27:14 DEBUG   : [helper.py:47] Installing python package avocado-framework-plugin-result-html using pip
2023-06-27 06:27:14 DEBUG   : [helper.py:49] Running pip3 install -U avocado-framework-plugin-result-html
2023-06-27 06:27:15 DEBUG   : [helper.py:55] Collecting avocado-framework-plugin-result-html
  Downloading avocado_framework_plugin_result_html-102.0-py3-none-any.whl (96 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 96.9/96.9 kB 1.4 MB/s eta 0:00:00
Collecting avocado-framework==102.0 (from avocado-framework-plugin-result-html)
  Using cached avocado_framework-102.0-py3-none-any.whl (439 kB)
Requirement already satisfied: jinja2 in /usr/lib/python3/dist-packages (from avocado-framework-plugin-result-html) (3.0.3)
Requirement already satisfied: setuptools in /usr/local/lib/python3.10/dist-packages (from avocado-framework==102.0->avocado-framework-plugin-result-html) (68.0.0)
Installing collected packages: avocado-framework, avocado-framework-plugin-result-html
  Attempting uninstall: avocado-framework
    Found existing installation: avocado-framework 101.0
    Uninstalling avocado-framework-101.0:
      Successfully uninstalled avocado-framework-101.0
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
avocado-framework-plugin-varianter-yaml-to-mux 101.0 requires avocado-framework==101.0, but you have avocado-framework 102.0 which is incompatible.
Successfully installed avocado-framework-102.0 avocado-framework-plugin-result-html-102.0

root@AMDtesthost_Ubuntu:~/tests# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04 LTS
Release:        22.04
Codename:       jammy

Installation Method : pip
@mr-avocado mr-avocado bot moved this to Triage in Default project Jun 27, 2023
@AYUSHJAIN951
Copy link
Contributor Author

https://pypi.org/project/avocado-framework-plugin-varianter-yaml-to-mux/#history
I am unable to see 102.0 release of the following plugin
@clebergnu Can you please look at this issue.

@clebergnu
Copy link
Contributor

https://pypi.org/project/avocado-framework-plugin-varianter-yaml-to-mux/#history I am unable to see 102.0 release of the following plugin @clebergnu Can you please look at this issue.

Hi @AYUSHJAIN951 ,

I can confirm the issue you are reporting is valid. The cause of the problem is that we moved the documentation to the plugin (in hope of giving them a better description). But in the specific case of the yaml-to-mux plugin, there's directives that are Sphinx-only, and not pure ResTructuredText (rst).

We'll add checks so that it doesn't happen anymore, probably something like:

$ twine check avocado_framework_plugin_varianter_yaml_to_mux-102.0-py3-none-any.whl 
Checking avocado_framework_plugin_varianter_yaml_to_mux-102.0-py3-none-any.whl: FAILED
ERROR    `long_description` has syntax errors in markup and would not be rendered on PyPI.                                                                      
         line 6: Error: Unknown interpreted text role "mod".

I'll check if there's anything we can do to mitigate this to version 102.0 without requiring a new release.

@clebergnu
Copy link
Contributor

Hi @AYUSHJAIN951 ,

I published a version of the plugins for 102.0 with just the long description modified. You can find it here. The installation should now proceed as expected.

I won't close this issue for now, because like I said before, we have to make sure this doesn't happen again.

Thanks!

@clebergnu clebergnu self-assigned this Jul 7, 2023
@AYUSHJAIN951
Copy link
Contributor Author

Hi @AYUSHJAIN951 ,

I published a version of the plugins for 102.0 with just the long description modified. You can find it here. The installation should now proceed as expected.

Thanks @clebergnu for the quick fix.

I won't close this issue for now, because like I said before, we have to make sure this doesn't happen again.

Sure let's keep the issue open for now until its having a proper fix.

Thanks!

Thanks!!

@clebergnu clebergnu moved this from Triage to Short Term (Current Q) Backlog in Default project Jul 12, 2023
@clebergnu clebergnu assigned richtja and unassigned clebergnu Oct 5, 2023
richtja added a commit to richtja/avocado that referenced this issue Oct 10, 2023
@richtja
Remove sphinx related tags from varianter description
11f1bf5 
With this small change, all the special tags will be removed from
yaml_to_mux varianter README file which is used for long description of
plugin. We have to do this because systems like pypi doesn't support
sphinx only basic ResTructuredText.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 10, 2023
@richtja
twine check in release process
1dadfdd 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
@richtja richtja mentioned this issue Oct 10, 2023
Merged
@richtja richtja linked a pull request Oct 10, 2023 that will close this issue
Twine check for release #5778
Merged
@richtja richtja moved this from Short Term (Current Q) Backlog to In progress in Default project Oct 11, 2023
richtja added a commit to richtja/avocado that referenced this issue Oct 12, 2023
@richtja
twine check in release process
9e46c2a 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 12, 2023
@richtja
Add long description to ansible plugin
7f72cf9 
It copies short description to the long description of ansible plugin,
because we don't have detailed description, but it is enforced by twine
check.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 12, 2023
@richtja
Remove sphinx related tags from varianters description
86a6f3e 
With this small change, all the special tags will be removed from
yaml_to_mux and pict varianters README files which is used for long
description of plugins. We have to do this because systems like pypi
doesn't support sphinx, only basic ResTructuredText.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 12, 2023
@richtja
twine check in release process
6d09ab4 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 13, 2023
@richtja
twine check in release process
7097a77 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 13, 2023
@richtja
twine check in release process
e85805d 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 16, 2023
@richtja
twine check in release process
09d9a36 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 16, 2023
@richtja
twine check in release process
9c652b8 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
richtja added a commit to richtja/avocado that referenced this issue Oct 16, 2023
@richtja
twine check in release process
c0286cc 
This commit adds twine check for each package, before publishing it.
With this change, we will avoid problems with publishing the packages in
the future.

Reference: avocado-framework#5713
Signed-off-by: Jan Richter <[email protected]>
@github-project-automation github-project-automation bot moved this from In progress to Done 103 in Default project Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@richtja richtja

Labels
bug
Projects
Default project
Archived in project
Milestone
#103 (LTS release - Sound of Freedom)
Development

Successfully merging a pull request may close this issue.

Twine check for release richtja/avocado
3 participants
@clebergnu @richtja @AYUSHJAIN951