Skip to content

Use knative.dev/hack/codegen-library.sh in hack/update-codegen.sh#734

Merged
knative-prow-robot merged 4 commits intoknative:mainfrom
nak3:fix-update-codegen
Aug 28, 2021
Merged

Use knative.dev/hack/codegen-library.sh in hack/update-codegen.sh#734
knative-prow-robot merged 4 commits intoknative:mainfrom
nak3:fix-update-codegen

Conversation

@nak3
Copy link
Copy Markdown
Contributor

@nak3 nak3 commented Aug 28, 2021

operator still does not use the common script knative.dev/hack/codegen-library.sh
so update deps failed as #726.

This patch fixes the script and includes #726.

/cc @houshengbo @evankanderson @mattmoor

knative-automation and others added 3 commits August 27, 2021 13:29
@knative-automation
upgrade to latest dependencies
4ed1c1f 
bumping golang.org/x/oauth2 a41e5a7...2bc19b1:
  > 2bc19b1 google/downscope: return body in error message
  > 7df4dd6 google/externalaccount: validate tokenURL and ServiceAccountImpersonationURL
  > faf39c7 google/downscope: additional examples
  > 6f1e639 google/downscope: update documentation
bumping knative.dev/hack e28525d...815cd31:
  > 815cd31 Stop performing pre-releases (# 77)
bumping go.uber.org/zap 120b08c...c8e813e:
  > c8e813e Release v1.19.0 (# 987)
  > 2198a43 Optimize size of BufferedWriteSyncer by moving stopped bool field (# 984)
  > 81879d1 Simplify check for writing errors to ErrorOutput (# 983)
  > 1cac10b go mod tidy (# 977)
  > 42e70dd fix typo (# 976)
  > 007a55e Discard counters for out-of-bounds levels (# 975)
bumping knative.dev/caching 4e553d2...f4b23a3:
  > f4b23a3 upgrade to latest dependencies (# 519)
  > caf7ff5 Update community files (# 518)
  > c9cb048 Update actions (# 517)
  > c1a1ca6 upgrade to latest dependencies (# 516)
  > 261907c upgrade to latest dependencies (# 515)
  > 0af7ed4 upgrade to latest dependencies (# 514)
  > 72a775e Update community files (# 513)
  > 389c38e upgrade to latest dependencies (# 512)
  > 09262d8 Update community files (# 509)
  > 75179c7 Update actions (# 508)
  > 69e71ab add owner (# 507)
bumping knative.dev/eventing 0c41e6d...348cfb2:
  > 348cfb2 upgrade to latest dependencies (# 5677)
  > d4721b6 gingko compatible (# 5676)
  > eb3b681 Update community files (# 5675)
  > 600659e upgrade to latest dependencies (# 5672)
  > 5a0aa5d [Broker] Always consider ObservedGeneration to determine readiness (# 5667)
  > a98548a Make usage of GetConditionSet, rather than direct access (# 5662)
  > de1b52b Use `controller.NewContext` (# 5661)
  > 4f6f307 Handful more cleanups (# 5660)
  > 5b10764 [IMC] Always consider ObservedGeneration to determine readiness (# 5655)
  > b2891a8 upgrade to latest dependencies (# 5659)
  > 1aa90e5 fixing minor things (# 5658)
  > d879d09 Plumb through using the `controller.Impl` tracker. (# 5656)
  > 0d6181f upgrade to latest dependencies (# 5657)
  > b2700c2 Generate new community contact table (# 5653)
  > da32c8e upgrade to latest dependencies (# 5652)
  > b289108 Update community files (# 5647)
  > 128a666 Create a new ListableTracker ctor taking a tracker.Interface (# 5651)
  > 3e584ed Use v1 CRD clients in tests (# 5650)
  > 19fcdf4 Fix DeadLetterSink on Trigger status (# 5642)
  > cfcab6f remove vaikas from roster (# 5645)
  > 3a4912d upgrade to latest dependencies (# 5635)
  > 9dd1419 Fix deprecation of controller.Run to controller.RunContext (# 5644)
  > 4b18ab8 Add myself to support roster (# 5640)
  > 5c08bd3 be more explicit about the semantic of ceOverrides (# 5641)
  > c183833 Resurrect source spec (# 5637)
  > 72fae58 Add trace exporting to mt broker (# 5634)
  > 41e36ee Update community files (# 5632)
  > 6296a81 Add tracing support to eventdisplay and heartbeats (# 5631)
bumping knative.dev/pkg 21eb4c1...4472e04:
  > 4472e04 Bump golangci-lint timeout to 10m for github actions (# 2241)
  > fac4920 Update assorted dependencies (# 2240)
  > 263ce40 Add chains to the downstream test (# 2237)
  > ddd97e9 Remove obsolete verbose log message for pre-release (# 2235)
  > a7f5b39 Addressable resolver now accepts custom resolvers (# 2195)
  > 2b66fdd Update community files (# 2239)
  > 51f5642 Update actions (# 2238)
  > 46a3611 Add Tekton downstream test (# 2233)
  > a70bb26 Add length limit for the generated names used in tests (# 2217)
  > a4ed097 Update version check comment (# 2234)
  > a94f5f0 Trim the space in configmap/registriesSkippingTagResolving's url. (# 2231)
  > 12d915e Update prometheus dep to 0.30.0 (# 2227)
  > 3bed4dc Make the configmap update logging Debugf (# 2226)
  > 44fffc8 This implements the EventExpansion needed to emit K8s events. (# 2224)
  > 6ef763d Remove long deprecated NewImplWithStats (# 2223)
  > 9c7fd8e Introduce `NewContext`, deprecate `NewImplFull`. (# 2222)
  > 766b8f7 Have the addressable resolver take a tracker. (# 2220)
  > 9b9bc2a Run codegen (# 2221)
  > 52d2a0d Deprecate ReadOnlyFinalizer interface over OnDeletionInterface (# 2212)
  > 44fc074 Update community files (# 2219)
  > bda81c0 upgrade to latest dependencies (# 2218)
  > 7b5ecbc Sketch out an alternate way of injecting clients/informers (# 2210)
  > 8c88fa2 Revert "Remove the unused file (# 1935)" (# 2213)
  > 50410e0 Use consistent case for "Deprecated" comments (# 2216)
  > 35bcd16 Drop redundant pointers and decoders (# 2211)
  > bf176d5 Sink two codegen passes below loop. (# 2209)
  > 2c8a7b6 Update community files (# 2208)
  > 23b0147 Update actions (# 2206)
bumping knative.dev/serving 5ad15df...64a145c:
  > 64a145c Update net-kourier nightly (# 11878)
  > d4bf1ad upgrade to latest dependencies (# 11875)
  > 26c4f72 Update net-kourier nightly (# 11873)
  > ce627e5 Manage endpoints when KIngress status contains an IP address  (# 11843)
  > 56845e2 Update community files (# 11874)
  > a53193a Update net-certmanager nightly (# 11871)
  > f0a7836 Update net-contour nightly (# 11872)
  > 95af728 Drop unused flag.Parse and simplify emptyDir image even more (# 11855)
  > 5845ae6 Update actions (# 11870)
  > 97234eb upgrade to latest dependencies (# 11869)
  > 0d9a49f Skip a few more conformance tests against optional API (# 11827)
  > 0be381e Update net-istio nightly (# 11868)
  > 8906177 Update net-certmanager nightly (# 11867)
  > 0bb8499 Add missing RootCA additions to checks (# 11865)
  > 03823ca Drop relatively useless logging (# 11863)
  > ac53b04 Probe properly after startup when periodSeconds greater than zero (# 11190)
  > 8ab7a60 upgrade to latest dependencies (# 11861)
  > 48f6457 Clean up build tags in .golangci.yaml (# 11860)
  > 62d2781 Update net-contour nightly (# 11857)
  > 83fa93d Update net-istio nightly (# 11859)
  > 3b15ee9 Update net-kourier nightly (# 11856)
  > 04a56dc Update net-certmanager nightly (# 11858)
  > 0bcde14 upgrade to latest dependencies (# 11851)
  > d9ef49f Use the `controller.Impl` tracker. (# 11853)
  > 35ed978 Simplify emptyDir image behavior (# 11841)
  > 4220792 Update net-contour nightly (# 11794)
  > 13175ca Update net-istio nightly (# 11850)
  > eb3ac4d Update net-kourier nightly (# 11849)
  > 6891f62 fix incorrect helper adopted (# 11845)
  > b9a86db upgrade to latest dependencies (# 11847)
  > b827fe7 Update net-kourier nightly (# 11838)
  > c6b1956 Update net-certmanager nightly (# 11840)
  > 21e0d8e Update net-istio nightly (# 11839)
  > aeb7290 Update community files (# 11836)
  > 7f0582d Remove deprected loadbalancer (# 11837)
  > 5ab9710 upgrade to latest dependencies (# 11830)
  > 7f32cde Add conformance test for periodSeconds>0 case (# 11814)
  > d932855 Update deps and fix deprecations (# 11824)
  > 265dfe5 Update net-istio nightly (# 11825)
  > 0629672 Update net-istio nightly (# 11816)
  > fa213a4 Update net-certmanager nightly (# 11815)
  > 3e7900e Remove deprecated ServiceName field (# 11817)
  > 6a54ed8 Adds a flag to skip conformance tests against non-required API (# 11769)
  > 32f12b6 Makes conformance test for error condition more genernal (# 11784)
  > b7abe27 Update net-istio nightly (# 11808)
  > 269555f Update net-kourier nightly (# 11807)
  > 3bf25ce Update net-kourier nightly (# 11804)
  > db6e59d Update net-istio nightly (# 11803)
  > 1f7f69c Update community files (# 11800)

Signed-off-by: Knative Automation <automation@knative.team>
@nak3
Fix hack/update-codegen.sh
1d2006c
@nak3
Fix lint
115b9d2
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Aug 28, 2021
@knative-prow-robot knative-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Aug 28, 2021
@nak3 nak3 mentioned this pull request Aug 28, 2021
Merged
evankanderson
evankanderson reviewed Aug 28, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 28, 2021
@knative-prow-robot
Copy link
Copy Markdown
Contributor

knative-prow-robot commented Aug 28, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evankanderson, nak3

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 28, 2021
@knative-prow-robot knative-prow-robot merged commit 455fabe into knative:main Aug 28, 2021
@nak3 nak3 mentioned this pull request Aug 28, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@evankanderson evankanderson evankanderson left review comments

@houshengbo houshengbo Awaiting requested review from houshengbo

@mattmoor mattmoor Awaiting requested review from mattmoor

Assignees

@evankanderson evankanderson

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nak3 @knative-prow-robot @evankanderson @knative-automation