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

Bump upstream #12

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

Bump upstream #12

wants to merge 32 commits into from

Conversation

arturo-skydio
Copy link

Pull in changes from upstream and resolve a few conflicts

  • Verified that all works in skyops using this git hash

kzadorozhny and others added 30 commits October 4, 2023 18:19
* fixes additional calls to kubectl that were always using default user

* skylib/k8s.bzl changes from michaelschiff/rules_gitops

* rename symlink to use_host_config per code review

Co-authored-by: Michael Schiff <schiff.michael@gmail.com>
* upgrade bazel deps

* buildifier-fix

* buildifier

* rules_go initialization

* always call go_register_toolchains

* use named parameter (buildifier)

* init go

* format

* readme
…ible form can be obtained by dropping arguments ["name"] (adobe#105)

(13:15:04) DEBUG: Repository k8s_test instantiated at:
  /root/repo/WORKSPACE:181:11: in <toplevel>
* --dry_run flag

* more info on dry run

* doc
* secrets example

* more commments

* buildifier

* use array for images
* update go and bazel dependencies
* make it work with bazel 6
* deprecate bazel 5.1.1 and switch to 6.0.0 in github actions
* Handle cases where returned object from a watch is not a Pod
If a YAML specifies an empty `initContainers:`, this will crash this
code if we also specify yamls. regardless of how pointless this is, this
is, it is valid k8s yaml that applies cleanly.
apesternikov and others added 2 commits October 4, 2023 18:28
* +use images or image pushes in k8s_deploy images parameter
+use dict {"name": ":image" } or [ ":image" ] for images parameter
+more tests

* correct handling of dependency runtimes

* test fixes

* use template to generate expected file

* cleanup

* buildifier

* buildifier nit

* explain execution_requirements rationale

* buildifier

Co-authored-by: Konstantin Zadorozhny <kzadorozhny@users.noreply.github.com>
- Tested by running gitops targets in skyops with this repo in local_repository and verifying no changes

Topic: upstream-changes
Reviewers: daniel-b,holt-w
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants