Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#1391 closed defect (fixed)

Resolve HG and Git names in ensure_dependencies.py

Reported by: matze Assignee: trev
Priority: P4 Milestone:
Module: Automation Keywords: infrastructure hg git
Cc: trev, fhd, AAlvz, poz2k4444 Blocked By:
Blocking: #1369 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: yes
Review URL(s):

Description

When creating a dependencies file for use with ensure_dependencies.py, the Mercurial and Git repositories of each module are currently required to have the exact same basename (e.g. hg.adblockplus.org/infrastructure and github.com/adblockplus/infrastructure).
Unfortunately, a bunch of these repositories are not named the same, e.g. hg.adblockplus.org/infrastructure-puppet-concat and github.com/adblockplus/puppet-concat).

Thus, we may either extend the aforementioned script to support different repository names (e.g. using the hg: and git: prefixes again), or simply re-name the repositories at GitHub.

Change History (5)

comment:1 Changed 5 years ago by trev

Renaming the GitHub repositories to have consistent names seems to be the better course of action.

I renamed the following repositories:

  • puppet-concat => infrastructure-puppet-concat
  • puppet-postgresql => infrastructure-puppet-postgresql
  • ActionBarSherlock => actionbarsherlock
  • puppetlabs-stdlib => infrastructure-puppetlabs-stdlib

It seems that infrastructure-puppet-spawn-fcgi, infrastructure-puppetlabs-apt, infrastructure-puppetlabs-firewall and infrastructure-puppetlabs-mysql are missing on GitHub, I will make sure these are created.

comment:2 Changed 5 years ago by trev

  • Ready set
  • Resolution set to fixed
  • Status changed from new to closed

Created the missing repositories as well, and made sure that our Mercurial repositories are in sync. Side-effect is that these Mercurial repositories are up-to-date now, the dependency still points to a pretty old revision however.

comment:3 Changed 5 years ago by trev

  • Owner set to trev
  • Priority changed from Unknown to P4

comment:4 Changed 5 years ago by trev

  • Blocking 1369 added

comment:5 Changed 5 years ago by matze

  • Verified working set
Note: See TracTickets for help on using tickets.