Skip to content

Fix integration test for events#786

Merged
mrunalp merged 1 commit into
opencontainers:masterfrom
hqhq:hq_fix_event_test
Apr 26, 2016
Merged

Fix integration test for events#786
mrunalp merged 1 commit into
opencontainers:masterfrom
hqhq:hq_fix_event_test

Conversation

@hqhq

@hqhq hqhq commented Apr 26, 2016

Copy link
Copy Markdown
Contributor

Signed-off-by: Qiang Huang h.huangqiang@huawei.com

Signed-off-by: Qiang Huang <h.huangqiang@huawei.com>
@hqhq

hqhq commented Apr 26, 2016

Copy link
Copy Markdown
Contributor Author

We missed this in #780 , we should add integration test to CI, @crosbymichael @mrunalp anyone can do that?

@cyphar

cyphar commented Apr 26, 2016

Copy link
Copy Markdown
Member

@hqhq #783 will enable the integration for all make test runs. This includes janky.

@crosbymichael

Copy link
Copy Markdown
Member

LGTM

1 similar comment
@mrunalp

mrunalp commented Apr 26, 2016

Copy link
Copy Markdown
Contributor

LGTM

@mrunalp mrunalp merged commit 7605fce into opencontainers:master Apr 26, 2016
@hqhq hqhq deleted the hq_fix_event_test branch April 27, 2016 01:08
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
These were added in f9dc90b (make link usage consistent across the
specification, 2017-08-09, opencontainers#687) to follow the new _N name-dedup
policy discussed in style.md.  They were removed in ea65eb3
(config-solaris.md: fix info, 2017-04-28, opencontainers#786), overlooking that
policy.  This commit brings them back.

Signed-off-by: W. Trevor King <wking@tremily.us>
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
With the same link update that config-solaris.md got in ea65eb3
(config-solaris.md: fix info, 2017-04-28, opencontainers#786), since the old link
404s.

Signed-off-by: W. Trevor King <wking@tremily.us>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants