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.

Sign up for GitHub

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

Jump to bottom

[CDF-21413] 😋 Function per file hashing #1327

Merged
merged 42 commits into from
Jan 6, 2025
Merged

[CDF-21413] 😋 Function per file hashing #1327

merged 42 commits into from
Jan 6, 2025
+307 −6

Conversation

doctrino
Copy link
Collaborator

@doctrino doctrino commented Jan 2, 2025 •
edited
Loading

Description

Example:

-    cdf-toolkit-function-hash: a218687fba5d77fcdea9a9601c5cdd1e9f82034f6b30c54f14daab9c0c4401a7
E       +    cognite-toolkit-hash: /=a218687f;config.py=f72f1ddb;extractor_config.yaml=b8f3d288;handler.py=675a6afe;ice_cream_factory_api.py=036a8ac9;requirements.txt=00847754

Note Cannot activate flag in local development as it would cause switching between running with/without flags to be difficult in the test suite. Tested it locally.

Checklist

doctrino and others added 27 commits January 1, 2025 13:11
@doctrino
test: extend testing
3e75bcd
@doctrino
fix: implemented emtpy
3518dcd
@doctrino
tests: updated mocking
3bc4d0c
@doctrino
fix: quote version key in location filter
84e9999
@doctrino
feat: added data modeling type
ca9c216
@doctrino
tests: updated test
59038a6
@doctrino
tests: regen
d5349d2
@doctrino
feat: support data modeling type
651497b
@doctrino
tests: update mock lookup
faa738a
@doctrino
build: changelog
e651803
@doctrino
fix: introduced bugs
23dfbde
@doctrino
fix; bug bug
84d0958
@doctrino
fix; bug bug bug
9cbd3be
@doctrino
tests: standardize
0d02410
@doctrino
tests: standardize 2
5530e99
@doctrino
tests: set cpu and memory outside limits
ce5129a
@doctrino
tests: try to recreate issue
4413d27
@doctrino
docs: documented case
9e241a4
@doctrino
fix: redeployment issue
c97e2d3
@doctrino
feat: added warning on mistake
710e03b
@doctrino
style: warning message
729d45d
@doctrino
feat; introduce ToolkitClientConfig
6e5d94d
@doctrino
style: better return value
d04e293
@doctrino
refactor: renaming
3041834
@doctrino
Update cognite_toolkit/_cdf_tk/client/_toolkit_client.py
c0c250a
@doctrino
docs; better docs
f38ff21
@doctrino
refactor: deal with special behavior
00d9ee8
@doctrino doctrino marked this pull request as ready for review January 2, 2025 08:14
@doctrino doctrino requested a review from a team as a code owner January 2, 2025 08:14
Base automatically changed from function-redeploy-issue to main January 6, 2025 11:02
@doctrino doctrino merged commit b102184 into main Jan 6, 2025
10 checks passed
@doctrino doctrino deleted the function-per-file-hashing branch January 6, 2025 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ronpal ronpal ronpal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@doctrino @ronpal