Permalink
Commits on May 14, 2014
  1. Add test cases surrounding tcp service_profile

    mikepea committed May 14, 2014
    As we are using individual schema files for tcp, http, and https
    service_profile sections, it is worth explicitly testing each
    section type.
    
    This adds the missing tests for tcp service_profile sections:
    
    * valid config
    * a bogus config with a 'persistence' section specified.
  2. Add http persistence section to our integ tests

    mikepea committed May 13, 2014
    Include the newly enabled persistence section in our
    integration tests, as we don't use it but the wider community
    does.
  3. Add missing schema for virtual_server persistence

    mikepea committed May 13, 2014
    There was no schema defined for virtual_server persistence method,
    which resulted in #36
    being raised.
    
    This adds the missing schema, and tests to confirm it is working.
    
    It reflects that COOKIE method is available via http, SSL_SESSION_ID via https, and
    nothing via tcp.
Commits on May 13, 2014
  1. Merge pull request #60 from alphagov/70161966-namespacing

    mikepea committed May 13, 2014
    [Fixes #70161966] Rename Vcloud::EdgeGatewayServices and Vcloud::Schema
Commits on May 2, 2014
  1. Switch to Core.logger

    mikepea committed May 2, 2014
Commits on Apr 30, 2014
  1. Improve CI credentials handling

    mikepea committed Apr 30, 2014
Commits on Apr 28, 2014
Commits on Apr 22, 2014
Commits on Mar 6, 2014
  1. Refactor out the 'test_cases' array

    mikepea committed Mar 6, 2014
    The test_cases array is not useful in the generators tests,
    and has caused problems.
    
    This refactors it into individual test cases, as per NatService
    generator.
  2. DRY out the ConfigurationDiffer classes

    mikepea committed Mar 6, 2014
    Make ConfigurationDiffer the parent of all per-service differs,
    since they are all sharing common behaviour, just altering
    what they consider to be 'the configuration for comparison'
  3. Add FirewallConfigurationDiffer class to ignore Id

    mikepea committed Mar 6, 2014
    (as per the NatConfigurationDiffer class in daebc34)
    
    When new firewall_rules are inserted, the resulting generator output
    dutifully increments the Id value of FirewallRules that follow.
    
    This creates a problem when comparing with ConfigurationDiffer,
    since the rules after our insertion have not changed, bar their
    Id value increasing in relation to the new rules.
    
    This commit adds a FirewallConfigurationDiffer class which strips out
    these Id values before comparison.
  4. Add NatConfigurationDiffer class to ignore Id

    mikepea committed Mar 6, 2014
    When new nat_rules are inserted, the resulting generator output
    dutifully increments the Id value of NatRules that follow.
    
    This creates a problem when comparing with ConfigurationDiffer,
    since the rules after our insertion have not changed, bar their
    Id value increasing in relation to the new rules.
    
    This commit adds a NatConfigurationDiffer class which strips out
    these Id values before comparison.
  5. DRY out differ tests before adding more classes

    mikepea committed Mar 6, 2014
    In anticipation of adding the Nat and Firewall differ classes
    (to handle Id fields), DRY out the ConfigurationDiffer and
    LoadBalancerConfigurationDiffer tests to have a common set
    of test cases that we can use to ensure all differ classes are
    obeying the same basic set of rules.
Commits on Mar 5, 2014
  1. Allow empty hashes for LoadBalancer services

    mikepea committed Mar 5, 2014
    All parameters underneath the per-service entries for
    virtual_server service_profile and pool service entries have
    defaults, thereby allowing an empty hash as an argument.
    
    The schema prevented this though, which this commit fixes.
    
    Now you can specify:
    
    pools:
      name: test-pool
      service:
        http: {}
    virtual_servers:
      name: test-vs
      service_profile:
        http: {}
  2. Update default healthcheck URI to be '/'

    mikepea committed Mar 5, 2014
    A healthcheck URI of '' (which technically is the defualt if no
    Uri is provided) results in a healthcheck request of 'OPTIONS *'.
    
    This is rejected with a 400 by Nginx and likely other servers, and
    so is not really a sensible default - vCloud Director considers the 400
    a bad request, and marks the member node as down.
    
    Instead, use '/' as the default healthcheck Uri, as this then maps
    to a 'GET /' call, generally returning ok by most sites.
    
    NB: This vCloud Director documentation suggests that the default is /
        regardless. This at least makes our tool match the docs.
  3. Updated README as per feedback

    mikepea committed Mar 5, 2014
Commits on Mar 4, 2014
  1. nat_service documentation

    mikepea committed Mar 4, 2014