New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add snapshotid to ec2 and os volumes extra #479

Merged
merged 1 commit into from Mar 11, 2015

Conversation

Projects
None yet
3 participants
@allardhoeve
Contributor

allardhoeve commented Mar 6, 2015

Add snapshot_id to extra dicts for StorageVolumes on EC2 and OpenStack.

@sebgoa

This comment has been minimized.

Show comment
Hide comment
@sebgoa

sebgoa Mar 10, 2015

Member

@allardhoeve you should be able to merge this ? you got your committer bit, correct ?

Member

sebgoa commented Mar 10, 2015

@allardhoeve you should be able to merge this ? you got your committer bit, correct ?

@allardhoeve

This comment has been minimized.

Show comment
Hide comment
@allardhoeve

allardhoeve Mar 10, 2015

Contributor

I thought it well to continue to pass my commits through community review, but yes, I could commit it.

Contributor

allardhoeve commented Mar 10, 2015

I thought it well to continue to pass my commits through community review, but yes, I could commit it.

Show outdated Hide outdated CHANGES.rst
@@ -15,6 +15,11 @@ General
Compute
~~~~~~~
- StorageVolume objects now have a key called snapshot_id in their extra
dicts containing the snapshot ID the volume was based on.

This comment has been minimized.

@sebgoa

sebgoa Mar 10, 2015

Member

Add that this is for ec2 and openstack

@sebgoa

sebgoa Mar 10, 2015

Member

Add that this is for ec2 and openstack

This comment has been minimized.

@allardhoeve

allardhoeve Mar 10, 2015

Contributor

Done

@allardhoeve

allardhoeve Mar 10, 2015

Contributor

Done

@@ -31,7 +31,7 @@
"id": "cfcec3bc-b736-4db5-9535-4c24112691b5",
"metadata": {},
"size": 50,
"snapshotId": null,
"snapshotId": "01f48111-7866-4cd2-986a-e92683c4a363",
"status": "available",
"volumeType": "None"
}

This comment has been minimized.

@sebgoa

sebgoa Mar 10, 2015

Member

no need for a fixture for the ec2 testing ?

@sebgoa

sebgoa Mar 10, 2015

Member

no need for a fixture for the ec2 testing ?

This comment has been minimized.

@allardhoeve

allardhoeve Mar 10, 2015

Contributor

The EC2 fixture already had snapshotId defined. I test this on line 785 of test_ec2.py.

@allardhoeve

allardhoeve Mar 10, 2015

Contributor

The EC2 fixture already had snapshotId defined. I test this on line 785 of test_ec2.py.

Allard Hoeve

@asfgit asfgit merged commit e90e759 into apache:trunk Mar 11, 2015

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@allardhoeve allardhoeve deleted the ByteInternet:add-snapshotid-to-ec2-and-os-volumes-extra branch Mar 11, 2015

MatthiasWiesnerCC pushed a commit to cloudControl/libcloud that referenced this pull request Mar 17, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment