- Notifications
You must be signed in to change notification settings - Fork1k
feat: Update generated schema for provisionerd#121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
d3a4eb8 to39ff6c7Comparecodecovbot commentedFeb 1, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## main #121 +/- ##==========================================+ Coverage 71.90% 72.24% +0.34%========================================== Files 91 91 Lines 3670 3751 +81 Branches 59 59 ==========================================+ Hits 2639 2710 +71- Misses 814 824 +10 Partials 217 217
Continue to review full report at Codecov.
|
39ff6c7 to4f8309eCompareThese changes were part of a bulk change to create provisionerd.They were ripped out for simpler reviewing.
4f8309e tob37edc2Compare| deferreader.Close() | ||
| deferwriter.Close() | ||
| gofunc() { | ||
| decoder:=json.NewDecoder(reader) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks like we're getting really close to being able to stream the terraform logs 🎉
| before_id uuid, | ||
| after_id uuid, | ||
| transition workspace_transitionNOT NULL, | ||
| initiatorvarchar(255)NOT NULL, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Unrelated to this PR, but it reminded me - what are values forinitiator like? Is it supposed to reference the user that initiated the transition, or something else?
These changes were part of a bulk change to create provisionerd.
They were ripped out for simpler reviewing.