Skip to content
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

Merge the master branch to Zun-exp #929

Closed
wants to merge 161 commits into from

Conversation

kevinzs2048
Copy link
Contributor

@kevinzs2048 kevinzs2048 commented Apr 13, 2018

For #700

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

This PR meet a openlab error which has been fixed in master branch recently. So merge the master to zun-exp according to comment 23 in below PR.
#858

ozerovandrei and others added 30 commits January 2, 2018 18:31
Add support to show a Neutron subnetpool through a GET request on
/v2.0/subnetpools/{subnetpool_id}.
The same command with OpenStack CLI is done with:
openstack subnet pool show
or
neutron subnetpool-show
…subnetpools-get

Add Neutron subnetpool get support
* Add Neutron subnetpool creation support

Add support to create a Neutron subnetpool through a POST request on
/v2.0/subnetpools.
The same command with OpenStack CLI is done with:
openstack subnet pool create
or
neutron subnetpool-create

* Add acceptance test for the subnetpool creation

Add the TestCreateSubnetPool acceptance test and a generic CreateSubnetPool function inside the
acceptance tests for sharing with other components.

* Rename TestCreateSubnetPool to TestSubnetPoolsCRUD

* Fix Subnetpools CreateOpts comments

Fix DefaultPrefixLen comment and remove empty line at the beginning of a
struct.
* Availability zones info support (gophercloud#320)

Add ability to get list and detailed AZs info

* Style fixes

* Change test fixture to proper

* Delete useless method of `ServerAvailabilityZoneExt`

* Code improvements

* Change `AvailabilityZoneInfo` type from `struct` to `slice`
* Add custom `UnmarshalJSON` to `AvailabilityZoneInfo`
* Get rid of 'OS' prefixes in variable's names

* Update doc.go

* Code refactoring

* Rename `ServiceOfState` to `ServiceState`
* Change type `Services` from `struct` to `map[string]ServiceState`
* Delete useless type `AvailabilityZoneInfo`
* Update doc.go

* Acceptance tests for gophercloud#704

* Change struct tag `updated_at` to `-`
Compute v2: Update docs for Availability Zones
* Fix "Content-Type" in orchestration env tests

We need to use "application/json" instead of "application/jason"

* Fix Orchestration's TestGetRRFileContents

Fix environments unit test "TestGetRRFileContents" by updating
the "expectedParsed" to be a valid representation of a Heat's
resource_registry definition.
Also update environment's fileMaps to sucessfully apply the
"fixFileRefs" method.

* Fix Orchestration TestGetRRFileContents comments
* Fix "Content-Type" in orchestration template tests

We need to use "application/json" instead of "application/jason"

* Fix flavor field in orchestration fixtures

Set "default" field to be a string in "ValidJSONTemplate",
"ValidYAMLTemplate", "InvalidTemplateNoVersion", "InvalidEnvironment",
"ValidJSONTemplateParsed" orchestration templates test fixtures.
That also fixes "TestTemplateParsing" orchestration template unit test.
* Add Neutron subnetpool update support

Add support to update a Neutron subnetpool through a PUT request on
/v2.0/subnetpools.
The same command with OpenStack CLI is done with:
openstack subnet pool set
or
neutron subnetpool-update

* Add a documentation about subnetpool updating

* Add subnetpool update to acceptance tests

* Cleanup Neutron subnetpools docs

Remove some garbage dots from titles because we don't really need them
in the documentation.

* Update Neutron subnetpools update fileds types

We need to use a "*string" to "AddressScopeID" and "Description" fileds,
"*bool" to a IsDefault field and "*int" to a DefaultQuota field.
That is because we wan't to have a possibility to update those fields to
empty or false or zero values.

* Fix TestSubnetPoolsCRUD update test

Variable newSubnetPool is new so it should be initialized.
Add a Delete request, unit test and acceptance tests with acceptance
reusable "DeleteSubnetPool" function.
…subnetpools-delete

Add Neutron subnetpool delete support
…get-acceptance-test

Add get request in subnetpools acceptance tests
* Hypervisors statistics support

* Add ability to get hypervisors stats
* Add unit tests
* Update doc.go
* Add acceptance tests

* Style fixes

* Rename few vars
* Add Compute service list support

* Remove ForcedDown field, add UpdatedAt field
* Add support to get specific hypervisor by ID

* Add unit tests

* Update doc.go

* Add acceptance tests

* Fix conversion `int` -> `string`
This commit prevents Gophercloud from performing a recursive
BuildRequestBody when the field's JSON tag is "-".
* Compute v2: Flavor Access Remove

* Correcting method names
* Add support to get hypervisor uptime specified by ID

* Add unit test

* Add acceptance test
* Add support for server live-migration

* Add unit test

* Fix 'Host' type `string` -> `*string` to be able to set it to nil

* Add acceptance test

* Add ability to turn-on/off acceptance test for server live-migration

* Get rid of 'strconv', add description when skip migrate test
* add simple tenant usage

* tenantusage time fields changed to time.Time

* removed multi, kept single

* lint

* fixed naming

* address comments in simpletenantusage

* renamed simpletenantusage to usage

* added acceptance test

* updated usage/doc.go to reflect package name

* renamed usage to tenantUsage to prevent shadowing

* renamed symbols in extension/usage
Nova returns the amount of ephemeral storage (in GB) associated with a
flavor as `OS-FLV-EXT-DATA:ephemeral`.
* Add Blockstorage service list support

* Add optional fields into List query
jtopjian and others added 24 commits April 7, 2018 05:31
* Add NewMessagingV2 client

* Add create function for queues

* Add unit tests for create queues

* Remove queueName from function signature & fix createOpts metadata

* Fix unit tests to use Extra and proper function signature

* Fix documentation

* Update urls to use contants

* Fix Extra comments

* Fix formatting

* Fix doc formatting

* Remove Client-ID from create function

* Add clientID parameter to NewMessagingV2
* Identity V3 roles: add function ListAssignmentsForUserOnProject()

* fix error

* fix JSON output

* create function ListAssignmentsOnResource()

* fix unit testing

* fix unit testing

* fix acceptance testing
There might be occasions when the API will return an actual
DELETED status. This is a short window of time, but it
could happen.
* Add create function for queues

* Add list function for queues

* Add unit tests for list queues

* Fix documentation

* Update commonURL to use constants

* Fix QueueDetails to properly handle misc key/values and update unit tests

* Remove Client-ID from list function

* Implement NextPageURL for list queues
* Clustering Policy Create Implementation

* Fix review comments

* Created Spec type
* Changed Policy unmarshal to handle both time with and without Z
* Added custom unmarshal to handle Version as either string or float

* Change UpdatedAt value in fixtures to test different code path

* fix variable name

* Fix review comments

- Removed custom Version type and unmarshal function
- Added Version assignment to policy unmarshal with custom logic

* added custom unmarshal for spec
* Add create function for queues

* Fix documentation

* Add list function for queues

* Add unit tests for list queues

* Fix QueueDetails to properly handle misc key/values and update unit tests

* Add update function for queues

* Add unit tests for update queue

* Fix formatting

* Update updateURL to use constants

* Fix UpdateQueueBody to use UpdateOp instead of a string

* Remove Client-ID from update function

* Fix unit tests to use proper path in update

* Remove Client-ID referneces in docs & fix unit tests

* Fix Update to use BatchUpdate
* initial commit

* fix X-OpenStack-Request-ID header

* fix review comments
* Senlin: Webhook Trigger (gophercloud#823)

Add webhooks trigger for senlin clustering

* Fixed example code in doc
Improved/cleanup comments to make it more readable
Refactored Webhook struct for Extract() to be cleaner and more readable

* Fixed trigger opts to properly construct the query strings for "V" and "params"
Added test cases for nil, empty, and params to webhook trigger
Added test case for testing query string construction

* Added acceptance test, but commented out because in order to invoke webhook trigger, requires cluster, profiles, and receiver to be created.

* Changed doc.go to use V param

* Merged from upstream with go fmt fix

* Fixed formatting for doc as well for testing for triggerOpts
…ates

Image Service v2: Unmarshal Fix and Acc Test Updates
…nto zun-exp

Signed-off-by: Kevin Zhao <kevin.zhao@arm.com>
@coveralls
Copy link

coveralls commented Apr 13, 2018

Coverage Status

Coverage increased (+4.9%) to 77.654% when pulling c9afdc1 on kevin-zhaoshuai:merge-master into 34d3d6f on gophercloud:zun-exp.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Apr 13, 2018

Build succeeded.

@jtopjian jtopjian mentioned this pull request Apr 18, 2018
@jtopjian
Copy link
Contributor

Implemented in #944

@jtopjian jtopjian closed this Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet