Yasuke/moto
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

rendering vpc_id in ec2 responses

Browse Source
This commit is contained in:
Jack Danger 2017-09-08 15:07:44 -07:00
parent a0283fc1ca
commit 5a03497367
2 changed files with 27 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
moto/ec2/responses/instances.py
View File

@ -255,17 +255,15 @@ EC2_RUN_INSTANCES = """<RunInstancesResponse xmlns="http://ec2.amazonaws.com/doc
<monitoring> <monitoring>
<state>enabled</state> <state>enabled</state>
</monitoring> </monitoring>
{% if instance.nics %} {% if instance.subnet_id %}
{% if instance.nics[0].subnet %} <subnetId>{{ instance.subnet_id }}</subnetId>
<subnetId>{{ instance.nics[0].subnet.id }}</subnetId> {% endif %}
<vpcId>{{ instance.nics[0].subnet.vpc_id }}</vpcId> {% if instance.vpc_id %}
<vpcId>{{ instance.vpc_id }}</vpcId>
{% endif %} {% endif %}
<privateIpAddress>{{ instance.private_ip }}</privateIpAddress> <privateIpAddress>{{ instance.private_ip }}</privateIpAddress>
{% if instance.public_ip %} {% if instance.nics[0].public_ip %}
<ipAddress>{{ instance.public_ip }}</ipAddress> <ipAddress>{{ instance.nics[0].public_ip }}</ipAddress>
{% endif %}
{% else %}
<subnetId>{{ instance.subnet_id }}</subnetId>
{% endif %} {% endif %}
<sourceDestCheck>{{ instance.source_dest_check }}</sourceDestCheck> <sourceDestCheck>{{ instance.source_dest_check }}</sourceDestCheck>
<groupSet> <groupSet>
@ -396,16 +394,16 @@ EC2_DESCRIBE_INSTANCES = """<DescribeInstancesResponse xmlns="http://ec2.amazona
<monitoring> <monitoring>
<state>disabled</state> <state>disabled</state>
</monitoring> </monitoring>
{% if instance.nics %} {% if instance.subnet_id %}
{% if instance.nics[0].subnet %} <subnetId>{{ instance.subnet_id }}</subnetId>
<subnetId>{{ instance.nics[0].subnet.id }}</subnetId> {% endif %}
<vpcId>{{ instance.nics[0].subnet.vpc_id }}</vpcId> {% if instance.vpc_id %}
<vpcId>{{ instance.vpc_id }}</vpcId>
{% endif %} {% endif %}
<privateIpAddress>{{ instance.private_ip }}</privateIpAddress> <privateIpAddress>{{ instance.private_ip }}</privateIpAddress>
{% if instance.nics[0].public_ip %} {% if instance.nics[0].public_ip %}
<ipAddress>{{ instance.nics[0].public_ip }}</ipAddress> <ipAddress>{{ instance.nics[0].public_ip }}</ipAddress>
{% endif %} {% endif %}
{% endif %}
<sourceDestCheck>{{ instance.source_dest_check }}</sourceDestCheck> <sourceDestCheck>{{ instance.source_dest_check }}</sourceDestCheck>
<groupSet> <groupSet>
{% for group in instance.dynamic_group_list %} {% for group in instance.dynamic_group_list %}

4
tests/test_ec2/test_instances.py
View File

@ -378,11 +378,15 @@ def test_get_instances_filtering_by_vpc_id():
reservations1.should.have.length_of(1) reservations1.should.have.length_of(1)
reservations1[0].instances.should.have.length_of(1) reservations1[0].instances.should.have.length_of(1)
reservations1[0].instances[0].id.should.equal(instance1.id) reservations1[0].instances[0].id.should.equal(instance1.id)
reservations1[0].instances[0].vpc_id.should.equal(vpc1.id)
reservations1[0].instances[0].subnet_id.should.equal(subnet1.id)
reservations2 = conn.get_all_instances(filters={'vpc-id': vpc2.id}) reservations2 = conn.get_all_instances(filters={'vpc-id': vpc2.id})
reservations2.should.have.length_of(1) reservations2.should.have.length_of(1)
reservations2[0].instances.should.have.length_of(1) reservations2[0].instances.should.have.length_of(1)
reservations2[0].instances[0].id.should.equal(instance2.id) reservations2[0].instances[0].id.should.equal(instance2.id)
reservations2[0].instances[0].vpc_id.should.equal(vpc2.id)
reservations2[0].instances[0].subnet_id.should.equal(subnet2.id)
@mock_ec2_deprecated @mock_ec2_deprecated