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

[RELEASE] 20230703154154 #3689

Merged
merged 4 commits into from Jul 4, 2023
Merged

[RELEASE] 20230703154154 #3689

merged 4 commits into from Jul 4, 2023

Conversation

nightfury1204
Copy link
Collaborator

@nightfury1204 nightfury1204 commented Jul 3, 2023

Pull Requests

Milestone Release

### What is the feature/fix?

https://app.asana.com/0/1203637156732418/1204875717950149/f

### Add screenshot or video (optional)

** Any screenshot or video capture using the feature **

### Does it has a breaking change?

** Describe the changes and if it has any breaking changes in any feature **

### How to use/test it?

- you need cli and rack of this version

### Checklist
- [ ] New coverage tests
- [ ] Unit tests passing
- [ ] E2E tests passing
- [ ] E2E downgrade/update test passing
- [ ] Documentation updated
- [ ] No warnings or errors on Deepsource/Codecov
### What is the feature/fix?

Create a role that will use the defined policies on that service. It overrides the [IamPolicy](https://docsv2.convox.com/reference/app-parameters#iampolicy) just for that service (it will not be used if `policies` is defined on convox.yml`).

Priority:

1. DedicatedRole (at service level, if specified)
2. IamPolicy

### Does it has a breaking change?

No, current ServiceRoles are not affected by the change.

### How to use/test it?

- Create a rack with the RC installed.
- Deploy an app with the following yml:

```
services:
  web:
    policies:
      - arn:aws:iam::aws:policy/AdministratorAccess
    build: .
    port: 3000
  api:
    build: .
    port: 3000
```

- Check on IAM if the policies were created correctly.

### Checklist
- [ ] New coverage tests
- [ ] Unit tests passing
- [ ] E2E tests passing
- [ ] E2E downgrade/update test passing
- [ ] Documentation updated
- [ ] No warnings or errors on Deepsource/Codecov
### What is the feature/fix?
https://app.asana.com/0/1203637156732418/1204402996040452/f

### Add screenshot or video (optional)

** Any screenshot or video capture using the feature **

### Does it has a breaking change?

** Describe the changes and if it has any breaking changes in any feature **

### How to use/test it?

** Describe how to test or use the feature **

### Checklist
- [ ] New coverage tests
- [ ] Unit tests passing
- [ ] E2E tests passing
- [ ] E2E downgrade/update test passing
- [ ] Documentation updated
- [ ] No warnings or errors on Deepsource/Codecov
@nightfury1204 nightfury1204 requested a review from ntner July 3, 2023 20:42
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Patch coverage: 39.81% and project coverage change: +0.03 🎉

Comparison is base (fcb6894) 35.78% compared to head (b3acf28) 35.82%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3689      +/-   ##
==========================================
+ Coverage   35.78%   35.82%   +0.03%     
==========================================
  Files         169      171       +2     
  Lines       18719    18933     +214     
==========================================
+ Hits         6699     6783      +84     
- Misses      10882    11006     +124     
- Partials     1138     1144       +6     
Impacted Files Coverage Δ
pkg/api/controllers.go 60.83% <0.00%> (-1.78%) ⬇️
pkg/manifest/service.go 41.02% <ø> (ø)
provider/aws/aws.go 22.64% <0.00%> (-0.55%) ⬇️
provider/k8s/system.go 0.00% <0.00%> (ø)
sdk/methods.go 73.84% <0.00%> (-2.68%) ⬇️
provider/aws/system.go 28.65% <2.94%> (-1.80%) ⬇️
provider/aws/builds.go 18.03% <20.00%> (+0.01%) ⬆️
pkg/cli/rack.go 55.94% <34.28%> (-2.27%) ⬇️
pkg/api/api.go 67.34% <50.00%> (-8.33%) ⬇️
provider/aws/processes.go 46.11% <50.00%> (+<0.01%) ⬆️
... and 4 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link

@ntner ntner left a comment

Choose a reason for hiding this comment

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

  • Install and update to rack version

  • Downgrade to previous version

  • Telemetry based param groupings at install

  • Common convox rack param set variations after install

  • New application install and running with multiple resources

  • Existing application working after upgrade

  • Review and Deploy Workflows working across update

  • General and build stress-testing

  • Service level policy, multiple variations tested

@nightfury1204 nightfury1204 merged commit ea06d48 into master Jul 4, 2023
12 checks passed
@nightfury1204 nightfury1204 deleted the 20230703154154 branch July 4, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants