| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This reintroduces the non-generic flag classes:
- a value() returns the primitive type for flags wrapping a primitive type
- easier to use in testing
- Serializer is moved to internals of typed class
Defines the flag backed by boolean BooleanFlag instead of FeatureFlag since not
all boolean flags are necessarily guarding a feature.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
- Exposes all defined flags at /flags/v1/defined.
- Defines all hosted flags in Flags. Perhaps later we can support hiding
hosted flags (aka support different set of flags for different teams).
- Categorizes /flags/v1 as a hosted vespa operation for /zone/v2.
Also fixes component definitions in services.xml. The affected bundles have
been verified at an example config server.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a new ZooKeeper backed flag source. It is defined in a new module
configserver-flags to allow as many as possible config server modules to depend
on it by minimizing dependencies.
The content of the ZK backed flag source can be viewed and modified through
REST API on the config server/controller. The data stored per flag looks like
{
"rules": [
{
"conditions": [
{ "type": "whitelist", "dimension": "hostname", "values": ["host1"] }
],
"value": true
}
]
}
typical for enabling a feature flag on host1.
2 types of conditions are so far supported: whitelist and blacklist. All the
conditions must match in order for the value to apply. If the value is null (or
absent), the default value will be used. At the time the flag's value is
retrieved, it is resolved against the conditions with the current zone,
hostname, and/or application.
The same data structure is used for FileFlagSource for files in
/etc/vespa/flags with the ".2" extension.
The FlagSource component injected in the config server is changed to:
1. Return the flag value if specified in /etc/vespa/flags, or otherwise
2. return flag value from ZooKeeper (same as REST API)
The current flags (module) is also changed:
- All flags must be defined in com.yahoo.vespa.flags.Flags. This allows the ZK
backed flag source additional sanity checking when modifying flags.
- If it makes sense to have different flag value depending on e.g. the
application, then at some point before the value is retrieved, one has to
bind the flag to that application (using with() to set up the fetch vector).
Future changes would be to 0. make a merged FlagSource in host admin, 1. add
support for viewing and modifying feature flags in dashboard, 2. in hv tool.
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
vespa-engine/hmusum/avoid-tests-writing-to-module-directory
Refactor to avoid writing test data to module directory
|
| | |
|
|/ |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
vespa-engine/hakonhall/notify-service-monitors-of-infra-apps
Make service monitors aware of infra applications in duper model.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Notify monitors of infrastructure application activation. Live-flipping the
content of the duper model is non-trivial and has been removed.
- Split out DuperModel as a simple mutable and thread-unsafe container of the
applications in the duper model, that also handles calls listeners on
changes. The previous DuperModel has been renamed to DuperModelManager.
- Replace SuperModelProvider::snapshot method (fast but difficult to use
right) with registerListener.
- Shorten the fully qualified package names by 1-2 levels for mosts classes.
Next steps:
- Make HA query the real orchestrator
- Start experimenting with health monitoring of infra apps
|
|/
|
|
| |
Default value for config is true (unchanged from how we do it today)
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
vespa-engine/hakonhall/add-infrastructure-applications-to-dupermodel
Add infrastructure applications to DuperModel
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
DuperModel is (will be) responsible for both active tenant applications
(through SuperModel) and infrastructure applications. This PR is one step
in that direction:
- All infrastructure applications (config, confighost, controller,
controllerhost, and proxyhost) are owned and managed by DuperModel.
- The InfrastructureProvisioner retrieves all possible infra apps from the
DuperModel (through a reduced API), and "activates" each of them if
target is set and there are any nodes etc.
- The InfrastructureProvisioner then notifies the DuperModel which
apps have been activated, and with which hosts.
- The DuperModel can then build delegate artificially create ApplicationInfo,
which gets translated into the application model, and finally the service
model.
- The resulting service model has NOT_CHECKED for each hostadmin service
instance. This is sufficient for goal 1 of this sprint.
- The config server application currently has health, so that's kept as-is
for now.
- Feature flags have been tried and works and allows 1. to disable adding the
infra apps in the DuperModel, and 2. to enable the infra configserver
instead of the currently created configserver w/health.
|
|/
|
|
| |
We have com.yahoo.component.Version, and one is enough.
|
|
|
|
|
|
|
| |
This allows us to access model importers (such as TensorFlow)
in config models without loading one instance per config model
instance, which is not possible with TensorFlow because it depends
on JNI code.
|
|
|
|
|
|
|
| |
Since we have extended the semantics of major-version to also mean
"upgrade this application also if you generally don't upgrade",
we need to make sure specifying this doesn't cause config models
on *earlier* versions to not load.
|
| |
|
|\
| |
| | |
Cleanup
|
| | |
|
|\ \
| |/
|/|
| |
| | |
vespa-engine/hmusum/more-robust-handling-of-errors-for-config-server-maintainers
More robust handling of exceptions for config server maintainers
|
| | |
|
|/ |
|
| |
|
| |
|
|
|
|
|
| |
Need to keep old constructor and make a temporary one with an ignored
argument to make this work (since arguments will be equal due to type erasure)
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
vespa-engine/hakonhall/enforce-cc-timeouts-in-orchestrator-4
Revert "Revert "Enforce CC timeouts in Orchestrator 4""
|
| | |
|
|\ \
| |/
|/| |
Add optional metrics-packets.xml file to configserver app.
|
| | |
|
|/ |
|
|\
| |
| |
| |
| | |
vespa-engine/revert-7490-revert-7472-revert-7470-revert-7424-hakonhall/enforce-cc-timeouts-in-orchestrator-2
Enforce CC timeouts in Orchestrator [4]
|
| | |
|
|/ |
|
| |
|
| |
|
| |
|