-
Notifications
You must be signed in to change notification settings - Fork 77
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
Fix issue with relation #428
base: master
Are you sure you want to change the base?
Conversation
BUILD on-tasks #1 : FAILURE BUILD on-tasks #1 Error Logs ▼Test Name: Redfish Registries "before all" hook: start HTTP server Error Details: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test. Stack Trace: Error: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test.Test Name: Http.Api.Notification "before all" hook: start HTTP server |
test this please |
BUILD on-tasks #2 : FAILURE |
test this please |
1 similar comment
test this please |
BUILD on-tasks #22 : FAILURE BUILD on-tasks #22 Error Logs ▼Test Name: test_node_workflows_get Error Details: (404) Reason: Not Found HTTP response headers: HTTPHeaderDict({'Content-Length': '116', 'X-Powered-By': 'Express', 'Connection': 'keep-alive', 'ETag': 'W/"74-pJFq8Hazf1VD2xd/kEOrhX/y1zg"', 'Date': 'Thu, 23 Mar 2017 03:21:20 GMT', 'Access-Control-Allow-Origin': '*', 'Content-Type': 'application/json; charset=utf-8'}) HTTP response body: {"message":"Could not find node with identifier fooey","status":"404","UUID":"1b373284-1ad7-4694-bd9b-24b1d6449d29"}Stack Trace: Traceback (most recent call last): |
@@ -134,7 +134,7 @@ function UcsServiceProfileDiscoveryJobFactory( | |||
.spread(function(newNode){ | |||
var relations = [{ | |||
relationType: 'associatedTo', | |||
targets: [newNode.associatedServer] | |||
targets: [members.associatedServer] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After map
, the member is not yet be a plural, so change members
to member
will make sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 after fix the plural variable name.
test this please |
1 similar comment
test this please |
BUILD Archiving » tian » PR_GATE #9 : FAILURE |
BUILD on-tasks #262 : FAILURE |
BUILD on-tasks #155 : FAILURE |
in serviceProfile discovery