Skip to content

Tuned CI/CD for FIPs build#218

Merged
Enmk merged 5 commits into
Altinity:22.8-fips-go1.19-boringsslfrom
zvonand:22.8-fips-go1.19-boringssl
Jan 17, 2023
Merged

Tuned CI/CD for FIPs build#218
Enmk merged 5 commits into
Altinity:22.8-fips-go1.19-boringsslfrom
zvonand:22.8-fips-go1.19-boringssl

Conversation

@zvonand

@zvonand zvonand commented Jan 12, 2023

Copy link
Copy Markdown
Member

This enables building via workflows of FIPS build (with BoringSSL)
Changes:

  • Different Docker image for building (installed Docker-in-Docker)
  • Additional flags for packager

Comment thread docker/packager/binary/Dockerfile Outdated
Comment thread docker/packager/packager Outdated
@Enmk Enmk force-pushed the 22.8-fips-go1.19-boringssl branch from 8c22555 to b1d5419 Compare January 12, 2023 20:50
@zvonand zvonand requested a review from Enmk January 12, 2023 20:51
@zvonand zvonand force-pushed the 22.8-fips-go1.19-boringssl branch from 0cee3a2 to c588824 Compare January 12, 2023 21:08
@vzakaznikov

Copy link
Copy Markdown
Collaborator

@zvonand, let's add a description to this PR :).

@Enmk Enmk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Needs some clarifications and also maybe cleanup?

Comment thread docker/packager/packager Outdated
# and on hung you can simply press Ctrl-C and it will spawn a python pdb,
# but on SIGINT dockerd will exit, so ignore it to preserve the daemon.
trap '' INT
dockerd --host=unix:///var/run/docker.sock --host=tcp://0.0.0.0:2375 --default-address-pool base=172.17.0.0/12,size=24 &

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Where do these values (172.17.0.0 and :2375) come from?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

2375 is a default port for docker.
this IP is semi-default IP pool for composing networks

@Enmk Enmk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ok

@Enmk Enmk merged commit e6212d9 into Altinity:22.8-fips-go1.19-boringssl Jan 17, 2023
Enmk added a commit that referenced this pull request Jun 30, 2023
Enmk added a commit that referenced this pull request Jul 17, 2023
@zvonand zvonand deleted the 22.8-fips-go1.19-boringssl branch July 19, 2023 11:09
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