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

feat(scalingengine): Use cf scalingengine in golangapi #3529

Closed
wants to merge 101 commits into from

Conversation

bonzofenix
Copy link
Contributor

Adds CF_INSTANCE_KEY and CF_INSTANCE_CERT in golangapi for Scaling Engine.

bonzofenix and others added 30 commits January 31, 2025 12:46
Adds xfcc cf endpoint support to scaling engine

Remove debug println

WIP

wip

Fix warning

Initial implementation of CF ServeR

Add .trunk to .gitignore for Devbox configuration.

Add .trunk to .gitignore for Devbox configuration.
 - Rename cfServer to cf-server in configuration properties
 - Add launch.json for VSCode Java debugging setup
 - Remove destroyMethod annotation from CFServerConfig bean
 - Update application.yml to reflect new property names for cf-server
 • Changed VSCode Java build configuration setting from automatic to interactive
 • Added new CFHTTPConfiguration class to configure additional Tomcat connector for HTTP on port 8090
 • Set server port to 8083 and added HTTP port configuration in application.yml
…date gorouterproxy to handle TLS and XFCC header; extend testhelpers with localhost SANs in certs.
 • Implement NewXfccAuthMiddleware constructor for initializing middleware
 • Add XFCCAuthenticationMiddleware method to handle authentication within middleware
 • Enhance checkAuth method to include XFCC header parsing and certificate validation
 • Introduce parseXFCCHeader utility method for extracting attributes from XFCC header
 • Add getSpaceGuid and getOrgGuid methods to extract GUIDs from certificate
 • Optimize mapFrom method for parsing organizational units from certificates
 • Extract CheckAuth function from xfccAuthMiddleware for standalone use with org and space parameters.
 • Replace instance method calls with standalone function calls in auth checks.
 • Update gorouterproxy tests to use the new CheckAuth function and pass org and space GUIDs directly.
 • Remove debug logging from mapFrom function.
 - Correct the missing type for the `portToForward` parameter in `GoRouterProxy` function.
 - Update command arguments to include `--port` and `--forwardTo` flags in `GoRouterProxy`.
 - Add `GoRouterProxy` port to the `PreparePorts` function.
 - Prefix a test with `FWhen` to focus on a specific scheduler test.
 - Introduce a new Makefile target `build-gorouterproxy` to compile the gorouterproxy component.
 - Mark `testsuite` as a PHONY target in the Makefile.
 - Update gorouterproxy tests to check for the new startup log message.
 - Add TLS certificate and key file flags to gorouterproxy and update the main function to use provided paths if available.
 - Include additional configuration for the Golang API server in the integration tests.
 - Implement AfterEach cleanup for stopping the Golang API server and gorouterproxy in integration tests.
 - Add GoRouterProxy to the list of compiled executables in the integration suite setup.
 - Allocate a new port for GolangAPICFServer in the integration test suite.
… Postgres container detached with corrected max_connections config
 - Introduce SchedulerCFServer component to handle CF server connections
 - Update integration tests to use temporary directories and URLs
 - Remove CFHTTPConfiguration class and adjust application.yml for scheduler
 - Refactor integration test code for clarity and consistency
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore(deps): update google.golang.org/genproto digest to 29210b9

* 🤖🦾🛠️ go mod tidy & make package-specs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…:main docker digest to a850ee8 (#3470)

* chore(deps): update ghcr.io/cloudfoundry/app-autoscaler-release-tools:main docker digest to a850ee8

* trigger new ci run

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: geigerj0 <112163019+geigerj0@users.noreply.github.com>
 • Extract default API server configuration into defaultGolangAPITestConfig function
 • Remove hardcoded file paths for API info, schema validation, and service catalog
 • Simplify PrepareGolangApiServerConfig by using the new default config function
 • Clean up commented-out password hash code and other unused variables
 • Update configuration structure to match new defaults and remove unnecessary URI settings
 • Refactor defaultGolangAPITestConfig to DefaultGolangAPITestConfig and add dbURI parameter.
 • Include Port in Server and BrokerServer configurations within DefaultGolangAPITestConfig.
 • Add Db and MetricsForwarder configurations to DefaultGolangAPITestConfig.
 • Simplify PrepareGolangApiServerConfig by removing explicit port parameters and using DefaultGolangAPITestConfig.
 • Rename writeYmlConfig to WriteYmlConfig and update its usages across test files.
 • Update test setup functions to reflect changes in configuration functions.
 • Remove redundant port and metrics forwarder URL parameters from test configurations.
# Issue

Redocly CLI is the recommended replacement for the deprecated
swagger-cli package

# Fix

See https://redocly.com/docs/cli/guides/migrate-from-swagger-cli
…ting

 - Remove container usage from the code-style job in GitHub Actions workflow
 - Add steps to install Devbox and make its shell environment available
 - Resolve merge conflict in devbox.json by including google-java-format
 - Add google-java-format to devbox.lock with support for multiple architectures
 - Create new Makefile in src/scheduler for Java formatting and checking
@bonzofenix bonzofenix force-pushed the 849-use-cf-scalingengine-in-golangapi branch from 47014c3 to a1ba120 Compare February 18, 2025 19:20
bonzofenix and others added 8 commits February 18, 2025 20:46
classes must contain no more than '1' consecutive capital letters.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…gengine specs

 - Deleted autoscaler/configutil/* from eventgenerator, operator, and scalingengine
 - Removed autoscaler/vendor/github.com/cloud-gov/go-cfenv/* from eventgenerator and operator
 - Eliminated autoscaler/vendor/github.com/mitchellh/mapstructure/* from eventgenerator and scalingengine
@bonzofenix bonzofenix force-pushed the 849-use-cf-scalingengine-in-golangapi branch from a1ba120 to 6a86a17 Compare February 19, 2025 12:56
@bonzofenix bonzofenix added allow-acceptance-tests This label needs to be added to enable the acceptance tests to run. and removed Do Not Merge labels Feb 20, 2025
renovate bot added 7 commits February 20, 2025 14:51
* chore(deps): update google.golang.org/genproto digest to 56aae31

* 🤖🦾🛠️ go mod tidy & make package-specs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
… digest to 4740a49 (#3546)

* fix(deps): update code.cloudfoundry.org/loggregator-agent-release/src digest to 4740a49

* 🤖🦾🛠️ go mod tidy & make package-specs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore(deps): update google.golang.org/genproto digest to 9fdb1ca

* 🤖🦾🛠️ go mod tidy & make package-specs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* fix(deps): update golang.org/x/exp digest to aa4b98e

* 🤖🦾🛠️ go mod tidy & make package-specs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@bonzofenix bonzofenix force-pushed the 849-use-cf-scalingengine-in-golangapi branch from adc78eb to a00ca28 Compare February 20, 2025 13:53
@bonzofenix bonzofenix closed this Feb 20, 2025
@asalan316
Copy link
Contributor

Closed infavor of #3555

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
allow-acceptance-tests This label needs to be added to enable the acceptance tests to run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants