Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 0 additions & 17 deletions tools/azure-sdk-tools/packaging_tools/code_report.py
Original file line number Diff line number Diff line change
Expand Up @@ -149,22 +149,6 @@ def merge_report(report_paths):
merged_report["operations"].update(report_json["operations"])
return merged_report

def filter_track2_versions(package_name, versions):
track2_versions = {
'azure-mgmt-appconfiguration': '1.0.0',
'azure-mgmt-compute':'17.0.0',
'azure-mgmt-eventhub':'8.0.0',
'azure-mgmt-keyvault':'7.0.0',
'azure-mgmt-monitor':'1.0.0',
'azure-mgmt-network':'16.0.0',
'azure-mgmt-resource':'15.0.0',
'azure-mgmt-storage':'16.0.0',
}
upbound = track2_versions.get(package_name)
if not upbound:
return versions
return list(filter(lambda x: x < upbound, versions))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this, which is added before

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The filter_track2_versions() is for track1 code_report.
And currently we are using release/v3 branch for track1.
This PR remove it from master.


def main(input_parameter: str, version: Optional[str] = None, no_venv: bool = False, pypi: bool = False, last_pypi: bool = False, output: str = None):
package_name, module_name = parse_input(input_parameter)
path_to_package = resolve_package_directory(package_name)
Expand All @@ -180,7 +164,6 @@ def main(input_parameter: str, version: Optional[str] = None, no_venv: bool = Fa
_LOGGER.info(f"Got {versions}")
if last_pypi:
_LOGGER.info(f"Only keep last PyPI version")
versions = filter_track2_versions(package_name, versions)
versions = [versions[-1]]

for version in versions:
Expand Down
19 changes: 16 additions & 3 deletions tools/azure-sdk-tools/packaging_tools/mgmt_release_stats.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,18 +46,26 @@ def add_provider(self, package_name, changelog_filename, start_date):
package_name) # set column title

with open(changelog_filename) as f:
in_stable_version = False
rls_date, rls_version = None, None
for line in f.readlines():
version, date = ReleaseStats.parse_release_event(line)
if version:
rls_date, rls_version = date, version
if rls_date >= start_date:
if re.match(r"^\d+\.\d+\.\d+$", version):
in_stable_version = True
else:
in_stable_version = False
if rls_date >= start_date and in_stable_version:
self._add_package_release(
self.provider_num+1, version, rls_date, package_name)

if rls_date and rls_date >= start_date and ReleaseStats.parse_breaking_head(line):
self._add_breaking_release(
self.provider_num+1, rls_version, rls_date, package_name)
if in_stable_version:
self._add_breaking_release(
self.provider_num+1, rls_version, rls_date, package_name)
else:
logging.info("skip breaking for {}".format(rls_version))

def gen_pivot_table(self):
logging.info("Generating pivot table")
Expand Down Expand Up @@ -173,6 +181,10 @@ def _add_package_release(self, col, version, rls_date, package_name):
row = self.find_or_create_row(rls_date, ws)
ws.cell(row=row, column=col, value=1)

ws = self.wb[SHEET_BREAKING_MATRIX]
row = self.find_or_create_row(rls_date, ws)
ws.cell(row=row, column=col, value=0)

ws = self.wb[SHEET_EVENT]
self.release_num += 1
ws.append([rls_date, package_name])
Expand Down Expand Up @@ -278,6 +290,7 @@ def build_release_stats(xls_file, threshold, start_date):
package_name, os.path.join(root, file), start_date)
stats.gen_latest_sum(90, SHEET_BREAKING_MATRIX, threshold-1)
stats.gen_latest_sum(180, SHEET_BREAKING_MATRIX, threshold)
stats.gen_latest_sum(90, SHEET_PACKAGE_MATRIX, threshold-1)
stats.gen_pivot_table()
stats.sum_matrix_sheets()
stats.save(xls_file)
Expand Down