-
Notifications
You must be signed in to change notification settings - Fork 59
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
Application Pipelines #2094
Application Pipelines #2094
Conversation
@Pezmc would appreciate some advice/input on this outstanding task. We still just repeating a HTTP request every X seconds, or did something more sophisticated get added that I haven't seen? |
Development work should now be complete, apart from:
For early next week, need to prioritise test coverage for the new work. I'm happy to take the E2E front-end testing, but will need assistance server-side. @Pezmc if you have the bandwidth, would be great to get the assist. TLDR for server-side changes and where test coverage is likely required:
|
Have added limited E2E testing as currently, we have no way of testing licensed features in the E2E tests - also the same issue with the Team Library. |
The unit test failure on postgres comes down to how CE/EE features co-exist in the database models and when/if migrations get applied. For a clean run of the forge app (no existing database), the code relies 100% on the sequelize models to generate the database structure. The migrations do not get used. This PR introduces a migration that adds a relation between the (CE) Projects table and (EE) PipelineStages table. However, in a clean run, that migration will not get applied. So the Projects table does not have the expected PipelineStageId column. Furthermore, the existing migration will fail when upgrading an existing instance of FF - because the migrations run before sequelize initiates the models, so the PipelineStages table won't exist at that point in time. I need to spend some more time working out how to fix all of this. In summary it involves:
This is a few hours work, including tests. |
Add Audit Events for DevOps Pipelines
Having been poking at this to understand the database requirements a bit better. Hit a couple UX things:
|
Move pipeline routes under ee src tree
Pipelines: Edit and Update
Pipelines: General Polish
Rename target to NextStageId and NextStage for clarity
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.
Approving for myself, see: https://flowforgeworkspace.slack.com/archives/C032Q63FGG1/p1685094359788189
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.
Approving for myself, see: https://flowforgeworkspace.slack.com/archives/C032Q63FGG1/p1685094359788189
Description
Introduces a new "Pipelines" view at the Application level.
Functionality:
To-do:
needsPermission
for new endpointsRelated Issue(s)
Closes #2082
Closes #2076
Checklist