public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: [pve-devel] applied:  [PATCH] add ceph quincy repositories
Date: Tue, 06 Sep 2022 09:39:23 +0200	[thread overview]
Message-ID: <1662449952.vxkhhoqo2u.astroid@nora.none> (raw)
In-Reply-To: <20220802125927.2341276-1-a.lauterer@proxmox.com>

On August 2, 2022 2:59 pm, Aaron Lauterer wrote:
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
>  src/repositories/mod.rs      |  2 ++
>  src/repositories/standard.rs | 29 +++++++++++++++++++++++++++++
>  tests/repositories.rs        |  2 ++
>  3 files changed, 33 insertions(+)
> 
> diff --git a/src/repositories/mod.rs b/src/repositories/mod.rs
> index 88b515d..d8848b8 100644
> --- a/src/repositories/mod.rs
> +++ b/src/repositories/mod.rs
> @@ -90,6 +90,8 @@ pub fn standard_repositories(
>  
>      if product == "pve" {
>          result.append(&mut vec![
> +            APTStandardRepository::from(APTRepositoryHandle::CephQuincy),
> +            APTStandardRepository::from(APTRepositoryHandle::CephQuincyTest),
>              APTStandardRepository::from(APTRepositoryHandle::CephPacific),
>              APTStandardRepository::from(APTRepositoryHandle::CephPacificTest),
>              APTStandardRepository::from(APTRepositoryHandle::CephOctopus),
> diff --git a/src/repositories/standard.rs b/src/repositories/standard.rs
> index 6b2cafd..17cfa8d 100644
> --- a/src/repositories/standard.rs
> +++ b/src/repositories/standard.rs
> @@ -46,6 +46,10 @@ pub enum APTRepositoryHandle {
>      NoSubscription,
>      /// The test repository.
>      Test,
> +    /// Ceph Quincy repository.
> +    CephQuincy,
> +    /// Ceph Quincy test repository.
> +    CephQuincyTest,
>      /// Ceph Pacific repository.
>      CephPacific,
>      /// Ceph Pacific test repository.
> @@ -75,6 +79,8 @@ impl TryFrom<&str> for APTRepositoryHandle {
>              "enterprise" => Ok(APTRepositoryHandle::Enterprise),
>              "no-subscription" => Ok(APTRepositoryHandle::NoSubscription),
>              "test" => Ok(APTRepositoryHandle::Test),
> +            "ceph-quincy" => Ok(APTRepositoryHandle::CephQuincy),
> +            "ceph-quincy-test" => Ok(APTRepositoryHandle::CephQuincyTest),
>              "ceph-pacific" => Ok(APTRepositoryHandle::CephPacific),
>              "ceph-pacific-test" => Ok(APTRepositoryHandle::CephPacificTest),
>              "ceph-octopus" => Ok(APTRepositoryHandle::CephOctopus),
> @@ -90,6 +96,8 @@ impl Display for APTRepositoryHandle {
>              APTRepositoryHandle::Enterprise => write!(f, "enterprise"),
>              APTRepositoryHandle::NoSubscription => write!(f, "no-subscription"),
>              APTRepositoryHandle::Test => write!(f, "test"),
> +            APTRepositoryHandle::CephQuincy => write!(f, "ceph-quincy"),
> +            APTRepositoryHandle::CephQuincyTest => write!(f, "ceph-quincy-test"),
>              APTRepositoryHandle::CephPacific => write!(f, "ceph-pacific"),
>              APTRepositoryHandle::CephPacificTest => write!(f, "ceph-pacific-test"),
>              APTRepositoryHandle::CephOctopus => write!(f, "ceph-octopus"),
> @@ -115,6 +123,13 @@ impl APTRepositoryHandle {
>                  "This repository contains the latest packages and is primarily used for test labs \
>                  and by developers to test new features."
>              }
> +            APTRepositoryHandle::CephQuincy => {
> +                "This repository holds the main Proxmox Ceph Quincy packages."
> +            }
> +            APTRepositoryHandle::CephQuincyTest => {
> +                "This repository contains the Ceph Quincy packages before they are moved to the \
> +                main repository."
> +            }
>              APTRepositoryHandle::CephPacific => {
>                  "This repository holds the main Proxmox Ceph Pacific packages."
>              }
> @@ -139,6 +154,8 @@ impl APTRepositoryHandle {
>              APTRepositoryHandle::Enterprise => "Enterprise",
>              APTRepositoryHandle::NoSubscription => "No-Subscription",
>              APTRepositoryHandle::Test => "Test",
> +            APTRepositoryHandle::CephQuincy => "Ceph Quincy",
> +            APTRepositoryHandle::CephQuincyTest => "Ceph Quincy Test",
>              APTRepositoryHandle::CephPacific => "Ceph Pacific",
>              APTRepositoryHandle::CephPacificTest => "Ceph Pacific Test",
>              APTRepositoryHandle::CephOctopus => "Ceph Octopus",
> @@ -155,6 +172,8 @@ impl APTRepositoryHandle {
>              }
>              APTRepositoryHandle::NoSubscription => "/etc/apt/sources.list".to_string(),
>              APTRepositoryHandle::Test => "/etc/apt/sources.list".to_string(),
> +            APTRepositoryHandle::CephQuincy => "/etc/apt/sources.list.d/ceph.list".to_string(),
> +            APTRepositoryHandle::CephQuincyTest => "/etc/apt/sources.list.d/ceph.list".to_string(),
>              APTRepositoryHandle::CephPacific => "/etc/apt/sources.list.d/ceph.list".to_string(),
>              APTRepositoryHandle::CephPacificTest => "/etc/apt/sources.list.d/ceph.list".to_string(),
>              APTRepositoryHandle::CephOctopus => "/etc/apt/sources.list.d/ceph.list".to_string(),
> @@ -200,6 +219,16 @@ impl APTRepositoryHandle {
>                  },
>                  format!("{}test", product),
>              ),
> +            APTRepositoryHandle::CephQuincy => (
> +                APTRepositoryPackageType::Deb,
> +                vec!["http://download.proxmox.com/debian/ceph-quincy".to_string()],
> +                "main".to_string(),
> +            ),
> +            APTRepositoryHandle::CephQuincyTest => (
> +                APTRepositoryPackageType::Deb,
> +                vec!["http://download.proxmox.com/debian/ceph-quincy".to_string()],
> +                "test".to_string(),
> +            ),
>              APTRepositoryHandle::CephPacific => (
>                  APTRepositoryPackageType::Deb,
>                  vec!["http://download.proxmox.com/debian/ceph-pacific".to_string()],
> diff --git a/tests/repositories.rs b/tests/repositories.rs
> index 70e0201..fd94e0d 100644
> --- a/tests/repositories.rs
> +++ b/tests/repositories.rs
> @@ -362,6 +362,8 @@ fn test_standard_repositories() -> Result<(), Error> {
>          APTStandardRepository::from(APTRepositoryHandle::Enterprise),
>          APTStandardRepository::from(APTRepositoryHandle::NoSubscription),
>          APTStandardRepository::from(APTRepositoryHandle::Test),
> +        APTStandardRepository::from(APTRepositoryHandle::CephQuincy),
> +        APTStandardRepository::from(APTRepositoryHandle::CephQuincyTest),
>          APTStandardRepository::from(APTRepositoryHandle::CephPacific),
>          APTStandardRepository::from(APTRepositoryHandle::CephPacificTest),
>          APTStandardRepository::from(APTRepositoryHandle::CephOctopus),
> -- 
> 2.30.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 




      reply	other threads:[~2022-09-06  7:39 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-02 12:59 [pve-devel] " Aaron Lauterer
2022-09-06  7:39 ` Fabian Grünbichler [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1662449952.vxkhhoqo2u.astroid@nora.none \
    --to=f.gruenbichler@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal