← Back to issue list

cli: implement `enable-ci travis --refresh` command

View original Github issue

Metadata

Project
snapcraft
Number
#932
Type
pull request
State
merged
Author
cprov
Labels
—
Created
Updated
Closed

Current evaluation

Merged in #932 via fe7e10826, adding `enable-ci travis --refresh` to renew expiring Travis credentials; follow-up commit 52c53dd fixed hangs and permission handling.

Resolution: closed resolved

Reason: PR #932 was merged as commit fe7e10826, implementing credential refresh for Travis. Follow-up commit 52c53dd addressed external-repository permission errors and a Python hang before completion.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This will allow users to refresh credentials bound to expire after 1 year (clock-ticking bomb) or more likely when there is a SSO password change (rotation). Travis 'deploy' phase will fail on expired credentials, so users will know when it's case to refresh. We could issue macaroons that never expire (age or passwd-changes), but I guess it ends up being a lot more difficult to handle in case of compromise. LP: #1646660

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in #932 via fe7e10826, adding `enable-ci travis --refresh` to renew expiring Travis credentials; follow-up commit 52c53dd fixed hangs and permission handling.
stealth/space-bunny-alpha — closed resolved Merged PR #932 added `enable-ci travis --refresh` to renew expiring Travis credentials; follow-up commit 52c53dd addressed hangs and external-repository permission errors.
qwen/qwen3.6-35b-a3b — — Merged. Adds the snapcraft enable-ci travis --refresh command to update expired Travis credentials. Addressed reviewer feedback on external repo permission errors, with follow-up fixes for hanging processes and permission checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged enable-ci travis --refresh to handle expired Travis credentials. Reviewers noted a hang and permission error on third-party repos. The author fixed both issues in a follow-up PR by adding proper permission checks and process handling.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1646660 Refresh `enable-ci` credentials snapcraft (launchpad) closed Feature delivered: `snapcraft enable-ci travis --refresh` added by merged PR snapcore/snapcraft#932 (commit fe7e10826); Launchpad bug closed by sergiusens on 2016-12-02 pointing at that PR.
76%