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

Finish contributing updates; thanks @acarbonetto #12981

Merged
merged 17 commits into from
Apr 5, 2024

Conversation

stephen-crawford
Copy link
Contributor

@stephen-crawford stephen-crawford commented Mar 29, 2024

Description

I was going over old PRs and found this one which just needed a little help over the finish line

If someone could add skip-changelog that would be appreciated.

Related Issues

#7107

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Stephen Crawford <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Copy link
Contributor

github-actions bot commented Mar 29, 2024

Compatibility status:

Checks if related components are compatible with change aa45cda

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/performance-analyzer.git]

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

There'a a lot of duplication with the content in developer guide, we should just add enough that links to it.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Contributor

✅ Gradle check result for 576a56e: SUCCESS

stephen-crawford and others added 8 commits April 4, 2024 09:41
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for 3bef242: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for 4750e99: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for b82607a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for c10c569: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for ef2ed75: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for 2820836: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@stephen-crawford
Copy link
Contributor Author

This is just a documentation change so I don't think the failure is related at all. @owaiskazi19 would you be able to approve? Thank you

@owaiskazi19
Copy link
Member

This is just a documentation change so I don't think the failure is related at all. @owaiskazi19 would you be able to approve? Thank you

Let's wait for the gradle check to pass. I just triggered it again.

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for ef2ed75: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@owaiskazi19
Copy link
Member

@scrawfor99 did you rebase your changes with latest main?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

❌ Gradle check result for ef2ed75: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Apr 4, 2024

✅ Gradle check result for aa45cda: SUCCESS

@owaiskazi19 owaiskazi19 merged commit 19d6c5c into opensearch-project:main Apr 5, 2024
31 checks passed
@owaiskazi19 owaiskazi19 added the backport 2.x Backport to 2.x branch label Apr 5, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 5, 2024
* Finish contributing updates; thanks @acarbonetto

Signed-off-by: Stephen Crawford <[email protected]>

* Fix indenting

Signed-off-by: Stephen Crawford <[email protected]>

* Readd build failure notes

Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING with review tips

Signed-off-by: Andrew Carbonetto <[email protected]>

* Update CONTRIBUTING with review tips

Signed-off-by: Andrew Carbonetto <[email protected]>

* Update contributing comment for review comment

Signed-off-by: Andrew Carbonetto <[email protected]>

* Fix contruibuting doc

Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

---------

Signed-off-by: Stephen Crawford <[email protected]>
Signed-off-by: Andrew Carbonetto <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
(cherry picked from commit 19d6c5c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
kotwanikunal pushed a commit that referenced this pull request Apr 9, 2024
* Finish contributing updates; thanks @acarbonetto



* Fix indenting



* Readd build failure notes



* Update CONTRIBUTING with review tips



* Update CONTRIBUTING with review tips



* Update contributing comment for review comment



* Fix contruibuting doc



* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




* Update CONTRIBUTING.md




---------






(cherry picked from commit 19d6c5c)

Signed-off-by: Stephen Crawford <[email protected]>
Signed-off-by: Andrew Carbonetto <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
…12981)

* Finish contributing updates; thanks @acarbonetto

Signed-off-by: Stephen Crawford <[email protected]>

* Fix indenting

Signed-off-by: Stephen Crawford <[email protected]>

* Readd build failure notes

Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING with review tips

Signed-off-by: Andrew Carbonetto <[email protected]>

* Update CONTRIBUTING with review tips

Signed-off-by: Andrew Carbonetto <[email protected]>

* Update contributing comment for review comment

Signed-off-by: Andrew Carbonetto <[email protected]>

* Fix contruibuting doc

Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>

---------

Signed-off-by: Stephen Crawford <[email protected]>
Signed-off-by: Andrew Carbonetto <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
Signed-off-by: Shivansh Arora <[email protected]>
@peternied
Copy link
Member

@scrawfor99 I think there was a bad merge conflict - it looks like this PR deleted the contribution [1] by @reta. Could you please restore the Troubleshooting Failing Builds section?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants