Skip to content
Snippets Groups Projects

Resolve "Change default docker registry"

Merged Jon Azpiazu requested to merge 27-change-default-docker-registry into master
All threads resolved!

Closes #27 (closed)

Edited by Jon Azpiazu

Merge request reports

Pipeline #39744 passed

Pipeline passed for 5e88ed01 on 27-change-default-docker-registry

Merged by Prada Sarasola, MiguelPrada Sarasola, Miguel 4 years ago (Jan 7, 2021 4:42pm UTC)

Loading

Pipeline #39748 passed

Pipeline passed for a721e972 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Prada Sarasola, Miguel
  • Jon Azpiazu added 1 commit

    added 1 commit

    • 12ef275f - Revert explicit docker registry address

    Compare with previous version

  • Jon Azpiazu added 1 commit

    added 1 commit

    • 58ba3c95 - Add login into artifactory registry from non-rosinstall template

    Compare with previous version

  • Jon Azpiazu resolved all threads

    resolved all threads

  • Test pipeline https://git.code.tecnalia.com/tecnalia_robotics/flexbotics/vision/vision_libraries_external_api/-/pipelines/39738

    However without the ability to look into the runner's logs, it is not possible to assure which registry is being used to fetch the images from.

    A solution (apart form asking Aitzol) might be to configure our runner in roboticsrunner to use that same registry (and authenticate).

    Another solution might be to just merge this MR, and monitorize if another error due to Docker's limitation on the number of pulls happens again.

  • Jon Azpiazu added 1 commit

    added 1 commit

    • 5e88ed01 - Add tag to job - collaterally fix yaml syntax

    Compare with previous version

  • Another solution might be to just merge this MR, and monitorize if another error due to Docker's limitation on the number of pulls happens again.

    Sounds good enough to me :rofl:

  • mentioned in commit a721e972

  • Please register or sign in to reply
    Loading