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

Reuse jobs of existing Github Actions Workflow #20

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

mohitsaxenaknoldus
Copy link

Fixes #18

@mohitsaxenaknoldus
Copy link
Author

@PierreZ can we please run this to see if it's working the right way?

Copy link
Member

@PierreZ PierreZ left a comment

Choose a reason for hiding this comment

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

Thanks @mohitsaxenaknoldus, this looks really nice, nice work 🥇

I left a couple of comments, mostly in order to split the work, and make the steps useable for other developers and workflows.

Feel free to push some commits 🚀

.github/workflows/cron-correctness.yml Outdated Show resolved Hide resolved
.github/workflows/cron-correctness.yml Outdated Show resolved Hide resolved
.github/workflows/cron-correctness.yml Outdated Show resolved Hide resolved
.github/workflows/pr-correctness.yml Outdated Show resolved Hide resolved
@PierreZ
Copy link
Member

PierreZ commented Dec 29, 2021

@PierreZ can we please run this to see if it's working the right way?

Unfortunately, I could run it, but it won't work, as we are still impacted by the official FDB site downtime. Downloading FDB from the official website is like the first step for every workflows 😞

I'm checking everyday the relevant issue and also the related posts on the forum, but no ETA for now.

@mohitsaxenaknoldus
Copy link
Author

Thanks @mohitsaxenaknoldus, this looks really nice, nice work 🥇

I left a couple of comments, mostly in order to split the work, and make the steps useable for other developers and workflows.

Feel free to push some commits 🚀

Done.
However, I am not aware how those parameters are passed. LMK if I missed something.

@mohitsaxenaknoldus
Copy link
Author

@PierreZ can we please run this to see if it's working the right way?

Unfortunately, I could run it, but it won't work, as we are still impacted by the official FDB site downtime. Downloading FDB from the official website is like the first step for every workflows disappointed

I'm checking everyday the relevant issue and also the related posts on the forum, but no ETA for now.

Is it working now?

@PierreZ
Copy link
Member

PierreZ commented Jan 6, 2022

@PierreZ can we please run this to see if it's working the right way?

Unfortunately, I could run it, but it won't work, as we are still impacted by the official FDB site downtime. Downloading FDB from the official website is like the first step for every workflows disappointed
I'm checking everyday the relevant issue and also the related posts on the forum, but no ETA for now.

Is it working now?

Website is up since yesterday, but artifacts have been moved. We are working on publishing a new version for the Github Action. Once this is published, we will merge #19, and everything should work.

I'll run the action once everything is ready. In the mean time, I think you have some indentation errors on your yaml, as Github is not setting some keys in green

@mohitsaxenaknoldus
Copy link
Author

@PierreZ can we please run this to see if it's working the right way?

Unfortunately, I could run it, but it won't work, as we are still impacted by the official FDB site downtime. Downloading FDB from the official website is like the first step for every workflows disappointed
I'm checking everyday the relevant issue and also the related posts on the forum, but no ETA for now.

Is it working now?

Website is up since yesterday, but artifacts have been moved. We are working on publishing a new version for the Github Action. Once this is published, we will merge #19, and everything should work.

I'll run the action once everything is ready. In the mean time, I think you have some indentation errors on your yaml, as Github is not setting some keys in green

Should be fixed now, thanks!

Copy link
Member

@PierreZ PierreZ left a comment

Choose a reason for hiding this comment

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

CI is now fixed, please rebase your PR. I also left some comments.

.github/workflows/correctness.yml Outdated Show resolved Hide resolved
.github/workflows/correctness.yml Outdated Show resolved Hide resolved
.github/workflows/cron-correctness.yml Show resolved Hide resolved
.github/workflows/pr-correctness.yml Show resolved Hide resolved
.github/workflows/cron-correctness.yml Show resolved Hide resolved
.github/workflows/pr-correctness.yml Show resolved Hide resolved
.github/workflows/pr-correctness.yml Outdated Show resolved Hide resolved
.github/workflows/cron-correctness.yml Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Jan 19, 2022

Pull Request Test Coverage Report for Build 1802989275

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 201 unchanged lines in 7 files lost coverage.
  • Overall coverage increased (+5.02%) to 75.378%

Files with Coverage Reduction New Missed Lines %
foundationdb/src/tuple/pack.rs 1 92.74%
foundationdb/src/tuple/subspace.rs 1 98.89%
foundationdb/src/transaction.rs 2 88.63%
foundationdb/src/api.rs 6 86.3%
foundationdb/src/tuple/hca.rs 7 67.83%
foundationdb/src/future.rs 31 80.43%
foundationdb-bindingtester/src/main.rs 153 89.69%
Totals Coverage Status
Change from base Build 1701316279: 5.02%
Covered Lines: 4084
Relevant Lines: 5418

💛 - Coveralls

@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2022

Codecov Report

Merging #20 (d8d99de) into main (d8a388b) will increase coverage by 1.93%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
+ Coverage   81.49%   83.43%   +1.93%     
==========================================
  Files          18       24       +6     
  Lines        3091     4714    +1623     
==========================================
+ Hits         2519     3933    +1414     
- Misses        572      781     +209     
Impacted Files Coverage Δ
foundationdb/src/future.rs 80.43% <0.00%> (-8.16%) ⬇️
foundationdb-bindingtester/src/main.rs 89.68% <0.00%> (-2.59%) ⬇️
foundationdb/src/lib.rs 100.00% <0.00%> (ø)
foundationdb/src/transaction.rs 88.62% <0.00%> (ø)
foundationdb/src/directory/node.rs 80.95% <0.00%> (ø)
foundationdb/src/directory/error.rs 0.00% <0.00%> (ø)
foundationdb/src/directory/directory_layer.rs 88.79% <0.00%> (ø)
foundationdb/src/directory/directory_partition.rs 81.34% <0.00%> (ø)
foundationdb/src/directory/mod.rs 94.85% <0.00%> (ø)
foundationdb/src/directory/directory_subspace.rs 90.41% <0.00%> (ø)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8a388b...d8d99de. Read the comment docs.

@PierreZ
Copy link
Member

PierreZ commented Jan 20, 2022

I think something is not working somewhere:

https://github.com/foundationdb-rs/foundationdb-rs/actions/runs/1717931608

error parsing called workflow "foundationdb-rs/foundationdb-rs/.github/workflows/correctness.yml@main": failed to fetch workflow: workflow was not found.

Copy link
Member

@PierreZ PierreZ left a comment

Choose a reason for hiding this comment

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

error parsing called workflow "foundationdb-rs/foundationdb-rs/.github/workflows/correctness.yml@main": failed to fetch workflow: workflow was not found.

@mohitsaxenaknoldus
Copy link
Author

error parsing called workflow "foundationdb-rs/foundationdb-rs/.github/workflows/correctness.yml@main": failed to fetch workflow: workflow was not found.

This is supposed to be the right syntax :/
I am not sure what is wrong.

@PierreZ
Copy link
Member

PierreZ commented Feb 2, 2022

error parsing called workflow "foundationdb-rs/foundationdb-rs/.github/workflows/correctness.yml@main": failed to fetch workflow: workflow was not found.

This is supposed to be the right syntax :/ I am not sure what is wrong.

This is true, the file does not yet exists on the main branch, only on your branch. Maybe you could resolve it locally(i.e in your own branch?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve CI by reusing steps
4 participants