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.

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

Deployment configurations #6

Merged
merged 23 commits into from
Feb 20, 2021
Merged

Deployment configurations #6

merged 23 commits into from
Feb 20, 2021

Conversation

petertrr
Copy link
Member

@petertrr petertrr commented Feb 11, 2021

What's done:

  • Added spring-boot-gradle-plugin to enable packaging of executable jars
  • Added docker-compose.yaml
  • Added Dockerfiles for backend and orchestrator
  • Added prometheus.yml
  • Expose prometheus metrics with spring-boot-actuator
  • Added gradle plugin to discover version based on git tag
  • Enabled build-info.properties generation for spring boot

TODO:

  • Finish dockerfiles
  • Test individual containers locally
  • Start registry before building images in compose file separate command for registry
  • Test compose & swarm locally
  • Add third party services configs
  • Consider building images with spring boot gradle plugin (also solves problem with image versions)
  • Endpoint with build info
  • Pushing images to local docker registry doesn't work, but it's fine for deploy on single machine now and needs to be tested properly on actual server
  • Set image version in docker-compose.yaml during build

### What's done:
* save-stack.yaml
* Dockerfiles
@codecov
Copy link

codecov bot commented Feb 11, 2021

Codecov Report

Merging #6 (7811767) into master (7b8dd1d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage     92.75%   92.75%           
  Complexity       11       11           
=========================================
  Files             4        4           
  Lines            69       69           
  Branches          1        1           
=========================================
  Hits             64       64           
  Misses            4        4           
  Partials          1        1           
Flag Coverage Δ Complexity Δ
unittests 92.75% <ø> (ø) 0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b8dd1d...7811767. Read the comment docs.

### What's done:
* spring-boot gradle plugin
* Dockerfiles
### What's done:
* spring-boot gradle plugin
* Dockerfiles
### What's done:
* Updated docker-compose.yaml
### What's done:
* Added prometheus.yml with docker swarm SD
* Added micromether-registry-prometheus for spring boot actuator
* Added save-deploy directory
@petertrr petertrr force-pushed the infra/deploy-configs branch 12 times, most recently from 11f85f4 to c0075b3 Compare February 16, 2021 11:58
### What's done:
* Common gradle task for spring configuration
* Wrapped deployment commands in gradle tasks
### What's done:
* Compile with coverage
### What's done:
* Added gradle plugin for setting version
### What's done:
* Enable build-info.properties generation for spring boot
### What's done:
* Changed Java version to 11
* Added Versions entries for java version
### What's done:
* Use docker-compose instead of swarm for now
### What's done:
* Fix for prometheus config
### What's done:
* Fix for prometheus config
### What's done:
* Fix for prometheus config
### What's done:
* Fix for prometheus config
@petertrr petertrr marked this pull request as ready for review February 18, 2021 08:08
server.port = 5100
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

create a table with ports

Copy link
Member

@orchestr7 orchestr7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

### What's done:
* Ports allocation
### What's done:
* Template docker-compose.yaml.template
* Set version during build
### What's done:
* Added node-exporter
### What's done:
* Use snapshot
### What's done:
* Fixes
@petertrr petertrr changed the title WIP: deployment configurations Вeployment configurations Feb 19, 2021
@petertrr petertrr changed the title Вeployment configurations Deployment configurations Feb 19, 2021
@petertrr petertrr merged commit e2cef0f into master Feb 20, 2021
@petertrr petertrr deleted the infra/deploy-configs branch February 20, 2021 07:23
@petertrr petertrr added this to the SAVE-cloud MVP milestone Feb 20, 2021
nulls added a commit that referenced this pull request Jul 26, 2022
nulls added a commit that referenced this pull request Jul 28, 2022
nulls added a commit that referenced this pull request Aug 3, 2022
nulls added a commit that referenced this pull request Aug 11, 2022
nulls added a commit that referenced this pull request Aug 11, 2022
nulls added a commit that referenced this pull request Aug 11, 2022
nulls added a commit that referenced this pull request Aug 11, 2022
nulls added a commit that referenced this pull request Aug 11, 2022
nulls added a commit that referenced this pull request Aug 12, 2022
nulls added a commit that referenced this pull request Aug 16, 2022
orchestr7 added a commit that referenced this pull request Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants