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

python scripts core dump with non-default locale #7408

Closed
1 task done
tarzanek opened this issue Oct 12, 2020 · 4 comments
Closed
1 task done

python scripts core dump with non-default locale #7408

tarzanek opened this issue Oct 12, 2020 · 4 comments
Assignees
Labels
Milestone

Comments

@tarzanek
Copy link
Contributor

This is Scylla's bug tracker, to be used for reporting bugs only.
If you have a question about Scylla, and not a bug, please ask it in
our mailing-list at scylladb-dev@googlegroups.com or in our slack channel.

  • I have read the disclaimer above, and I am reporting a suspected malfunction in Scylla.

Installation details
Scylla version (or git commit hash): 4.2.rc4-0.20200907.bf0c493c28
Cluster size: 3
OS (RHEL/CentOS/Ubuntu/AWS AMI): Centos 7

Hardware details (for performance issues) Delete if unneeded
Platform (physical/VM/cloud instance type/docker):
Hardware: sockets= cores= hyperthreading= memory=
Disks: (SSD/HDD, count)

[scylla@scylla ~]$  sudo /usr/lib/scylla/node_exporter_install --help
usage: node_exporter_install [-h] [-F]

Download and install prometheus node_exporter

optional arguments:
  -h, --help   show this help message and exit
  -F, --force  Force re-installation when node_exporter is already installed
[scylla@scylla ~]$ scylla --version
4.2.rc4-0.20200907.bf0c493c28
[scylla@scylla ~]$ locale
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=
[scylla@scylla ~]$ export LC_ALL=en_US
[scylla@scylla ~]$ locale
LANG=en_US.UTF-8
LC_CTYPE="en_US"
LC_NUMERIC="en_US"
LC_TIME="en_US"
LC_COLLATE="en_US"
LC_MONETARY="en_US"
LC_MESSAGES="en_US"
LC_PAPER="en_US"
LC_NAME="en_US"
LC_ADDRESS="en_US"
LC_TELEPHONE="en_US"
LC_MEASUREMENT="en_US"
LC_IDENTIFICATION="en_US"
LC_ALL=en_US
[scylla@scylla ~]$  sudo /usr/lib/scylla/node_exporter_install --help
Segmentation fault
@tarzanek
Copy link
Contributor Author

I suppose the same problem happens for any other relocatable python scripts

@tarzanek
Copy link
Contributor Author

quick workaround is of course
export LC_ALL=en_US.UTF-8

@avikivity
Copy link
Member

Yes, we should just put that into the thunk.

@slivne slivne added this to the 4.4 milestone Oct 12, 2020
syuu1228 added a commit to syuu1228/scylla that referenced this issue Oct 12, 2020
scylla-python3 causes segfault when non-default locale specified.
As workaround for this, we need to set LC_ALL=en_US.UTF_8 on python3 thunk.

Fixes scylladb#7408
avikivity pushed a commit that referenced this issue Oct 18, 2020
scylla-python3 causes segfault when non-default locale specified.
As workaround for this, we need to set LC_ALL=en_US.UTF_8 on python3 thunk.

Fixes #7408

Closes #7414

(cherry picked from commit ff129ee)
avikivity pushed a commit that referenced this issue Oct 18, 2020
scylla-python3 causes segfault when non-default locale specified.
As workaround for this, we need to set LC_ALL=en_US.UTF_8 on python3 thunk.

Fixes #7408

Closes #7414

(cherry picked from commit ff129ee)
avikivity pushed a commit that referenced this issue Oct 18, 2020
scylla-python3 causes segfault when non-default locale specified.
As workaround for this, we need to set LC_ALL=en_US.UTF_8 on python3 thunk.

Fixes #7408

Closes #7414

(cherry picked from commit ff129ee)
@avikivity
Copy link
Member

Backported to 4.0, 4.1, 4.2.

syuu1228 added a commit to syuu1228/scylla that referenced this issue Nov 28, 2022
Since scylladb#7408 does not occur on current python3 version (3.11.0), let's drop
the workarond.
avikivity pushed a commit to scylladb/scylla-python3 that referenced this issue Nov 28, 2022
Since scylladb/scylladb#7408 does not occur on current python3 version (3.11.0), let's drop
the workarond.

Closes #32
avikivity pushed a commit that referenced this issue Nov 28, 2022
Since #7408 does not occur on current python3 version (3.11.0), let's drop
the workarond.

Closes #12097
gusev-p pushed a commit to gusev-p/scylla that referenced this issue Feb 27, 2023
The patch introduces the _partition_is_open flag,
inject partition_end only if there was some data
in the input reader.

A simple unit test has been added for
the nonforwardable reader which checks this
new behaviour.
Merge "Cqlsh serverless v2" from Karol Baryla

This PR adds serverless support for CQLSH. It was previosuly merged, but dtests later discovered a bug in python driver. The bug should be fixed now, so let's merge this again.

@fruch As you had more luck reproducing the issue with dtests than us, could you try and run them with this PR to make sure it works now?

Fixes scylladb#317 (again)
Closes scylladb#323

* github.com:scylladb/scylla-tools-java:
  Add Scylla Cloud serverless support
  Switch cqlsh to use scylla-driver
install.sh: drop locale workaround from python3 thunk

Since scylladb#7408 does not occur on current python3 version (3.11.0), let's drop
the workarond.

Closes scylladb#32
tls: add missing include <map>

std::multimap is later used in the file.

Closes scylladb#1314
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants