Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@cj-chung
Copy link
Contributor

@cj-chung cj-chung commented Sep 29, 2020

Create RouterConfigurationInfo and its related entities as follow:

In package com.futurewei.alcor.common.enumClass:

  • OperationType;
  • MessageType

In package com.futurewei.alcor.web.entity.route:

  • InternalRouterInfo;
  • InternalRouterConfiguration;
  • InternalSubnetRoutingTable;
  • InternalRoutingRule;
  • InternalRoutingRuleExtraInfo

@xieus xieus added the contract new or modified service interfaces label Sep 29, 2020
@xieus xieus added this to the Version 0.9.2020.09.30 milestone Sep 29, 2020
@cj-chung cj-chung requested a review from haboy52581 September 29, 2020 22:00
/*
Copyright 2019 The Alcor Authors.

Licensed under the Apache License, Version 2.0 (the "License");
Copy link
Contributor

Choose a reason for hiding this comment

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

@xieus - maybe we should start using the new license for new files? What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

It is okay. We will do a one-time change across the whole repo. Thanks for the reminder.

@xieus xieus changed the title RouterConfigurationInfo and its related entities [Contract] Router Internal Configuration and Related Entity Classes Sep 30, 2020
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

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

LGTM.

@xieus xieus changed the title [Contract] Router Internal Configuration and Related Entity Classes [Contract] Router Internal Config and Related Entity Classes Sep 30, 2020
@xieus xieus merged commit 83f950b into futurewei-cloud:master Sep 30, 2020
@cj-chung cj-chung deleted the routerinfo branch September 30, 2020 17:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contract new or modified service interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants