-
Notifications
You must be signed in to change notification settings - Fork 2k
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
migrate code from googleapis/nodejs-containeranalysis #2816
Conversation
NimJay
commented
Nov 9, 2022
- added directory structure
- npm init
- moved over samples
- passing tests for create, get, delete note and create occurrence
- passing get occurrence test
- passing delete occurrence
- passing test for occurrences for note, flaky
- passing occurrences for image
- samples
- passing get Discovery Info
- passing find vulnerabilities for image
- passing high severity vulnerability test
- pass all vulnerabilities for image
- failing pubsub
- passing pubsub tests
- passing polling test
- updated imageUrl to https://gcr.io/my-project/my-image:123 for more realistic use case
- removed escape slashes from filter strings, not needed
- updated description of occurrenceId
- added comment to explain second filter statement, needed for testing
- explicity state length greater than zero
- removed system-test, moved tests into test directory
- removed async functions from tests without await
- removed unused exec
- removed explicit project id from pub sub
- ran fix
- replaced promise/timeout with p-retry
- changed all samples to be wrapped with an outer main function, added params as comments for user
- added after step to delete occurrences and notes
- updated polling sample and test
- ran fix, dangling errs for symlink and polling
- updated comment on occurrenceId
- fixed typo in discovery test
- grabs projectId from client, update projectId and vars the refer to it in before hooks, added uuid to test resourceUrl
- removed commented code from polling sample
- move end region tag to inside main function
- removed symlinks from test script, removed commented test from polling block
- deleted unused import from pub sub sample
- changed grafeas to container analysis v1beta1 client
- fixed typo in client, removed comment
- added retry logic to get occurrence test
- changed assert match to assert include
- added create topic logic to fix ci test error
- removed regexp from assert include
- changed tests to include arrow functions
- moved containeranalysis and pubsub into prod deps
- added retry logic for all occs of note, check if topic exists and fixed typo on delete occ test
- added sample-metadata
- fixed occurrence typo
- added params to sample-metadata usage
- added p-retry to prod deps, updated poll sample
- working poll discovery occurrence finished sample
- updated default value of retries and timeoutSeconds to be more readable
- refactor: changes formatting of various statements
- chore: release 1.0.0 (Added back datastore v1beta3 samples. #33)
- fix(deps): update dependency @google-cloud/pubsub to ^0.29.0 (Upgrade to latest nodejs LTS release #36)
- chore: release 1.0.1 (Pin engines to Node 4.2. Closes #36. #37)
- chore: release 1.0.2 (Simplify app.yaml files, reduce # of e2e tests, and cleanup for webpa… #40)
- chore: release 1.1.0 (Skip node_modules for better deployments. #44)
- feat: adds getGrafeasClient() method on ContainerAnalysisClient instance (Change 'sampletask' to 'sampleTask' for consistency #46)
- chore: release 1.2.0 (Add property query filtering snippet. #47)
- chore: update samples to reflect v1 Grafeas API (Datastore: Use new Key object to easily reference parts of a key path #48)
- chore: release 1.2.1 (Added instructions for running the tests. Fixes #49. #50)
- chore: release 1.2.2 (Add Prediction API sample. #52)
- docs(samples): Update Topic ID in PubSub sample (Added Pub/Sub samples. #55)
- chore: release 1.3.0 (Move back to container-based Travis builds and removed some redundant tests. #58)
- fix(deps): update dependency @google-cloud/pubsub to ^0.30.0 (Add logging example and link to instructions for Compute Engine as well #59)
- chore: release 1.3.1 (Datastore / CloudSQL samples should only store first two octtets of IP #61)
- chore: release 1.3.2 (Changes for Datastore v1beta3. #68)
- fix(deps): update dependency @google-cloud/pubsub to ^0.31.0 (memcache env vars #71)
- chore: release 1.3.3 (Add sample for brunch #72)
- fix(deps): update dependency @google-cloud/pubsub to ^0.32.0 (Samples/gulp #81)
- chore: release 1.4.0 (Provided comment about done() #78)
- fix(deps): update dependency @google-cloud/pubsub to v1 (Fix metric filter call #84)
- chore: release 1.5.1 (I can commit directly to master #97)
- chore: add license headers for samples (Remove skip_files from app.yaml files #106)
- chore: release 1.6.0 (Add Cloud Speech sample. #110)
- chore: release 1.6.1 (Add Cloud Vision text detection sample. #113)
- refactor: use explicit mocha imports (Remove skip_files from app.yaml files, as it's no longer needed. #119)
- chore: release 1.7.0 (gcloud-node datastore no longer supports "autoPaginate" option #115)
- chore(deps): update dependency mocha to v7 (getApplicationDefault is undefined. #121)
- chore: release 1.7.1 (Add HTTP Cloud Function. #131)
- feat: move API to typescript code generation (Changed helloworld to camel case #135)
- chore: release 1.8.0 (Add missing sample that got inlined into the docs. #137)
- chore(deps): update dependency uuid to v7
- chore: release 1.9.0 (Add Speech API streaming sample. #144)
- chore: release 1.10.0 (Switch CloudSQL sample to 2nd gen. Added unit tests. #153)
- feat!: drop node8 support, support for async iterators (Update CI settings. #167)
- test: there's a race condition emptying queue (Logging API samples #175)
- test: retry pubsub test, as it relies on many moving parts (Export data from BigQuery samples #186)
- test: retry queue deletion (Update all dependencies 🌴 #191)
- build: adopt CI changes and generator formatting (Add remaining Translate samples. #197)
- test: make tests idempotent (Add browsing data sample #201)
- fix(samples): should use effective_severity field (Fix typo #200)
- chore: release 2.0.0 (Remove output only fields #170)
- build: modified export in protos.js
- chore(deps): update dependency uuid to v8 (Fix asserts and related stubs. #206)
- fix(deps): update dependency @google-cloud/pubsub to v2 (Add quickstart tests #215)
- chore(deps): update dependency mocha to v8 (Fix pagination #222)
- test: attempt retry on note delete (Upgrade tests and samples to ES2015 #224)
- chore: release 2.1.0 (Convert tasks.js to yargs #210)
- chore: release 2.1.1 (Error: Audio 'content' cannot be longer than 1 min. For long audio, use the 'uri' parameter to specify a file. #232)
- docs(samples): update sample to illustrate AR style urls
- chore: release 3.0.0 (Update app.yaml to use instance connection name #264)
- chore: release 3.0.1 (Updating additional swagger API references #268)
- chore: release 3.1.0 (Recognize stream freezes (from example) #278)
- build: update gapic-generator-typescript to v1.2.10. (AssertionError: undefined deepEqual 1 when porting functions/datastore to local setup #284)
- chore: release 3.1.1 (Vision snippets. #294)
- chore: release 3.1.2 (New cloud functions example #299)
- chore: new owl bot post processor docker image (Google Cloud Functions - Private Github repo in package.json #300)
- chore: release 3.1.3 (Sample command's response take too long time #303)
- chore: release 3.1.4 (Removed deprecated flag from app.yaml #309)
- chore: release 3.1.5 (Video samples #319)
- chore: release 3.1.6 (Fix issue with read-only transaction #325)
- chore: release 3.1.7 (speech FLAC encoding not work #327)
- chore: release 3.1.8 (Parallelize DLP tests #337)
- chore: release 3.2.0 (Update translate dependency #340)
- chore: release 3.3.0 (Semantic Text Matching like quora using Google NLP API (question) #353)
- chore(main): release 3.4.0 (Fix CONTRIBUTING.md links not rendering correctly #389)
- fix: Flaky pubsub test (Initial commit of sample #394)
- build!: update library to use Node 12 (Error reporting #408)
- chore(main): release 4.0.0 (Stackdriver samples #407)
- fix(deps): update dependency @google-cloud/pubsub to v3 (Remove "entire video" detection from faces #410)
- chore(main): release 4.1.0 (sendgrid example memory potential leaks #416)
- fix(issue-417): adding additional asserts to track flaky error (Fix bigquery samples #421)
- chore(main): release 4.2.0 (New changes added #MAKE CODE GREAT AGAIN! #422)
- chore(deps): update dependency uuid to v9 (Upgrades language samples to semi-gapic client #435)
- test(samples): disable samples tests for this repository (Added enableWordTimeOffsets: true to async (file) #440)
- chore(main): release 4.3.0 (Fix iot document link #429)
- chore(main): release 4.4.0 ([Speech] README needs autogen update (new Words CLI options are not shown in the README) #444)
…ealistic use case
params as comments for user
🤖 I have created a release *beep* *boop* --- ## [4.2.0](googleapis/nodejs-containeranalysis@v4.1.0...v4.2.0) (2022-07-04) ### Features * support regapic LRO ([1290f13](googleapis/nodejs-containeranalysis@1290f13)) ### Bug Fixes * **deps:** update dependency @google-cloud/grafeas to v4 ([#420](googleapis/nodejs-containeranalysis#420)) ([6e40ebe](googleapis/nodejs-containeranalysis@6e40ebe)) * **issue-417:** adding additional asserts to track flaky error ([#421](googleapis/nodejs-containeranalysis#421)) ([23cd446](googleapis/nodejs-containeranalysis@23cd446)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [uuid](https://togithub.com/uuidjs/uuid) | [`^8.0.0` -> `^9.0.0`](https://renovatebot.com/diffs/npm/uuid/8.3.2/9.0.0) | [![age](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/compatibility-slim/8.3.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/confidence-slim/8.3.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>uuidjs/uuid</summary> ### [`v9.0.0`](https://togithub.com/uuidjs/uuid/blob/HEAD/CHANGELOG.md#​900-httpsgithubcomuuidjsuuidcomparev832v900-2022-09-05) [Compare Source](https://togithub.com/uuidjs/uuid/compare/v8.3.2...v9.0.0) ##### ⚠ BREAKING CHANGES - Drop Node.js 10.x support. This library always aims at supporting one EOLed LTS release which by this time now is 12.x which has reached EOL 30 Apr 2022. - Remove the minified UMD build from the package. Minified code is hard to audit and since this is a widely used library it seems more appropriate nowadays to optimize for auditability than to ship a legacy module format that, at best, serves educational purposes nowadays. For production browser use cases, users should be using a bundler. For educational purposes, today's online sandboxes like replit.com offer convenient ways to load npm modules, so the use case for UMD through repos like UNPKG or jsDelivr has largely vanished. - Drop IE 11 and Safari 10 support. Drop support for browsers that don't correctly implement const/let and default arguments, and no longer transpile the browser build to ES2015. This also removes the fallback on msCrypto instead of the crypto API. Browser tests are run in the first supported version of each supported browser and in the latest (as of this commit) version available on Browserstack. ##### Features - optimize uuid.v1 by 1.3x uuid.v4 by 4.3x (430%) ([#​597](https://togithub.com/uuidjs/uuid/issues/597)) ([3a033f6](https://togithub.com/uuidjs/uuid/commit/3a033f6bab6bb3780ece6d645b902548043280bc)) - remove UMD build ([#​645](https://togithub.com/uuidjs/uuid/issues/645)) ([e948a0f](https://togithub.com/uuidjs/uuid/commit/e948a0f22bf22f4619b27bd913885e478e20fe6f)), closes [#​620](https://togithub.com/uuidjs/uuid/issues/620) - use native crypto.randomUUID when available ([#​600](https://togithub.com/uuidjs/uuid/issues/600)) ([c9e076c](https://togithub.com/uuidjs/uuid/commit/c9e076c852edad7e9a06baaa1d148cf4eda6c6c4)) ##### Bug Fixes - add Jest/jsdom compatibility ([#​642](https://togithub.com/uuidjs/uuid/issues/642)) ([16f9c46](https://togithub.com/uuidjs/uuid/commit/16f9c469edf46f0786164cdf4dc980743984a6fd)) - change default export to named function ([#​545](https://togithub.com/uuidjs/uuid/issues/545)) ([c57bc5a](https://togithub.com/uuidjs/uuid/commit/c57bc5a9a0653273aa639cda9177ce52efabe42a)) - handle error when parameter is not set in v3 and v5 ([#​622](https://togithub.com/uuidjs/uuid/issues/622)) ([fcd7388](https://togithub.com/uuidjs/uuid/commit/fcd73881692d9fabb63872576ba28e30ff852091)) - run npm audit fix ([#​644](https://togithub.com/uuidjs/uuid/issues/644)) ([04686f5](https://togithub.com/uuidjs/uuid/commit/04686f54c5fed2cfffc1b619f4970c4bb8532353)) - upgrading from uuid3 broken link ([#​568](https://togithub.com/uuidjs/uuid/issues/568)) ([1c849da](https://togithub.com/uuidjs/uuid/commit/1c849da6e164259e72e18636726345b13a7eddd6)) ##### build - drop Node.js 8.x from babel transpile target ([#​603](https://togithub.com/uuidjs/uuid/issues/603)) ([aa11485](https://togithub.com/uuidjs/uuid/commit/aa114858260402107ec8a1e1a825dea0a259bcb5)) - drop support for legacy browsers (IE11, Safari 10) ([#​604](https://togithub.com/uuidjs/uuid/issues/604)) ([0f433e5](https://togithub.com/uuidjs/uuid/commit/0f433e5ec444edacd53016de67db021102f36148)) - drop node 10.x to upgrade dev dependencies ([#​653](https://togithub.com/uuidjs/uuid/issues/653)) ([28a5712](https://togithub.com/uuidjs/uuid/commit/28a571283f8abda6b9d85e689f95b7d3ee9e282e)), closes [#​643](https://togithub.com/uuidjs/uuid/issues/643) ##### [8.3.2](https://togithub.com/uuidjs/uuid/compare/v8.3.1...v8.3.2) (2020-12-08) ##### Bug Fixes - lazy load getRandomValues ([#​537](https://togithub.com/uuidjs/uuid/issues/537)) ([16c8f6d](https://togithub.com/uuidjs/uuid/commit/16c8f6df2f6b09b4d6235602d6a591188320a82e)), closes [#​536](https://togithub.com/uuidjs/uuid/issues/536) ##### [8.3.1](https://togithub.com/uuidjs/uuid/compare/v8.3.0...v8.3.1) (2020-10-04) ##### Bug Fixes - support expo>=39.0.0 ([#​515](https://togithub.com/uuidjs/uuid/issues/515)) ([c65a0f3](https://togithub.com/uuidjs/uuid/commit/c65a0f3fa73b901959d638d1e3591dfacdbed867)), closes [#​375](https://togithub.com/uuidjs/uuid/issues/375) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-containeranalysis). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xOTQuMiIsInVwZGF0ZWRJblZlciI6IjMyLjE5NC4yIn0=-->
* chore(main): release 4.3.0 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [4.4.0](https://togithub.com/googleapis/nodejs-containeranalysis/compare/v4.3.0...v4.4.0) (2022-10-06) ### Features * Add new analysis status and cvss version fields ([#443](https://togithub.com/googleapis/nodejs-containeranalysis/issues/443)) ([2b4db8e](https://togithub.com/googleapis/nodejs-containeranalysis/commit/2b4db8ee9859823e32de8256853429adf88cf207)) --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Here is the summary of changes. You are about to add 14 region tags.
This comment is generated by snippet-bot.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once repo-root CODEOWNERS
is updated. 👍
Updated |
Rebase failed
Closing. I will redo this pull-request and make sure we do not update via merge commits. |