feat: reuseable workflow

[ChaudharyRaman]

Fixes #700
Please briefly answer these questions:

• what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
  Added reuseable workflow and implement in benchmark.yml and validation.yml

• what changes does this pull request make?

• are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

[Phoenix500526]

The validation ci test is failed. Please fix it.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.19%. Comparing base (85e7891) to head (58f001b).
Report is 6 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #738      +/-   ##
==========================================
- Coverage   75.19%   75.19%   -0.01%     
==========================================
  Files         175      175              
  Lines       26803    26803              
  Branches    26803    26803              
==========================================
- Hits        20155    20154       -1     
- Misses       5438     5440       +2     
+ Partials     1210     1209       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ChaudharyRaman]

@Phoenix500526 @liangyuanpeng Please review and suggest for any changes and addition/fix in the script.

[Phoenix500526]

LGTM. Please modify the third commit message. BTW, you don't need to manually merge the master branch when there are no conflicts. GitHub will automatically rebase and merge it. Therefore, please remove the second commit and squash the first, third, and fourth commits into one."

[ChaudharyRaman]

@Phoenix500526 Just pushed, Please check

[Phoenix500526]

LGTM

[liangyuanpeng]

otherwise LGTM.

[liangyuanpeng]

This PR resolved this issue - #700

Update to Fixes #700, and then it will auto close the issue when this PR is merged.

[liangyuanpeng]

LGTM! Thanks for your work :)

[liangyuanpeng]

requested a review from liangyuanpeng

LGTM and thanks, (I don`t have owners here.)

[ChaudharyRaman]

LGTM! Thanks for your work :)

Got it. Thanks! @bsbds Please Review.

[Phoenix500526]

LGTM

[liangyuanpeng]

@ChaudharyRaman
I think we still update here, do you want to continue it?

Xline/.github/workflows/pull_request.yml

Lines 128 to 151 in eadf2d9

	- name: Build xline image
	run: |
	docker run -q --rm -v $(pwd):/xline \
	-e SCCACHE_GHA_ENABLED=on \
	-e ACTIONS_CACHE_URL=${ACTIONS_CACHE_URL} \
	-e ACTIONS_RUNTIME_TOKEN=${ACTIONS_RUNTIME_TOKEN} \
	ghcr.io/xline-kv/build-env:latest \
	cargo build --release --bin xline --bin benchmark --bin validation_lock_client
	sudo apt-get install -y --force-yes expect
	cd scripts
	cp ../target/release/{xline,benchmark,validation_lock_client} .
	ldd ./xline
	ldd ./benchmark
	cp ../fixtures/{private,public}.pem .
	docker build . -t ghcr.io/xline-kv/xline:master
	cd ..
	pwd
	mkdir -p _output
	docker save -o _output/xline.tar ghcr.io/xline-kv/xline:master
	ls _output
	- uses: actions/upload-artifact@v4

[ChaudharyRaman]

@liangyuanpeng Yeah right, I would do that. Currently, I am working on GSOC proposal with dead-line so close, Will create a PR by tomorrow. Thank You.

[ChaudharyRaman]

@liangyuanpeng, Previously while implementing what i thought is the for actions/upload-artifact@v4 , we are using for upload Log and Benchmark, and since Upload logs depends on failure() || cancelled(). Its better to let it be there and so instead of creating new Upload, lets create a general Case for rest including Benchmark and _output/xline.tar. What do you say.