-
Notifications
You must be signed in to change notification settings - Fork 53
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
- bubble up the errors to consumers of the cf config util.
47014c3
to
a1ba120
Compare
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
a1ba120
to
6a86a17
Compare
asalan316
approved these changes
Feb 20, 2025
* 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>
adc78eb
to
a00ca28
Compare
|
Closed infavor of #3555 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds CF_INSTANCE_KEY and CF_INSTANCE_CERT in golangapi for Scaling Engine.