Project

Profile

Help

Refactor #990

Rework the Vagrant provisioning script to install the dependencies from the spec files instead of installing/removing Pulp

Added by rbarlow over 5 years ago. Updated over 1 year ago.

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

100%

Estimated time:
Platform Release:
2.8.0
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
June 2015
Quarter:

Description

We have some code to pull dependencies out of the spec files that is used by Jenkins. We should use this same code in our Vagrant provisioning script as well so that we can shave a little time off of the dependency installation. Currently, Vagrant installs Pulp and then removes it in order to get the dependencies.

Associated revisions

Revision 6f5290b5 View on GitHub
Added by rbarlow over 5 years ago

Provision a virtualenv for Crane if present.

Our Vagrant provisioning didn't do anything for Crane. This commit creates basic support for creating a virtualenv and installing Crane's dependencies.

In order to avoid copying the RPM dependency information, I determined how to query Crane's spec file for the RPMs that Crane depends on. Since this line was also useful for our other projects, I converted our dependencies to all be installed this way. By doing this, I also solved #990 and so this commit also fixes that issue.

https://pulp.plan.io/issues/990

fixes #990

Revision 6f5290b5 View on GitHub
Added by rbarlow over 5 years ago

Provision a virtualenv for Crane if present.

Our Vagrant provisioning didn't do anything for Crane. This commit creates basic support for creating a virtualenv and installing Crane's dependencies.

In order to avoid copying the RPM dependency information, I determined how to query Crane's spec file for the RPMs that Crane depends on. Since this line was also useful for our other projects, I converted our dependencies to all be installed this way. By doing this, I also solved #990 and so this commit also fixes that issue.

https://pulp.plan.io/issues/990

fixes #990

History

#1 Updated by rbarlow over 5 years ago

We could do something similar to:

PLUGIN_PACKAGE_LIST=$(rpmspec -q --requires *.spec --queryformat '%{NAME} ')

which is very similar to the --provides line from:

https://github.com/pulp/pulp_packaging/blob/3ab4aca16800b952e3a3db6533cc5b1617bf7225/ci/jobs/ci-continuous.yaml#L218

#2 Updated by rbarlow over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to rbarlow
  • Sprint/Milestone set to 17
  • Platform Release set to master

#3 Updated by rbarlow over 5 years ago

  • Status changed from ASSIGNED to POST

#4 Updated by rbarlow over 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#5 Updated by rbarlow about 5 years ago

  • Platform Release changed from master to 2.8.0

#6 Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from MODIFIED to 5

#7 Updated by rbarlow almost 5 years ago

  • Status changed from 5 to 6

The Vagrant environment works and uses the specs for deps.

#8 Updated by dkliban@redhat.com over 4 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#10 Updated by bmbouter over 2 years ago

  • Sprint set to June 2015

#11 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (17)

#12 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF