Skip to content
This repository has been archived by the owner on Oct 21, 2020. It is now read-only.

Add namespace extended attributes to directory #672

Merged

Conversation

poidag-zz
Copy link
Contributor

Add extended attribute to directory, this can be used for metrics/retention cleanup which I intend to contribute to this repository. The idea here is to build a relation for a directory to a namespace without the need for string parsing for unknown namespace names. i.e:

ns
ns-1
ns-1-dev

etc

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 15, 2018
@poidag-zz poidag-zz force-pushed the feature/add-xattr-to-directory branch from 70b5ef1 to 8f109fd Compare March 15, 2018 17:36
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 15, 2018
@rootfs
Copy link
Contributor

rootfs commented Mar 16, 2018

/assign @wongma7

@poidag-zz
Copy link
Contributor Author

Hi @wongma7

Do you have any thoughts on this?

@poidag-zz
Copy link
Contributor Author

/remove-lifecycle rotten

@poidag-zz
Copy link
Contributor Author

Hi @rootfs @wongma7,

let me know if there is any more required on this one?

@poidag-zz poidag-zz force-pushed the feature/add-xattr-to-directory branch from 8f109fd to 1023a9f Compare June 5, 2018 14:26
@poidag-zz
Copy link
Contributor Author

@rootfs @wongma7

I have rebased against glide.yaml to fix conflict. Are you able to add some input on this PR or merge?

@wongma7
Copy link
Contributor

wongma7 commented Jun 14, 2018

/lgtm
thank you and apologies for delay

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 14, 2018
@wongma7 wongma7 merged commit 473d67c into kubernetes-retired:master Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/nfs-client cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants