Project

Profile

Help

Task #2829

As a developer, Pulp2 Vagrant file uses the libvirt name 'pulp2_devel' and Pulp3 Vagrant file uses 'pulp3_devel'

Added by amacdona@redhat.com over 4 years ago. Updated almost 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
2.14.0
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Dev Environment, Pulp 2
Sprint:
Sprint 22
Quarter:

Associated revisions

Revision a0975951 View on GitHub
Added by amacdona@redhat.com over 4 years ago

Avoid collision with Pulp2 dev env

re 2829

Revision 262c6c99 View on GitHub
Added by amacdona@redhat.com over 4 years ago

Avoid collisions with Pulp 3 dev env

re #2829

History

#1 Updated by amacdona@redhat.com over 4 years ago

Some roles may work for both Pulp 2 and Pulp 3. Others could be converted to do the parts that are common between the 2, and call other tasks for Pulp2/Pulp3. This would be nice, but it would increase the "backwards compatibility tax", so I think we should keep them completely separate. Pulp 2 will have a decreasing importance, and we don't want to waste resources maintaining our dev-setup to work for both.

#2 Updated by bmbouter over 4 years ago

+1 to keeping the roles totally separate.

One question I've been wondering about is how many pulp checkouts will there be. Say a "set of checkouts: consists of all the repos that back a single Vagrant environment, i.e. pulp, pulp_rpm, pulp_puppet, etc. I think if we have to have two sets of checkouts then the practical benefit goes down because if you have two sets of checkouts you can just checkout the devel folder twice also and you'll also have two Vagrantfiles.

I think the only issue there is that they both register the same libvirtd name, so maybe we should just change the libvirtdname insead?

If we did want to use just one set of checkouts I imagine you can switch your branches to master with checkout.py, run `vagrant up pulp2`, leave it on, switch to 3.0-dev with checkout.py, and run `vagrant up pulp3`. My only concern is that each Vagrant up needs to not disturb the NFS mount of the other machine so we would need to test this. If we did this, for each environment you leave you, you would just make sure to have the branches checked out correctly and then run `prestart` before you use it.

Given that is how this would be used, maybe just making the two libvirt names be pulp2_devel and pulp3_devel would be easier.

What do you think?

#3 Updated by amacdona@redhat.com over 4 years ago

  • Description updated (diff)

bmbouter wrote:

Given that is how this would be used, maybe just making the two libvirt names be pulp2_devel and pulp3_devel would be easier.

Yeah. I think thats all we need.

#4 Updated by bmbouter over 4 years ago

  • Subject changed from As a developer I can easily work on Pulp 2 and Pulp 3 to As a developer, Pulp2 Vagrant file uses the libvirt name 'pulp2_devel' and Pulp2 Vagrant file uses 'pulp3_devel'
  • Sprint Candidate changed from No to Yes

#5 Updated by mhrivnak over 4 years ago

  • Subject changed from As a developer, Pulp2 Vagrant file uses the libvirt name 'pulp2_devel' and Pulp2 Vagrant file uses 'pulp3_devel' to As a developer, Pulp2 Vagrant file uses the libvirt name 'pulp2_devel' and Pulp3 Vagrant file uses 'pulp3_devel'

#6 Updated by bmbouter over 4 years ago

  • Groomed changed from No to Yes

#7 Updated by daviddavis over 4 years ago

This will be a huge help. Thanks!

#8 Updated by mhrivnak over 4 years ago

  • Sprint/Milestone set to 41

#9 Updated by amacdona@redhat.com over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com

#10 Updated by amacdona@redhat.com over 4 years ago

  • Status changed from ASSIGNED to MODIFIED

#13 Updated by pcreech over 4 years ago

  • Platform Release set to 2.14.0

#14 Updated by pcreech over 4 years ago

  • Status changed from MODIFIED to 5

#15 Updated by pcreech over 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#16 Updated by bmbouter almost 4 years ago

  • Sprint set to Sprint 22

#17 Updated by bmbouter almost 4 years ago

  • Sprint/Milestone deleted (41)

#18 Updated by bmbouter almost 3 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF