Ravenbrook / Projects / Perforce Defect Tracking Integration / Version 1.1 Product Sources / Design
Perforce Defect Tracking Integration Project
This document sets out the high-level requirements for a system to move an organization from use of Perforce jobs, to use of Perforce jobs and defect tracker issues together with the P4DTI. We call this process migration. The document explains the consequences of these requirements for the design of the migration process and tools.
The P4DTI needs to support migration to meet requirement 95. This is Ravenbrook's job000022 and Perforce's job005387.
The intended readership is project developers and management.
This document is not confidential.
These are the high-level requirements for migration:
The migration procedure must not lose knowledge about jobs and fixes (requirement 1).
The migration procedure must inspire confidence in its results (requirements 1 and 2).
The administrator must be able to carry out the migration (requirements 114 and 115).
The migrated system must make the jobs of the developers and managers easier (requirement 2).
It must be possible to extend the P4DTI so that it supports migration to new defect trackers (requirement 20).
This analysis is based in part on [NB 2001-04-19].
Jobnames are an important part of an organization's knowledge about its product. If an organization has been using Perforce jobs for some time, then these jobnames will appear in plans, code, design, manuals, e-mails, printouts, and people's memories. The migration must not lose this information (migration requirement 1).
It's also important that users of the integrated system be able to tell which job in Perforce corresponds to which defect in the defect tracker. These are conflicting requiurements.
Here are some possible solutions:
Use Perforce jobnames for jobs in Perforce, defect tracker names
for defects in the defect tracker. Users find the defect corresponding
to a job by looking at the P4DTI-issue-id
field in the job.
Users find the job corresponding to a defect by looking at the "Perforce
jobname" field in the defect.
This solution imposes a burden on the user. (Fails migration requirement 4).
Use Perforce jobnames for jobs created in Perforce, but a defect created in the defect tracker gets a job name in Perforce that matches its name in the defect tracker.
This creates an artificial distinction between issues created in the two systems as well as imposing the burden identified for solution 1. (Fails migration requirement 4).
When a job is created in Perforce, give it a name in the defect tracker that is similar or identical to the name of the corresponding job in Perforce. And vice versa for defects created in the defect tracker.
This creates an artificial distinction between issues created in the two systems. This is not possible in some defect trackers. (Fails migration requirements 4 and 5.)
Rename all the Perforce jobs so that their names match the names of the corresponding defects in the defect tracker.
This loses information. (Fails migration requirement 1).
None of these solutions meets all the requirements. However, the first solution is the best and also the simplest.
Decision 1. In the migrated system, Perforce jobs will have Perforce jobnames and defect tracker issues will have defect tracker names.
The P4DTI's default mode of operation is to replace the Perforce jobspec with one of its own choosing. However, it is very likely that the organization has configured their jobspec carefully to match their process, and may have tools that operate on that jobspec. This is certainly true of both Ravenbrook and Perforce.
This means that we can't replace the jobspec without breaking migration requirement 1. The most we can do is to add our own fields to the jobspec.
Decision 2. We will add the P4DTI fields to the jobspec but not otherwise change it.
It follows from decision 2 that we can't use the P4DTI's automatic configuration to work out how the fields in the defect tracker correspond to the fields in Perforce. (The automatic configuration generator constructs a Perforce job field for each defect tracker field that the administrator chooses to replicate.) Each organization must be able to choose how the fields correspond.
Decision 3. Each organization must choose the correspondence between the fields in the defect tracker and the fields in Perforce.
It follows from decision 3 that we're in an "Advanced Configuration" situation [RB 2000-10-16, 8.6]. An organization must construct a field mapping between Perforce and the defect tracker.
This section lists situations that complicate the process of configuration and migration.
Fields may not be one-to-one between Perforce and the defect tracker.
Example. You have a single "Description" field (as in the default Perforce jobspec). But TeamTrack requires you to have a "Title" field containing a single line of text, and a "Description" field. In this case, you'd have to come up with some translation scheme: perhaps splitting the Perforce "Description" field into its first line (or paragraph?), forming the TeamTrack "Title", and the remaining lines, forming the TeamTrack "Description" ).
Perforce data may not correspond well to defect tracker data.
Example. You have specified "Priority: high/medium/low" in Perforce, but Bugzilla specifies priorities "P1" to "P5". We're not going to be able to automatically translate these fields: the organization must decide on a translation and implement it.
Different concepts in the two systems force you to change some data during migration.
Example. You have a job "Status" with values "open", "fixed", "ignored", "duplicate", "not_a_bug" and so on. But TeamTrack's workflow support requires you that status be "open" or "closed", with a separate "Resolution" field containing the reason why the issue was closed. So during migration you decide to add a "Resolution" field to Perforce and translate the old "Status" field into a combination of the new "Status" and "Resolution" fields.
Not all jobs need to be migrated.
Many organizations will only want to migrate jobs that are currently open; old jobs should be left alone. But other organizations will want to migrate all jobs so that reports and metrics show the true picture.
Defaults need to be provided for defect tracker fields.
Example. A defect tracker issue generally has more fields than a Perforce job. For example, TeamTrack issues have fields for "Issue Type" (bug, enhancement request, etc.), "Manager", "Tester", "Engineer", "Version", "How Found" and others that are unlikely to be in any organization's jobspec. The organization will need to specify sensible defaults for these fields.
Our experience of testing release 0.3.0 (which used advanced configuration) with potential customers was that this kind of configuration was too complicated and error-prone for the average administrator. So any solution requiring "advanced configuration" or something similar will fail migration requirement 3. We believe that many organizations will be unable to use this method to migrate from Perforce to a defect tracker without expert help.
This failing is severe, so we spent some time trying to come up with ways to work round it. However, we couldn't find any way to do this that took account of the complications in section 3.3 and which is affordable.
An approach that might meet all the user requirements would be to have a graphical user interface which presents fields from Perforce and the defect tracker. The administrator will drag the fields around until they match up. Then for each pair of corresponding fields the administrator picks a method for translation; this may also involve pairing up values. Finally, when all is ready, the administrator hits the big "Migrate" button and off it goes. Further widgets would help work around each of the various complications listed in section 3.3.
We estimate that this solution would require at least 12 months of effort for design, implementation and testing (that's around $150,000). We don't believe that this expenditure will be cost-effective.
This is the sequence of steps to migrate from Perforce to the defect tracker.
Make a test setup with a test Perforce server containing a copy of your jobs, and a test defect tracker.
Develop an advanced configuration [RB 2000-10-16, 8.6].
In the advanced configuration, specify both a migration field map and a replication field map. We need two field maps because (1) during migration we need to provide defaults for some defect tracker fields (complication 5) and because data may be being changed during migration (complication 3). Usually the two maps will be similar but subtly different.
For a defect tracker field which reproduces the contents of multiple Perforce fields, write a single translator which uses the job argument to get at other job fields.
For a Perforce field which reproduces the contents of multiple defect tracker fields, write multiple translators (i.e. have multiple entries in the field_map list).
Specify a migrate_p
function to select the jobs to
migrate.
Test that migration works (it's probably best to start out with
a handful of jobs by setting the migrate_p
appropriately)
then run the consistency checker.
When migration works, test that replication works by taking some issues through their life cycles.
When everything works satisfactorily, repeat on your real data.
Our experience (based on alpha testing the P4DTI when all configuration was "advanced") is that the steps in section 4.1 will be very hard to follow and possibly beyond the ability of some organizations.
We can make success more likely if we can make the migration robust: in particular, it must be possible to get migration done in one pass, even if there are problems.
This is especially important when an organization has many jobs. TeamTrack can process cases at about 2 per second, so 100,000 jobs may take 54 hours to migrate (each case gets written once and read once). It will not be acceptable to set this going and find out after 53 hours that one of the jobs has a bogus value in it that can't be translated and that the whole thing has to be done again.
Because Perforce isn't very strict about the contents of job fields
(and anyway you can override the checking with p4 job -f
)
we must expect many bogus fields in jobs. These must be translated
anyway, with warnings. The administrator can always tidy these up
afterwards.
The migration design needs to be idempotent. We should be able to run the migration script several times, each time migrating some more jobs.
These are the possibilities:
Advise the organization not to migrate, but to continue using Perforce jobs.
This isn't an acceptable solution. Organizations get substantial benefits from a defect tracker, including workflow enforcement and reporting.
Leave people to cope with the difficulties of advanced configuration.
We don't like this. It reflects badly on the P4DTI. It fails requirement 75.
Improve the migration process so that it is easy enough.
This is unacceptably expensive; see section 3.4.
Provide consultancy for each organization.
This suggests a number of questions:
Would Perforce be happy about this?
Who's going to pay for the work?
I think the end user is going to have to pay in each case.
Who's going to do the work?
Ravenbrook would be willing to do it for at least the first year. It should be possible to train Perforce consulting partners and TeamShare PSG to do this.
Migration should be idempotent. You should be able to run the migration script many times over the same set of jobs and it should do no harm.
This should not create huge slews of new issues (multiple copies of the same issue). One way to do this would be to check to see if there's already an issue which replicates to a given job.
We should be able to use the migration design to allow people to create new jobs in Perforce and have them replicated.
[This is a rough sketch at present. NB, please complete. GDR 2001-06-25.]
This design is based on [NB 2001-05-10a] and [NB 2001-05-10b].
The new command-line script migrate.py
just creates a
replicator instance and calls its migrate()
method.
These configuration items are needed to support replication (eventually these descriptions will be replaced with links to the Integrator's Guide):
use_perforce_jobnames
: when this is true, the
replicator uses ordinary Perforce jobnames for replicated
issues.
migrate_p
: this function is called for each job
during migration; iff it returns true, then job is migrated.
migration_field_map
: this is the field map that is
used during migration. It has the same specification as the
field_map
configuration parameter.
migrated_jobspec_description
: this is the new
Perforce jobspec. After copying the jobs to the defect tracker, the
Perforce jobspec is changed to this (if supplied) and issues are
replicated back to Perforce.
These methods are added or changed in the replicator class:
__init__()
: makes a client for the replicator to use
when issuing p4 change -o
commands. Calls
dt.init()
.
check_jobspec()
: checks that the Perforce jobspec
has the P4DTI fields.
create_issue()
: applies the migration field map and
creates a new issue in the defect tracker.
init()
: removed. Initializtion has been moved to
appropriate places so that the migrate script can run without the
replicator checking that it has the right jobspec. Creation of client
and calling defect_tracker.init
moved to
__init__()
method. Checking of jobspec moved to new method
check_jobspec()
and called from the check()
,
refresh()
and run()
methods.
migrate()
: carries out migration.
replicate_many()
: implements the
use_perforce_jobnames
feature.
update_job()
: supports the
use_perforce_jobnames
feature by recording the jobname of a
newly created Perforce job so that it can be passed to
setup_for_replication()
and thus recorded in the defect
tracker.
migrate_users()
: copies users from Perforce to the
defect tracker.
These methods has been added to the defect tracker class (eventually these descriptions will be replaced with links to the Integrator's Guide):
add_user()
: adds a user to the defect tracker if not
already present, and if this is possible given the constraints of the
defect tracker (licences etc.).
new_issue()
: creates a new issue in the defect
tracker.
[NB 2001-04-19] | "Re: PERFORCE job job005387 for review" (e-mail message); Nick Barnes; Ravenbrook Limited; 2001-04-19. |
[NB 2001-05-10a] | "migration design notes" (e-mail message); Nick Barnes; Ravenbrook Limited; 2001-05-10. |
[NB 2001-05-10b] | "Re: migration design notes" (e-mail message); Nick Barnes; Ravenbrook Limited; 2001-05-10. |
[RB 2000-10-16] | "Perforce Defect Tracking Integration Integrator's Guide"; Richard Brooksby; Ravenbrook Limited; 2000-10-16. |
2001-06-01 | GDR | Created. Added requirements. |
2001-06-25 | GDR | Added analysis and design. |
Copyright © 2001 Ravenbrook Limited. This document is provided "as is", without any express or implied warranty. In no event will the authors be held liable for any damages arising from the use of this document. You may make and distribute verbatim copies of this document provided that you do not charge a fee for this document or for its distribution.
$Id: //info.ravenbrook.com/project/p4dti/branch/2001-04-20/migrate-bugzilla/design/migration/index.html#2 $
Ravenbrook / Projects / Perforce Defect Tracking Integration / Version 1.1 Product Sources / Design