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

Added tests at the end of build process, updated containers and command line program #481

Merged
merged 47 commits into from
Jun 7, 2024
Merged

Added tests at the end of build process, updated containers and command line program #481

merged 47 commits into from
Jun 7, 2024

Conversation

masesdevelopers
Copy link
Contributor

@masesdevelopers masesdevelopers commented Jun 5, 2024

Description

Added tests at the end of build process using an Apache Kafka server hosted in a service which uses the container based on KNet:

  • the container was updated to host:
    • Server through environment variable KNET_RUNNING_MODE:
      • a standalone zookeeper/kafka server (KNET_RUNNING_MODE=standalone)
      • a zookeeper node (KNET_RUNNING_MODE=zookeeper)
      • a kafka node (KNET_RUNNING_MODE=broker)
    • CLI (KNET_RUNNING_MODE not set): the same container can start a CLI able to execute CLI commands or, with the right parameters, the Server side commands
  • the container now uses a bash script as main entry point which manages the usage
  • KNetCLI: removed the constraint on ClassToRun, if the switch is not available the first argument is used to decide which class invokes

Related Issue

closed #480

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@masesdevelopers masesdevelopers added enhancement New feature or request KNet KNet related issue KNetCLI KNetCLI related issue github_actions Pull requests that update GitHub Actions code labels Jun 5, 2024
@masesdevelopers masesdevelopers self-assigned this Jun 5, 2024
@masesdevelopers masesdevelopers changed the title Added tests at the end of build process Added tests at the end of build process, updated containers and command line program Jun 7, 2024
@masesdevelopers masesdevelopers merged commit aa2ef03 into masesgroup:master Jun 7, 2024
7 checks passed
@masesdevelopers masesdevelopers deleted the 480-add-test-execution-at-the-end-of-build-process branch June 7, 2024 18:40
@masesdevelopers
Copy link
Contributor Author

The current test behavior fails in many execution and some one of them waits undefinitively for a subscription to become available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request github_actions Pull requests that update GitHub Actions code KNet KNet related issue KNetCLI KNetCLI related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add test execution at the end of build process
1 participant