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

Feat: add deamon #67

Merged
merged 11 commits into from
Feb 1, 2024
Merged

Feat: add deamon #67

merged 11 commits into from
Feb 1, 2024

Conversation

akitaSummer
Copy link
Contributor

@akitaSummer akitaSummer commented Jan 23, 2024

add a daemon for restart nydusd and overlay when reboot

Copy link

socket-security bot commented Jan 23, 2024

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] filesystem Transitive: environment, shell +3 79.3 kB 4ver

View full report↗︎

integration/index.2.test.js Outdated Show resolved Hide resolved
integration/index.2.test.js Outdated Show resolved Hide resolved
integration/index.2.test.js Outdated Show resolved Hide resolved
integration/index.2.test.js Outdated Show resolved Hide resolved
integration/index.2.test.js Outdated Show resolved Hide resolved
packages/deamon/src/main.rs Outdated Show resolved Hide resolved
packages/deamon/src/main.rs Show resolved Hide resolved
packages/deamon/src/pid.rs Outdated Show resolved Hide resolved
pid,
self.config.get_project_path()
);
match signal::kill(unistd::Pid::from_raw(pid.clone() as i32), Signal::SIGKILL) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Mixing synchronous and asynchronous methods.

packages/deamon/src/pid.rs Outdated Show resolved Hide resolved
@akitaSummer akitaSummer force-pushed the feat/deamon branch 18 times, most recently from cbbc951 to 0bf7a42 Compare January 24, 2024 16:23
@akitaSummer akitaSummer force-pushed the feat/deamon branch 4 times, most recently from 06f1207 to 8e47b5d Compare January 24, 2024 19:46
assert.strictEqual(require(path.join(cwd, 'node_modules', 'esbuild/package.json')).version, '0.15.14');
const nodeModulesDir = path.join(cwd, 'node_modules');

await execa.command(`umount -f ${nodeModulesDir}`);

Check warning

Code scanning / CodeQL

Shell command built from environment values Medium

This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
This shell command depends on an uncontrolled
absolute path
.
integration/index.2.test.js Dismissed Show dismissed Hide dismissed
@akitaSummer akitaSummer changed the title Feat/deamon Feat: add deamon Jan 25, 2024
@elrrrrrrr
Copy link
Member

LGTM

@elrrrrrrr elrrrrrrr merged commit b6246dc into master Feb 1, 2024
14 checks passed
@elrrrrrrr elrrrrrrr deleted the feat/deamon branch February 1, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants