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

The example: 'camera management' got errors with pipeline status #180

Closed
jiekechoo opened this issue Feb 1, 2023 · 3 comments · Fixed by #182
Closed

The example: 'camera management' got errors with pipeline status #180

jiekechoo opened this issue Feb 1, 2023 · 3 comments · Fixed by #182
Labels
question Further information is requested

Comments

@jiekechoo
Copy link

clone from main branch or v2.3.0, here is the error message in EVAM service:

/pipelines/object_detection/person_vehicle_bike/9/pipelines/object_detection/person_vehicle_bike/9
the url from 'camera management' app service, it's duplicate. how to fix it ?

WARNING:tornado.access:404 GET /pipelines/object_detection/person_vehicle_bike/9/pipelines/object_detection/person_vehicle_bike/9/status (172.20.0.1) 0.70ms
WARNING:tornado.access:404 GET /pipelines/object_detection/person_vehicle_bike/7/pipelines/object_detection/person_vehicle_bike/7/status (172.20.0.1) 0.74ms
WARNING:tornado.access:404 GET /pipelines/object_detection/person_vehicle_bike/8/pipelines/object_detection/person_vehicle_bike/8/status (172.20.0.1) 0.73ms
@jiekechoo jiekechoo added the bug Something isn't working label Feb 1, 2023
@lenny-goodell
Copy link
Member

@ajcasagrande , @presatish, FYI

@lenny-goodell lenny-goodell added question Further information is requested and removed bug Something isn't working labels Feb 1, 2023
@lenny-goodell lenny-goodell moved this to New Issues in Application Services Feb 1, 2023
@jiekechoo
Copy link
Author

in file: util.go, comment these code, it works.

func issueGetRequest(ctx context.Context, res interface{}, baseUrl string, requestPath string) (err error) {
	// u, err := url.Parse(baseUrl)
	// if err != nil {
	// 	return err
	// }
	// requestPath = path.Join(u.Path, requestPath)
	return utils.GetRequest(ctx, &res, baseUrl, requestPath, nil)
}

func issueDeleteRequest(ctx context.Context, res interface{}, baseUrl string, requestPath string) (err error) {
	// u, err := url.Parse(baseUrl)
	// if err != nil {
	// 	return err
	// }
	// requestPath = path.Join(u.Path, requestPath)
	return utils.DeleteRequest(ctx, &res, baseUrl, requestPath)
}

because the file: evam.go has combined the URL

func (info PipelineInfo) getPipelineUrl(evamBaseUrl string) (string, error) {
	uri, err := url.Parse(evamBaseUrl)
	if err != nil {
		return "", err
	}
	uri.Path = path.Join(uri.Path, "pipelines", info.Name, info.Version, info.Id)
	return uri.String(), nil
}

@github-project-automation github-project-automation bot moved this from New Issues to Done in Application Services Feb 2, 2023
@ajcasagrande
Copy link

This is due to the following issue having been fixed, and the workaround code not being removed:
edgexfoundry/go-mod-core-contracts#719

@lenny-goodell lenny-goodell moved this from Done to Previously Done in Application Services Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants