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

BREAKING CHANGE: merge processes into a single process #140

Merged
merged 32 commits into from
Aug 27, 2023
Merged

BREAKING CHANGE: merge processes into a single process #140

merged 32 commits into from
Aug 27, 2023

Conversation

pueding
Copy link
Member

@pueding pueding commented Aug 27, 2023

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • DevOps process (non-breaking change which improves efficiency and reliability for CI/CD)
  • Documentation only

What this PR does / why we need it:

This PR merge wave-cli, wave-metrics, api-server, controller into wave-autoscale binary project. It simplifies the process so users can run Wave Autoscale by executing wave-autoscale binary.

Which issue/s this PR fixes

#135

How Has This Been Tested?

  • unit tests in wave-autoscale project

Checklist:

@pueding
refactor(wave-cli): extract functions to get them tested
8d6ea2e
@pueding
refactor(azure_virtual_machine_scale_sets.rs): rename unused variable…
3999528 
… 'result_text' to '_result_text' to remove compiler warning
@pueding
feat(data-layer): separate creating an instance and syncing with the …
da3db15 
…database
@pueding
fix(api-server): add synchronization of data layer before returning a…
f8fb818 
…pp state in get_app_state_for_test function
@pueding
fix(data-layer): remove unnecessary check for empty definition path a…
1ad6a61 
…nd handle error when loading definition file into database
@pueding
chore(moon.yml): add 'test' as a dependency for build tasks in wave-a…
59204d8 
…utoscale, wave-cli, and wave-metrics
@pueding
chore: arhicver
2d5915a
@pueding
refactor(wave-cli): replace hard-coded sleep duration with INTERVAL c…
bc9e060 
…onstant for better control and flexibility
@pueding
feat(wave_config): update default values and struct fields in WaveConfig
31d81a3
@pueding
fix(controller): create DataLayer in main and reuse it
035ebc3
@pueding
Merge remote-tracking branch 'origin/133-wave-cli-reliability' into 1…
f458450 
…35-merge-processes-into-wave-autoscale
@pueding
feat(api-server): make api-server as a module, not process
9dc240a
@pueding
refactor(data_layer): rename the watch method to `watch_definitions…
4b4cc46 
…` for better clarity and semantics
@pueding
chore(wave_config.rs): add Clone trait to WaveConfig struct for bette…
053b836 
…r usability and flexibility
@pueding
feat(wave-autoscale): merge api-server and wave-metrics into the cont…
7e4ab1f 
…roller
@pueding
BREAKING CHANGE: merge wave-cli, wave-metrics, api-server into wave-a…
7a5d6c3 
…utoscale
@pueding
fix: wave_config path
f220eae
@pueding
chore: package-lock.json
9be7ca5
@pueding
chore(README.md): improve table readability by bolding column headers…
eb82ded 
… and aligning table cells
@pueding
Merge remote-tracking branch 'origin/131-metric-statistic' into 135-m…
251a07a 
…erge-processes-into-wave-autoscale
@pueding
chore(api-server): change the type from "application" to "library"
fd58690
@pueding
chore(moon): remove archiver:audit lint task from pre-push hook
74d8cef
@pueding
docs(README.md): update links and formatting for Telegraf and Vector …
cc227ea 
…sections
@pueding
Merge remote-tracking branch 'origin/131-metric-statistic' into 135-m…
b4077bf 
…erge-processes-into-wave-autoscale
@pueding
Merge remote-tracking branch 'origin/main' into 135-merge-processes-i…
e7f31ed 
…nto-wave-autoscale
@pueding
Merge remote-tracking branch 'origin/main' into 135-merge-processes-i…
c1e25b7 
…nto-wave-autoscale
@pueding
chore: remove archiver:audit in pre-commit
8e78a80
@pueding
chore: pre-commit changed
35a0a72
@pueding
feat(wave-autoscale): handle ctrl+c signal
a49003d
@pueding
chore(wave-autoscale): debug-aarch64-apple-darwin
fef64c4
@pueding pueding linked an issue Aug 27, 2023 that may be closed by this pull request
Merge processes into wave-autoscale #135
Closed
@pueding pueding merged commit 3ebccce into main Aug 27, 2023
1 check passed
@pueding pueding deleted the 135-merge-processes-into-wave-autoscale branch August 27, 2023 14:03
@pueding pueding changed the title 135 merge processes into wave autoscale BREAKING CHANGE: merge processes into a single process Aug 27, 2023
@pueding pueding added this to the v0.0.12 milestone Aug 27, 2023
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
v0.0.12
Development

Successfully merging this pull request may close these issues.

Merge processes into wave-autoscale
1 participant
@pueding