Anna Song
eb51117adf
Fix flaky
...
Disable submodules and increase timeout to prevent `localize` test that uses testdata from flaking.
2022-09-15 18:05:25 -07:00
Christian Provenzano
ca71736140
Wrap error in exported func
2022-09-09 20:27:28 -04:00
Christian Provenzano
e7d5f665b8
Fix redundant rnode metadata parsing
2022-09-09 19:25:59 -04:00
yugo kobayashi
a54226e0a9
fix error handling
2022-09-03 00:29:10 +00:00
yugo kobayashi
f086269d6e
be build fail when parse failed to FunctionSpec
2022-09-03 00:29:10 +00:00
Kubernetes Prow Robot
cb2636335f
Merge pull request #4779 from annasong20/locloader
...
Reinstate #4652 without url-related code
2022-08-30 08:13:01 -07:00
Anna Song
f80cf9f433
Reinstate #4652 without url-related code
...
Remove code that changes remotes-loading code path, as mandated by #4756
2022-08-29 16:57:15 +00:00
Yunchi Luo
d917876949
expand and clean up repospec_test
2022-08-27 14:30:10 -04:00
Natasha Sarkar
8868d91670
Revert "Load and calculate Destination of files for kustomize localize ( #4652 )"
...
This reverts commit bf17fe1d8f .
2022-08-26 13:26:23 -05:00
Kubernetes Prow Robot
b6fae2a959
Merge pull request #4759 from mightyguava/enable-ssh-tests
...
Enable SSH tests
2022-08-26 09:14:25 -07:00
natasha41575
08918ea352
run openapi tests with other krusty tests
2022-08-26 09:01:44 -05:00
Yunchi Luo
5dcbd35e40
Enable SSH tests
2022-08-25 16:57:21 -04:00
Kubernetes Prow Robot
2e2674fa24
Merge pull request #4727 from KnVerey/improve_fieldspec_errors
...
Improve invalid node kind errors
2022-08-25 12:44:07 -07:00
Anna Song
bf17fe1d8f
Load and calculate Destination of files for kustomize localize ( #4652 )
...
* Implement file localization for localize
Implement file localization for new command kustomize localize
* Patch file localization
Fix lint errors, address feedback, begin transition to file loader from
localizer
* Add Repo() to Loader
* Implement locLoader factory + cleanup
* Add domain to RepoSpec
Fix ssh relative url host parsing for non-github domain on the side
* Implement Load(), New(), Root(), Dst() for LocLoader
* Address repospec code review comments
* Address 1st round of code review feedback
* Address feedback #2
Removed localized path calculations to cover edge case and improve readability.
* Remove ldr Cleanup logging
* Address code review round #3
* Address code review feedback #4
2022-08-19 14:33:53 -07:00
Katrina Verey
f8d1e778c7
Commit go.sum updates
2022-08-16 18:26:04 -04:00
Katrina Verey
79a9154cf8
Improve error message when namespace transformer is given invalid fieldspecs
...
Also remove invalid+ignored fieldspecs from the defaults
2022-08-16 18:18:44 -04:00
Katrina Verey
55a37de686
Fix other linters on modified lines
2022-08-10 19:34:25 -04:00
Katrina Verey
f6b72077c8
Stop using deprecated ioutil functions
2022-08-10 18:22:46 -04:00
Katrina Verey
5762794793
Revert "[address #4717 ] create template/metadata when using includeTemplates if not present"
2022-08-10 14:06:33 -04:00
Agustina Barbetta
15545cc228
create template/metadata in includeTemplates if not present
2022-08-07 20:48:51 -03:00
Natasha Sarkar
1b0fe2a078
update openapi scripts to fetch protobuffer schemas ( #4582 )
...
* update openapi scripts to fetch protobuffer schema
* code review
* code review
* restore makeOpenApiInfoDotGo.sh
* code review
2022-08-05 15:00:32 -07:00
natasha41575
001b0c5fe5
pin to kyaml v0.13.9
2022-08-02 10:20:35 -05:00
natasha41575
f6a1e4af3b
pin to kyaml v0.13.8
2022-07-29 14:44:06 -05:00
natasha41575
afa2b52807
merge go.sum files
2022-07-29 13:42:32 -05:00
Katrina Verey
2a3b645758
Emit a warning when env loading bug is being relied on
2022-07-28 18:04:48 -04:00
Kubernetes Prow Robot
c4b4a41913
Merge pull request #4704 from KnVerey/ns_transformer_options
...
Option to customize NamespaceTransformer role binding subject handling
2022-07-28 08:27:11 -07:00
Katrina Verey
0c37ee89af
Option to customize NamespaceTransfomer role binding subject handling
2022-07-27 23:12:55 -04:00
Natasha Sarkar
2f2ba40876
replacements: fix issue with create: true option when there is an existing field ( #4667 )
...
* replacements: demonstrate broken behavior when using 'create: true' with an already existing field
* replacements: fix issue with 'create: true' option when there is an existing field
* Suggestion for PR 4667
* code review
* lint error
Co-authored-by: Katrina Verey <katrina.verey@shopify.com >
2022-07-15 14:31:49 -07:00
Katrina Verey
0c6e827ab8
Option to customize NamespaceTransformer overwrite behaviour ( #4708 )
...
* Option to customize NamespaceTransformer overwrite behaviour
* Code review feedback
2022-07-14 12:00:58 -07:00
Kubernetes Prow Robot
0fce7d53a9
Merge pull request #4703 from KnVerey/name-ref-annotations
...
Fix NameReference transformer handling of self-references in annotations
2022-07-11 17:22:50 -07:00
Katrina Verey
68780b4c0c
Fix flakey test
2022-07-11 20:03:13 -04:00
Katrina Verey
04e1663b70
appendCsVAnnotation must mutate the annotations RNode in place
...
Otherwise, it is incompatible with filters like fieldSpec.Filter that recurse through a tree of RNodes. In the case of the bug this commit fixes, the leaf RNode is a metadata.annotation value field, and appendCsvAnnotation is called immediately before updating that leaf's value. If appendCsvAnnotation replaces the entire metadata.annotation RNode on the resource, the leaf RNode that gets updated is no longer attached to the resource.
Alternatively, `func (f Filter) setScalar` could be updated to change
the value of the leaf RNode BEFORE calling appendCsvAnnotation. However,
the modification of the entire metadata segment of the RNode in a
function that nominally just edits an annotation feels like a
side-effect prone to creating other difficult-to-debug situations,
beyond this ordering dependency.
2022-07-11 19:58:49 -04:00
Katrina Verey
77814ac12b
NameReferenceTransformer misses self references in annotations
2022-07-08 22:07:15 -04:00
Kubernetes Prow Robot
e1b6aa393a
Merge pull request #4692 from koba1t/chore/add_support_for_go_workspace_mode
...
[proposal] add support for go workspace mode
2022-07-08 10:29:47 -07:00
yugo kobayashi
3229c810cc
restore require sigs.k8s.io/kustomize/* in go.mod
2022-07-08 16:58:01 +00:00
Kubernetes Prow Robot
344d0eec83
Merge pull request #4700 from KnVerey/ns_transformer_err_on_ambiguous_ns
...
Correctly detect ambiguity between potential referrers when targeting a name+namespace reference
2022-07-07 15:48:37 -07:00
Katrina Verey
387c95be1f
Correctly detect ambiguity between potential referrers when targeting a name+namespace reference
2022-07-07 18:33:48 -04:00
yugo kobayashi
6fb96b7d53
update go 1.18
2022-07-07 20:11:29 +00:00
yugo kobayashi
670cef4fd1
remove 'replace sigs.k8s.io/kustomize/*' and cleanup
2022-07-07 18:30:49 +00:00
koba1t
a6449aa03f
try to fix e2econtainer test
2022-07-05 03:44:56 +09:00
koba1t
e201dbb31a
go work sync
2022-07-02 15:31:24 +09:00
koba1t
bc981fcbc3
merge upstream master
2022-07-02 15:18:26 +09:00
Kubernetes Prow Robot
8a729459aa
Merge pull request #4694 from annasong20/add-loc-tdata-remote
...
Add `localize` test data with remote reference
2022-06-30 14:22:44 -07:00
Anna Song
d1102fb726
Move demandDirectoryRoot into kyaml/filesys ( #4677 )
...
* Move demandDirectoryRoot into kyaml/filesys
* Make root directory platform-agnostic
Support windows root directory. Dogfood own error package.
* Use cleaner windows support implementation
* Address feedback
* Address feedback x2
* Re-apply go.sum changes to avoid CI errors
2022-06-30 11:12:07 -07:00
Anna Song
edb6294741
Add kustomization with remote reference
2022-06-30 17:21:23 +00:00
koba1t
8c8a2125b7
update all go.mod to go1.18
2022-06-29 05:14:33 +09:00
koba1t
f39f914a98
add many pkgs for go.work
2022-06-29 03:32:54 +09:00
koba1t
c583f4164c
add support for go workspace mode
2022-06-28 23:42:56 +09:00
Nuno Anselmo
d8efc15169
Adds commentary on expected/unexpected test output
2022-06-21 23:52:39 +01:00
Nuno Anselmo
86d48b2a95
Implements TestNamedspacedServiceAccounts with and without overlap
2022-06-21 23:07:54 +01:00