Skip to content

Conversation

@falkzoll
Copy link
Contributor

  • An akka upgrade in apache/openwhisk required changes here to successfully build and run the test cases.

- An akka upgrade in apache/openwhisk required changes here to successfully build and run the test cases.
@falkzoll
Copy link
Contributor Author

Closing/Opening PR to force re-check in travis.

@falkzoll falkzoll closed this Jul 16, 2021
@falkzoll falkzoll reopened this Jul 16, 2021
@falkzoll
Copy link
Contributor Author

Two tests are still failing. Checked the travis build logs and they fail since longer time already.

runtime.actionContainers.PythonActionLoopContainerTests > actionloop-python-v3.7 should run zipped Python action containing a virtual environment FAILED
runtime.actionContainers.PythonActionLoopContainerTests > actionloop-python-v3.7 should run zipped Python action containing a virtual environment with non-standard entry point FAILED

I will try to find the root cause for them to get the build unblocked again.
In the meantime I will close this PR. Will open it again, when I have an update.

@falkzoll falkzoll closed this Jul 16, 2021
- Python package `random-useragent` seem to be removed from pypi.org. This python package is used in the virtualenv tests as a sample package to install.
  Because of the now missing package, the generated virtualenvs are invalid and cause the testcases to fail.
  Moved to the similar package `random_user_agent` instead and fixed the related action code to unblock the build.
@falkzoll falkzoll reopened this Jul 16, 2021
@falkzoll falkzoll changed the title Resolve akka versions explicitly. Resolve akka versions explicitly to unblock build. Jul 16, 2021
@falkzoll
Copy link
Contributor Author

The python package random-useragent has been removed from pypi.org. This package is used in as sample in some of the test cases. Updated the test actions to use the similar package random_user_agent package.

@falkzoll falkzoll closed this Jul 16, 2021
@falkzoll falkzoll reopened this Jul 16, 2021
@sciabarracom
Copy link
Contributor

I included it in #111 (it was necessary) with lot of rework of the tests

@falkzoll
Copy link
Contributor Author

Closing this PR as its content is included in the now merged PR #111 .

@falkzoll falkzoll closed this Jul 19, 2021
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.

3 participants