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

Fix symlink #1798

Merged
merged 8 commits into from
Jun 9, 2022
Merged

Fix symlink #1798

merged 8 commits into from
Jun 9, 2022

Conversation

Strikerzee
Copy link
Contributor

@Strikerzee Strikerzee commented May 24, 2022

Fix the symlink. (Cherry-picked from personal/nitin/xdatamove)

  • Calling filepath.evalsymlinks in case of linux.
  • Explicit checking "." (current directory) for symlink.

Remove snippet of code not relevant to the cherry-picked commits.

ste/mgr-JobMgr.go Outdated Show resolved Hide resolved
jobsAdmin/init.go Outdated Show resolved Hide resolved
@zezha-msft zezha-msft added this to the xdm-integration milestone Jun 2, 2022
@Strikerzee Strikerzee changed the title Introduce changes for job progress status and fix symlink Fix symlink Jun 6, 2022
@mohsha-msft mohsha-msft requested a review from nakulkar-msft June 7, 2022 10:27
nitin-deamon and others added 7 commits June 7, 2022 12:52
Changes for jobProgress status.

- If job marked complete, drain all the messages from jobStatus channel.

Fix the symlink.

- Calling filepath.evalsymlinks in case of linux.
- Explicit checking "." (current directory) for symlink.

Remove snippet of code not relevant to the cherry-picked commits.
- Calling filepath.evalsymlinks in case of linux.
- Explicit checking "." (current directory) for symlink.
@Strikerzee Strikerzee force-pushed the personal/arnav/job-progress-fix-symlink branch from c820eb8 to ec04d04 Compare June 7, 2022 13:07
ste/mgr-JobMgr.go Outdated Show resolved Hide resolved
cmd/zc_traverser_local.go Show resolved Hide resolved
@Strikerzee Strikerzee merged commit 583e6e7 into dev Jun 9, 2022
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.

5 participants