-
Notifications
You must be signed in to change notification settings - Fork 659
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
[Monorepo] - Proclaim monorepo #4014
Conversation
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
…nd-go-replace Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
* Add dockerfiles Signed-off-by: Eduardo Apolinario <[email protected]> * Build component using reusable workflow Signed-off-by: Eduardo Apolinario <[email protected]> * Add "Components checks" workflow Signed-off-by: Eduardo Apolinario <[email protected]> * Fix typo Signed-off-by: Eduardo Apolinario <[email protected]> * Rename gh workflow Signed-off-by: Eduardo Apolinario <[email protected]> * Use the correct dockerfile Signed-off-by: Eduardo Apolinario <[email protected]> * Enable endtoend tests Signed-off-by: Eduardo Apolinario <[email protected]> * Use correct path to end2end reusable workflow Signed-off-by: Eduardo Apolinario <[email protected]> * Use unique prefixes for the cache Signed-off-by: Eduardo Apolinario <[email protected]> * Use tmp/tmp Signed-off-by: Eduardo Apolinario <[email protected]> * Be more explicit about the path components docker images are saved to Signed-off-by: Eduardo Apolinario <[email protected]> * Test only overriding datacatalog image Signed-off-by: Eduardo Apolinario <[email protected]> * Comment out actual helm upgrades (i.e. simply run tests) Signed-off-by: Eduardo Apolinario <[email protected]> * Comment out end2end and bring integration tests Signed-off-by: Eduardo Apolinario <[email protected]> * Fix typo in definition of priorities Signed-off-by: Eduardo Apolinario <[email protected]> * Hardcode go version to 1.19 Signed-off-by: Eduardo Apolinario <[email protected]> * Use correct working directory in integration.yml Signed-off-by: Eduardo Apolinario <[email protected]> * Enable flytepropeller integration tests Signed-off-by: Eduardo Apolinario <[email protected]> * Unpack envvars prior to calling reusable workflow Signed-off-by: Eduardo Apolinario <[email protected]> * Enable go_generate.yml Signed-off-by: Eduardo Apolinario <[email protected]> * Enable push_docker_image Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flytecopilot go generate Signed-off-by: Eduardo Apolinario <[email protected]> * Fix image tags Signed-off-by: Eduardo Apolinario <[email protected]> * Enable lint and unit tests Signed-off-by: Eduardo Apolinario <[email protected]> * Pass component to lint and unit-tests jobs Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flytestdlib unit test Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flyteplugins test Signed-off-by: Eduardo Apolinario <[email protected]> * Build flytescheduler image Signed-off-by: Eduardo Apolinario <[email protected]> * Monorepo ci checks fix lint (#4032) * Fix flyteplugins lint errors Signed-off-by: Eduardo Apolinario <[email protected]> * Fix datacatalog lint errors Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flyteadmin lint errors Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flyteaidl lint errors Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flytepropeller lint errors Signed-off-by: Eduardo Apolinario <[email protected]> * Comment flytecopilot lint and flyteidl unit tests Signed-off-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]>
* Fix single-binary Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flyteplugins references in single-binary Signed-off-by: Eduardo Apolinario <[email protected]> * Point to local flyteidl in single-binary Signed-off-by: Eduardo Apolinario <[email protected]> * Fix flytecopilot references Signed-off-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]>
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4014 +/- ##
=========================================
Coverage ? 59.19%
=========================================
Files ? 549
Lines ? 39512
Branches ? 0
=========================================
Hits ? 23388
Misses ? 13822
Partials ? 2302 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Eduardo Apolinario <[email protected]>
bb101c0
to
4fdf255
Compare
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.
LGTM, but there is a failing test
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Describe your changes
This PR proclaims the 7 modules introduced by #4012 as the new place for contribution.
The list of changes:
Let's talk about each change separately.
Rename the modules
The rules that govern the publishing of multi-module repositories are described in Go Modules Reference - The Go Programming Language, more specifically:
This essentially means that in a subsequent PR, the one where we deal with release, we'll need to generate a few more tags, one per-component.
This PR simply fixes the names of the modules and ensures that
go replace
directives point to each module. I manually tested the compilation of each component. A subsequent PR is going to fix the single-binary and the components Dockerfiles.Build single-binary using local modules
Done in this stacked PR.
Component CI checks
Done as part of this stacked PR.
Changes to releases
edit: this is going to come in a separate PR. For now merges to master will not trigger image builds nor releases.
We're going to tag merges to master using semver versionsing, so these new tags will update the patch version and use that to create a new release. Exactly how we do currently in each component. These automatic releases are not going to create new helm releases.