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

Temporarily switch to dockerhub for CI jobs #1908

Merged
merged 2 commits into from
Jan 29, 2023

Conversation

vatanaksoytezer
Copy link
Contributor

CI has been failing for a while and I have a feeling that we run out of pulls or ghcr is just broken now (though I haven't find any bug reports with a quick search). This PR switches CI jobs to use dockerhub instead of ghcr, I will try to switch back in the first of Feb to see if that was a pull issue.

@codecov
Copy link

codecov bot commented Jan 29, 2023

Codecov Report

Base: 50.34% // Head: 50.34% // No change to project coverage 👍

Coverage data is based on head (30e42ec) compared to base (5838ce8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1908   +/-   ##
=======================================
  Coverage   50.34%   50.34%           
=======================================
  Files         374      374           
  Lines       31395    31395           
=======================================
  Hits        15802    15802           
  Misses      15593    15593           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@tylerjw tylerjw merged commit 39d3714 into moveit:main Jan 29, 2023
@vatanaksoytezer vatanaksoytezer deleted the vatan/dont_use_ghcr branch January 30, 2023 02:32
henningkayser added a commit to henningkayser/moveit2 that referenced this pull request Mar 9, 2023
@tylerjw tylerjw added the backport-humble Mergify label that triggers a PR backport to Humble label Mar 23, 2023
mergify bot pushed a commit that referenced this pull request Mar 23, 2023
Co-authored-by: Vatan Aksoy Tezer <[email protected]>
(cherry picked from commit 39d3714)

# Conflicts:
#	.docker/ci-testing/Dockerfile
#	.github/workflows/ci.yaml
adlarkin pushed a commit to adlarkin/moveit2 that referenced this pull request Apr 5, 2023
Co-authored-by: Vatan Aksoy Tezer <[email protected]>
(cherry picked from commit 39d3714)
adlarkin pushed a commit to adlarkin/moveit2 that referenced this pull request Apr 5, 2023
Co-authored-by: Vatan Aksoy Tezer <[email protected]>
(cherry picked from commit 39d3714)
@adlarkin adlarkin mentioned this pull request Apr 5, 2023
6 tasks
vatanaksoytezer pushed a commit that referenced this pull request Apr 5, 2023
zultron added a commit to zultron/hw_device_mgr that referenced this pull request May 15, 2023
Following upstream.  Possibly temporary.

moveit/moveit2#1908
zultron added a commit to zultron/hw_device_mgr that referenced this pull request May 15, 2023
Following upstream.  Possibly temporary.

moveit/moveit2#1908
zultron added a commit to zultron/hw_device_mgr that referenced this pull request May 15, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Remove `clang` tests & ccache config; fixes failure on Ubuntu
  Jammy, with no `clang-format-10` package; no C code here anyway

- Update pre-commit GH action tag to latest 3.0.0

[1]: moveit/moveit2#1908
zultron added a commit to zultron/hw_device_mgr that referenced this pull request May 15, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Remove `clang` tests & ccache config; fixes failure on Ubuntu
  Jammy, with no `clang-format-10` package; no C code here anyway

- Update pre-commit GH action tag to latest 3.0.0

[1]: moveit/moveit2#1908
zultron added a commit to zultron/hw_device_mgr that referenced this pull request May 15, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Remove `clang` tests & ccache config; fixes failure on Ubuntu
  Jammy, with no `clang-format-10` package; no C code here anyway

- Update pre-commit GH action tag to latest 3.0.0

[1]: moveit/moveit2#1908
zultron added a commit to zultron/hal_ros_control that referenced this pull request May 30, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Fix `clang-format` package name for Jammy

- Update pre-commit GH action tag to latest 3.0.0

[1]: moveit/moveit2#1908
zultron added a commit to zultron/hal_ros_control that referenced this pull request May 30, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Fix `clang-format` package name for Jammy

- Update pre-commit GH action tag to latest 3.0.0

- Fix Cloudsmith.io repo install

[1]: moveit/moveit2#1908
zultron added a commit to zultron/hal_ros_control that referenced this pull request May 30, 2023
- Fix CI Docker image namespace, following upstream [1] (possibly
  temporary)

- Update CI Docker image and CI config to Humble/Jammy

- Fix `clang-format` package name for Jammy

- Update pre-commit GH action tag to latest 3.0.0

- Fix Cloudsmith.io repo install

- Fix `python-attrs-pip` rosdep key while waiting on [2] and [3]

[1]: moveit/moveit2#1908
[2]: ros/rosdistro#36888
[3]: ros/rosdistro#36893
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble Mergify label that triggers a PR backport to Humble
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants