Skip to content

Conversation

@osipovartem
Copy link
Contributor

@osipovartem osipovartem commented Feb 17, 2025

  • removed USE_FILE_SYSTEM_INSTEAD_OF_CLOUD, USE_FS flags
  • optional bucket, region and credentials

To create FS (file system) storage profile just change your request to

### STORAGE PROFILE
payload = json.dumps({
    "type": "fs"
})
headers = {'Content-Type': 'application/json'}
response = requests.request("POST", f'{url}/storage-profiles', headers=headers, data=payload).json()

We don't need any other variables or additional logic

Closes #262
Closes #187

Combined docker image (init)

Set variable, set entity within session (#232)

* Set variable, set entity

* Fix comments

Rename multipart field to camelCase (#238)

* Rename multipart field to camelCase

* small improvement

Refine http status codes (#237)

* fix dbt retry

* fmt

* more retries fixed

Try to get catalog name from session with fallback to default (#239)

less redundancy in manifest (#242)

Fixing CORS to allow Cookies from UI (#235)

* Fixing CORS to allow Cookies from UI

* Formatting fix

* Clippy fixes

* Adding flag to enable CORS

* Adjusting CORS arg name

Add CONTIBUTING.md draft (#240)

* Add CONTIBUTING.md draft

* Updating CONTRIBUTING and CODE_OF_CONDUCT

* Renaming

---------

Co-authored-by: Scott Lyons <scottalyons@gmail.com>

Renaming icehut -> icebucket (#244)

* Rename IceHut -> IceBucket

* Formatting fix

Adding workflow to generator a list of contributors (#246)

Updating README structure with new name (#245)

* Initial README restructuring

* Update README.md (#234)

* Updating README with new name

* Adding LICENSE file

* Fixing LICENSE

---------

Co-authored-by: DanCodedThis <94703934+DanCodedThis@users.noreply.github.com>

Changing contributor script (#248)

* Adding workflow to generator a list of contributors

* Changing contributor script

Make SP endpoint optional (#250)

Tmp tyr to build

Dockerfile update

Add file system storage profile

Add file system storage profile
@osipovartem osipovartem force-pushed the issues/262_local_fs_profile branch from 6f4d286 to 2755b36 Compare February 17, 2025 10:41
@osipovartem osipovartem merged commit 9b7a65b into main Feb 17, 2025
3 of 4 checks passed
@osipovartem osipovartem deleted the issues/262_local_fs_profile branch February 17, 2025 20:21
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.

Add local FS storage profile Retire USE_FILE_SYSTEM_INSTEAD_OF_CLOUD

3 participants