Opened on 09/18/2014 at 08:06:14 AM

Closed on 09/18/2014 at 10:36:57 AM

Last modified on 09/18/2014 at 03:26:00 PM

#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.

Attachments (0)

Change History (5)

comment:1 Changed on 09/18/2014 at 10:22:05 AM 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 on 09/18/2014 at 10:36:57 AM 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 on 09/18/2014 at 10:37:19 AM by trev

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

comment:4 Changed on 09/18/2014 at 10:51:42 AM by trev

  • Blocking 1369 added

comment:5 Changed on 09/18/2014 at 03:26:00 PM by matze

  • Verified working set

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
 
Note: See TracTickets for help on using tickets.