added ext filesystem capabilities

[AidenCohen31]

No description provided.

[brianhlin]

@LincolnBryant this is one of our students who's adding some functionality so that we can use this at the CHTC. Who would be a good person to give this a review?

[LincolnBryant]

Sorry it took so long to take a gander at this. I will test it out tomorrow or so! Thank you for the contribution.

[LincolnBryant]

Hi, trying to build this package I'm not having any luck.

[centos@ext-test osgconnect-quota]$ rpm -qa | grep golang
golang-bin-1.18.4-1.el7.x86_64
golang-src-1.18.4-1.el7.noarch
golang-1.18.4-1.el7.x86_64

[centos@ext-test osgconnect-quota]$ go build
# github.com/maniaclab/osgconnect-quota
./qs2.go:213:23: cannot use email (variable of type *string) as type string in argument to utilizationBar
./qs2.go:219:23: cannot use email (variable of type *string) as type string in argument to utilizationBar
./qs2.go:224:23: cannot use email (variable of type *string) as type string in argument to utilizationBar

maybe it ought to be

[centos@ext-test osgconnect-quota]$ git diff
diff --git a/qs2.go b/qs2.go
index 02fa118..ca3317a 100644
--- a/qs2.go
+++ b/qs2.go
@@ -210,18 +210,18 @@ func main() {
                cq := cephQuota(username, *cephPathPtr)
 
                fmt.Printf("%-10s: ", *cephPathPtr)
-               utilizationBar(&cq, email)
+               utilizationBar(&cq, *email)
        }
        if *pathPtr != "" {
                xq := xfsQuota(username, *pathPtr)
 
                fmt.Printf("%-10s: ", *pathPtr)
-               utilizationBar(&xq, email)
+               utilizationBar(&xq, *email)
        }
        if *extPtr != "" {
                eq := extQuota(username, *extPtr)
                fmt.Printf("%-10s: ", *extPtr)
-               utilizationBar(&eq, email)
+               utilizationBar(&eq, *email)
 
        }
 }

Is it working for you @AidenCohen31 ?

[AidenCohen31]

Looks like I made some changes that need to be reverted, I'll look into it and get back to you tomorrow

…

On Tue, Jan 17, 2023, 2:55 PM Lincoln Bryant ***@***.***> wrote: Hi, trying to build this package I'm not having any luck. ***@***.*** osgconnect-quota]$ rpm -qa | grep golang golang-bin-1.18.4-1.el7.x86_64 golang-src-1.18.4-1.el7.noarch golang-1.18.4-1.el7.x86_64 ***@***.*** osgconnect-quota]$ go build # github.com/maniaclab/osgconnect-quota ./qs2.go:213:23: cannot use email (variable of type *string) as type string in argument to utilizationBar ./qs2.go:219:23: cannot use email (variable of type *string) as type string in argument to utilizationBar ./qs2.go:224:23: cannot use email (variable of type *string) as type string in argument to utilizationBar maybe it ought to be ***@***.*** osgconnect-quota]$ git diff diff --git a/qs2.go b/qs2.go index 02fa118..ca3317a 100644 --- a/qs2.go +++ b/qs2.go @@ -210,18 +210,18 @@ func main() { cq := cephQuota(username, *cephPathPtr) fmt.Printf("%-10s: ", *cephPathPtr) - utilizationBar(&cq, email) + utilizationBar(&cq, *email) } if *pathPtr != "" { xq := xfsQuota(username, *pathPtr) fmt.Printf("%-10s: ", *pathPtr) - utilizationBar(&xq, email) + utilizationBar(&xq, *email) } if *extPtr != "" { eq := extQuota(username, *extPtr) fmt.Printf("%-10s: ", *extPtr) - utilizationBar(&eq, email) + utilizationBar(&eq, *email) } } Is it working for you @AidenCohen31 <https://github.com/AidenCohen31> ? — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AI3ZOTPLHPPWKQH2Z3CW6F3WS4BMDANCNFSM5UXLDY5A> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[AidenCohen31]

finished with my changes, but may need some feedback on if using a sudo command is allowed eg. sudo repquota in order to get ext quotas.

[brianhlin]

@AidenCohen31 I would assume that this runs as a privileged user so it shouldn't need sudo

[LincolnBryant]

Yep, sudo not needed here.