Skip to content

Conversation

@iceiilin
Copy link
Member

@iceiilin iceiilin commented Jan 8, 2016

PR to 1.1.0 branch, the same as #102 that has been merged into master.

Without this PR, the enclosure node that doesn't have "encloses" relationType previously will not be updated to include information about compute nodes. This problem is only exposed with the fix of ODR-314 (RackHD/on-http#88, RackHD/on-core#62) about adding the deleting and updating enclosure/compute node.

Choose a reason for hiding this comment

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

'helper' is not defined.

@heckj
Copy link
Member

heckj commented Jan 8, 2016

👍

@jlongever
Copy link
Contributor

Can one of the admins verify this patch?

@anhou
Copy link
Member

anhou commented Jan 14, 2016

👍

anhou pushed a commit that referenced this pull request Jan 14, 2016
modify generate enclosure job to not override other relation type
@anhou anhou merged commit adbcabd into RackHD:release/1.1.0-branch Jan 14, 2016
@iceiilin iceiilin deleted the 1.1.0 branch May 31, 2016 13:12
kellylu2sym pushed a commit to kellylu2sym/on-tasks that referenced this pull request Aug 8, 2017
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.

5 participants