Snowflake-Labs / schemachange

A Database Change Management tool for Snowflake
Apache License 2.0
481 stars 219 forks source link

The license_file parameter is deprecated, use license_files instead. #183

Closed kplgrwl closed 11 months ago

kplgrwl commented 11 months ago

We recently started getting this error on our Schemachange (version 3.5.2). Is there any workaround to fix it?

image

opodoprigora commented 11 months ago

Same issue, running schemachange: pip install -q schemachange==3.5.2

kplgrwl commented 11 months ago

We were using 3.4.1 in our production and this start showing up on our CI/CD pipeline since last 7/14 (Friday)

opodoprigora commented 11 months ago

@kplgrwl same but started today, this is our workflow

      - name: Checkout repo
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      - name: Use Python 3.10
        uses: actions/setup-python@v4
        with:
          python-version: '3.10'

      - name: Install Python Packages
        run: |
          echo "pip install pytest==7.1.3"
          pip install pytest==7.1.3

          echo "Installing Snowflake Python Connector: pip install -q snowflake-connector-python==2.8.0"
          pip install snowflake-connector-python==2.8.0

          echo "Installing schemachange: pip install -q schemachange==3.4.1"
          pip install -q schemachange==3.4.1  -- errors out
...

          echo "Setting temporary directory to store data by connector"
          export TMPDIR=/tmp
mjsqu commented 11 months ago

Caused by this issue with Cython3 in pyyaml? https://github.com/yaml/pyyaml/issues/724

👀

ogawade commented 11 months ago

Same issue

mjsqu commented 11 months ago

These changes will work-around the issue by updating PyYAML to 6.0 in the dependencies: https://github.com/mjsqu/schemachange/commit/41b3b5cad472799292d17dbf895650b1e73e8f36

vikas-iquanti commented 11 months ago

Same issue. The PyYAML dependency needs to be upgraded to 6.0

ogawade commented 11 months ago

Can someone who is collaborator add pyyaml = 6.0 and merge it into schemachange master?

snk commented 11 months ago

Same issue

opodoprigora commented 11 months ago

Thanks @mjsqu , using forked method as an interim solution.

sfc-gh-jhansen commented 11 months ago

Included in the new 3.5.3 release. Thanks!