Skip to content

Initial version of presubmit-tests.sh helper in prow-tests image#12

Merged
google-prow-robot merged 2 commits intoknative:masterfrom
adrcunha:presubmit-helper
Jul 18, 2018
Merged

Initial version of presubmit-tests.sh helper in prow-tests image#12
google-prow-robot merged 2 commits intoknative:masterfrom
adrcunha:presubmit-helper

Conversation

@adrcunha
Copy link
Copy Markdown
Contributor

This helper manages flag parsing and presubmit skipping, leaving the real script in the repos with only the important stuff (i.e., running the tests themselves).

This will reduce boilerplate code and unify the testing strategy and features for presubmits across ther repos.

This helper manages flag parsing and presubmit skipping, leaving the real script in the repos with only the important stuff (i.e., running the tests themselves).

This will reduce boilerplate code and unify the testing strategy and features for presubmits across ther repos.
@google-prow-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrcunha

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

@google-prow-robot google-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 18, 2018
@google-prow-robot google-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 18, 2018
@adrcunha adrcunha requested review from jessiezcc and removed request for steuhs July 18, 2018 03:19
@jessiezcc
Copy link
Copy Markdown
Contributor

/lgtm

@google-prow-robot google-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 18, 2018
@google-prow-robot google-prow-robot merged commit afba074 into knative:master Jul 18, 2018
@adrcunha adrcunha deleted the presubmit-helper branch July 18, 2018 22:38
google-prow-robot pushed a commit to knative/build that referenced this pull request Jul 19, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`presubmit-tests.sh` helper is implemented by knative/test-infra#12
google-prow-robot pushed a commit to knative/serving that referenced this pull request Jul 20, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

presubmit-tests.sh helper is implemented by knative/test-infra#12
google-prow-robot pushed a commit that referenced this pull request Jul 21, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`presubmit-tests.sh` helper was implemented by #12
google-prow-robot pushed a commit to knative/eventing that referenced this pull request Jul 21, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

presubmit-tests.sh helper is implemented by knative/test-infra#12
google-prow-robot pushed a commit to knative/docs that referenced this pull request Jul 21, 2018
* Use helpers from prow-tests image for tests

We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`presubmit-tests.sh` helper is implemented by knative/test-infra#12
`e2e-tests.sh` helper is implemented by knative/test-infra#17

* Explicitly fail test if setup fails
google-prow-robot pushed a commit to knative/docs that referenced this pull request Jul 21, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

presubmit-tests.sh helper is implemented by knative/test-infra#12
adrcunha added a commit that referenced this pull request Aug 6, 2018
* Initial version of presubmit-tests.sh helper in prow-tests image

This helper manages flag parsing and presubmit skipping, leaving the real script in the repos with only the important stuff (i.e., running the tests themselves).

This will reduce boilerplate code and unify the testing strategy and features for presubmits across ther repos.

* Reorganize source and clarify comments.
bbrowning pushed a commit to bbrowning/serving that referenced this pull request Oct 5, 2018
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

presubmit-tests.sh helper is implemented by knative/test-infra#12
vdemeester pushed a commit to vdemeester/knative-build that referenced this pull request Apr 3, 2019
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`presubmit-tests.sh` helper is implemented by knative/test-infra#12
vdemeester pushed a commit to vdemeester/knative-build that referenced this pull request Apr 3, 2019
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`presubmit-tests.sh` helper is implemented by knative/test-infra#12
Cynocracy pushed a commit to Cynocracy/test-infra that referenced this pull request Jul 22, 2020
* Add domain and realm CRDs

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

* Update copyright headers

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

* Add unit tests for domain and realm lifecycle

Signed-off-by: Andrew Su <andrew.su.01@gmail.com>

* Add unit tests for domain and realm types

Signed-off-by: Andrew Su <andrew.su.01@gmail.com>

* Add unit tests for realm and domain defaults

Signed-off-by: Andrew Su <andrew.su.01@gmail.com>

* Update copyright header

Signed-off-by: Andrew Su <andrew.su.01@gmail.com>

* Address comments

* Update CRD verison to v1

* Make a copy of LBIngressStatus as LBImgressspec

* gen reconciler changes

* Update tests

* Use IngressConfig struct instead of map type

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

* Address comments

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

* Address comments

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

* Update codegen

Signed-off-by: Shash Reddy <shashwathireddy@gmail.com>

Co-authored-by: Shash Reddy <shashwathireddy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants