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

this is to fix github actions error #176

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

Conversation

nit-in
Copy link
Contributor

@nit-in nit-in commented Sep 10, 2021

the LEAVE function is being used for stopping the script gracefully
but it is also being used in the end for exiting after script have ran successfully,
this might not break the build in local host but causing an error in Github actions
as the script exits with exit 1, which is considered as failure (in bash) even when the output gsi image is created successfully,
this https://github.com/nit-in/erfan_gsi_mmx_6746/actions/runs/1206555548 run might give an idea
as the gsi is created but workflow fails
Solution: using other function BYE or exit 0 at the end

Signed-off-by: nit-in [email protected]

the LEAVE function is being used for stopping the script gracefully
but it is also being used in the end for exiting after script have ran successfully,
this might not break the build in local host but causing an error in Github actions
as the script exits with exit 1, which is considered as failure even when the output gsi image is created successfully,
this https://github.com/nit-in/erfan_gsi_mmx_6746/actions/runs/1206555548 run might give an idea
as the gsi is created but workflow fails
Solution: using other function BYE or exit 0 at the end

Signed-off-by: nit-in <[email protected]>
as the script runs successfully, unmount the partitions and clean the working directory
before exiting same as LEAVE function

Signed-off-by: nit-in <[email protected]>
@Velosh
Copy link
Contributor

Velosh commented Sep 10, 2021

Why is this necessary? Since you can fork it instead of pushing dummy code to official repo. In my opinion, no need to do it. You can try to use || (if fail) to exit 0 code, maybe.

@erfanoabdi
Copy link
Owner

make it dependent on CLEAN argument

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.

3 participants