Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

ci: Exclude *.service.in files from license static check#1101

Closed
sboeuf wants to merge 1 commit intokata-containers:masterfrom
sboeuf:fix_license_chk
Closed

ci: Exclude *.service.in files from license static check#1101
sboeuf wants to merge 1 commit intokata-containers:masterfrom
sboeuf:fix_license_chk

Conversation

@sboeuf
Copy link
Copy Markdown

@sboeuf sboeuf commented Jan 30, 2019

In order to prevent CI failures when systemd service template
files are modified, this commit excludes *.service.in files from
the license static check script.

Fixes #1100

Signed-off-by: Sebastien Boeuf sebastien.boeuf@intel.com

In order to prevent CI failures when systemd service template
files are modified, this commit excludes *.service.in files from
the license static check script.

Fixes kata-containers#1100

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
@sboeuf
Copy link
Copy Markdown
Author

sboeuf commented Jan 30, 2019

/test

sboeuf pushed a commit to sboeuf/agent that referenced this pull request Jan 30, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
@jodh-intel
Copy link
Copy Markdown

I guess we could do this, but since systemd units support standard hash comments, couldn't (in fact shouldn't) we also just add the copyright header to the following?:

@sboeuf
Copy link
Copy Markdown
Author

sboeuf commented Jan 30, 2019

@jodh-intel Fair enough, let's drop this PR and add the license.

sboeuf pushed a commit to sboeuf/agent that referenced this pull request Jan 30, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
@sboeuf sboeuf closed this Jan 30, 2019
jodh-intel pushed a commit to jodh-intel/agent-1 that referenced this pull request Feb 25, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
jodh-intel pushed a commit to jodh-intel/agent-1 that referenced this pull request Mar 8, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
jodh-intel pushed a commit to jodh-intel/agent-1 that referenced this pull request Mar 8, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
jodh-intel pushed a commit to jodh-intel/agent-1 that referenced this pull request Mar 13, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
egernst pushed a commit to egernst/agent-1 that referenced this pull request Apr 16, 2019
The semantic around the agent is that it should be a passive
component, hence it should not implicitly shut down the VM.

Instead, we expect the kata-runtime to be responsible for this,
using the appropriate VM interface to stop it.

Fixes kata-containers#449

Depends-on: github.com/kata-containers/tests#1101

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
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.

2 participants