Skip to content

Commit

Permalink
Merge pull request #2461 from influxdata/rem-386
Browse files Browse the repository at this point in the history
chore: remove support for 386
  • Loading branch information
docmerlin authored Jan 6, 2021
2 parents ea128fe + 65371d8 commit 84194d6
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 74 deletions.
2 changes: 1 addition & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
build:
docker:
- image: nathanielc/docker-client
parallelism: 3
parallelism: 2
working_directory: ~/kapacitor
steps:
- checkout
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
- [#2441](https://github.com/influxdata/kapacitor/pull/2441): Preallocate GroupIDs for increased performance by reducing allocations.
- [#2456](https://github.com/influxdata/kapacitor/pull/2456): Gzip data by default that is sent to influxdb.
- [#2454](https://github.com/influxdata/kapacitor/pull/2454): Add PrimaryProperty and SecondaryProperty methods to BigPanda AlertNode.
- [#2461](https://github.com/influxdata/kapacitor/pull/2461): BREAKING: we are forced to remove support for 386 builds as go doesn't support them anymore.

## v1.5.7 [2020-10-27]

Expand Down
58 changes: 0 additions & 58 deletions Dockerfile_build_ubuntu32

This file was deleted.

14 changes: 5 additions & 9 deletions build.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,9 @@
}

supported_builds = {
'darwin': [ "amd64", "i386" ],
'linux': [ "amd64", "i386", "armhf", "arm64", "armel", "static_i386", "static_amd64" ],
'windows': [ "amd64", "i386" ]
'darwin': [ "amd64" ],
'linux': [ "amd64", "armhf", "arm64", "armel", "static_amd64" ],
'windows': [ "amd64" ]
}

supported_packages = {
Expand Down Expand Up @@ -382,8 +382,6 @@ def get_system_arch():
arch = os.uname()[4]
if arch == "x86_64":
arch = "amd64"
elif arch == "386":
arch = "i386"
elif arch == "aarch64":
arch = "arm64"
elif 'arm' in arch:
Expand Down Expand Up @@ -531,9 +529,7 @@ def build(version=None,

# Handle variations in architecture output
fullarch = arch
if arch == "i386" or arch == "i686":
arch = "386"
elif arch == "aarch64" or arch == "arm64":
if arch == "aarch64" or arch == "arm64":
arch = "arm64"
elif "arm" in arch:
arch = "arm"
Expand Down Expand Up @@ -913,7 +909,7 @@ def main(args):
type=str,
help='Name to use for package name (when package is specified)')
parser.add_argument('--arch',
metavar='<amd64|i386|armhf|arm64|armel|all>',
metavar='<amd64|armhf|arm64|armel|all>',
type=str,
default=get_system_arch(),
help='Target architecture for build output')
Expand Down
7 changes: 1 addition & 6 deletions test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ then
fi

# Update this value if you add a new test environment.
ENV_COUNT=3
ENV_COUNT=2

# Default return code 0
rc=0
Expand Down Expand Up @@ -118,11 +118,6 @@ case $ENVIRONMENT_INDEX in
run_test_docker Dockerfile_build_ubuntu64 test_64bit_race --test --generate $no_uncommitted_arg --race
rc=$?
;;
2)
# 32 bit tests
run_test_docker Dockerfile_build_ubuntu32 test_32bit --test --generate $no_uncommitted_arg --arch=i386
rc=$?
;;
"count")
echo $ENV_COUNT
;;
Expand Down

0 comments on commit 84194d6

Please sign in to comment.