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

fix: wrong struct used to marshal body into release #4128

Merged

Conversation

vLia
Copy link
Contributor

@vLia vLia commented Jul 5, 2023

Pull request description

Creating triggers and webhooks via yaml was throwing error

Checklist (choose whats happened)

  • breaking change! (describe)
  • tested locally
  • tested on cluster
  • added new dependencies
  • updated the docs
  • added a test

Breaking changes

Changes

Fixes

@vLia vLia merged commit e745002 into release/v1.13.0 Jul 5, 2023
@vLia vLia deleted the lilla/fix/create-trigger-webhook-yaml-release branch July 5, 2023 09:01
vsukhin added a commit that referenced this pull request Jul 7, 2023
* test commit (#3753)

* feat: cronjob template (#3739)

* feat: add cronjob template to model

* feat: cron job temaplte field

* fix: dep update

* fix: test and test suite template typo

* doc: cron job template

* fix: dep update

* trigger ci flow (#3754)

* trigger flow (#3756)

* Update flow (#3766)

ci changes for develop and release branches

* feat: use command parameters and deprecate args (#3694)

* feat: use command parameters and deprecate args

* fix: go fmt

* fix: command mode field

* feat:command mode field

* fix: agrs mode field

* fix: use args mode

* fix: rmove deprecated

* fix: golint

* fix: unit test

* fix: unit test

* fix: support command and args in curl

* fix: args mode parameter

* fix: remove log

* fix: go lint

* fix: remove unused const

* feat: artillery command and args

* fix: update model for artifcts

* feat: change artifact support

* fix: refresh docs

* fix: scrape curl dirs

* fix: unit test

* feat: postman command and args

* fix: unit test

* feat: kubepug args and command

* fix: unit  test

* fix: int test

* fix: postman int

* feat: soapui command and args

* fix: unit test

* feat: add gradle command and arguments

* fix: unit test

* fix: unit test

* fix: int test

* feat: maven args and command

* feat: jmeter command and args

* fix: unit test

* fix: unit test

* feat: cypress command and args

* feat: command and args for ginkgo

* fix: golint

* fix: report file for ginkgo

* fix: ginkgo vars arg

* feat: k6 args and command

* feat: playwright command and args

* fix: ci flow

* doc: executor default args and command

* fix: escape <> for docs

* fix: dep update

* doc: explain parameters

* doc: executor command and args example

* add tests for develop branch (#3769)

* fix: upload variables file when it is too big for the command line (#3767)

* fix: increase http timeout (#3772)

* add checking for release branch (#3778)

* fix typo (#3779)

* fix typo in ci

* fix ldflags (#3790)

* docs: new information architecture (#3791)

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>
Co-authored-by: aabedraba <aabedraba@aabedrabas-MacBook-Pro.local>

* feat: add develop branch to docs workflow! (#3793)

* docs: trigger docs build

* docs: add docs redirects

* feat: container init data for executors (#3768)

* feat: curl container init data

* feat: init container data for template

* feat: init container data for soapui

* feat: init container data for cypress

* feat: cypress init container data

* feat: init container data for postman

* fix: unit test

* fix: logs

* fix: test

* fix: unit test

* fix: unit test

* fix: unit test

* fix: unit test

* fix: unit test

* fix: int test

* feat: init container data for kubepug

* fix: unit tests

* fix: unit tests

* fix: unit test

* fix: int test

* fix: test yaml

* feat: artillery init container data

* feat: init container data for ginkgo

* fix: gingko int test

* fix: unit test

* fix: int test

* fix: int test

* fix: int test

* fix: unit test

* feat: int test

* fix: unit test

* fix: data dir

* fix: int test

* fix: unit test

* fix: unit test

* Revert "fix: unit test"

This reverts commit 901a8df.

* fix: unit test

* docs: add more redirects (#3798)

* docs: add redirect for test variable

* docs: respect system theme for docs

* docs: variables file changes (#3794)

* docs: variables file changes

* docs: Apply suggestions from code review

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

---------

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* comment out automatic release branch merge for now

* build: update makefile (#3803)

* update makefile

* add cypress12 (#3812)

* docs: JMeter memory consumption (#3811)

* docs: JMeter memory consumption

* docs: apply suggestions from code review

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

---------

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* exclude building docs on every commit (#3817)

* trigger workflow

* trigger ci

* feat: detect more kubernetes distributions (#3818)

* chore: do not run test pipeline on docs

* Revert "chore: do not run test pipeline on docs"

This reverts commit 10f03f5.

* feat: add spinner to testkube init (#3788)

* feat: add spinner for testkube-init

Signed-off-by: Sindhu Inti <iamsindhuinti23@gmail.com>

* feat:add spinner to testkube init

Signed-off-by: Sindhu Inti <iamsindhuinti23@gmail.com>

* fix: import time package

* linting

Signed-off-by: Sindhu Inti <iamsindhuinti23@gmail.com>

---------

Signed-off-by: Sindhu Inti <iamsindhuinti23@gmail.com>

* chore: update broken docs links and change to new docs links everywhere (#3819)

* docs: enable doc search from /search page (#3824)

* chore: do not run test pipeline on docs (#3825)

* docs: redirects (#3826)

* fix added missing version for cypress

* docs: create a Testkube Dashboard category in  docs (#3834)

* Dashboard E2E tests - selector fixed (#3836)

* feat: added cloud connect command (#3777)

* fix: http timeout (#3773)

* feat: added cloud connect command

* add check for release branch (#3780)

* fix typo

* feat: added cloud connect command

* added env cloud client

* build: fix ldflag (#3789)

* update ldflags

* comment out automatic release branch merge for now

* fix typo

* feat: almost done

* feat: statuses update

* chore: moving some parts to common package

* chore connect

* fix: messages

* fix: dashboard

* feat: add CloudLogin function

* fix: flags wasnt propagating

* fix: cleaning

* fix: added env id and org id as well have already env created

* fix: env and org env variable

* fix: removed unused code

* fix: flags naming

* fix: added cloud uris to testkube init and upgrade

* fix: passing root domain

* fix: tests

* fix: tests

* fix: golangci lint

* fix: rollbacl some strange changes

* fix: rollbacl some strange changes

* fix: go mod tidy

* fix: extract agent port to the const

* fix: unused code

* fix: unused code

---------

Co-authored-by: Vladislav Sukhin <vladislav@kubeshop.io>
Co-authored-by: ypoplavs <45286051+ypoplavs@users.noreply.github.com>
Co-authored-by: ypoplavs <yulia.poplavska@kubeshop.io>
Co-authored-by: Povilas Versockas <p.versockas@gmail.com>

* docs: cloud docs improvement (#3833)

* feat: Test run script - branch overwrite (#3838)

* Test run script - branch param

* Test run script - branch param

* E2E tests moved to testkube-dashboard repo, test CRD updated (#3839)

* fix: do not use object that is nil on error to register an error with on update test (#3848)

* feat: ZAP executor (#3821)

* feat: zap executor

* rename release branch (#3845)

* docs: trigger ci

* docs: fix link

* docs: re-enable search

* docs: disable search page

* fix: remove dev dockerfile (#3855)

* feat: add clusterid to agent (#3858)

* docs: Migrate Web Images to Local Images (#3853)

* Migrate Web Images to Local Images

In the Testkube Cloud doc pages.

* Update architecture.md

* Update environment-management.md

* Update installing-agent.md

* Update organization-management.md

* Update managing-cli-context.md

* feat: cd event emiiter (#3806)

* feat: cd event emiiter

* fix: lintt

* fix: change cd events mapping

* feat: add cdevents env vars

* feat: send cd event for scraper

* fix: detect mime type

* fix: test type

* fix: unit tests

* fix: error message

* fix: add source field

* fix: typo

* fix: add error

* fix: check for empty namespace

* feat: map test types for cd event

* feat: map test types for cd event

* fixL change notifying about files

* fix: double file opening

* fix: remove duplicated notification

* feat: unit test for event listener

* feat: cd event loader test

* feat: cd event integration test

* fix: unit test for extractor

* fix: minio integrattion test

* feat: extractor integration tests

* fix: int tests

* fix: int test

* fix: int test

* fix: int test

* fix: int test

* fix: run tests in parallel

* fix: unit tests

* docs: redirects and link fix

* docs: improve examples in curl executor

* fix: docs redirect

* docs: add timeout docs (#3863)

* docs: fix timeout and add to sidebar

* Cypress 12 tests - updated (#3866)

* feat: Custom container executor for Playwright - artifacts (#3867)

* Playwright project - lock file aligned

* Playwright project - lock file aligned

* playwright artifacts tests

* custom container executor - playwright - artifacts

* custom container executor - cypress - artifacts (#3868)

* Fixing the typo (#3865)

Co-authored-by: Shiva Kumar, Harsha <harsha.shivakumar@lowes.com>

* docs: fix timeout page

* docs: add tracetest executor (#3878)

* feat: dashboard uri for cd event (#3879)

* feat: dashboard uri for cd event

* fix: unit tests

* fix: unit test

* docs: dashboard uri

* fix: unit test

* fix: dep

* fix: dep

* fix: use latest version

* docs: telemetry and metrics refinements (#3885)

* docs: refine telemetry docs

Refine services to which telemetry are sent

* docs: refine metrics doc

Relates to kubeshop/helm-charts#515 (comment)

* ci: add alpine image parametrization (#3891)

* add alpine image parametrization

* fix typo in release flow

* feat: condition ttl (#3890)

* feat: add ttl to model

* feat: ttl condition support

* fix: dep update

* fix: ttl units

* fix: dep update

* ci: rename gh token var (#3893)

* fix: http proxy

* ci: fix gc token var

* docs: add zap executor (#3897)

* docs: add zap executor

* docs: apply suggestions from code review

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

---------

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* ci: add docker registry description (#3898)

* add GH action for uploading readme to docker registry

* fix: check username and token for update (#3896)

* fix: check username and token for update

* fix: update secret by field

* fix: dep update

* fix: dep update

* docs: zap references (#3902)

* feat: default artifacts dir (#3905)

* feat: default artifacts dir

* fix: docs

* fix: dynamic artifacts diiirectory

* fix: golint

* ci: add SBOM and cosign for all binaries and docker images (#3910)

* ci: add sboms and cosign

* fix: support plain secret var for postman (#3911)

* ci: add workflow dispatch

* ci: add permission contents for release-dev flow

* Break Dashboard doc into separate pages (#3927)

* fix: support multiline commands (#3937)

* fix: optional working dir

* fix: remove arg

* fix: support multiline commands for executors

* fix: ginkgo run command

* fix: zap config path

* fix: remove duplicated declaration

* fix: log command args

* fix: golint

* docs: update zap examples (#3943)

* fix: update examples in docs

* maven executor tests - negative test name fixed (#3945)

* docs: update Ingress set-up (#3947)

* docs: update Ingress set-up

* fix editing suggestions

* fix: link of Docs at README.md (#3918)

* fix: dep update (#3929)

* fix: dep update

* fix: dep update

* fix: dep update

* docs: update the s3 with IAM setup (#3954)

* docs: update the s3 with IAM setup

* fix: Apply suggestions from code review

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

---------

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* fix: use executor job template for container executor (#3959)

* fix: use executor job template for container executor

* fix: use executor job template

* fix: check for empty template

* fix: use proper mapping

* fix: sync telemetry for cli (#3963)

* fix: sync server and cli telemetry flag

* fix: add debug logs

* e2e tests - crd - separate DASHBOARD_API_URL (#3966)

* feat: webhook template and headers (#3978)

* feat: webhook template and headers

* fix: escape text

* docs: cli

* feat: add unit test

* fix: unit test

* fix: remove reset

* fix: template

* fix: lint

* fi

* docs: webhook fields description

* fix: dep

* Update api/v1/testkube.yaml

Co-authored-by: Lilla Vass <vLia@users.noreply.github.com>

* Update pkg/api/v1/testkube/model_webhook.go

Co-authored-by: Lilla Vass <vLia@users.noreply.github.com>

* Update pkg/api/v1/testkube/model_webhook_create_request.go

Co-authored-by: Lilla Vass <vLia@users.noreply.github.com>

---------

Co-authored-by: Lilla Vass <vLia@users.noreply.github.com>

* docs: update zap docs (#3989)

* chore: back port agent changes to develop (#3995)

* docs: add docs for cd events· (#3996)

* docs: add operator api reference (#3997)

* docs: improve slack docs (#3994)

* docs: refine schema for content.type (#4004)

* fix: docs pre-run script not supported in container executor (#4001)

* fix: dep update (#4011)

* fix: use testkube logger for telemetry

* Revert "fix: dep update (#4011)"

This reverts commit 7d20507.

* Revert "Revert "fix: dep update (#4011)""

This reverts commit 3404bec.

* Revert "fix: use testkube logger for telemetry"

This reverts commit 7e3b0c3.

* Dashboard E2E tests - CRD - storageClassName added (#4018)

* fix: refactor get client method (#4019)

* fix: refactor get client method

* fix: error message

* fix: use Testkube logs for telemetry (#4014)

* fix: use Testkube logs for telemetry

* fix: golint

* docs: Add Run Tests from Github Actions Page (#4021)

* Add Run Tests from Github Actions Page

* Update docs/docs/articles/run-tests-with-github-actions.md

Co-authored-by: Dawid Rusnak <dawid@drcode.pl>

* Update run-tests-with-github-actions.md

Fix table formatting.

* Update run-tests-with-github-actions.md

Add dashboardUrl to Cloud and Enterprise and Own Instance tables

* Update docs/docs/articles/run-tests-with-github-actions.md

Co-authored-by: Dawid Rusnak <dawid@drcode.pl>

* Update run-tests-with-github-actions.md

Update yaml segments and code snippet.

* Update run-tests-with-github-actions.md

minor editing

---------

Co-authored-by: Dawid Rusnak <dawid@drcode.pl>

* feat: test trigger condition list (#4022)

* feat: test trigger condition list

* fix: dep update

* fix: dep update

* feat: add testkube cloud login (#4026)

* docs: updated getting started section for cloud (#4023)

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* feat: testkube cloud connect populate user context (#4032)

* fix: Custom cypress executor CRDs - `latest` version for Cypress 8-10 (#4027)

* Cypress executor definitions - updated to latest version for Cypress 8-10

* Cypress custom executor definitions updated

* docs: update docs dependencies (#4037)

* docs: fix broken build

* feat: parallel testsuite (#3075)

* feat: test suite batch step

* feat: switch to testsuite v3

* fix: go lint

* feat: test suite batch execution models

* feat; remove batch step execution

* fix: mapping for batch testsuite execution

* fix: print batch results

* feat: change testsuite template

* fix: update batch test execution

* fix: add batch step for reference

* fix: change testsuite scheduling test processing

* fix: concurrency level parameter

* fix: go fmt

* fix: type conversion

* fix: dep update

* fix: add nil check

* fix: nil check

* fix: update version for template

* fix: dep update

* fix: tune test suite execution

* fix: tune abortion check

* fix: quote messages

* dep update

* fix: dep update

* fix: test suite examples

* fix: test suite docs

* fix: doc update

* add a separate workflow to test parallel testsuite images

* fix: check for empty steps

* fix: remove debugging

* fix: pass executor id for test suites

* fix: add test suite v2 object

* fix: test suite v2

* fix: backward compatibility

* fix: make steps optional

* fix:backward compatibility for api

* fix: remove test workflow

* fix: docs

* fix: rename test suite fields

* fix: update template

* fix: dep update

* fix: format string

* fix: golint

* fix: golint

* fix: change test suite step object

* fix: use refactored test suite step object

* fix: unit test

* fix: change delay type

* fix: enable caching

* fix: change example

* fix: test suite json refresh

* fix: add test suite step v2 model

* fix: doc update

* fix: backward compatibility

* fix: dep update

---------

Co-authored-by: ypoplavs <yulia.poplavska@kubeshop.io>

* feat: validate cloud context on root cmd (#4033)

* feat: validate cloud context on root cmd

* chore: organize

* chore: organize

* chore: organize

* feat: added org-env names to context on set context

* feat: added org-env names fallback

* fix: golangci lint

* fix: codeql alert

* fix: utf character fails build

* feat: add testkube cloud init (#4042)

* docs: put CLI commands in tabs (#4045)

* docs: docs improvements" (#4047)

* docs: small fixes (#4048)

* fix: removed header from completion script (#4049)

* fix: removed header from completion script

* fix: removed printf

* fix: populate org and env (#4055)

* fix: populate org and env

* fix: populate org and env

* feat: add testkube cloud login alias (#4061)

* docs: update docs for helm charts (#4062)

* docs: Testkube CLI docker (#4064)

* docs: Testkube CLI docker

* docs: Apply suggestions from code review

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

---------

Co-authored-by: Julianne Fermi <julianne@kubeshop.io>

* feat: add orgid and envid to telemetry (#4065)

* chore(deps): bump golang.org/x/net from 0.6.0 to 0.7.0 (#4041)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.6.0 to 0.7.0.
- [Commits](golang/net@v0.6.0...v0.7.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: released testkube cli docker image (#3928)

* feat: yaml content type (#4070)

* feat: create yaml support

* feat: test source yaml update

* feat: update test suite yaml

* feat: test trigger yaml update

* docs: openapi yaml input

* Revert "docs: openapi yaml input"

This reverts commit 1710b0e.

* Revert "Revert "docs: openapi yaml input""

This reverts commit 90f7697.

* fix: update flow token

* fix: lint version

* fix: check linting

* fix: lint version check

* fix: change lint version

* fix: test linter

* fix: openapi linter

* fix: var shadow

* fix: delete test returns error only when deleting test CR fails otherwise it returns warnings (#3813)

* fix: delete test returns error only when deleting test CR fails otherwise it returns warnings

* fix: upgrade open api lint action

* fix: open api error

* fix: test

* test: update install.sh script (#4071)

* update install script to include beta arg

* feat: post run script (#4075)

* feat: post run script

* docs: cli reference

* fix: lint check

* fix: go fmt

* Revert "fix: lint check"

This reverts commit ed82bd5.

* fix: optional args mode

* fix: dep update

* cypress 12 example setup - step skipped (#4078)

* fix: check for nil parent (#4079)

* feat: basic reconciler (#4082)

* feat: basic reconciler

* fix: remove break

* fix: check job pod state

* fix: break loop

* feat: add reconciler to api sservice

* fix: add reconclier to config

* fix: nil check

* fix: test suite steps status

* ci: remove release branch creation in helm-charts repo (#4090)

* feat: cluster name for events (#4091)

* feat: cluster name for events

* fix: unit tests

* docs: fixing indentation around testkube ingress sample (#4087)

* feat: add refresh token for cloud (#4095)

* fix: dep update

* fix: wrong struct used to marshal body into (#4128)

* fix: map test suite YAML to proper update structure (#4129)

* fix: add hashtag to shebang in example script (#4136)

* docs: update example to be appliable on copy-paste (#4141)

* fix: dep update

* fix: dep update

* fix: go lint

* fix: lint

* Revert "fix: go lint"

This reverts commit 15a2c41.

* fix: merge

* fix: dep update

---------

Signed-off-by: Sindhu Inti <iamsindhuinti23@gmail.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: ypoplavs <45286051+ypoplavs@users.noreply.github.com>
Co-authored-by: Lilla Vass <vLia@users.noreply.github.com>
Co-authored-by: Abdallah Abedraba <aabedraba@gmail.com>
Co-authored-by: Julianne Fermi <julianne@kubeshop.io>
Co-authored-by: aabedraba <aabedraba@aabedrabas-MacBook-Pro.local>
Co-authored-by: ypoplavs <yulia.poplavska@kubeshop.io>
Co-authored-by: Sindhu  Inti <89198083+Sindhuinti@users.noreply.github.com>
Co-authored-by: Tomasz Konieczny <tomasz.konieczny@kubeshop.io>
Co-authored-by: Jacek Wysocki <jacek.wysocki@gmail.com>
Co-authored-by: Povilas Versockas <p.versockas@gmail.com>
Co-authored-by: Harsha Shiva Kumar <harshaskumar05@gmail.com>
Co-authored-by: Shiva Kumar, Harsha <harsha.shivakumar@lowes.com>
Co-authored-by: Ale <93217218+alelthomas@users.noreply.github.com>
Co-authored-by: Guillaume Berche <guillaume.berche@orange.com>
Co-authored-by: Richard Kovacs <kovacsricsi@gmail.com>
Co-authored-by: nicufk <89570185+nicufk@users.noreply.github.com>
Co-authored-by: Dawid Rusnak <dawid@drcode.pl>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Dylan Jones <85761518+djonesz@users.noreply.github.com>
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.

2 participants