search

servo / saltfs

Salt Stack Filesystem
Apache License 2.0
56 stars 107 forks source link

Add kunalmohan to try users #1002

Closed kunalmohan closed 4 years ago

kunalmohan commented 4 years ago

I will be working on WebGPU CTS and having "try" permissions will be quite helpful.

jdm commented 4 years ago

Filed https://github.com/servo/saltfs/issues/1003 about the CI failures.

jdm commented 4 years ago

Try changing https://github.com/servo/saltfs/blob/90036502c8fac2291fba372b3309bafef605f5b2/salt/common.sls#L6-L8 as well.

kunalmohan commented 4 years ago

Some success, but still 2 failures

kunalmohan commented 4 years ago

I fail to understand why some tasks return gpg: no valid OpenPGP data found. and others Ok

jdm commented 4 years ago

@bors-servo r+

bors-servo commented 4 years ago

:pushpin: Commit af9d6b4 has been approved by jdm

bors-servo commented 4 years ago

:hourglass: Testing commit af9d6b4dfcb084d7472b6a28f1a81941091d18f0 with merge 29a06798b67ee7f3cb7897a3752117dc4ccc86ec...

bors-servo commented 4 years ago

:broken_heart: Test failed - checks-travis

jdm commented 4 years ago

Ok, I figured out why those two jobs are failing - the URLs for the archived salt versions don't include the patch release. I'm going to try upgrading us to a non-archived version instead in #1004.

jdm commented 4 years ago

@bors-servo retry

jdm commented 4 years ago

@bors-servo r+

bors-servo commented 4 years ago

:pushpin: Commit c91daf2 has been approved by jdm

bors-servo commented 4 years ago

:hourglass: Testing commit c91daf2c70acc1dbb781d3d6bb2ddbfd3149ee42 with merge 4cbef3b82b66592d3d59f190593ae5fc4a53e099...

bors-servo commented 4 years ago

:sunny: Test successful - checks-travis Approved by: jdm Pushing 4cbef3b82b66592d3d59f190593ae5fc4a53e099 to master...

jdm commented 4 years ago

This change has been deployed; you should be able to use try commands now.