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

ossm: add merge_upstream.sh to do merges of upstream commits #14

Merged

Conversation

dgn
Copy link

@dgn dgn commented Apr 23, 2024

Sadly, just merging commits isn't enough, we also need to re-generate some files. To not over-complicate the automator script, I put it into a separate script that we can call from the automator.

Makefile.core.mk Outdated Show resolved Hide resolved
dgn added a commit to dgn/sail-operator that referenced this pull request Apr 23, 2024
This is required for openshift-service-mesh#14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
@dgn dgn force-pushed the add-merge-script branch from cfceff3 to df3ac1c Compare April 23, 2024 12:35
@FilipB FilipB self-requested a review April 23, 2024 12:41
@dgn
Copy link
Author

dgn commented Apr 23, 2024

/retest

istio-testing pushed a commit to istio-ecosystem/sail-operator that referenced this pull request Apr 23, 2024
This is required for openshift-service-mesh#14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
ossm/merge_upstream.sh Outdated Show resolved Hide resolved
ossm/merge_upstream.sh Show resolved Hide resolved
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
@dgn dgn force-pushed the add-merge-script branch from df3ac1c to e82fd06 Compare April 23, 2024 14:09
Copy link

openshift-ci bot commented Apr 23, 2024

@dgn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-4-16 e82fd06 link false /test e2e-ocp-4-16

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@dgn
Copy link
Author

dgn commented Apr 23, 2024

/test e2e-ocp-4-16

@FilipB FilipB self-requested a review April 23, 2024 15:31
Copy link

@FilipB FilipB left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-merge-bot openshift-merge-bot bot merged commit 7cefe83 into openshift-service-mesh:main Apr 23, 2024
8 of 9 checks passed
@dgn dgn deleted the add-merge-script branch April 23, 2024 15:37
openshift-merge-bot bot pushed a commit that referenced this pull request Apr 24, 2024
…ator@main (#17)

* Use INFO level to log missing namespace in IstioRevision/IstioCNI (#64)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Ensure control plane is deployed immediately when the target namespace is created (#65)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (#66)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

* Update SetDefaultEventuallyTimeout for e2e test (#69)

Signed-off-by: frherrer <frherrer@redhat.com>

* Ensure CI highlights failure in lint-watches.sh (#70)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Make lint-watches ignore ValidatingAdmissionPolicy(Binding) (#71)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Ensure update-istio.sh updates version field in versions.yaml (#72)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Wrap errors to provide context (#68)

* Wrap errors to provide context

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Add called function instead of the caller to error message

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix lint errors

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix test

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Make bundle generation work during merges (#73)

This is required for #14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>

* Refactor (#76)

* Merge if statements

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Replace multiple ifs with switch

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (#67)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>
Signed-off-by: istio-testing <istio-testing-bot@google.com>
Signed-off-by: frherrer <frherrer@redhat.com>
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: Marko Lukša <mluksa@redhat.com>
Co-authored-by: Istio Automation <istio-testing-bot@google.com>
Co-authored-by: Francisco Herrera <frherrer@redhat.com>
Co-authored-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: openshift-service-mesh-bot <null>
openshift-merge-bot bot pushed a commit that referenced this pull request Apr 24, 2024
…ator@main (#18)

* Use INFO level to log missing namespace in IstioRevision/IstioCNI (#64)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Ensure control plane is deployed immediately when the target namespace is created (#65)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (#66)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

* Update SetDefaultEventuallyTimeout for e2e test (#69)

Signed-off-by: frherrer <frherrer@redhat.com>

* Ensure CI highlights failure in lint-watches.sh (#70)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Make lint-watches ignore ValidatingAdmissionPolicy(Binding) (#71)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Ensure update-istio.sh updates version field in versions.yaml (#72)

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Wrap errors to provide context (#68)

* Wrap errors to provide context

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Add called function instead of the caller to error message

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix lint errors

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Fix test

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Make bundle generation work during merges (#73)

This is required for #14

Signed-off-by: Daniel Grimm <dgrimm@redhat.com>

* Refactor (#76)

* Merge if statements

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Replace multiple ifs with switch

Signed-off-by: Marko Lukša <mluksa@redhat.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>

* Automator: Update dependencies in istio-ecosystem/sail-operator@main (#67)

Signed-off-by: istio-testing <istio-testing-bot@google.com>

---------

Signed-off-by: Marko Lukša <mluksa@redhat.com>
Signed-off-by: istio-testing <istio-testing-bot@google.com>
Signed-off-by: frherrer <frherrer@redhat.com>
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: Marko Lukša <mluksa@redhat.com>
Co-authored-by: Istio Automation <istio-testing-bot@google.com>
Co-authored-by: Francisco Herrera <frherrer@redhat.com>
Co-authored-by: Daniel Grimm <dgrimm@redhat.com>
Co-authored-by: openshift-service-mesh-bot <null>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants