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

Add slugid command #18

Closed
walac opened this issue Sep 15, 2016 · 8 comments
Closed

Add slugid command #18

walac opened this issue Sep 15, 2016 · 8 comments

Comments

@walac
Copy link
Contributor

walac commented Sep 15, 2016

A slugid is an uuid used to uniquely identify tasks. The goal here is to make slugid-go interface into a taskcluster-cli command:

taskcluster slugid v4
taskcluster slugid nice
taskcluster slugid from-uuid <uuid>
taskcluster slugid to-uuid <slugid>
@ApheleiaS
Copy link

Hello,
Can you please assign me to this bug?
Thanks :)

@walac
Copy link
Contributor Author

walac commented Sep 18, 2016

It is yours, please accept team member invitation to make the assignment.

@walac
Copy link
Contributor Author

walac commented Sep 19, 2016

Update: sorry, but you are already assigned to bug #20, once you are done there, you can pick this.

@djmitche
Copy link
Contributor

@petemoore points out this already exists in the https://github.com/taskcluster/pulse-go project

@djmitche djmitche reopened this Jan 11, 2017
@djmitche
Copy link
Contributor

Although now that I see #24, this makes sense, too:

TASKID=$(taskcluster slugid)
cat <<EOF | taskcluster queue createTask $TASKID
...

@petemoore
Copy link
Member

@petemoore points out this already exists in the https://github.com/taskcluster/pulse-go project

It actually exists here:
https://github.com/taskcluster/slugid-go#command-line-usage

I tend to think we don't need anything here, and we can close this issue.
@djmitche ok with you?

@djmitche
Copy link
Contributor

Oops, yes, I copied the wrong URL.

I think this would fit well into the formulation above for creating a new task from a shell script. Installing a separate binary just to generate a slugid goes against the everything-in-one-package approach for taskcluster-cli. And this is going to be a very simple plugin!

@yannlandry yannlandry self-assigned this Jan 20, 2017
@yannlandry
Copy link
Contributor

On it, replacing from-uuid to encode and to-uuid to decode.

yannlandry added a commit that referenced this issue Jan 21, 2017
yannlandry added a commit that referenced this issue Jan 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants