X Tutup
Skip to content

Build scan plugin as separate package#537

Merged
tiborvass merged 9 commits intodocker:masterfrom
thaJeztah:scan_plugin_package
Apr 1, 2021
Merged

Build scan plugin as separate package#537
tiborvass merged 9 commits intodocker:masterfrom
thaJeztah:scan_plugin_package

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Mar 31, 2021

Need to separate this more, because currently, the "deb" variant will use the same pacakge version for all packages produces, and inherit the package version from the cli/engine. fixed

replaces / closes #517
replaces / closes #528
closes #530

@thaJeztah thaJeztah force-pushed the scan_plugin_package branch 8 times, most recently from f92b878 to 472089e Compare March 31, 2021 14:23
glours and others added 9 commits April 1, 2021 01:10
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
Alternative solution was to update git on centos7 in /rpm/SPEC/docker-ce-cli.spec: 
```
if [ "$SUITE" == "7" ]; then
    yum install -y http://opensource.wandisco.com/centos/7/git/x86_64/wandisco-git-release-7-2.noarch.rpm
    yum install -y git
fi
```

But rather not changing what is installed on the CEnTOS bistro for the entire CI pipeline

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
This allows building from a source tar.gz if the needed variables (COMMIT,
TAG_NAME, PLATFORM_BINARY) are set.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Need to separate this more, because currently, it will inherit
the package version from the cli/engine.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Override the package version manually until we have properly separated the
control files (etc.) for each package that we build.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the scan_plugin_package branch from 472089e to 3497992 Compare March 31, 2021 23:10
@thaJeztah thaJeztah changed the title [wip] build scan plugin as separate package Build scan plugin as separate package Apr 1, 2021
@thaJeztah thaJeztah marked this pull request as ready for review April 1, 2021 11:41
@tiborvass tiborvass merged commit 308e055 into docker:master Apr 1, 2021
@thaJeztah thaJeztah deleted the scan_plugin_package branch April 1, 2021 20:08
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.

4 participants

X Tutup