Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Add CentOS 8 Package Builder Step #9776

Merged
merged 1 commit into from
Dec 17, 2020
Merged

Add CentOS 8 Package Builder Step #9776

merged 1 commit into from
Dec 17, 2020

Conversation

kj4ezj
Copy link
Contributor

@kj4ezj kj4ezj commented Dec 11, 2020

Change Description

From AUTO-279, the CI system and build scripts have been updated to support CentOS 8. However, where most of the steps in the CI system are auto-generated based on what platforms are defined in the .cicd/platforms folder, the package builder steps must be added to the CI system by hand because every operating system is like a snowflake...lol.

Tested Working

To verify the new binary works, I downloaded eosio-2.1.0-alpha2.x86_64.rpm from the CentOS 8 Package Builder step in eosio build 25943. Then, in the folder with that artifact:

docker pull centos:8
docker run -v "$(pwd):/eos" -w /eos -it centos:8 /bin/bash

Finally, inside the container:

yum update
yum install -y /eos/eosio-2.1.0-alpha2.x86_64.rpm
nodeos --version
nodeos --help

The output is omitted here for brevity, but it worked as expected.

See Also

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes
    None.

API Changes

  • API Changes
    None.

Documentation Additions

  • Documentation Additions
    None.

@kj4ezj kj4ezj marked this pull request as ready for review December 11, 2020 21:04
@kj4ezj kj4ezj requested a review from nksanthosh December 11, 2020 21:07
@nksanthosh
Copy link
Contributor

This PR will be merged in after the upcoming release of eos. Thank you.

@kj4ezj kj4ezj mentioned this pull request Dec 17, 2020
12 tasks
@kj4ezj kj4ezj merged commit 8d70ed5 into develop Dec 17, 2020
@kj4ezj kj4ezj deleted the zach-centos-8 branch December 17, 2020 15:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants