Prepare v1.7.0 release #7809
Conversation
Signed-off-by: Johan Fylling <johan.dev@fylling.se>
Signed-off-by: Johan Fylling <johan.dev@fylling.se>
✅ Deploy Preview for openpolicyagent ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Johan Fylling <johan.dev@fylling.se>
srenatus
left a comment
There was a problem hiding this comment.
Thanks for putting these together. Noted a couple of small things 👇
| - rego: Pass along `TracingOpts` into `EvalContext` ([#](https://github.com/open-policy-agent/opa/pull/7778)) authored by @srenatus | ||
| - runtime: add `ExtraDiscoveryOpts` to `runtime.Params` ([#7766](https://github.com/open-policy-agent/opa/pull/7766)) authored by @srenatus | ||
| - sdk: Allow for setting default options for all instances ([#7760 B](https://github.com/open-policy-agent/opa/pull/7760)) authored by @srenatus | ||
| - sdk: Options store takes precedence over ext store ([#7800](https://github.com/open-policy-agent/opa/pull/7800)) authored by @srenatus |
There was a problem hiding this comment.
I'd drop that, it's an adjustment to one of the other merged PRs.
| - server: Add hooks wiring + new hooks for inter-query caches ([#](https://github.com/open-policy-agent/opa/pull/7775)) authored by @srenatus | ||
| - server: Ensure that wrapped middlewares all support `http.Flusher` ([#7772](https://github.com/open-policy-agent/opa/pull/7772)) authored by @srenatus | ||
| - server/authorizer: Allow adding paths to validator ([#7792](https://github.com/open-policy-agent/opa/pull/7792)) authored by @philipaconrad | ||
| - server+plugins: Allow plugins to inject http handler middlewares ([#7789](https://github.com/open-policy-agent/opa/pull/7789)) authored by @srenatus |
There was a problem hiding this comment.
This was recently mentioned here, #7676 (comment), so we could also say
reported by @deeglaze
| - format: Not bracketing keywords in imports ([#7742](https://github.com/open-policy-agent/opa/issues/7742)) authored by @johanfylling | ||
| - loader: Add bundle lazy loading mode across the runtime. ([#7768](https://github.com/open-policy-agent/opa/pull/7768)) authored by @philipaconrad | ||
| - loader: Pass bundle name in `AsBundle()` ([#7798](https://github.com/open-policy-agent/opa/pull/7798)) authored by @srenatus | ||
| - opa exec: stop plugins before exit ([#7760 A](https://github.com/open-policy-agent/opa/pull/7760)) authored by @srenatus |
There was a problem hiding this comment.
| - opa exec: stop plugins before exit ([#7760 A](https://github.com/open-policy-agent/opa/pull/7760)) authored by @srenatus | |
| - opa exec: stop plugins before exit ([#7760](https://github.com/open-policy-agent/opa/pull/7760)) authored by @srenatus |
| - perf: AST compiler optimizations ([#7740](https://github.com/open-policy-agent/opa/pull/7740)) authored by @anderseknert | ||
|
|
||
| ### Docs, Website, Ecosystem | ||
|
|
There was a problem hiding this comment.
Let's perhaps call out that really soon (#7781), what's shown on the website will be different, based on the latest release? It's not in these PRs, but it's meant to happen right after this release. (ping @charlieegan3)
There was a problem hiding this comment.
added a suggestion below
| - build: Add top-level token permissions for workflows ([#7795](https://github.com/open-policy-agent/opa/pull/7795)) authored by @timothyklee | ||
| - build: Show a warning when .manifest is ignored ([#7807](https://github.com/open-policy-agent/opa/pull/7807)) authored by @charlieegan3 | ||
| - cli: Avoid os.Exit() in Run() funcs ([#7788](https://github.com/open-policy-agent/opa/pull/7788)) authored by @srenatus |
There was a problem hiding this comment.
These three should perhaps go into Runtime/Tooling?
| - build(deps): bump golang.org/x/net from 0.41.0 to 0.42.0 | ||
| - build(deps): bump golang.org/x/time from 0.11.0 to 0.12.0 | ||
| - build(deps): bump google.golang.org/grpc from 1.73.0 to 1.74.2 | ||
| - build(deps): bump sigs.k8s.io/yaml from 1.4.0 to 1.6.0 |
There was a problem hiding this comment.
I think only opentelemetry and gRPC and gqlparser and maybe containerd are worth taking note of. 🧐
| - plugins/discovery: Make `Factories()` merge the factories ([#7777](https://github.com/open-policy-agent/opa/pull/7777)) authored by @srenatus | ||
| - plugins/discovery: Replace environment variables after evaluation ([#7787](https://github.com/open-policy-agent/opa/pull/7787)) authored by @philipaconrad | ||
| - plugins/logs: Add experimental intermediate results field ([#7796](https://github.com/open-policy-agent/opa/pull/7796)) authored by @philipaconrad | ||
| - report: Fetching latest OPA release version from GitHub ([#7756](https://github.com/open-policy-agent/opa/pull/7756)) authored by @johanfylling |
There was a problem hiding this comment.
Worth noting that OPA no longer sends any telemetry data? 💭
Signed-off-by: Johan Fylling <johan.dev@fylling.se>
Co-authored-by: Charlie Egan <git@charlieegan3.com> Signed-off-by: Johan Fylling <johan.dev@fylling.se>
Co-authored-by: Charlie Egan <git@charlieegan3.com> Signed-off-by: Johan Fylling <johan.dev@fylling.se>
Co-authored-by: Charlie Egan <git@charlieegan3.com> Signed-off-by: Johan Fylling <johan.dev@fylling.se>
|
|
||
| - build: Show a warning when .manifest is ignored ([#7807](https://github.com/open-policy-agent/opa/pull/7807)) authored by @charlieegan3 | ||
| - cli: Avoid os.Exit() in Run() funcs ([#7788](https://github.com/open-policy-agent/opa/pull/7788)) authored by @srenatus | ||
| - cmd: Use command.RunE to return errors and perform orderly shutdown of OPA ([#7788](https://github.com/open-policy-agent/opa/pull/7788)) authored by @srenatus |
There was a problem hiding this comment.
link to the same PR as above?
There was a problem hiding this comment.
Let's just combine it. The release notes don't need to reflect every commit... :)
There was a problem hiding this comment.
Yeah. This PR was a rebase/merge so both its commits ended up on main instead of a squash. I added a A/B suffix to them first, but that got rejected (probably wouldn't help the reader much).
There was a problem hiding this comment.
@srenatus , I combined the commits for another rebased PR, so we could do that here to. I just thought these looked like two distinct features.
There was a problem hiding this comment.
what should we combine them under? cli: avoid os.Exit() in Run() funcs is the PR headline.
Signed-off-by: Johan Fylling <johan.dev@fylling.se>
No description provided.