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

Automatic update is broken #65

Closed
1 task done
traversaro opened this issue Aug 24, 2024 · 10 comments
Closed
1 task done

Automatic update is broken #65

traversaro opened this issue Aug 24, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@traversaro
Copy link
Contributor

Solution to issue cannot be found in the documentation.

  • I checked the documentation.

Issue

No idea why.

Installed packages

.

Environment info

.
@traversaro traversaro added the bug Something isn't working label Aug 24, 2024
@traversaro
Copy link
Contributor Author

traversaro commented Aug 24, 2024

Triggering an update with conda-forge/gz-sensors-feedstock#37 in gave us an error in https://github.com/conda-forge/gz-sensors-feedstock/actions/runs/10538400204/job/29200871549 :

2024-08-24 12:17:28,030 INFO     webservices_dispatch_action.version_updater || getting latest version
2024-08-24 12:17:41,638 INFO     webservices_dispatch_action.version_updater || curr version|latest version: 8.1.0|8.2.0
2024-08-24 12:17:41,658 INFO     conda_forge_tick.update_recipe.version || selector: None
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || jinja2 context: {'component_name': 'sensors', 'version': '8.2.0'}
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || evaluated jinja2 vars: {'repo_name': 'gz-sensors', 'major_version': '8.2.0', 'version_package': '', 'name': 'gz-sensors8.2.0', 'component_version': 'sensors8.2.0', 'cxx_name': 'libgz-sensors8.2.0'}
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || updated jinja2 context: {'component_name': 'sensors',
 'component_version': 'sensors8.2.0',
 'cxx_name': 'libgz-sensors8.2.0',
 'major_version': '8.2.0',
 'name': 'gz-sensors8.2.0',
 'repo_name': 'gz-sensors',
 'version': '8.2.0',
 'version_package': ''}
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || url key: url
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || hash key: sha256
2024-08-24 12:17:41,663 INFO     conda_forge_tick.update_recipe.version || hashing URL template: [https://github.com/gazebosim/{{](https://github.com/gazebosim/%7B%7B) repo_name }}/archive/{{ repo_name }}{{ version }}.tar.gz
2024-08-24 12:17:41,664 INFO     conda_forge_tick.update_recipe.version || rendered URL: https://github.com/gazebosim/gz-sensors/archive/gz-sensors8.2.0.tar.gz
2024-08-24 12:17:43,910 INFO     conda_forge_tick.update_recipe.version || new URL hash: None
2024-08-24 12:17:43,911 INFO     conda_forge_tick.update_recipe.version || updated|errors: False|{"could not hash URL template '[https://github.com/gazebosim/{{](https://github.com/gazebosim/%7B%7B) repo_name }}/archive/{{ repo_name }}{{ version }}.tar.gz'"}
2024-08-24 12:17:43,911 CRITICAL conda_forge_tick.update_recipe.version || {"could not hash URL template '[https://github.com/gazebosim/{{](https://github.com/gazebosim/%7B%7B) repo_name }}/archive/{{ repo_name }}{{ version }}.tar.gz'"}
2024-08-24 12:17:43,912 CRITICAL conda_forge_tick.update_recipe.version || Recipe did not change in version migration!
2024-08-24 12:17:43,912 CRITICAL webservices_dispatch_action.version_updater || errors when updating the recipe: {"could not hash URL template '[https://github.com/gazebosim/{{](https://github.com/gazebosim/%7B%7B) repo_name }}/archive/{{ repo_name }}{{ version }}.tar.gz'"}
2024-08-24 12:17:43,912 ERROR    webservices_dispatch_action.version_updater || error while updating the recipe!
Traceback (most recent call last):
  File "/webservices_dispatch_action/webservices_dispatch_action/version_updater.py", line 97, in update_version
    raise RuntimeError("Error updating the recipe!")
RuntimeError: Error updating the recipe!
INFO:webservices_dispatch_action.utils:token can change workflows: False
INFO:webservices_dispatch_action.utils:pushing and commenting: branch|owner|repo = conda_forge_admin_37|conda-forge-admin|gz-sensors-feedstock
Traceback (most recent call last):
  File "/opt/conda/bin/run-webservices-dispatch-action", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/webservices_dispatch_action/webservices_dispatch_action/__main__.py", line 175, in main
    raise RuntimeError(
RuntimeError: Updating version failed! error in push|version update: False|True

@traversaro
Copy link
Contributor Author

No idea why, but apparently to fix this we need to align the recipe style with gz-math: conda-forge/gz-sensors-feedstock@5bbc306 .

traversaro added a commit to traversaro/gz-common-feedstock that referenced this issue Aug 24, 2024
The gz-math recipe style works fine with the autotick-bot: conda-forge/gz-sim-feedstock#65 (comment) .
@traversaro
Copy link
Contributor Author

We need to review and align all repos in https://github.com/orgs/conda-forge/repositories?q=gz- .

@Tobias-Fischer
Copy link
Contributor

It looks like all of these have been fixed @traversaro?

@traversaro
Copy link
Contributor Author

To be honest, I do not know. We should check if a autotick-bot PR was opened in one of this repos after the cleanup PRs.

@Tobias-Fischer
Copy link
Contributor

Nope .. there is gz-physics 7.3.0 but there was no autotick-bot :(

@traversaro
Copy link
Contributor Author

Nope .. there is gz-physics 7.3.0 but there was no autotick-bot :(

Yes, interesting. There is no error in versions error in conda-forge status, and if you check the bot-versions output it outputs (see https://github.com/regro/cf-scripts/actions/runs/11007210867/job/30562973811):

2024-09-24T05:29:51.8730580Z 2024-09-24 05:29:51,872 INFO     conda_forge_tick.update_upstream_versions || itr  2236 - eta  2226s: gz-physics - 7.2.0 -> 7.3.0

so the correct version is detected, but somehow the PR is not opened.

@traversaro
Copy link
Contributor Author

Opened an upstream issue at regro/cf-scripts#3010 .

@traversaro
Copy link
Contributor Author

Ok, somehow all the recent release for Ionic unblocked the situation. The only missing repo is gz-sim, that will be fixed once #64 is merged.

@traversaro
Copy link
Contributor Author

The only missing repo is gz-sim, that will be fixed once #64 is merged.

Actually it is now fixed, see #67 . Let's close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants