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

Remove berkeley codename from CI #16160

Closed
wants to merge 268 commits into from

Conversation

dkijania
Copy link
Member

Port CI fixes from dev:

  • rosetta fix
  • berkeley codename removal
  • shellcheck for build scripts

dkijania and others added 30 commits August 2, 2024 15:41
…_to_develop_0208

merge compatible to develop 02/08
…_deb_dev

[DEV] Removed Test executive dockers and buster debian
swapping out the s3 bucket link for snark-keys
normalizing snarky yojson serializers to match kimchi common
dkijania and others added 22 commits September 23, 2024 23:32
…berkeley_removal

port compatible berkeley removal to dev
…ve_test_dev

[Dev] Port Patch archive test again
[Dev] Update logo and description for README.md
Still getting the following warnings/errors:
```
In scripts/debian/build.sh line 12:
  BRANCH_NAME="${BRANCH_NAME}" source "${SCRIPTPATH}"/../export-git-env-vars.sh
                                      ^-- SC1091 (info): Not following: ./../export-git-env-vars.sh was not specified as input (see shellcheck -x).

In scripts/debian/build.sh line 14:
  source "${SCRIPTPATH}"/../export-git-env-vars.sh
         ^-- SC1091 (info): Not following: ./../export-git-env-vars.sh was not specified as input (see shellcheck -x).

In scripts/debian/build.sh line 19:
source "${SCRIPTPATH}"/builder-helpers.sh
       ^-- SC1091 (info): Not following: ./builder-helpers.sh was not specified as input (see shellcheck -x).
```
I am still getting:
```
In scripts/debian/publish.sh line 60:
) && break || scripts/debian/clear-s3-lockfile.sh; done
  ^-- SC2015 (info): Note that A && B || C is not if-then-else. C may run when A is true.

In scripts/debian/publish.sh line 80:
  for _i in {1..10}; do (verify_o1test_repo_has_package "${deb}") && break || sleep 60; done
                                                                  ^-- SC2015 (info): Note that A && B || C is not if-then-else. C may run when A is true.

For more information:
  https://www.shellcheck.net/wiki/SC2015 -- Note that A && B || C is not if-t...
```
I am still getting the following message:
```
In scripts/debian/reversion.sh line 63:
source scripts/debian/publish.sh --names "${NEW_NAME}_${NEW_VERSION}.deb" --version "${NEW_VERSION}" --codename "${CODENAME}" --release "${NEW_RELEASE}"
       ^-----------------------^ SC1091 (info): Not following: scripts/debian/publish.sh was not specified as input (see shellcheck -x).

For more information:
  https://www.shellcheck.net/wiki/SC1091 -- Not following: scripts/debian/pub...
```
I am still getting:
```
In scripts/debian/verify.sh line 47:
  && echo 'OK: ALL WORKED FINE!' || (echo 'KO: ERROR!!!' && exit 1)
  ^-- SC2015 (info): Note that A && B || C is not if-then-else. C may run when A is true.

For more information:
  https://www.shellcheck.net/wiki/SC2015 -- Note that A && B || C is not if-t...
```
…witches-develop

reformat: ignore opam-switches [develop]
…elop

[easy/develop] Make shellcheck happier on scripts/debian/*.sh
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania changed the title Port CI improvements from dev Port CI improvements from dev to comp Sep 29, 2024
@dkijania
Copy link
Member Author

dkijania commented Oct 7, 2024

!ci-build-me

@dkijania dkijania changed the title Port CI improvements from dev to comp Remove berkeley codename from CI Oct 23, 2024
@dkijania dkijania closed this Oct 28, 2024
@dkijania dkijania deleted the dkijania/port_ci_fixes_from_dev branch October 28, 2024 21:40
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.

7 participants