Skip to content

Execute yamlRestTest in integration job#4838

Merged
ykmr1224 merged 1 commit into
opensearch-project:mainfrom
ykmr1224:github-workflow-test
Nov 21, 2025
Merged

Execute yamlRestTest in integration job#4838
ykmr1224 merged 1 commit into
opensearch-project:mainfrom
ykmr1224:github-workflow-test

Conversation

@ykmr1224

@ykmr1224 ykmr1224 commented Nov 20, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

  • In Github workflow, execute yamlRestTest in integration job instead of unit
    • Just for consistency (I felt weird to see the yamlRestTest failure in unit result)
  • Checked workflow execution result.

Related Issues

n/a

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • New PPL command checklist all confirmed.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff or -s.
  • 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.

@ykmr1224
Execute yamlRestTest in integration job
7314f42 
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
@ykmr1224 ykmr1224 self-assigned this Nov 20, 2025
@ykmr1224 ykmr1224 added the maintenance Improves code quality, but not the product label Nov 20, 2025
@ykmr1224 ykmr1224 marked this pull request as ready for review November 20, 2025 21:25
@ykmr1224 ykmr1224 enabled auto-merge (squash) November 20, 2025 21:26
@ykmr1224 ykmr1224 merged commit f005865 into opensearch-project:main Nov 21, 2025
38 checks passed
asifabashar pushed a commit to asifabashar/sql that referenced this pull request Dec 10, 2025
@ykmr1224 @asifabashar
Execute yamlRestTest in integration job (opensearch-project#4838)
cdb15fa 
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LantaoJin LantaoJin LantaoJin approved these changes

@Swiddis Swiddis Swiddis approved these changes

@ps48 ps48 Awaiting requested review from ps48 ps48 is a code owner

@kavithacm kavithacm Awaiting requested review from kavithacm

@derek-ho derek-ho Awaiting requested review from derek-ho

@joshuali925 joshuali925 Awaiting requested review from joshuali925 joshuali925 is a code owner

@dai-chen dai-chen Awaiting requested review from dai-chen dai-chen is a code owner

@YANG-DB YANG-DB Awaiting requested review from YANG-DB

@mengweieric mengweieric Awaiting requested review from mengweieric mengweieric is a code owner

@vamsimanohar vamsimanohar Awaiting requested review from vamsimanohar vamsimanohar is a code owner

@penghuo penghuo Awaiting requested review from penghuo penghuo is a code owner

@seankao-az seankao-az Awaiting requested review from seankao-az

@MaxKsyunz MaxKsyunz Awaiting requested review from MaxKsyunz

@Yury-Fridlyand Yury-Fridlyand Awaiting requested review from Yury-Fridlyand

@anirudha anirudha Awaiting requested review from anirudha anirudha is a code owner

@forestmvey forestmvey Awaiting requested review from forestmvey

@acarbonetto acarbonetto Awaiting requested review from acarbonetto acarbonetto is a code owner

@GumpacG GumpacG Awaiting requested review from GumpacG

@noCharger noCharger Awaiting requested review from noCharger noCharger is a code owner

@qianheng-aws qianheng-aws Awaiting requested review from qianheng-aws qianheng-aws is a code owner

@yuancu yuancu Awaiting requested review from yuancu yuancu is a code owner

@RyanL1997 RyanL1997 Awaiting requested review from RyanL1997 RyanL1997 is a code owner

Assignees

@ykmr1224 ykmr1224

Labels

maintenance Improves code quality, but not the product

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ykmr1224 @LantaoJin @Swiddis