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

feat: Argo Rollouts api-server and UI #1015

Merged
merged 116 commits into from
Apr 9, 2021
Merged

feat: Argo Rollouts api-server and UI #1015

merged 116 commits into from
Apr 9, 2021

Conversation

rbreeze
Copy link
Member

@rbreeze rbreeze commented Feb 27, 2021

Signed-off-by: Remington Breeze remington@breeze.software

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
  • The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
  • I've signed my commits with DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My builds are green. Try syncing with master if they are not.
  • My organization is added to USERS.md.

@rbreeze
Add scaffolding for api server
2aaec17 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add stub for rollouts server command
1ec117a 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
More work on server
9b2a147 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add working watch and get endpoints for rollout. Add UI stub.
7ef81ac 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Functional UI list. API streaming endpoints
5352e39 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix streaming. More UI progress
cd36f60 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Replace RolloutInfo with API def. More UI progress, add restart endpoint
8795442 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix rollouts watch endpoint. UI watch is smarter. Light/dark mode in …
0c1630b 
…UI. More UI progress

Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add server action to promote rollout, add more info to rollout UI view
c0e6c57 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add kubectl plugin dashboard command. Add ability to set image in UI.…
45950d3 
… Smaller UI tweaks and additions

Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Remove static files
73f3fc4 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Makefile modification for CI, replace replicasets with revisions
75fcde9 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Improve set image UX and other improvements suggested in standup
04b6a28 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
UI improvements
58db5b2 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add color coding to images
77c0b85 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Address some feedback from standup
41d4ea1 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Address more feedback
6706c36 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Batch edit container images, tooltip for last restarted
bea7616 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add rollback to api-server and UI
925b3a1 
Signed-off-by: Remington Breeze <remington@breeze.software>
@jessesuen jessesuen mentioned this pull request Mar 18, 2021
Closed
@jessesuen jessesuen added this to the v1.0 milestone Mar 19, 2021
@rbreeze
Begin adding analysis run UI. Lots more UI improvements
816b666
@rbreeze
Improve search for rollouts list. Add keyboard shortcuts for search a…
58a6d69 
…nd more for list

Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
UI fixes. Add Dockerfile for kubectl plugin / UI
246d42f 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix rollouts list streaming in UI
db48b46 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix Dockerfile for kubectl plugin / UI
33180c7 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
ui bug fixes
de9db08 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Merge branch 'master' into ui
4dfb130
@rbreeze
Add comment for linter to ignore static file import
944dea6 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Merge branch 'ui' of github.com:rbreeze/argo-rollouts into ui
cbb2bdf
@rbreeze
Rerun codegen
84795e5 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix ui issues
8087816 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Resolve merge conflicts
4fb95d5 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Remove standalone server make target and cmd stub
518e81d 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Tweak rollouts list. Fix plugin make target
55e2cda 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add duration unit labels for steps
51e1f5a 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Update mkdocs
cb2e6cb 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Update mkdocs
7d97df6 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Replace v1alpha1.RolloutInfo with rollout.RolloutInfo (generated)... …
c3efeb4 
…reverts types.go changes

Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Fix UI crash due to updated types
d5484ca 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Lint
44e0deb 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Add revision to replicasets on rollouts list page
121c3a0 
Signed-off-by: Remington Breeze <remington@breeze.software>
jessesuen
jessesuen reviewed Apr 8, 2021
View reviewed changes
server/server.go Outdated Show resolved Hide resolved
jessesuen
jessesuen requested changes Apr 8, 2021
View reviewed changes
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
@rbreeze
Add docs page for dashboard
f0353a9 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Address code review comments
2667ce9 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Resolve merge conflict
75eeb71 
Signed-off-by: Remington Breeze <remington@breeze.software>
jessesuen
jessesuen reviewed Apr 8, 2021
View reviewed changes
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
pkg/apiclient/rollout/rollout.proto Outdated Show resolved Hide resolved
@rbreeze
Address code review comments
fa2ba5f 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
lint
c1333cd 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
Minor UI improvements and fixes
51c53c4 
Signed-off-by: Remington Breeze <remington@breeze.software>
@rbreeze
More ui enhancements
f1b4800 
Signed-off-by: Remington Breeze <remington@breeze.software>
@sonarcloud
Copy link

sonarcloud bot commented Apr 9, 2021

SonarCloud Quality Gate failed.

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 739 Code Smells

No Coverage information No Coverage information
36.3% 36.3% Duplication

@rbreeze rbreeze requested a review from jessesuen April 9, 2021 17:00
jessesuen
jessesuen approved these changes Apr 9, 2021
View reviewed changes
Copy link
Member

@jessesuen jessesuen left a comment

Choose a reason for hiding this comment

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

Great work!

@rbreeze rbreeze merged commit 7facfaa into argoproj:master Apr 9, 2021
@khhirani khhirani mentioned this pull request Apr 27, 2021
Merged
@JuozasVainauskas JuozasVainauskas mentioned this pull request May 29, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jessesuen jessesuen jessesuen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rbreeze @jessesuen