Skip to content

Commit

Permalink
Adding more clean up in the GH workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
renan-souza committed Dec 3, 2024
1 parent 2ed40df commit a119e68
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 11 deletions.
9 changes: 0 additions & 9 deletions .github/workflows/run-tests-kafka.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,12 +47,3 @@ jobs:
- name: Test notebooks
run: pytest --ignore=notebooks/zambeze.ipynb --nbmake "notebooks/" --nbmake-timeout=600 --ignore=notebooks/dask_from_CLI.ipynb

# - name: Test notebooks
# run: |
# pip install -e .[all] # Installing stuff again may not be needed
# export MQ_TYPE=kafka
# export MQ_PORT=9092
# python -c 'from flowcept.configs import MQ_TYPE, MQ_PORT; print(f"MQ_TYPE={MQ_TYPE}"); print(f"MQ_PORT={MQ_PORT}")'
# python -c 'from flowcept import Flowcept; assert Flowcept.services_alive()'
# pytest --ignore=notebooks/zambeze.ipynb --nbmake "notebooks/" --nbmake-timeout=600 --ignore=notebooks/dask_from_CLI.ipynb
5 changes: 4 additions & 1 deletion .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,4 +106,7 @@ jobs:
make tests
- name: Stop docker compose with kafka
run: docker compose -f deployment/compose-kafka.yml down
run: docker compose -f deployment/compose-kafka.yml down

- name: Clean up log files again
run: find /home/runner/runners/ -type f -name "*.log" -exec sh -c 'echo {}; >"{}"' \;
2 changes: 1 addition & 1 deletion src/flowcept/commons/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ def get_utc_minutes_ago(minutes_ago=1):
def perf_log(func_name, t0: float, logger=None):
"""Configure the performance log."""
if PERF_LOG:
_logger = logger or FlowceptLogger()
t1 = time()
_logger = logger or FlowceptLogger()
_logger.debug(f"[PERFEVAL][{func_name}]={t1 - t0}")
return t1
return None
Expand Down

0 comments on commit a119e68

Please sign in to comment.