Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nobl9/nobl9-go
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.84.0-rc5
Choose a base ref
...
head repository: nobl9/nobl9-go
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.84.0-rc6
Choose a head ref
  • 2 commits
  • 3 files changed
  • 2 contributors

Commits on Aug 21, 2024

  1. chore: Update dependency cspell to v8.14.2 (#524)

    [![Mend
    Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
    
    This PR contains the following updates:
    
    | Package | Change | Age | Adoption | Passing | Confidence |
    |---|---|---|---|---|---|
    | [cspell](https://streetsidesoftware.github.io/cspell/)
    ([source](https://togithub.com/streetsidesoftware/cspell/tree/HEAD/packages/cspell))
    | [`8.14.1` ->
    `8.14.2`](https://renovatebot.com/diffs/npm/cspell/8.14.1/8.14.2) |
    [![age](https://developer.mend.io/api/mc/badges/age/npm/cspell/8.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
    |
    [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/cspell/8.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
    |
    [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/cspell/8.14.1/8.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
    |
    [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/cspell/8.14.1/8.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
    |
    
    ---
    
    ### Release Notes
    
    <details>
    <summary>streetsidesoftware/cspell (cspell)</summary>
    
    ###
    [`v8.14.2`](https://togithub.com/streetsidesoftware/cspell/blob/HEAD/packages/cspell/CHANGELOG.md#small8142-2024-08-20-small)
    
    [Compare
    Source](https://togithub.com/streetsidesoftware/cspell/compare/v8.14.1...v8.14.2)
    
    - chore: Update Integration Test Performance Data
    ([#&#8203;6126](https://togithub.com/streetsidesoftware/cspell/issues/6126))
    ([012c897](https://togithub.com/streetsidesoftware/cspell/commit/012c897)),
    closes
    [#&#8203;6126](https://togithub.com/streetsidesoftware/cspell/issues/6126)
    
    </details>
    
    ---
    
    ### Configuration
    
    📅 **Schedule**: Branch creation - "after 10pm every weekday,before 5am
    every weekday,every weekend" (UTC), Automerge - At any time (no schedule
    defined).
    
    🚦 **Automerge**: Enabled.
    
    ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
    rebase/retry checkbox.
    
    🔕 **Ignore**: Close this PR and you won't be reminded about this update
    again.
    
    ---
    
    - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
    this box
    
    ---
    
    This PR was generated by [Mend
    Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
    [repository job log](https://developer.mend.io/github/nobl9/nobl9-go).
    
    <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwiamF2YXNjcmlwdCIsInJlbm92YXRlIl19-->
    
    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
    renovate[bot] authored Aug 21, 2024
    Copy the full SHA
    f2f7faf View commit details

Commits on Aug 22, 2024

  1. feat: PC-13831 unique SLOs validation and test (#526)

    ## Release Notes
    
    Added BudgetAdjustment validation for unique SLOs in filters
    jakubpieta authored Aug 22, 2024

    Verified

    This commit was signed with the committer’s verified signature.
    IvanGoncharov Ivan Goncharov
    Copy the full SHA
    b988a29 View commit details
Showing with 50 additions and 2 deletions.
  1. +4 −1 manifest/v1alpha/budgetadjustment/validation.go
  2. +45 −0 manifest/v1alpha/budgetadjustment/validation_test.go
  3. +1 −1 package.json
5 changes: 4 additions & 1 deletion manifest/v1alpha/budgetadjustment/validation.go
Original file line number Diff line number Diff line change
@@ -54,7 +54,10 @@ var filtersValidationRule = validation.New(
validation.ForSlice(func(f Filters) []SLORef { return f.SLOs }).
WithName("slos").
Rules(validation.SliceMinLength[[]SLORef](1)).
IncludeForEach(sloValidationRule),
IncludeForEach(sloValidationRule).
Rules(validation.SliceUnique(func(s SLORef) string {
return s.Project + s.Name
}, "SLOs must be unique")),
)

var sloValidationRule = validation.New(
45 changes: 45 additions & 0 deletions manifest/v1alpha/budgetadjustment/validation_test.go
Original file line number Diff line number Diff line change
@@ -273,6 +273,51 @@ func TestValidate_Spec(t *testing.T) {
},
},
},
{
name: "duplicate slo",
spec: Spec{
FirstEventStart: time.Now(),
Duration: "1m",
Rrule: "FREQ=WEEKLY;INTERVAL=2",
Filters: Filters{
SLOs: []SLORef{
{Name: "test", Project: "project"},
{Name: "test", Project: "project"},
},
},
},
expectedErrors: []testutils.ExpectedError{
{
Prop: "spec.filters.slos",
Code: validation.ErrorCodeSliceUnique,
},
},
},
{
name: "duplicate slo with multiple others",
spec: Spec{
FirstEventStart: time.Now(),
Duration: "1m",
Rrule: "FREQ=WEEKLY;INTERVAL=2",
Filters: Filters{
SLOs: []SLORef{
{Name: "test1", Project: "project"},
{Name: "test1", Project: "project"},
{Name: "test2", Project: "project"},
{Name: "test3", Project: "project"},
{Name: "test4", Project: "project"},
{Name: "test5", Project: "project"},
{Name: "test6", Project: "project"},
},
},
},
expectedErrors: []testutils.ExpectedError{
{
Prop: "spec.filters.slos",
Code: validation.ErrorCodeSliceUnique,
},
},
},
{
name: "proper spec",
spec: Spec{
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@
"private": true,
"type": "module",
"devDependencies": {
"cspell": "8.14.1",
"cspell": "8.14.2",
"markdownlint-cli": "0.41.0",
"yaml": "2.5.0"
},