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

Ensure the AzureFunction and "fat" Python script are in sync #79

Open
cudeso opened this issue Feb 24, 2024 · 2 comments
Open

Ensure the AzureFunction and "fat" Python script are in sync #79

cudeso opened this issue Feb 24, 2024 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@cudeso
Copy link
Owner

cudeso commented Feb 24, 2024

There's a difference between the AzureFunction and the "fat" Python app installed locally.

Need to find ways on how to keep both in sync.

I'm focussed on the Python app locally, but open to ideas from @Kaloszer @lnfernux @jusso-dev

@cudeso cudeso added enhancement New feature or request help wanted Extra attention is needed labels Feb 24, 2024
@Kaloszer
Copy link
Contributor

Kaloszer commented Feb 26, 2024

  1. For logging just have a custom function that takes 'environment' flag - that has either local/azurefunction as a param and depending on that do console.log/context.log
  2. Merge both functions to keep the logic from both, I believe it was mentioned that the local one has the old method, but no reason to drop it if it hadn't been removed/deprecated yet.
  3. The local function can then just be a script reference to the 'merged' one from the azure function with the 'local' switch, default value of the switch would be azurefunction

PS: I'm quite busy at work at the moment and don't have much time, but hope that it will change, at least for the better in a couple of weeks :)

@lnfernux
Copy link
Collaborator

lnfernux commented Mar 5, 2024

Same situation as @Kaloszer

I think 1. is a good idea, same with 2. as long as we are able to do that.

cudeso added a commit that referenced this issue Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants