Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

scripts: Fix collect script bug URL #972

Merged
merged 2 commits into from
Jan 31, 2018

Conversation

jodh-intel
Copy link
Contributor

Correct the URL displayed when the user runs
{cc|kata}-collect-data.sh -h.

Fixes #971.

Signed-off-by: James O. D. Hunt james.o.hunt@intel.com

Correct the URL displayed when the user runs
`{cc|kata}-collect-data.sh -h`.

Fixes clearcontainers#971.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Show bug URL when user runs `make help`.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm


# build type for Kata Containers
KATA_TYPE = kata

KATA_PROJECT_NAME = Kata Containers
KATA_PROJECT_TAG = kata-containers
KATA_PROJECT_URL = https://github.com/kata-containers
KATA_BUG_URL = $(KATA_PROJECT_URL)/kata-containers/issues/new
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess that is in theory the best, most generic and neutral, URL we have. Shame 'orgs' don't have an Issues section on github.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah. I suspect we'll end up with a bunch of issues both at this URL and at the runtime one but from a triage perspective, it would be good if we could get them reported in the "main" repo as defined here.

@clearcontainersbot
Copy link

kubernetes qa-passed 👍

@jodh-intel jodh-intel merged commit d9671a7 into clearcontainers:master Jan 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants