Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

agent: Remove duplicate function #2181

Merged
merged 2 commits into from
Aug 26, 2020
Merged

agent: Remove duplicate function #2181

merged 2 commits into from
Aug 26, 2020

Conversation

utopiabound
Copy link
Contributor

@utopiabound utopiabound commented Aug 24, 2020

Signed-off-by: Nathaniel Clark nclark@whamcloud.com


This change is Reviewable

Signed-off-by: Nathaniel Clark <nclark@whamcloud.com>
@utopiabound utopiabound requested review from jgrund and a team August 24, 2020 14:41
@utopiabound utopiabound self-assigned this Aug 24, 2020
beevans
beevans previously approved these changes Aug 24, 2020
johnsonw
johnsonw previously approved these changes Aug 24, 2020
nlinker
nlinker previously approved these changes Aug 24, 2020
@@ -3,6 +3,7 @@
// license that can be found in the LICENSE file.

use crate::{
action_plugins::stratagem::util::search_rootpath,
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't this be better suited in lustre.rs? Working with fids isn't inherently stratagem specific

Signed-off-by: Nathaniel Clark <nclark@whamcloud.com>
@jgrund jgrund merged commit 74b2bda into master Aug 26, 2020
@jgrund jgrund deleted the action-duplicate-fn branch August 26, 2020 18:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants