Setup logging #616
Closed
Setup logging #616
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: ezgidemirel <[email protected]>
Fix "xpkg dep" with function dependencies
Signed-off-by: ezgidemirel <[email protected]>
Signed-off-by: Hasan Turken <[email protected]>
Signed-off-by: Philippe Scorsolini <[email protected]>
Signed-off-by: Philippe Scorsolini <[email protected]>
Signed-off-by: Philippe Scorsolini <[email protected]>
The configuration command is no longer useful and has been stubbed out for a few releases. Time to remove it entirely. Signed-off-by: Adam Wolfe Gordon <[email protected]>
Bring private fork up to date with public fork
Signed-off-by: Christopher Haar <[email protected]>
* fix(xpkg): func xpkg dep for functions Signed-off-by: Christopher Haar <[email protected]> * fix(xpkg): func xpkg dep for functions Signed-off-by: Christopher Haar <[email protected]> * feat(bump): update native list because of new resource-kinds after bump k8 versions Signed-off-by: Christopher Haar <[email protected]> * feat(xpkg): dep tests adoption Signed-off-by: Christopher Haar <[email protected]> * fix(xpkg): func xpkg dep for functions Signed-off-by: Christopher Haar <[email protected]> --------- Signed-off-by: Christopher Haar <[email protected]>
…und#16) Signed-off-by: Christopher Haar <[email protected]>
extend run to include creating a kind cluster and installing uxp (with wait)
add some better error handling to ensure repeated commands are more idempotent
…-with [devex] Initial `up local start` and `up local stop` subcommands
Signed-off-by: Adam Wolfe Gordon <[email protected]>
* feat(xrd): add option to generate xrd from xr or claim Signed-off-by: Christopher Haar <[email protected]> * style(json): use k8s YamlToJson instead of own function and switch printer Signed-off-by: Christopher Haar <[email protected]> --------- Signed-off-by: Christopher Haar <[email protected]>
Make cli-review-team the default codeowner
Signed-off-by: ezgidemirel <[email protected]>
Signed-off-by: Christopher Haar <[email protected]>
- Starting from Spaces v1.17.0, we would like to remove the mxp-authz-webhook Chart. Signed-off-by: Alper Rifat Ulucinar <[email protected]>
Signed-off-by: Christopher Haar <[email protected]>
fix(configuration): add isCRD as linter for Configurations for cache hit
Happy 2024! Signed-off-by: Adam Wolfe Gordon <[email protected]>
The Project type holds the metadata and configuration for an Upbound development project, which can be built by `up build` into a Crossplane configuration. Signed-off-by: Adam Wolfe Gordon <[email protected]>
The new `up project build` command builds an Upbound project into a Crossplane package. For now we support only configuration projects, and don't support embedded functions.
Introduce `up project build`
Our development artifacts going forward are projects, which get compiled into Crossplane packages (configurations, functions, or providers). Rename `up configuration init` to `up project init` and update it to work on projects instead of configurations. Toward upbound#27 Signed-off-by: Adam Wolfe Gordon <[email protected]>
project: Rename `configuration init` to `project init`
When given a directory, go:embed excludes files starting with `.`. We intentionally include some dotfiles in our build test, so we want those included. Append `/**` to the embed pattern to capture the dotfiles. Signed-off-by: Adam Wolfe Gordon <[email protected]>
`up project build` is intended to ignore any YAML files that don't contain an XRD or composition. We do this by parsing the YAML to get its `apiVersion` and `kind` and collect only the relevant files into an in-memory filesystem that we hand off to the package parser (which will choke if it encounters an unexpected type). Previously, we were parsing the YAMLs as `unstructured.Unstructured`; this throws an error if the `kind` field isn't found, defeating the purpose of our pre-parsing. This wasn't caught by the tests due to the bug fixed in the previous commit. Parse into a `TypeMeta` instead, since it holds all the fields we care about anyway and doesn't care if `kind` is missing. While we're here, add validation in the tests to ensure we have the right number of resources in the package (i.e., didn't ignore ones we should include). Signed-off-by: Adam Wolfe Gordon <[email protected]>
Signed-off-by: Christopher Haar <[email protected]>
project: Correctly ignore non-KRM YAML files in build
feat(bump): bump actions/upload-artifacts to v4
…ound#35) Signed-off-by: Christopher Haar <[email protected]>
…) (upbound#33) Signed-off-by: Christopher Haar <[email protected]>
…rent pre-reqs (upbound#610) (upbound#34) Signed-off-by: Christopher Haar <[email protected]>
…pbound#39) Signed-off-by: Christopher Haar <[email protected]>
Signed-off-by: Christopher Haar <[email protected]>
Signed-off-by: Christopher Haar <[email protected]>
Signed-off-by: Dr. Stefan Schimanski <[email protected]>
Signed-off-by: Dr. Stefan Schimanski <[email protected]>
Signed-off-by: Dr. Stefan Schimanski <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pick of #614.