Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Trac Trac
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Issues 246
    • Issues 246
    • List
    • Boards
    • Service Desk
    • Milestones
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
  • Wiki
    • Wiki
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar
  • Legacy
  • TracTrac
  • Issues
  • #32969

Closed (moved)
(moved)
Open
Created Jan 15, 2020 by opara@opara🎄

Chutney 'bootstrap-network.sh' shouldn't overwrite 'CHUTNEY_DATA_DIR'

When you set up a tor network with something like:

tools/test-network.sh --flavor basic --net-dir /tmp/chutney-net

the tools/test-network.sh script will set $CHUTNEY_DATA_DIR based on the value of --net-dir and then call tools/bootstrap-network.sh. This bootstrapping script will then overwrite the variable $CHUTNEY_DATA_DIR if that directory doesn't exist or if the path is relative. This causes unexpected behavior when the user explicitly sets the --net-dir option. For example, Chutney works fine if you provide it with a directory that doesn't exist (it will create that directory automatically), so there is no need for the tools/bootstrap-network.sh script to unexpectedly change it to $CHUTNEY_PATH/net. The tools/bootstrap-network.sh also does not need to change it to an absolute path since Chutney does this automatically.

My suggestion is to remove the following lines from tools/bootstrap-network.sh:

# Get a working net path
if [ ! -d "$CHUTNEY_DATA_DIR" ]; then
    # looks like a broken path: use the chutney path as a base
    export CHUTNEY_DATA_DIR="$CHUTNEY_PATH/net"
fi
if [ -d "$PWD/$CHUTNEY_DATA_DIR" ]; then
    # looks like a relative path: make chutney path absolute
    export CHUTNEY_DATA_DIR="$PWD/$CHUTNEY_DATA_DIR"
fi
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking