Remove JSON examples from artifacts
Pull Request
#440
Summary
Remove data source and location source JSONs from artifacts.
These JSONs are no longer provided from both CLI and WebAPI includes container image, but we can use them of course if we need because JSONs are still kept in the repo.
Changes
Essense of this PR is following:
- src/CarbonAware.DataSources/CarbonAware.DataSources.Json/src/CarbonAware.DataSources.Json.csproj
- Remove JSONs in
data-sources/fromItemGroup, anddata-sources/jsonwould be created in publish directory.
- Remove JSONs in
- src/CarbonAware.LocationSources/src/CarbonAware.LocationSources.csproj
- Remove JSONs in
location-sources/fromItemGroup, andlocation-sources/jsonwould be created in publish directory.
- Remove JSONs in
- src/CarbonAware.DataSources/CarbonAware.DataSources.Json/src/Configuration/JsonDataSourceConfiguration.cs
-
test-data-azure-emissions.jsonis no longer set by default.
-
- src/CarbonAware.WebApi/src/Dockerfile
- WebAPI container is no longer started without any configuration, so set JSON data source before
dotnet tool runfor generating OpenAPI document.
- WebAPI container is no longer started without any configuration, so set JSON data source before
- src/CarbonAware.WebApi/src/appsettings.json
- Remove data source configuration.
- src/GSF.CarbonAware/src/GSF.CarbonAware.csproj
- Remove both
data-sourcesandlocation-sources.
- Remove both
- src/GSF.CarbonAware/src/GSF.CarbonAware.targets
- Remove both
data-sourcesandlocation-sources, but they would be created into publish directory.
- Remove both
This PR has other changes, but they are for test cases because they depends JSON data source and location source.
Note that I commented out test case for ElectricityMapsFree in both src/CarbonAware.WebApi/test/integrationTests/CarbonAwareControllerTests.cs and src/CarbonAware.WebApi/test/integrationTests/LocationsControllerTests.cs because IntegrationTestingBase.cs does not have configuration for ElectricityMapsFree. I will fix it after this PR.
Checklist
- [x] Local Tests Passing?
- [x] CICD and Pipeline Tests Passing?
- [ ] Added any new Tests?
- [ ] Documentation Updates Made?
- [ ] Are there any API Changes? If yes, please describe below.
- [ ] This is not a breaking change. If it is, please describe it below.
Are there API Changes?
No
Is this a breaking change?
This PR breaks current behavior. CASDK is no longer refer test-data-azure-emissions.json for data source and azure-regions.json for location source. CASDK requires the user to configure data source at least.
This a stale pull request. Please review, update or/and close as necessary.
This PR conflicts current dev branch. I will fix after #555.