Skip to content

Add Parent Field to Resource Manager#2941

Merged
waprin merged 5 commits intogoogleapis:masterfrom
waprin:add_parents
Jan 18, 2017
Merged

Add Parent Field to Resource Manager#2941
waprin merged 5 commits intogoogleapis:masterfrom
waprin:add_parents

Conversation

@waprin
Copy link
Contributor

@waprin waprin commented Jan 18, 2017

Fixes #2940 by including parent field in the update.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 18, 2017
sir-buckyball
sir-buckyball reviewed Jan 18, 2017
View reviewed changes
resource_manager/unit_tests/test_project.py Outdated
PROJECT_NUMBER = 12345678
PROJECT_LABELS = {'env': 'prod'}
PROJECT_LIFECYCLE_STATE = 'ACTIVE'
PARENT = {'type': 'organization', 'id': 433637338579}

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@daspecster daspecster added the api: cloudresourcemanager Issues related to the Resource Manager API. label Jan 18, 2017
daspecster
daspecster reviewed Jan 18, 2017
View reviewed changes
resource_manager/google/cloud/resource_manager/project.py Outdated
client = self._require_client(client)

data = {'name': self.name, 'labels': self.labels}
data = {}

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

dhermes
dhermes approved these changes Jan 18, 2017
View reviewed changes
Copy link
Contributor

@dhermes dhermes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

resource_manager/google/cloud/resource_manager/project.py Outdated
data = {
'name': self.name,
'labels': self.labels,
'parent': self.parent

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

resource_manager/google/cloud/resource_manager/project.py
self.labels = resource.get('labels', {})
self.status = resource['lifecycleState']
if 'parent' in resource:
self.parent = resource['parent']

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

resource_manager/unit_tests/test_project.py Outdated
'lifecycleState': 'DELETE_REQUESTED',
'parent': {
'type': 'organization',
'id': '433637338579'

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

resource_manager/unit_tests/test_project.py Outdated
'parent': {
'type': 'organization',
'id': '433637338579'
}

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes
Copy link
Contributor

dhermes commented Jan 18, 2017

@waprin Don't merge until CI is green

@waprin waprin merged commit e9b503a into googleapis:master Jan 18, 2017
@dhermes dhermes mentioned this pull request Feb 24, 2017
Merged
19 tasks
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
@waprin
Add Parent Field to Resource Manager (googleapis#2941)
f0ed002 
* Add Parent Field to Resource Manager
parthea pushed a commit that referenced this pull request Jun 4, 2023
@waprin
Add Parent Field to Resource Manager (#2941)
626d935 
* Add Parent Field to Resource Manager
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@daspecster daspecster daspecster left review comments

@sir-buckyball sir-buckyball sir-buckyball left review comments

@dhermes dhermes dhermes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@tseaver tseaver

Labels

api: cloudresourcemanager Issues related to the Resource Manager API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@waprin @dhermes @daspecster @sir-buckyball @tseaver @googlebot