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.

Sign up for GitHub

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

Jump to bottom

Fuzzy ci split workflows #9

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fuzzy ci split workflows #9

wants to merge 7 commits into from

Conversation

pitag-ha
Copy link
Owner

No description provided.

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from b0efcf9 to 10cc61f Compare December 19, 2023 16:36
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 10cc61f to bd3b42e Compare December 19, 2023 16:57
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha
Copy link
Owner Author

hi

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from bd3b42e to 20942bf Compare December 19, 2023 18:23
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch 2 times, most recently from 01a1359 to dee94cd Compare December 19, 2023 19:40
pitag-ha and others added 6 commits December 20, 2023 12:43
@pitag-ha
Add query num to output to observe server crashes
8af039f
@pitag-ha
First POC with dummy data
12b2951 
First working POC

A few updates

Checking out GH action artefacts

Find common ancestor

Useless commit to test GH action

Create data in parallel, save as artefacts

Save shas in job output

Add diff step

Try sending a comment to the PR

Improve save shas workflow

Upload diff as artifact

Upload diff as artifact
@pitag-ha @abbysmal
Fix conditional and extract shared variables
ddb4f6f 
Co-authored-by: Enguerrand Decorne <[email protected]>

Fix label logic conditional

Co-authored-by: Enguerrand Decorne <[email protected]>

Do some stuff

Co-authored-by: Enguerrand Decorne <[email protected]>
@pitag-ha
Fix approaval logic and others
e224abd 
Restructure

Remove timestamp from diff

Follow up on approval workflow

Fix artifact download

Use curl to download diff artifact

Debug why the gh api rate limit gets exceeded

Fix the approve workflow, i.e. sending a long msg contianing hash etc

Use GITHUB_TOKEN again (works when using the right syntax)

Update comment msg

Add output logic

Debug comparison of diff hashes

Handle both full reponses data and distilled data

Try to be more coherent with quotes

Fix URLs

Improve env hygiene and remove artifact_names job

Improve comment on approval

Create real data

Update merl-an

Update base branch name

Fix diff name and diff output

Disable the other gh actions for now

Fix the approval msg
@pitag-ha @abbysmal
Document env vars and pull in Irmin
ee29c9e 
Co-authored-by: Enguerrand Decorne <[email protected]>

Document the env variables

Co-authored-by: Enguerrand Decorne <[email protected]>

Pull in Irmin

Co-authored-by: Enguerrand Decorne <[email protected]>
@pitag-ha
Create significant data and readable diff
d0bb39f 
Install Irmin deps and build Irmin

Improve the diff output

Update the permissions

Format the diffs in a readable way

Move Irmin's lockfile

Fix diff creation

Debug why ocamlopt isn't in the PATH

Create more data

Reduce dif context

Build test/irmin-pack as well

Make sure Irmin deps are solved from its lock file

Use opam-monorepo to pull Irmin's dependencies instead of opam

Debugging and first irmin build cache implementation

More debugging and first merl-an cache implementation

Try both irmin and merl-an cache

Clean up the PR comment and stdout msgs

Last polishing of the diff output

Avoid triggering the workflow when unnecessary

Clean up commented code and debug steps

Make create_diff script more readable

Clean up and format the yml-file
@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from dee94cd to 7044f79 Compare December 20, 2023 11:44
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 7044f79 to 3e5b05f Compare December 20, 2023 12:01
@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 3e5b05f to d693484 Compare December 20, 2023 12:34
@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from a80b40a to 9179475 Compare December 20, 2023 16:12
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 9179475 to 41343bc Compare December 20, 2023 16:30
@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 41343bc to 1549a64 Compare December 20, 2023 16:48
@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 1549a64 to 2d0e1d8 Compare December 20, 2023 17:17
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch 3 times, most recently from 253c714 to 507a2c1 Compare December 20, 2023 19:56
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from ed1afeb to 507a2c1 Compare December 20, 2023 21:21
@pitag-ha
Split the workflow into two workflows
c1c8175 
With this, the Fuzzy CI workflow does not do anything that requires
permissions. Permissions are not granted when the workflow is triggered
by a PR from a fork.
@github-actions GitHub Actions
Copy link

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

@pitag-ha pitag-ha force-pushed the fuzzy-ci-split-workflows branch from 507a2c1 to c1c8175 Compare December 20, 2023 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fuzzy-diff-looks-good
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pitag-ha