Project

General

Profile

Feature #1871

VPC support on EC2 compute

Added by Romain Vrignaud almost 7 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Compute resources
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The patch prosposed on blog entry (http://blog.theforeman.org/2012/05/ec2-provisioning-using-foreman.html)
http://pastie.org/4002125 doesn't work.
With this modification proposed by ohad '<%= select_f f, :subnet_id , vpcs, :id, :cidr_block, {}, :label => "VPC" %>'
I get this in debug mode :
http://pastebin.com/ZEFNbWyc


Related issues

Related to Foreman - Feature #1868: Foreman should ask which VPC to use when creating a new Instance on EC2Duplicate2012-09-17
Related to Foreman - Bug #1879: Foreman fails to acquire IP when using AWS VPC with private IPResolved2012-10-02

Associated revisions

Revision beed05de (diff)
Added by Mark O'Shea almost 6 years ago

fixes #1871, #1879 - add VPC support to EC2 compute resource

Based on patch from Romain Vrignaud <>

History

#1 Updated by Ohad Levy almost 7 years ago

  • Tracker changed from Bug to Feature

#2 Updated by Ohad Levy almost 7 years ago

@Romain, do you have the latest patch that worked? IMHO the only thing left here is to have a logic choosing which ip address to use (public vs private) for ssh

#3 Updated by Romain Vrignaud almost 7 years ago

Here is the paste that I use in production : http://pastie.org/5078094

For +super.merge({ :ip => :private_ip_address }) this is related to #1879

#4 Updated by Ohad Levy over 6 years ago

  • Target version changed from 1.1 to 1.2.0

#5 Updated by Will Sterling over 6 years ago

Romain Vringnaud has submitted a commit request that looks promising but I am having trouble getting it to merge with 1.2. app/views/compute_resources_vms/form/_ec2.html.erb fails to render and only one of four VPC subnets appear in the network tab. Romain thinks the problem comes from the recently added multi-nic support. Any ideas on what change this commit would need to work with 1.2? https://github.com/theforeman/foreman/pull/237/files

#6 Updated by Glen Ogilvie over 6 years ago

Any progress on this? We would really like to use this feature in a project that is coming up soon.

#7 Updated by Dominic Cleal over 6 years ago

  • Assignee deleted (Ohad Levy)
  • Target version deleted (1.2.0)

Glen, there hasn't been any progress that I'm aware of, so removing the 1.2 tracker. If somebody is able to rebase the patch, we can include it in a future release.

#8 Updated by Glen Ogilvie over 6 years ago

Dominic Cleal wrote:

Glen, there hasn't been any progress that I'm aware of, so removing the 1.2 tracker. If somebody is able to rebase the patch, we can include it in a future release.

This is something that would be really helpful to us if it worked. I don't think I have the Ruby skills however, to rebase this patch myself.

#9 Updated by Dominic Cleal about 6 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.3.0

#10 Updated by Anonymous almost 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF