Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Getting ready for v0.1.0 #73

Merged
merged 76 commits into from
Aug 29, 2024
Merged

Getting ready for v0.1.0 #73

merged 76 commits into from
Aug 29, 2024

Conversation

ohuu
Copy link
Collaborator

@ohuu ohuu commented Aug 29, 2024 •
edited
Loading

This is a big refactor of the project after @seal integrated Cardamon with the frontend. The refactor includes:

  • Replacing sqlx with sea_orm
  • Support for mysql & postgres
  • Removal of all Remote and Local DOA, replaced with a single DAO
  • Control database access and logging level through env vars
  • Release workflow (builds to linux, mac and windows and publishes to crates.io)
  • Merges the cardamon server with the cardamon binary so we have a single binary to release

seal and others added 30 commits July 2, 2024 11:49
@seal @ohuu
Start of ui-server
3deb28c
@ohuu
Start of openapi spec
8476324
@seal @ohuu
Finished swagger, start of logic
6aacae6
@ohuu
/api/runs endpoint
fc478f4
@seal @ohuu
Update for timing logic
8215467
@ohuu
Get database ready for ui server
4cbc60c 
- Added "run" table
- Added foreign key for "run_id" to cpu_metrics and scenario_iterations
  tables
- Added dao for run table
- Cleaned up some stuff
@ohuu
updated tests
846b5a8
@ohuu
made stop_time in Run an optional value
8e1348c
@ohuu
Add dataset classes
cc03414 
- Renamed scenario_iteration to just iteration
- Renamed cpu_metrics to just metrics
- Removed run from the database so there are only 2 tables in the database
- Added DatasetBuilder objects for building a dataset
@seal
Starting re-write of feature/ui-server
0cbe359
@seal
Updated routes and data access
46c31f5 
- Started on creating routes
- Did state management for routes
@seal
Added most of first ui-route
36e680c 
Need to add ability to return count for pages.
@seal
Ui -server improvement
548af05
@seal
Cors + search query for route
bf97cf9
@seal
Git ignore
5bc5a9d
@seal
Update main.rs to use local dao updated name
716cc6b
@seal
Merge with main
f0dad90
@seal
Remove flake lock and nix from ignore
de8ebbc
@seal
Ui re-write
531c795
@seal
Added
a758a95 
 - Commands to dataset for
    - Fetching unique run id's
    - Fetching by scenario name *and run id*
    - Total unique scenarios ( unique by name, not run_id)
    - Paginated unique scenarios
 - Pagination for both endpoints
    - On all scenarios endpoint by *name*
    - On specific scenario by *run_id*
@seal
Testing sh scripts for ui_server routes - must be unix based
b60bd82
@seal
- Made cpu_util be calculated before db insertion
ecf4ccc 
- Changed all times to UTC - ui's job
- Added run insertion to DB to remove foreign key failures
- Altereted unqiue scenarios to be div by 0 safe
- Made all scenario ep give last 5 average
- Made avg_cpu calculate properly
@seal
Forgot cargo
d50bcac
@seal
Updated example and updated .gitignore
a14d8ef
@seal
Fix fixtures test case - syntax
133f1f4
@seal
Changed stdout file to append, previously would over-write previous run
8154079
@seal
Cleanup stdout and err before each run
8fb3644
@seal
Prepare main.rs for remote server, allowing for both metrics remote URl
a9a0038 
and local DB
Change migrations timestamps so run is first ( Postgres integration
won't allow for table to be created with run being made first )
@seal
Db prepare
2b537dc 
Remove dependancies
@seal
Fix deps and sqlx prepare
228aeb0
ohuu added 26 commits August 27, 2024 10:05
@ohuu
testing windows build
6fedf3a
@ohuu
bumped to v0.0.3
4dadc62
@ohuu
trying to get ui assets into crates
a186b79
@ohuu
trying to get ui assets into crates
8f7d7f5
@ohuu
ignoring public ui assets in cardamon
602e87b
@ohuu
removed public assets from cardamon
50c62bf
@ohuu
removed stuff from the crate when packaging
9cc2c8a
@ohuu
include src/public files in crate when packaging
3acb2aa
@ohuu
added build:release job to workflow
301d55b
@ohuu
added support for windows
090ce5c
@ohuu
got workflow pipeline correct
f3f689e
@ohuu
trying to fix windows build
c54546a
@ohuu
try again
0ad59ba
@ohuu
added extension to cardamon executable
71da953
@ohuu
stopped clash of artifact name
2812bf8
@ohuu
add macos to release workflow
4a63741
@ohuu
bumped to v0.0.4
0d41b0d
@ohuu
adding checksums
6bf3736
@ohuu
bumped to v0.0.5
6a53fb9
@ohuu
adding checksum
e68bbb2
@ohuu
bumped to v0.0.6
5c90b5c
@ohuu
fixed line endings for long gh command
c4ba101
@ohuu
finished checksums in release
e863035
@ohuu
removed version number from archives to make it easier for install sc…
e342e08 
…ript to install cardamon
@ohuu
removed version number from archives to make it easier for install sc…
47f83a8 
…ript to install cardamon
@ohuu
improved README
f75091a
@ohuu
Copy link
Collaborator Author

ohuu commented Aug 29, 2024

closes #67, #68

@ohuu ohuu merged commit 1419a80 into main Aug 29, 2024
3 of 4 checks passed
@ohuu ohuu deleted the release/0.1.0 branch August 29, 2024 16:10
@anewmandev anewmandev mentioned this pull request Aug 30, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ohuu @seal