-
Notifications
You must be signed in to change notification settings - Fork 582
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Update generated schema for provisionerd #121
Conversation
d3a4eb8
to
39ff6c7
Compare
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
to
4f8309e
Compare
These changes were part of a bulk change to create provisionerd. They were ripped out for simpler reviewing.
4f8309e
to
b37edc2
Compare
defer reader.Close() | ||
defer writer.Close() | ||
go func() { | ||
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 🎉
@@ -22,14 +23,16 @@ CREATE TABLE workspace_history ( | |||
completed_at timestamptz, | |||
workspace_id uuid NOT NULL REFERENCES workspace (id) ON DELETE CASCADE, | |||
project_history_id uuid NOT NULL REFERENCES project_history (id) ON DELETE CASCADE, | |||
name varchar(64) NOT NULL, | |||
before_id uuid, | |||
after_id uuid, | |||
transition workspace_transition NOT NULL, | |||
initiator varchar(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 for initiator
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.